Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(self-shutdown): abort frozen ios when unsharing shutdown nexus #1788

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

tiagolobocastro
Copy link
Contributor

These frozen IOs prevent the nexus from shutting down. We don't have any hooks today to do this whilsts the target is stopping so we add a simple loop which tries a number of times. Ideally we should get some sort of callback to trigger this.

These frozen IOs prevent the nexus from shutting down.
We don't have any hooks today to do this whilsts the target is stopping
so we add a simple loop which tries a number of times.
Ideally we should get some sort of callback to trigger this.

Signed-off-by: Tiago Castro <[email protected]>
@dsharma-dc
Copy link
Contributor

dsharma-dc commented Dec 19, 2024

The frozen IOs are held up in our NexusChannel and not yet (re)submitted into the child device io channel, because in self shutdown path we haven't brought back io mode to normal. So wondering why they hold up nexus nvmf subsystem unshare/stop? Perhaps since the IOs have entered the nexus subsystem and not ack'ed?

@tiagolobocastro
Copy link
Contributor Author

The frozen IOs are held up in our NexusChannel and not yet (re)submitted into the child device io channel, because in self shutdown path we haven't brought back io mode to normal. So wondering why they hold up nexus nvmf subsystem unshare/stop? Perhaps since the IOs have entered the nexus subsystem and not ack'ed?

Haven't looked into the subsystem for a while but IIRC it wont stop because there's outstanding IO. When the last IO completes the subsystem is stopped.

@tiagolobocastro
Copy link
Contributor Author

bors merge

@bors-openebs-mayastor
Copy link

Build succeeded:

@bors-openebs-mayastor bors-openebs-mayastor bot merged commit 02e79b6 into release/2.7 Dec 19, 2024
4 checks passed
@bors-openebs-mayastor bors-openebs-mayastor bot deleted the self-shutdown branch December 19, 2024 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants