virtio_blk: Limit use of indirect descriptors
ClosedPublic
Actions

Authored by andrew on Jun 17 2025, 3:18 PM.

Tags

None

Referenced Files

	F166941095: D50909.id157184.diff
	Mon, Aug 17, 10:43 PM

	Unknown Object (File)
	Sun, Aug 9, 9:36 AM

	Unknown Object (File)
	Sat, Aug 8, 5:47 PM

	Unknown Object (File)
	Thu, Aug 6, 7:43 PM

	Unknown Object (File)
	Sun, Aug 2, 5:29 AM

	Unknown Object (File)
	Sun, Aug 2, 1:27 AM

	Unknown Object (File)
	Sat, Aug 1, 6:54 AM

	Unknown Object (File)
	Tue, Jul 28, 5:31 PM

View All 84 Files

Subscribers

imp

Details

Reviewers

bryanv
harry.moulton_arm.com

Commits

rGcbc2e34613c4: virtio_blk: Limit use of indirect descriptors

Summary

Pass 0 as the number of indirect descriptors when they are not
supported.

This fixes an issue on the Arm FVP model where we don't negotiate
them, however pass a number of segments greater than
VIRTIO_MAX_INDIRECT. This leads to virtqueue_alloc failing and
virtio_blk failing to attach.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

andrew created this revision.Jun 17 2025, 3:18 PM

Herald added a subscriber: imp. · View Herald TranscriptJun 17 2025, 3:18 PM

andrew requested review of this revision.Jun 17 2025, 3:18 PM

Harbormaster completed remote builds in B64944: Diff 157184.Jun 17 2025, 3:18 PM

harry.moulton_arm.com accepted this revision.Jun 19 2025, 11:59 AM

This revision is now accepted and ready to land.Jun 19 2025, 11:59 AM

Closed by commit rGcbc2e34613c4: virtio_blk: Limit use of indirect descriptors (authored by andrew). · Explain WhyJun 23 2025, 1:20 PM

This revision was automatically updated to reflect the committed changes.

andrew added a commit: rGcbc2e34613c4: virtio_blk: Limit use of indirect descriptors.

Revision Contents
Changeset List

Path

Size

sys/

dev/

virtio/

block/

virtio_blk.c

6 lines

Diff 157452

View Options

sys/dev/virtio/block/virtio_blk.c

virtio_blk: Limit use of indirect descriptorsClosedPublicActions