Skip to content

compat (fd): Adds cross-platform-compatible file descriptor type#3314

Closed
bgoing-micron-oss wants to merge 1 commit into
linux-nvme:masterfrom
Micron-TPG-OSS:bgoing/libnvme-fd-compatibility
Closed

compat (fd): Adds cross-platform-compatible file descriptor type#3314
bgoing-micron-oss wants to merge 1 commit into
linux-nvme:masterfrom
Micron-TPG-OSS:bgoing/libnvme-fd-compatibility

Conversation

@bgoing-micron-oss
Copy link
Copy Markdown
Contributor

@bgoing-micron-oss bgoing-micron-oss commented Apr 23, 2026

Creates and uses new libnvme_fd_t to handle differences in file descriptor types across platforms. Also defines platform-specific values for invalid file descriptors and test file descriptors.

Comment thread libnvme/src/nvme/lib-compat.h Outdated
@igaw
Copy link
Copy Markdown
Collaborator

igaw commented May 13, 2026

Could you rebase and move the typedefs to lib-types.h? Rest looks good, not too much noise and code churn which is good news :)

Thanks!

Creates and uses new libnvme_fd_t to handle differences in
file descriptor types across platforms.  Also defines platform-specific
values for invalid file descriptors and test file descriptors.

Signed-off-by: Broc Going <bgoing@micron.com>
Signed-off-by: Brandon Capener <bcapener@micron.com>
@bgoing-micron-oss bgoing-micron-oss force-pushed the bgoing/libnvme-fd-compatibility branch from 346c568 to cc570ac Compare May 13, 2026 15:58
@igaw
Copy link
Copy Markdown
Collaborator

igaw commented May 13, 2026

Same here, I tweaked the commit message slightly to use the different style for the prefix and applied it directly. Thanks!

@igaw igaw closed this May 13, 2026
@bgoing-micron-oss bgoing-micron-oss deleted the bgoing/libnvme-fd-compatibility branch May 13, 2026 17:41
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.

2 participants