Skip to content

Sndk plugin fixes#3119

Open
jeff-lien-sndk wants to merge 3 commits intolinux-nvme:masterfrom
jeff-lien-sndk:sndk-plugin-fixes
Open

Sndk plugin fixes#3119
jeff-lien-sndk wants to merge 3 commits intolinux-nvme:masterfrom
jeff-lien-sndk:sndk-plugin-fixes

Conversation

@jeff-lien-sndk
Copy link
Contributor

This PR contains fixes or changes for the following issues:

  • Add Support for SN655 UUID checking in sandisk plugin
  • Add Clear Assert Capability for new drives
  • vs-internal-log Sandisk plugin command does not set data area value correctly
  • Updates to vs-internal-log plugin command

The UUID used by the SN640 and SN655 drives was not
being checked in the sandisk plugin code.

Signed-off-by: jeff-lien-sndk <jeff.lien@sandisk.com>
2 Sandisk drives do support the Clear Assert VUC but did
not capability enabled.  This change will enable it.

Signed-off-by: jeff-lien-sndk <jeff.lien@sandisk.com>
The data-area parameter was being overwritten and set
to the default data area value for all cases.  This change
will only set it when it's not specified.

This set of changes fully deprecates a few of the optional parameters
for this command across both WDC and Sandisk plugins, and tidies up some
of the logic around how telemetry or vendor defined debug is returned.

This adds a new --type value in the vs-internal-log command for
capturing Host-initiated and Controller-initated telemetry in one
command.

Signed-off-by: Brandon Paupore <brandon.paupore@sandisk.com>

Fixes a compile warning on tar file creation.
Fixes error that creates an empty file.

Signed-off-by: jeff-lien-sndk <jeff.lien@sandisk.com>
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