https://jira.whamcloud.com This file is an XML representation of an issue en-us 9.4.14 940014 05-12-2023 https://jira.whamcloud.com/browse/LU-6245 Lustre <p>In times past libcfs was built for user space and kernel space. Today we no longer need to build a user land version of libcfs. This ticket will track the work of removing the libcfs abstraction to support both user space and kernel space. In the end we should have a minimal libcfs that only supports kernel space.</p> LU-6245

Untangle userland and kernel space support for libcfs

Improvement Major Resolved Fixed James A Simmons James A Simmons patch Fri, 13 Feb 2015 18:50:16 +0000 Fri, 2 Mar 2018 15:55:57 +0000 Sun, 13 Aug 2017 17:44:11 +0000 Lustre 2.8.0 Lustre 2.11.0 0 13 <p>James Simmons (uja.ornl@gmail.com) uploaded a new patch: <a href="http://review.whamcloud.com/13759" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13759</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove tcpip abstraction from libcfs<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 81c7924e9b27f2416c19dba5aeeb6d368ae6f79c</p> <p>James,</p> <p>Of course there are several ways to go about this but in my analysis the next step would be to sort out the LNet headers. Most are not needed for userspace and should be amended to generate an error if _<em>KERNEL</em>_ is not defined. Others like lib-types.h contain a mixture of wire types, definitions needed by kernel and user space, and definitions only needed by kernel space. This header and similar ones "hold on to" some of the libcfs cruft.</p> <p>James Simmons (uja.ornl@gmail.com) uploaded a new patch: <a href="http://review.whamcloud.com/13760" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13760</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove tcpip abstraction from libcfs<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 2f4a50f14a596a4f7bb86ed19a84cfc421d7a80a</p> <p>I have some patches that cleanup the LNET headers. The confusing thing with those headers is that user land definitions are located in a couple of headers. We need to talk to the LNET guys to discuss what goes where.</p> <p>James Simmons (uja.ornl@gmail.com) uploaded a new patch: <a href="http://review.whamcloud.com/13783" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13783</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: move lucache from libcfs to lustre<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 51427528b599bcbc4f7ec365eecab0dfc1b735f9</p> <p>James Simmons (uja.ornl@gmail.com) uploaded a new patch: <a href="http://review.whamcloud.com/13792" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13792</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> lnet: remove kernel defines in userland headers<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 0a1792b46812ac6c944993d1de7f5da5181f0f7f</p> <p>James Simmons (uja.ornl@gmail.com) uploaded a new patch: <a href="http://review.whamcloud.com/13793" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13793</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: cleanup libcfs lock handling<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 1bb798a89148ddd8377291d3976d4c0806a0770c</p> <p>James Simmons (uja.ornl@gmail.com) uploaded a new patch: <a href="http://review.whamcloud.com/13835" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13835</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: create userland and kernel string operations<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 2a49e29a102182bde900dbb8a76e2666bca61833</p> <p>James Simmons (uja.ornl@gmail.com) uploaded a new patch: <a href="http://review.whamcloud.com/13841" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13841</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove mem wrappers for libcfs<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 6d0640de868bf863303c73c5c0816e1e2411f832</p> <p>James Simmons (uja.ornl@gmail.com) uploaded a new patch: <a href="http://review.whamcloud.com/13874" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13874</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove prim wrappers for libcfs<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 27834b07967580fa9c55aefe3d0f7a2da3d938c1</p> <p>James Simmons (uja.ornl@gmail.com) uploaded a new patch: <a href="http://review.whamcloud.com/13899" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13899</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove bitops wrappers for libcfs<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: e4faab0020cc3f1face65cac1194d310d9915c6d</p> <p>James Simmons (uja.ornl@gmail.com) uploaded a new patch: <a href="http://review.whamcloud.com/13917" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13917</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove time wrappers for libcfs<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: b4813e50a58023fd9c3f96c75293fda7b43718c6</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/13760/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13760/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove tcpip abstraction from libcfs<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 0b868add80281c085ce1b297d1cb078deaab802a</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/13783/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13783/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: move lucache from libcfs to lustre<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: e2e7ea06c66d57ba214cd76e3d19420632aba8dd</p> <p>John L. Hammond (john.hammond@intel.com) uploaded a new patch: <a href="http://review.whamcloud.com/14180" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/14180</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove libcfsutil.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 1df4745ab0640465c253264056259d5b6685276f</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/13792/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13792/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> lnet: remove kernel defines in userland headers<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: c740130066cf618f4fd31eae716affbb98ecf308</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/13793/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13793/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: cleanup libcfs lock handling<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 7e95d67f81f1a1a2613809541f30ad4e06618df5</p> <p>frank zago (fzago@cray.com) uploaded a new patch: <a href="http://review.whamcloud.com/14270" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/14270</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> utils: split kernel comm between user and kernel<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 83ab3ecd1900f562117712ce7f784813b8e32432</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/13835/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13835/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: create userland and kernel string operations<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 05d13ca333a3158a98f0c0e048a85564c6b9ed88</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/14270/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/14270/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> utils: split kernel comm between user and kernel<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: fd44a6d8c71e59ed54b36b2b15e2bdd2b9a74d38</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/14512" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/14512</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> lustre: include kthread.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: e2f2976d9bf490674b5ab5c216a60beae34ec221</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/14624" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/14624</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: Sync up libcfs hash code with upstream<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: eff0abe0ba40fae2b8dc694ca3782b1d598c333f</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/14180/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/14180/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove libcfsutil.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 3b84a1ee5213563945225854a50e9037bb9646db</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/13841/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13841/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove mem wrappers for libcfs<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 6ea4de1b7f237d331c9e0d66f4cc53365d036e5d</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/13874/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13874/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove prim wrappers for libcfs<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: f6995cf04407dff15d6ca79ca44cfa97dc6eb014</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/13899/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13899/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove bitops wrappers for libcfs<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 9e9225acb37dc7e0ceb02c15e4597deab8f25e9a</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/14629" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/14629</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> tests: remove libcfs.h from test applications<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 5edf2f85241709e1873016de9e42ce4058b10e9c</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/14629/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/14629/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> tests: remove libcfs.h from test applications<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 6ac2f14f0f622863c330b8b96da9373bad1abeb5</p> <p>I have started the work on removing the random wrappers. Most of the cleanups are straight forward except for cfs_srand which is used to add to the entropy pool. Now starting in kernel 3.6 we have add_device_randomness which is only missing for RHEL6 kernels. The question is do we even need it. Should we just remove cfs_srand?</p> <p>Hi Andreas,<br/> Could you kindly please comment on the question from James? Thank you.</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/15083" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15083</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: move struct netstrfns to nidstr.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 0f2fab0dcba0941935c4f4b841627c27739a1abf</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/15085" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15085</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: move cfs_ip_addr_* function from libcfs to LNet<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 5c1427c6cbeced8715f05baf44bfd6bb77da3a5c</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/15086" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15086</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> lnet: Avoid nodemap related forward declarations in nidstring.c<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: e5a7758074b7807922d8c54968e1b955151d7088</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/15083/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15083/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: move struct netstrfns to nidstr.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: d231bc874075672d3302727e53baef8f4dd03287</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/14624/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/14624/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: cleanup up libcfs hash code for upstream<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: d8278c699434fd7975609e121a1a75820595a601</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/15197" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15197</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove last of user land code in libcfs module<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 866f45e4f9067c60d8fa14388b1f471bc52a1db5</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/15200" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15200</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: cleanup list handling<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 420773df8d5c2dcdb4819a33a4a60a46f02ba00a</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/15208" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15208</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> tests: remove libcfs.h from lctl code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 7fdcb5c40158c49c2f5c77b6f3a8ef7499ddc7aa</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/15217" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15217</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> utils: remove libcfs.h from lustre utilities code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 33afb72e4b370f32ccd2f655d7e9637608513b82</p> <p>The last two patches remove libcfs.h usage from the user land utilities. The libcfs.h header is still left in user land code via the lustre include headers. That will next on the list for cleanup and after that libcfs.h user land code can be removed.</p> <p>James, to answer your question on <tt>cfs_rand()</tt> and <tt>cfs_srand()</tt> and <tt>cfs_get_random_bytes()</tt> - these were very important for the Cray Catamount machine, which had tens of thousands of cores booting simultaneously with a single-threaded interrupt-free kernel. There was not any kind of h/w random number generator to use. That sometimes resulted in the identical random number entropy state across multiple nodes, which would cause problems with e.g. client UUIDs being the same on two clients, as well as lesser issues with initial lustre_handle, and initial XID value and other items that should be unique (if at least for debugging purposes).</p> <p>The solution was to include the NID into the random number pool, which was one of the very few things that was unique about each node. I think that if it is possible to call <tt>add_device_randomness()</tt> to include the NID into the entropy pool that would allow us to use the kernel PRNG with some assurance that the number sequences will be different across nodes.</p> <p>I suspect that for existing RHEL 6 installations that the chance of having the identical state across many nodes is low, most machines have h/w RNG these days via the CPU, and for the few places it matters we can still fold in the NID(s) directly. If the <tt>add_device_randomness()</tt> API <em>is</em> available with newer kernels then it should be used to include the NID(s) into the RNG pool, to avoid potential problems in the future if a Catamount-like system is revived again. </p> <p>These "serious" places all use <tt>cfs_get_random_bytes()</tt>. The <tt>cfs_rand()</tt> users in lod_qos.c and elsewhere doesn't need this special treatment, and obd_alloc_fail() could be removed entirely (I have an in-progress patch for that, if you want to take it over).</p> <p>One potential option is to have the RHEL6 <tt>add_device_randomness()</tt> wrapper hash the input bytes into a 64-bit mask that we can XOR into the values returned by the kernel's get_random_bytes() in cfs_get_random_bytes() (which can be dropped entirely once we drop RHEL6 support). Likewise <tt>ll_generate_random_uuid()</tt> could also be converted to generate_random_uuid() at that time.</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/15085/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15085/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: move cfs_ip_addr_* function from kernel libcfs to LNet<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 31114c83e492a1f831165bb2b96496560967b42d</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/15368" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15368</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> gss: remove libcfs.h from gss utilities code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: e1d230b3fb58498ca8e63b971a824593dd3009f2</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/15086/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15086/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> lnet: Avoid nid range related forward declarations in nidstring.c<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: cb611b97b7c43b6d93bc86d4fbde5ba01bb4c8e6</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/15208/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15208/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> tests: remove libcfs.h from lctl code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: a3b9617c740f063a198bd6d81b7860af5799f755</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/15217/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15217/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> utils: remove libcfs.h from lustre utilities code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: a8012629fa66bfeab28b41e23130d020d2367001</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/15368/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15368/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> gss: remove libcfs.h from gss utilities code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 9bcf83c0a84d7cbb86f4bcd55cc01b0ab7c72250</p> <p>James: How is this ticket progressing? With all the patch activity, I'm not sure where it stands.</p> <p>Doug</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/15197/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15197/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove last of user land code in libcfs module<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: b8e7ad5d6441c48aac6b4a0334735abe9f56ab28</p> <p>With the landing of the last patch the libcfs module code no longer has any user land code in it. The libcfs headers are left to cleanup.</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/15909" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15909</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove err.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 3f0e9c3ac45bb5879241b72e2af91e8d468bbabb</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/15910" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15910</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove linux-utils.c<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: d038659f07bfaf194c86434659b188ad489f698f</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/15913" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15913</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove atomic cpt allocations<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 0e3ce713faed609d4f7d6715ae5219898525fb51</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/13917/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/13917/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove unused cfs_timer_done<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 8e91e60a197b8700e1de4043b62aa63112a3eef3</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/15909/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15909/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove err.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: a1dabf7958d29cb75e67e1a4ddd1f220f713dceb</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/15913/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15913/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove atomic cpt allocations<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 2de6f02a3e1a4c5f17d4baf3044795453ac52009</p> <p>James, it looks like <a href="http://review.whamcloud.com/15910" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15910</a> is the last unlanded patch for this ticket. Are there more cleanup patches needed for this? If yes, can you please file a separate bug for 2.9.0 and we can close this one as fixed for 2.8.0, which simplifies tracking.</p> <p>Not currently but many more patches are needed. Wouldn't it be better for this ticket to be independent of any version. It is going to be a while before this is finished. </p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/15910/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15910/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove linux-utils.c<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 561bd1db8620a3166100b95f25cf10f1b0211e70</p> <p>I agree with James - removed fix version</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/16879" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/16879</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove types.h from userland code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 946bd6dc26a28f1f21b657d1725af56ee426f42a</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/16914" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/16914</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove userland headers from libcfs.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 160008ed3a36c8b74b7ea66136b11d597b9f0b27</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/16916" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/16916</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove byteorder.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 337eb855c27786b30d7c3493578f1d61234bbb03</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/17202" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/17202</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove typedefs in libcfs source code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: f94e740856388cb6ab3c23d9f9a6c27a61c52b39</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/17375" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/17375</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: delete some unused code in libcfs<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 61b195190ae25738143fad9e52462ea83cd73482</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/17376" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/17376</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove cfs_fs_time handling<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 531eaa2c776283f876dd73114e3901441479ec7d</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/17643" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/17643</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: make libcfs_ioctl.h uapi compliant<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 57a6fd6b13a9cdbfe4ca728e0bc5b99662a9f750</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/17643/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/17643/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: make libcfs_ioctl.h and lnetctl.h uapi compliant<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: cf7df5976769bbaa8cab1fc7b8fb80296bfef833</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/16914/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/16914/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove userland headers from libcfs.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 56782fa6e0cdd208d07480ad4b284ada1f09c60c</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/16879/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/16879/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove types.h from userland code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 4a0ec1d7621cc42e3139d067969e37b8285f7c06</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/17202/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/17202/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove typedefs in libcfs source code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: d40b3f9663c7627f0f08c4e3b5efa6cd206ca0f0</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/18964" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/18964</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: fix racey module handling<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: a148dad7c59842b72d1fec210b891d1fbdf325f3</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/17376/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/17376/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove cfs_fs_time handling<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: c3ddfae1b795385aa14db61df1a416ab57bfc066</p> <p>Oleg Drokin (oleg.drokin@intel.com) uploaded a new patch: <a href="http://review.whamcloud.com/19094" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/19094</a><br/> Subject: Revert "<a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove cfs_fs_time handling"<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 77f3810c080cdea09e64108d9aa812d53a29eaf9</p> <p>Patch 17376 was done before RHEL7 picked up the time64_t changes in later kernels. That API is now in the latest RHEL7 kernels which caused the breakage we are seeing.</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/19094/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/19094/</a><br/> Subject: Revert "<a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove cfs_fs_time handling"<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: c8e9c0cdd3f82385f5f088b61f677e62f063517f</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/19113" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/19113</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove libcfs userland time abstraction<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: e5bdc857dff4b158def5c9b51a167115d7dbbf94</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/18964/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/18964/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: fix racey module handling<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 42cb962d3f15f827be739ac5d02692d2643c3e07</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/19383" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/19383</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: Remove kp30.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 739742b9c9c294493f29b217c25156f7ca2738d6</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/19113/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/19113/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove libcfs userland time abstraction<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 8caff5d019674aea7d98e225d0136bf4694d5157</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/19459" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/19459</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove sysctl module handling<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: b5bd09da04e5b110c6444560a51422e211fa113d</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/19460" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/19460</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: Remove kp30.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: bf755ecc66c58d0c6691caabf6030579465ea1a1</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/19383/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/19383/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: replace CFS_MODULE_PARAM with linux kernel module api<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: d96a9248708d4da02728c9976a9a90ba29bd2bc0</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/19459/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/19459/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove sysctl module handling<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 9766f8e746731c74096c86fcc9514c3e3219e674</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/19460/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/19460/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: Remove kp30.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: b8bddf048483d3ed95201de06f2da0925c82cd54</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/20204" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/20204</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove types abstraction from kernel code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 006f47fbd02ede01d43f5c7b4ab55a3b78260e6d</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/20590" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/20590</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> client: remove types abstraction from client code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: be56a6f5d75d84b173c0cf8e8ac2297bb0ea2816</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/20204/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/20204/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove types abstraction from libcfs/LNet code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 2b8c4566a6a63cdf09e555a744ce6a1453651b98</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/21187" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/21187</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> server: remove types abstraction from server code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 93cddebe2466c6dc2a9595feb1816ab005ef847c</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/20590/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/20590/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> client: remove types abstraction from client code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 500f334631c6ebec72f5791472f21603da3e0ef9</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/22138" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/22138</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: use libcfs_private.h only for kernel space<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: b0a2501d6bcb98eab17ade7751bfbced500e7d5a</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/22139" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/22139</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: move uid handling to linux directory<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 8929f66a11e8decec27dcd3ed1e45a493058ec2f</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/15200/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/15200/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: cleanup list handling<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 91ef75669c7e41091378a8401ca0c093b7f17174</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/22139/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/22139/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: move uid handling to linux directory<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 74cecb5aad2243d1173405dc0063f4b46396f92f</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/22370" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/22370</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> server: remove types abstraction from lfsck and OSS code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 0f1ca20ddba2efde7c97a81160bdcb77a69b17e8</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/22371" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/22371</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> server: remove types abstraction from MDS/MGS code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 6df9e57abf89df139b04e4eed7efb7f2d1145e07</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/22370/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/22370/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> server: remove types abstraction from lfsck and OSS code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 520631764c0e9d1bf5c70f8d060b81a2f322d5dc</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/22371/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/22371/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> server: remove types abstraction from MDS/MGS code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 86e61f80cf6921b537a465b2f6b28914eaac8168</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/16916/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/16916/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove byteorder.h<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: e8965be4e135b55f0a900446e4a1c74905eaccd1</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/21187/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/21187/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> server: remove types abstraction from quota/target/nodemap code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 676c2fe47c8de0825db3001f1e964347be3dc479</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/22732" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/22732</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove last of types abstractions<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: f24f93e86f5c20d5475d8fb23f8040bc329c055e</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/22857" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/22857</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove cfs_fs_time handling<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: c79aec76c42cb1da4fdcd576e039fbd21f2d9435</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/22866" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/22866</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: add ktime_get_real_seconds support<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 61400a14e7c2032a805efdfd8e8623f8eed10d76</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/22732/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/22732/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove last of types abstractions<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 4f4622c417429053bad7943da02b1965ae178a37</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="http://review.whamcloud.com/22857/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/22857/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove cfs_fs_time handling<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: c6ed413f09efa7776ea78c7010cb2e71f8c966b3</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="http://review.whamcloud.com/23733" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/23733</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove wrappers for timer functions<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: fb75d58e5ed00d25e63ed1de7a37e85d2b7e87d2</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="https://review.whamcloud.com/22866/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/22866/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: add ktime_get_real_seconds support<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 07ff616a18cc602460724e6693e142782cad0fcc</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="https://review.whamcloud.com/24575" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/24575</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: replace IS_PO2 with is_power_of_2 in server code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: 2751e6b8954dc2e0150c9927d4f497dd9d0dcee5</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="https://review.whamcloud.com/23733/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/23733/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: remove wrappers for timer functions<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: 5801dad47e4727a44b6343a3f7f875d7992f29a3</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="https://review.whamcloud.com/22138/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/22138/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: use libcfs_private.h only for kernel space<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: f431777cf5cbb0f1493e91a35bbc2a26e1d14db9</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="https://review.whamcloud.com/24575/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/24575/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> libcfs: replace IS_PO2 with is_power_of_2 in server code<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: fc117cc92d224466c1ec45de422f9f6624bbb1da</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="https://review.whamcloud.com/28089" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/28089</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> uapi: move libcfs/lnet UAPI header into own uapi directory<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: b5d9cfa6508d6f65897690c8409155c827285155</p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="https://review.whamcloud.com/28089/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/28089/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> uapi: move libcfs/lnet UAPI header into own uapi directory<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: d92bd22523aa9bb1cdb91ded3d46fb180f68a93c</p> <p>I have just one patch left that I will submit shortly. Then this work will finish!!! <img class="emoticon" src="https://jira.whamcloud.com/images/icons/emoticons/biggrin.png" height="16" width="16" align="absmiddle" alt="" border="0"/></p> <p>The only work left is the test application it_test.c pulling in interval_tree.c from the lustre kernel source. This shouldn't be happening so I was looking at implementing the functions from interval_tree.c that is needed for it_test. Now when I looked to see how to test these changes I noticed it_test isn' even used anywere!!! Is it<br/> safe to delete it if this is the case?</p> <p>James Simmons (uja.ornl@yahoo.com) uploaded a new patch: <a href="https://review.whamcloud.com/28406" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/28406</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> tests: remove it_test<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: 1<br/> Commit: d7622179968c6fedb9019c2d55a5bea18533b95f</p> <p>Last patch for this ticket push!!!! <img class="emoticon" src="https://jira.whamcloud.com/images/icons/emoticons/tongue.png" height="16" width="16" align="absmiddle" alt="" border="0"/></p> <p>Oleg Drokin (oleg.drokin@intel.com) merged in patch <a href="https://review.whamcloud.com/28406/" class="external-link" target="_blank" rel="nofollow noopener">https://review.whamcloud.com/28406/</a><br/> Subject: <a href="https://jira.whamcloud.com/browse/LU-6245" title="Untangle userland and kernel space support for libcfs" class="issue-link" data-issue-key="LU-6245"><del>LU-6245</del></a> tests: remove it_test<br/> Project: fs/lustre-release<br/> Branch: master<br/> Current Patch Set: <br/> Commit: ebd0885c544dfec44c303361222decb9fe3d82d2</p> <p>After 5 years and 10K+ lines of code changes this project is over.</p> Blocker LU-6401 Related LU-5000 LU-6501 LU-5011 LU-9679 LU-10707 LU-8904 Development End date Thu, 2 Jun 2016 18:50:16 +0000 Rank 1|hzx6c7: Rank (Obsolete) 17491 Start date Fri, 13 Feb 2015 18:50:16 +0000