[LU-10855] llog cleanups Created: 27/Mar/18 Updated: 28/Aug/18 Resolved: 28/Aug/18 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.12.0 |
| Type: | Bug | Priority: | Minor |
| Reporter: | John Hammond | Assignee: | John Hammond |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||
| Severity: | 3 | ||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||
| Description |
|
There is no need to implement llog_cancel() as an llog operation. (We do not need to have a lop_cancel member in struct llog_operations. This method only used by changelogs now and the prototype is abused to make it work. With lop_cancel removed, all uses of hsm_actions_logops, changelog_orig_logops, and osp_mds_ost_orig_logops can be replaces with llog_common_cat_ops. LLOG_AGENT_ORIG_CTXT should be called LLOG_ACTIONS_ORIG_CTXT. LLOG_AGENT_ORIG_CTXT setup and cleanup should be moved from the MDD layer to the MDT layer. Then there will be no need for the MDT to clone this context. |
| Comments |
| Comment by Gerrit Updater [ 27/Mar/18 ] |
|
John L. Hammond (john.hammond@intel.com) uploaded a new patch: https://review.whamcloud.com/31795 |
| Comment by Gerrit Updater [ 28/Mar/18 ] |
|
John L. Hammond (john.hammond@intel.com) uploaded a new patch: https://review.whamcloud.com/31812 |
| Comment by Gerrit Updater [ 30/Apr/18 ] |
|
John L. Hammond (john.hammond@intel.com) uploaded a new patch: https://review.whamcloud.com/32202 |
| Comment by Gerrit Updater [ 17/May/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/31795/ |
| Comment by Gerrit Updater [ 29/May/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/32202/ |
| Comment by Gerrit Updater [ 06/Jun/18 ] |
|
Andreas Dilger (andreas.dilger@intel.com) uploaded a new patch: https://review.whamcloud.com/32651 |
| Comment by Gerrit Updater [ 06/Jun/18 ] |
|
Andreas Dilger (andreas.dilger@intel.com) uploaded a new patch: https://review.whamcloud.com/32653 |
| Comment by Gerrit Updater [ 06/Jun/18 ] |
|
Andreas Dilger (andreas.dilger@intel.com) uploaded a new patch: https://review.whamcloud.com/32654 |
| Comment by Gerrit Updater [ 18/Jul/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32651/ |
| Comment by Gerrit Updater [ 18/Jul/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32653/ |
| Comment by Gerrit Updater [ 18/Jul/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32654/ |
| Comment by Gerrit Updater [ 28/Aug/18 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/31812/ |
| Comment by Peter Jones [ 28/Aug/18 ] |
|
Landed for 2.12 |