[LU-8066] Move lustre procfs handling to sysfs and debugfs. Created: 25/Apr/16 Updated: 08/Jan/24 |
|
| Status: | Open |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Upstream |
| Type: | Improvement | Priority: | Minor |
| Reporter: | James A Simmons | Assignee: | James A Simmons |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | patch, upstream | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Description |
|
For lustre 2.8 the tools have been updated to work with the upstream client which has replaced procfs with sysfs and debugfs. Now that we fully support this setup we can safely backport all the changes done upstream to support sysfs and debugfs. |
| Comments |
| Comment by Oleg Drokin [ 26/Apr/16 ] |
|
This is a good thing to do, but there's also server side code that needs the conversion and with 2.9 feature freeze we are probably best aiming this to a release next after 2.9 and we want all of this going to a single release. The other thing we need to do is an email to lustre-devel and lustre-discuss to warn people this is coming and they need to adjust their scripts/monitoring tools/whatever to check both locations so that once hte change comes - they are ready. |
| Comment by James A Simmons [ 26/Apr/16 ] |
|
Can't change it to Lustre 2.10 since it doesn't exist as an option. I understand the concern since this is a visible change to the users. Perhaps this might be worthy of a branch. I would want to wait until feature freeze so once 2.10 opens up we can land it right away. |
| Comment by Gerrit Updater [ 29/Aug/16 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: http://review.whamcloud.com/22209 |
| Comment by Gerrit Updater [ 27/Oct/16 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: http://review.whamcloud.com/23425 |
| Comment by Gerrit Updater [ 27/Oct/16 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: http://review.whamcloud.com/23427 |
| Comment by Gerrit Updater [ 27/Oct/16 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: http://review.whamcloud.com/23428 |
| Comment by James A Simmons [ 30/Nov/16 ] |
|
While porting this work I noticed newer kernels have a function string_get_size() which pretty much does the same thing as lprocfs_[seq_]read_frac_helper(). The difference is that string_get_size() appends the unit at the end. So using this we would [root@ninja06 lustre-ffff88105afb9800]# lctl get_param llite.lustre-*/max_read_ahead_mb Should we chop off this last bit or instead allow it to be printed out. Will this break existing scripts out there? |
| Comment by Gerrit Updater [ 30/Nov/16 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: http://review.whamcloud.com/24031 |
| Comment by Gerrit Updater [ 30/Nov/16 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: http://review.whamcloud.com/24033 |
| Comment by Gerrit Updater [ 30/Nov/16 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: http://review.whamcloud.com/24034 |
| Comment by Gerrit Updater [ 01/Jan/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/22209/ |
| Comment by Gerrit Updater [ 01/Jan/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/23425/ |
| Comment by Gerrit Updater [ 03/Jan/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/24688 |
| Comment by Gerrit Updater [ 18/Jan/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/23427/ |
| Comment by Gerrit Updater [ 24/Jan/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/23428/ |
| Comment by Gerrit Updater [ 24/Jan/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/25049 |
| Comment by Gerrit Updater [ 24/Jan/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/25073 |
| Comment by Gerrit Updater [ 30/Jan/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/25160 |
| Comment by Gerrit Updater [ 07/Feb/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/24033/ |
| Comment by Gerrit Updater [ 07/Feb/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/25049/ |
| Comment by Gerrit Updater [ 07/Feb/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/24034/ |
| Comment by Gerrit Updater [ 08/Feb/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/25316 |
| Comment by Gerrit Updater [ 18/Feb/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/25160/ |
| Comment by Gerrit Updater [ 25/Feb/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/25631 |
| Comment by Gerrit Updater [ 25/Feb/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/25632 |
| Comment by Gerrit Updater [ 16/Mar/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/26020 |
| Comment by Gerrit Updater [ 06/Apr/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/25632/ |
| Comment by Gerrit Updater [ 06/Apr/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/26430 |
| Comment by Gerrit Updater [ 16/Apr/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/26650 |
| Comment by Gerrit Updater [ 16/Apr/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/26651 |
| Comment by Gerrit Updater [ 19/Jul/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/26020/ |
| Comment by Gerrit Updater [ 19/Jul/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/25073/ |
| Comment by Gerrit Updater [ 19/Jul/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/28108 |
| Comment by Gerrit Updater [ 19/Jul/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/28109 |
| Comment by Gerrit Updater [ 04/Aug/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/28366 |
| Comment by Gerrit Updater [ 05/Aug/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/24688/ |
| Comment by Gerrit Updater [ 10/Aug/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/28460 |
| Comment by Bob Glossman (Inactive) [ 11/Aug/17 ] |
|
I think the very recently landed is having very negative consequences on sles12sp2. Hung commands are unkillable. If I back out https://review.whamcloud.com/24688 everything works fine again. |
| Comment by James A Simmons [ 11/Aug/17 ] |
|
Only on SLES12sp2 ?? That is strange. Let me take a look |
| Comment by Bob Glossman (Inactive) [ 11/Aug/17 ] |
|
I think it may have something to do with the particular set of autoconf results seen in sles12sp2. I see the following in lustre's config.h: #define HAVE_KERNEL_PARAM_LOCK 1 |
| Comment by James A Simmons [ 11/Aug/17 ] |
|
Thanks for the info. Now I see what is going on. For RHEL6 its so old it doesn't do any module param locking. So I provide our own which is determined by HAVE_MODULE_PARAM_LOCKING. This test if _kernel_param[un]lock is available. That is what was introduced at first. Then in later kernels it was made internal and kernel_param_[un]lock was introduced. So on SLES12SP2 you have kernel_param_lock visible and __kernel_param_lock invisible. I think I know how to fix that. I have a Ubuntu 17 system so let me try it there. |
| Comment by Gerrit Updater [ 11/Aug/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/28498 |
| Comment by Bob Glossman (Inactive) [ 11/Aug/17 ] |
|
James, |
| Comment by Bob Glossman (Inactive) [ 11/Aug/17 ] |
|
Direct test convinces me https://review.whamcloud.com/28498 fixes the problem. Can't say for sure that correct and appropriate locking is still being done. Can say that simple commands that were hanging don't do so anymore with the patch. |
| Comment by Gerrit Updater [ 13/Aug/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/26650/ |
| Comment by Gerrit Updater [ 19/Aug/17 ] |
|
Hongchao Zhang (hongchao.zhang@intel.com) uploaded a new patch: https://review.whamcloud.com/28612 |
| Comment by Hongchao Zhang [ 19/Aug/17 ] |
|
Encounter the above problem when testing the master on 2.6.32-696.6.3.el6.x86_64. |
| Comment by Bob Glossman (Inactive) [ 22/Aug/17 ] |
|
just tried https://review.whamcloud.com/28498 on el6.9. it doesn't fix the problem there. may fix it on sles12sp2, but still broken on el6. I see command hangs just like on sles12. The only remedy appears to be to back out the change: https://review.whamcloud.com/24688/ autoconf results seen on el6: /* #undef HAVE_KERNEL_PARAM_LOCK */ |
| Comment by James A Simmons [ 22/Aug/17 ] |
|
We also need patch https://review.whamcloud.com/#/c/28612 for el6 to work properly. I will rebase the patch on top of 28498 patch. |
| Comment by Bob Glossman (Inactive) [ 22/Aug/17 ] |
|
https://review.whamcloud.com/#/c/28612 is broken due to incorrect #ifdef. can't be used as is in general. maybe ok for el6, I will try it. |
| Comment by James A Simmons [ 22/Aug/17 ] |
|
I just updated it. Give it a try. |
| Comment by Bob Glossman (Inactive) [ 22/Aug/17 ] |
|
maybe https://review.whamcloud.com/28498 and https://review.whamcloud.com/#/c/28612 ought to be merged. They seem very closely related and tightly coupled. Easier to track one mod than two. |
| Comment by James A Simmons [ 22/Aug/17 ] |
|
True but lets see if 28498 is going to landed in the next few hours. If not I can merge both patches. |
| Comment by Bob Glossman (Inactive) [ 22/Aug/17 ] |
|
the 2 mods together do work on el6 |
| Comment by Gerrit Updater [ 28/Aug/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/28498/ |
| Comment by Gerrit Updater [ 31/Aug/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/28612/ |
| Comment by Gerrit Updater [ 31/Aug/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/28818 |
| Comment by Gerrit Updater [ 12/Sep/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/28948 |
| Comment by Gerrit Updater [ 21/Sep/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/28818/ |
| Comment by Gerrit Updater [ 28/Sep/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/29250 |
| Comment by Gerrit Updater [ 29/Sep/17 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/29255 |
| Comment by Gerrit Updater [ 02/Oct/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/26430/ |
| Comment by Gerrit Updater [ 24/Oct/17 ] |
|
Mike Pershin (mike.pershin@intel.com) uploaded a new patch: https://review.whamcloud.com/29723 |
| Comment by Gerrit Updater [ 01/Nov/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/28948/ |
| Comment by Gerrit Updater [ 21/Jan/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/30960 |
| Comment by Gerrit Updater [ 21/Jan/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/30961 |
| Comment by Gerrit Updater [ 21/Jan/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/30962 |
| Comment by Gerrit Updater [ 21/Jan/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/30963 |
| Comment by Gerrit Updater [ 30/Jan/18 ] |
|
Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/31091 |
| Comment by Gerrit Updater [ 30/Jan/18 ] |
|
Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/31092 |
| Comment by Gerrit Updater [ 31/Jan/18 ] |
|
Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/31107 |
| Comment by Gerrit Updater [ 06/Mar/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/28108/ |
| Comment by Gerrit Updater [ 06/Mar/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/28366/ |
| Comment by Gerrit Updater [ 09/Mar/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/31618 |
| Comment by Gerrit Updater [ 14/Mar/18 ] |
|
James Nunez (james.a.nunez@intel.com) uploaded a new patch: https://review.whamcloud.com/31652 |
| Comment by Gerrit Updater [ 15/Mar/18 ] |
|
James Nunez (james.a.nunez@intel.com) uploaded a new patch: https://review.whamcloud.com/31658 |
| Comment by Gerrit Updater [ 29/Apr/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32198 |
| Comment by Gerrit Updater [ 02/May/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/29250/ |
| Comment by Gerrit Updater [ 02/May/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/29255/ |
| Comment by Gerrit Updater [ 02/May/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/30962/ |
| Comment by Gerrit Updater [ 06/May/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/28460/ |
| Comment by Gerrit Updater [ 06/May/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/30960/ |
| Comment by Gerrit Updater [ 12/May/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/26651/ |
| Comment by Gerrit Updater [ 12/May/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/30961/ |
| Comment by Gerrit Updater [ 13/May/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32377 |
| Comment by Gerrit Updater [ 22/May/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32495 |
| Comment by Gerrit Updater [ 22/May/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32496 |
| Comment by Gerrit Updater [ 22/May/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32497 |
| Comment by Gerrit Updater [ 22/May/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32498 |
| Comment by Gerrit Updater [ 22/May/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32499 |
| Comment by Gerrit Updater [ 22/May/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32500 |
| Comment by Gerrit Updater [ 22/May/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32501 |
| Comment by Gerrit Updater [ 22/May/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32502 |
| Comment by Gerrit Updater [ 22/May/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32503 |
| Comment by Gerrit Updater [ 22/May/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32504 to sysfs |
| Comment by Gerrit Updater [ 22/May/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32505 |
| Comment by Gerrit Updater [ 22/May/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32506 |
| Comment by Gerrit Updater [ 22/May/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32507 |
| Comment by Gerrit Updater [ 23/May/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32515 |
| Comment by Gerrit Updater [ 23/May/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32516 |
| Comment by Gerrit Updater [ 23/May/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32517 |
| Comment by Gerrit Updater [ 07/Jun/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/24031/ |
| Comment by Gerrit Updater [ 14/Jun/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32719 |
| Comment by James A Simmons [ 14/Jun/18 ] |
|
The latest patch also handles the issue of people still shoving things into proc. Ugh!!! |
| Comment by Gerrit Updater [ 14/Jun/18 ] |
|
John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/32495/ |
| Comment by Gerrit Updater [ 14/Jun/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) uploaded a new patch: https://review.whamcloud.com/32721 |
| Comment by Gerrit Updater [ 14/Jun/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/32721/ |
| Comment by Gerrit Updater [ 14/Jun/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32722 |
| Comment by Gerrit Updater [ 12/Jul/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32810 |
| Comment by Gerrit Updater [ 18/Jul/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32719/ |
| Comment by Gerrit Updater [ 18/Jul/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32722/ |
| Comment by Gerrit Updater [ 19/Jul/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32837 |
| Comment by Gerrit Updater [ 24/Jul/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32377/ |
| Comment by Gerrit Updater [ 29/Jul/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/32896 |
| Comment by Gerrit Updater [ 06/Aug/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32496/ |
| Comment by Gerrit Updater [ 06/Aug/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32497/ |
| Comment by Gerrit Updater [ 06/Aug/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32498/ |
| Comment by Gerrit Updater [ 06/Aug/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32499/ |
| Comment by Gerrit Updater [ 06/Aug/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32896/ |
| Comment by Gerrit Updater [ 15/Aug/18 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/33005 |
| Comment by Gerrit Updater [ 15/Aug/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/33009 |
| Comment by Gerrit Updater [ 15/Aug/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/33010 |
| Comment by Gerrit Updater [ 15/Aug/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/33011 |
| Comment by Gerrit Updater [ 17/Aug/18 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/33019 |
| Comment by Gerrit Updater [ 18/Aug/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32198/ |
| Comment by Gerrit Updater [ 23/Aug/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32501/ |
| Comment by Gerrit Updater [ 23/Aug/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33019/ |
| Comment by Gerrit Updater [ 23/Aug/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/33059 |
| Comment by Gerrit Updater [ 28/Aug/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33005/ |
| Comment by Gerrit Updater [ 10/Sep/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32502/ |
| Comment by Gerrit Updater [ 17/Sep/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/33180 |
| Comment by Gerrit Updater [ 21/Sep/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/33213 |
| Comment by Gerrit Updater [ 21/Sep/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32503/ |
| Comment by Gerrit Updater [ 21/Sep/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32504/ to sysfs |
| Comment by Gerrit Updater [ 21/Sep/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33009/ |
| Comment by Gerrit Updater [ 01/Oct/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33059/ |
| Comment by Gerrit Updater [ 01/Oct/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33010/ |
| Comment by Gerrit Updater [ 05/Oct/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32505/ |
| Comment by Gerrit Updater [ 05/Oct/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32506/ |
| Comment by Gerrit Updater [ 05/Oct/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32507/ |
| Comment by Gerrit Updater [ 07/Oct/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33011/ |
| Comment by Gerrit Updater [ 10/Oct/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32515/ |
| Comment by Gerrit Updater [ 12/Oct/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32516/ |
| Comment by Gerrit Updater [ 29/Oct/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33213/ |
| Comment by Gerrit Updater [ 06/Nov/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32517/ |
| Comment by Gerrit Updater [ 09/Nov/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/33632 |
| Comment by Gerrit Updater [ 17/Nov/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33180/ |
| Comment by Gerrit Updater [ 20/Nov/18 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/33686 |
| Comment by James A Simmons [ 27/Nov/18 ] |
|
Patch https://review.whamcloud.com/#/c/31618 restores lctl dl for non root users so its ready if you want to land it to the LTS release. Since not every subsystem is ported to sysfs everything will not show up with lctl dl but its better than it not working at all. |
| Comment by James A Simmons [ 29/Nov/18 ] |
|
Li from the previous discussion you stated in The problem is why the -ENOSYS failure of class_process_proc_param(PARAM_OST) is not ignored in osd_process_config()? I am not sure about the llog processing codes, but I guess the process is: class_process_proc_param(PARAM_MDT) -> class_process_proc_param(PARAM_HSM) -> class_process_proc_param(PARAM_HSM) -> class_process_proc_param(PARAM_OSD) -> class_process_proc_param(PARAM_OST). So if osd_process_config() doesn't understand the llog record, it should ignore the -ENOSYS. Right? I don't think this is best approach. The source of the problem is that lustre_process_log() and lustre_end_log() are not tolerant of any errors. The question is what errors should we consider as critical and what can we just report to the user but not fail the system? |
| Comment by Gerrit Updater [ 04/Dec/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/33774 |
| Comment by Gerrit Updater [ 04/Dec/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/33776 |
| Comment by Gerrit Updater [ 06/Dec/18 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/33799 |
| Comment by Gerrit Updater [ 06/Dec/18 ] |
|
Joseph Gmitter (jgmitter@whamcloud.com) merged in patch https://review.whamcloud.com/33686/ |
| Comment by Gerrit Updater [ 14/Dec/18 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/33874 |
| Comment by James A Simmons [ 18/Dec/18 ] |
|
We final have run into the sysfs uuid name issue with newer kernel. Currently we are using obd_name_"address_pointer" but newer kernels filter out the address pointer being exposed to user land. BTW this will also impact procfs files as well with newer kernels. |
| Comment by Gerrit Updater [ 27/Dec/18 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/33920 |
| Comment by Gerrit Updater [ 04/Jan/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33774/ |
| Comment by Gerrit Updater [ 04/Jan/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33632/ |
| Comment by Gerrit Updater [ 04/Jan/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33776/ |
| Comment by Gerrit Updater [ 04/Jan/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33874/ |
| Comment by Gerrit Updater [ 30/Jan/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33920/ |
| Comment by Gerrit Updater [ 24/Feb/19 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/34292 |
| Comment by Gerrit Updater [ 27/Feb/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32810/ |
| Comment by Gerrit Updater [ 03/Mar/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/33799/ |
| Comment by Gerrit Updater [ 04/Mar/19 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/34372 |
| Comment by Gerrit Updater [ 15/Mar/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/34372/ |
| Comment by Gerrit Updater [ 01/Apr/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/25631/ |
| Comment by Gerrit Updater [ 01/Apr/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/34292/ |
| Comment by Gerrit Updater [ 01/Apr/19 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/34557 |
| Comment by Gerrit Updater [ 08/Apr/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/34557/ |
| Comment by Gerrit Updater [ 08/Apr/19 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/34612 |
| Comment by Gerrit Updater [ 09/Apr/19 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/34624 |
| Comment by Gerrit Updater [ 19/Apr/19 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/34715 |
| Comment by Gerrit Updater [ 19/Apr/19 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/34716 |
| Comment by Gerrit Updater [ 19/Apr/19 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/34717 |
| Comment by Gerrit Updater [ 19/Apr/19 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/34718 |
| Comment by Gerrit Updater [ 30/Apr/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/34624/ |
| Comment by Gerrit Updater [ 30/Apr/19 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/34779 |
| Comment by Gerrit Updater [ 30/Apr/19 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/34781 |
| Comment by Gerrit Updater [ 30/Apr/19 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/34782 |
| Comment by Gerrit Updater [ 08/May/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/34612/ |
| Comment by Gerrit Updater [ 11/May/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/34782/ |
| Comment by Gerrit Updater [ 21/May/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/30963/ |
| Comment by Gerrit Updater [ 07/Jun/19 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/35096 |
| Comment by Gerrit Updater [ 11/Jun/19 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/35185 |
| Comment by Gerrit Updater [ 20/Jun/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/34715/ |
| Comment by Gerrit Updater [ 25/Jun/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35185/ |
| Comment by Gerrit Updater [ 25/Jun/19 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/35313 |
| Comment by Gerrit Updater [ 27/Jun/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/34716/ |
| Comment by Gerrit Updater [ 28/Jun/19 ] |
|
James Simmons (uja.ornl@yahoo.com) uploaded a new patch: https://review.whamcloud.com/35349 |
| Comment by Gerrit Updater [ 03/Jul/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/34781/ |
| Comment by Gerrit Updater [ 03/Jul/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35313/ |
| Comment by Gerrit Updater [ 12/Jul/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/34717/ |
| Comment by Gerrit Updater [ 17/Jul/19 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/35542 |
| Comment by Gerrit Updater [ 19/Jul/19 ] |
|
James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/35575 |
| Comment by Gerrit Updater [ 27/Jul/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35575/ |
| Comment by Gerrit Updater [ 30/Jul/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35542/ |
| Comment by Gerrit Updater [ 30/Jul/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/34779/ |
| Comment by Gerrit Updater [ 03/Aug/19 ] |
|
James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/35687 |
| Comment by Gerrit Updater [ 11/Aug/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35349/ |
| Comment by Gerrit Updater [ 03/Sep/19 ] |
|
James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/36043 |
| Comment by Gerrit Updater [ 04/Oct/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/34718/ |
| Comment by Gerrit Updater [ 08/Nov/19 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/36715 |
| Comment by Gerrit Updater [ 18/Nov/19 ] |
|
James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/36791 |
| Comment by Gerrit Updater [ 06/Dec/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35096/ |
| Comment by Gerrit Updater [ 13/Dec/19 ] |
|
Minh Diep (mdiep@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/37005 |
| Comment by Gerrit Updater [ 16/Dec/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/36682/ |
| Comment by Gerrit Updater [ 16/Dec/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/36791/ |
| Comment by Gerrit Updater [ 16/Dec/19 ] |
|
James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/37033 |
| Comment by Gerrit Updater [ 20/Dec/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35687/ |
| Comment by Gerrit Updater [ 03/Jan/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37005/ |
| Comment by Gerrit Updater [ 18/Jan/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/36715/ |
| Comment by Gerrit Updater [ 18/Jan/20 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/37275 |
| Comment by Gerrit Updater [ 03/Feb/20 ] |
|
James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/37407 |
| Comment by Gerrit Updater [ 06/Aug/20 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/39578 |
| Comment by Gerrit Updater [ 09/Aug/20 ] |
|
Andreas Dilger (adilger@whamcloud.com) merged in patch https://review.whamcloud.com/39578/ |
| Comment by Gerrit Updater [ 27/Aug/20 ] |
|
Neil Brown (neilb@suse.de) uploaded a new patch: https://review.whamcloud.com/39740 |
| Comment by Gerrit Updater [ 12/Sep/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/39740/ |
| Comment by Gerrit Updater [ 14/Dec/20 ] |
|
Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/40968 |
| Comment by Gerrit Updater [ 22/Dec/20 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/40968/ |
| Comment by Gerrit Updater [ 05/Apr/21 ] |
|
James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/43212 |
| Comment by Gerrit Updater [ 15/Apr/21 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/43212/ |
| Comment by Gerrit Updater [ 14/Jul/21 ] |
|
James Simmons (jsimmons@infradead.org) uploaded a new patch: https://review.whamcloud.com/44311 |
| Comment by Gerrit Updater [ 10/Aug/21 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/44311/ |
| Comment by Gerrit Updater [ 30/Sep/21 ] |
|
"Oleg Drokin <green@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/45096 |
| Comment by Gerrit Updater [ 10/Oct/21 ] |
|
"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/45096/ |