[LU-5061] add lnb_ prefix to members of struct niobuf_local Created: 14/May/14 Updated: 07/Mar/17 Resolved: 07/Mar/17 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.6.0 |
| Fix Version/s: | Lustre 2.7.0 |
| Type: | Improvement | Priority: | Minor |
| Reporter: | John Hammond | Assignee: | John Hammond |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Rank (Obsolete): | 13979 |
| Description |
|
From http://review.whamcloud.com/#/c/9784/3/lustre/osd-ldiskfs/osd_io.c: Andreas Dilger Apr 24 11:59 PM (improvement) It would be nice to get a separate patch that changed the niobuf_local "page", "dentry", "len", and "flags" to "lnb_page", "lnb_dentry", etc. so that these things can be found in a reasonable manner. Same for "niobuf_remote" field names. Seems there is a bit of a start on this in ofd_internal.h to define compat macros, but a single cleanup patch would be better. At the same time, some consistency in naming the variables used for "niobuf_local" would be good. Sometimes it is "nl", sometimes "lnb", sometimes "local", others "nb", or "res", etc. Probably "lnb" and "rnb" are the most commonly used names, though they are not totally consistent with the struct name (which would imply "nbl" and "nbr"). |
| Comments |
| Comment by John Hammond [ 27/May/14 ] |
|
Please see http://review.whamcloud.com/10451. |
| Comment by John Hammond [ 27/May/14 ] |
| Comment by John Hammond [ 25/Jul/14 ] |
|
Both patches landed to master. |