Rework controller reconfiguration #1060
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Separate out __nvme_deconfigure_ctrl() to make clear which attributes are to be saved during nvme_reconfigure_ctrl(). Signed-off-by: Hannes Reinecke <[email protected]>
The 'address' attribute is required in nvme_reconfigure_ctrl(), so we need to update it before calling the function and must not modify it in nvme_reconfigure_ctrl(). Signed-off-by: Hannes Reinecke <[email protected]>
The controller attributes have a different scope across reconfiguration; some are immutable, some persist across reconfiguration, and some will be reset when calling nvme_reconfigure_ctrl(). Signed-off-by: Hannes Reinecke <[email protected]>
…lers When a controller is disconnected it will just clear the volatile attributes, but remain part of the tree structure. So when traversing the tree one could access a disconnected controller, but then c->name is NULL. So fixup all places calling nvme_ctrl_get_name() to ensure that they don't crash on an empty name. Signed-off-by: Hannes Reinecke <[email protected]>
hreinecke
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Calling nvme_reconfigure_ctrl() requires some attributes to be cleared and other to remain constant. This patchset documents the scope of the controller attributes, and also ensures that the code does not trip over an empty 'name' attribute (which is volatile and only set after 'connect' succeeded).
Fixes: #951