Skip to content

nvme.i: decode 'tsas' discovery log page entry#1057

Merged
igaw merged 1 commit intolinux-nvme:masterfrom
hreinecke:py-tsas
Aug 29, 2025
Merged

nvme.i: decode 'tsas' discovery log page entry#1057
igaw merged 1 commit intolinux-nvme:masterfrom
hreinecke:py-tsas

Conversation

@hreinecke
Copy link
Copy Markdown
Collaborator

Decode the 'tsas' discovery log page entry to get information about any TCP 'sectype' settings.

Decode the 'tsas' discovery log page entry to get information about
any TCP 'sectype' settings.

Signed-off-by: Hannes Reinecke <[email protected]>
@igaw igaw merged commit 86ceb42 into linux-nvme:master Aug 29, 2025
12 checks passed
@igaw
Copy link
Copy Markdown
Collaborator

igaw commented Aug 29, 2025

Thanks!

Comment thread libnvme/nvme.i
break;
}
PyDict_SetItemStringDecRef(tsas, "cms", val);
PyDict_SetItemStringDecRef(entry, "tsas", val);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should this be using the tsas object instead of val?
PyDict_SetItemStringDecRef(entry, "tsas", tsas);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@hreinecke
@igaw
Fixed it with this PR: #1058

@hreinecke hreinecke deleted the py-tsas branch September 4, 2025 12:06
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.

3 participants