[LU-9523] unravel #DEFINEs in cl_page.c Created: 17/May/17 Updated: 07/Jun/17 Resolved: 07/Jun/17 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.10.0 |
| Type: | Improvement | Priority: | Minor |
| Reporter: | Ben Evans (Inactive) | Assignee: | Ben Evans (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Epic/Theme: | cleanup |
| Rank (Obsolete): | 9223372036854775807 |
| Description |
|
CL_PAGE_INVOKE, CL_PAGE_INVOID and CL_PAGE_INVOID_REVERSE all simply call list_for_each_entry (or list_for_each_entry_reverse) and call one of the functions from cl_page_operations. It's much simpler and easy to read to unroll these #defines, along with CL_PAGE_OP and use the correct code. |
| Comments |
| Comment by Gerrit Updater [ 17/May/17 ] |
|
Ben Evans (bevans@cray.com) uploaded a new patch: https://review.whamcloud.com/27168 |
| Comment by Gerrit Updater [ 07/Jun/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/27168/ |
| Comment by Peter Jones [ 07/Jun/17 ] |
|
Landed for 2.10 |