Skip to content

Added vendor specific subsystem pause/resume#74

Closed
susobhandey wants to merge 3 commits into
openebs:v24.01.x-mayastorfrom
susobhandey:subsys_reset_pause
Closed

Added vendor specific subsystem pause/resume#74
susobhandey wants to merge 3 commits into
openebs:v24.01.x-mayastorfrom
susobhandey:subsys_reset_pause

Conversation

@susobhandey

Copy link
Copy Markdown

if vendor sets flag during subsystem pause
we keep allowing keep alive for 120 sec even though subsystem is not active There is no change in fabric command. Only we allow admin cmds for 120 sec

if vendor sets flag during subsystem pause
we keep allowing keep alive for 120 sec even though subsystem is not active
There is no change in fabric command. Only we allow admin cmds for 120 sec

Signed-off-by: Susobhan Dey <susobhan.dey@datacore.com>
Comment thread lib/nvmf/ctrlr.c Outdated
Comment thread lib/nvmf/subsystem.c Outdated
Susobhan Dey added 2 commits May 7, 2026 20:07
scripts/rpc.py nvmf_subsystem_set_pause_timeout <nqn> <time in sec>

Signed-off-by: Susobhan Dey <susobhan.dey@datacore.com>
Signed-off-by: Susobhan Dey <susobhan.dey@datacore.com>
@tiagolobocastro

Copy link
Copy Markdown
Member

This is against the wrong base branch @susobhandey, should be v25.05.x-mayastor

@tiagolobocastro tiagolobocastro left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change the base branch of the commit

Comment thread lib/nvmf/ctrlr.c
struct spdk_nvmf_subsystem_poll_group *sgroup = NULL;
struct spdk_nvmf_subsystem_pg_ns_info *ns_info;
uint32_t nsid;
struct spdk_nvmf_subsystem *subsystem = NULL;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can leave it unset as well, since it's init before being used (also seems to be the practice in spdk AFAIK)

@susobhandey

Copy link
Copy Markdown
Author

Closed as we have another one

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.

2 participants