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

Introduce Stack Reset #455

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ntnx-alay
Copy link

Stack reset helps reset situation where user may press Ctrl-C in the middle of command and current established session does not close properly. When operating in single session mode, drive shall not let establish another session until sessionTimeout is reached or drive is power cycled (whichever occurs first).

Stack Reset helps reset any on going session and able to re-establish session without reaching timeout or physically removing drive (or power cycling the drive).

Use of timeout was interesting idea - but a few drives that tested showed min and max SessionTimeout, when configured value in between, session could not be established due to invalid parameters.

SessionTimeout is optional implementation, while Stack Reset is required as per spec.

Stack reset helps reset situation where user may press Ctrl-C in the
middle of command and current established session does not close
properly. When operating in single session mode, drive shall not let
establish another session until sessionTimeout is reached or drive
is power cycled (whichever occurs first).

Stack Reset helps reset any on going session and able to re-establish
session without reaching timeout or physically removing drive (or
power cycling the drive).

Use of timeout was interesting idea - but a few drives that tested
showed min and max SessionTimeout, when configured value in between,
session could not be established due to invalid parameters.

SessionTimeout is optional implementation, while Stack Reset is required
as per spec.
Note: No functionality change expected.
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.

1 participant