Thanks to visit codestin.com
Credit goes to github.com

Skip to content

block: allow VIRTIO_BLK_T_GET_ID for read-only devices - #7546

Merged
rbradford merged 1 commit into
cloud-hypervisor:mainfrom
cbrewster:cb-virtio-blk-ro-getdeviceid
Dec 9, 2025
Merged

rbradford merged 1 commit into
cloud-hypervisor:mainfrom
cbrewster:cb-virtio-blk-ro-getdeviceid

Conversation

@cbrewster

Copy link
Copy Markdown
Contributor

#7294 adjusted the checks for read-only requests made to virtio-blk devices and started rejecting VIRTIO_BLK_T_GET_ID requests. These requests do not perform any writes and are needed in order to access device serials from within the guest.

@cbrewster
cbrewster requested a review from a team as a code owner December 6, 2025 18:58
@cbrewster
cbrewster force-pushed the cb-virtio-blk-ro-getdeviceid branch 2 times, most recently from a1a450e to 6503daf Compare December 6, 2025 19:20
@rbradford

Copy link
Copy Markdown
Member

@cbrewster Thank you for change - looks good. Please make sure you read CONTRIBUTING.md. In particular the note about DCO sign off on the commits.

cloud-hypervisor#7294 adjusted
the checks for read-only requests made to virtio-blk devices and started
rejecting VIRTIO_BLK_T_GET_ID requests. These requests do not perform
any writes and are needed in order to access device serials from within
the guest.

Signed-off-by: Connor Brewster <[email protected]>
@cbrewster
cbrewster force-pushed the cb-virtio-blk-ro-getdeviceid branch from 6503daf to be17dca Compare December 8, 2025 01:37
@cbrewster

Copy link
Copy Markdown
Contributor Author

@rbradford I updated the commit with the DCO sign off, thanks for taking a look!

@rbradford
rbradford enabled auto-merge December 9, 2025 14:52
@rbradford
rbradford added this pull request to the merge queue Dec 9, 2025
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Dec 9, 2025
@rbradford
rbradford added this pull request to the merge queue Dec 9, 2025
@likebreath likebreath added the bug-fix Bug fix to include in release notes label Dec 9, 2025
Merged via the queue into cloud-hypervisor:main with commit 41a8dcd Dec 9, 2025
42 of 44 checks passed
@github-project-automation github-project-automation Bot moved this from 🆕 New to ✅ Done in Cloud Hypervisor Roadmap Dec 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix Bug fix to include in release notes

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

4 participants