[LU-8054] Split LFSCK local notification from lfsck_in_notify() API Created: 22/Apr/16 Updated: 17/May/17 Resolved: 17/May/17 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.10.0 |
| Type: | Bug | Priority: | Minor |
| Reporter: | nasf (Inactive) | Assignee: | nasf (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Severity: | 3 |
| Rank (Obsolete): | 9223372036854775807 |
| Description |
|
Currently, the lfsck_in_notify() contains too much functionalities, including both the remote LFSCK control and local LFSCK notification. These different LFSCK requirements share the same structure "lfsck_request" to parse kinds of data. It is difficult to enhance the "lfsck_request" with more local LFSCK requirements, because enlarging or adjusting the lfsck_request fields will affect the LFSCK RPC on-wire protocol and caused interoperability issues. We will split the local LFSCK notifications from the lfsck_in_notify() with new LFSCK API, then even if we have more local LFSCK requirements in future, we can enhance such API freely without affecting LFSCK RPC protocol. |
| Comments |
| Comment by Gerrit Updater [ 26/Apr/16 ] |
|
Fan Yong (fan.yong@intel.com) uploaded a new patch: http://review.whamcloud.com/19785 |
| Comment by Gerrit Updater [ 13/Jan/17 ] |
|
Fan Yong (fan.yong@intel.com) uploaded a new patch: https://review.whamcloud.com/24881 |
| Comment by Gerrit Updater [ 03/Feb/17 ] |
|
Fan Yong (fan.yong@intel.com) uploaded a new patch: https://review.whamcloud.com/25236 |
| Comment by Gerrit Updater [ 21/Mar/17 ] |
|
Andreas Dilger (andreas.dilger@intel.com) merged in patch https://review.whamcloud.com/24881/ |
| Comment by nasf (Inactive) [ 17/May/17 ] |
|
The patch has been landed to master. |