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

Check MBR Shadow during TakeOwnership #456

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

Conversation

ntnx-alay
Copy link

At this point in time, utility is taking ownership with provided password successfully with two classes of drives.

  1. That supports MBRShadowing
  2. That does not support MBRShadowing

However, for those drives that does not support MBRSHadowing, it reports error and tool's return code is non zero despite taking ownership.

This patch is to improve experience by checking MBRShadow capability (decoded with discovery0) prior to enable MBR.

At this point in time, utility is taking ownership with provided password successfully
with two classes of drives.
  1.  That supports MBRShadowing
  2.  That does not support MBRShadowing

However, for those drives that does not support MBRSHadowing, it reports error and
tool's return code is non zero despite taking ownership.

This patch is to improve experience by checking MBRShadow capability (decoded with
discovery0) prior to enable MBR.
Note: No functionality change expected with this commit.
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