Skip to content

fabrics: Do not swap bytes for system uuid#264

Merged
igaw merged 1 commit intolinux-nvme:masterfrom
igaw:fix-system-uuid
Mar 1, 2022
Merged

fabrics: Do not swap bytes for system uuid#264
igaw merged 1 commit intolinux-nvme:masterfrom
igaw:fix-system-uuid

Conversation

@igaw
Copy link
Copy Markdown
Collaborator

@igaw igaw commented Mar 1, 2022

The kernel is taking care of swapping the system uuid bytes since
f1d8e614d74b ("drivers/firmware/dmi_scan.c: check dmi version when get
system uuid"). Hence we should not swap them again. Instead just read
/sys/class/dmi/id/product_uuid and store the bytes directly into the
system_uuid buffer.

Signed-off-by: Daniel Wagner dwagner@suse.de

Fixes: #263

The kernel is taking care of swapping the system uuid bytes since
f1d8e614d74b ("drivers/firmware/dmi_scan.c: check dmi version when get
system uuid"). Hence we should not swap them again. Instead just read
/sys/class/dmi/id/product_uuid and store the bytes directly into the
system_uuid buffer.

Signed-off-by: Daniel Wagner <dwagner@suse.de>
@igaw igaw force-pushed the fix-system-uuid branch from 92e0a1f to 224967d Compare March 1, 2022 13:49
@igaw
Copy link
Copy Markdown
Collaborator Author

igaw commented Mar 1, 2022

Reordered variables declarations.

@igaw igaw merged commit db0c577 into linux-nvme:master Mar 1, 2022
@igaw igaw deleted the fix-system-uuid branch March 1, 2022 13:51
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.

wrong hostnqn uuid

2 participants