[LU-3159] proc filenames are broken in pre-2.4 Created: 11/Apr/13 Updated: 16/Apr/13 Resolved: 16/Apr/13 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.4.0 |
| Fix Version/s: | Lustre 2.4.0 |
| Type: | Bug | Priority: | Blocker |
| Reporter: | Christopher Morrone | Assignee: | Jian Yu |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | LB | ||
| Severity: | 3 |
| Rank (Obsolete): | 7666 |
| Description |
|
The proc directory naming of the filter-* namespaces is pretty broken on master. In 2.1, it would look like this: /proc/fs/lustre/ldlm/namespaces/filter-lsa-OST0000_UUID but as of 2.3.63 it not looks like this: /proc/fs/lustre/ldlm/namespaces/filter-ffff88081233c000 The previous form was human readable, and tools could relatively easily match those stats up with the correct OST, because the OST name is right in the directory name. The new form is not human readable, and I can't find any mention of that pointer value anywhere else in /proc, so it is pretty useless. Not to mention is is very likely a bad idea security wise to be exposing internal kernel pointer values to user-space like this. Since LMT is currently the only way that our admins have the slightest idea what is going on with a Lustre filesystem and this significantly degrades LMT's functionality, I would argue that this needs to be fixed before 2.4.0. |
| Comments |
| Comment by Andreas Dilger [ 12/Apr/13 ] |
|
The LDLM namespace naming was fixed (along with some other things) in my patch at http://review.whamcloud.com/4618. I'd be happy if someone extracted those changes out of my patch and landed them for 2.4.0. |
| Comment by Peter Jones [ 12/Apr/13 ] |
|
Yu, Jian Could you please do as Andreas suggests to address this one? Thanks Peter |
| Comment by Jian Yu [ 12/Apr/13 ] |
|
Patch for Lustre master branch is in http://review.whamcloud.com/6039. |
| Comment by Peter Jones [ 16/Apr/13 ] |
|
Landed for 2.4 |