virtio_blk: Fix initialisation of dump request structure
ClosedPublic
Actions

Authored by sarah.walker2_arm.com on Mar 30 2026, 9:21 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, May 26, 4:45 PM

	Unknown Object (File)
	Mon, May 18, 2:16 AM

	Unknown Object (File)
	Mon, May 18, 2:08 AM

	Unknown Object (File)
	Thu, May 14, 5:30 AM

	Unknown Object (File)
	Thu, May 14, 2:54 AM

	Unknown Object (File)
	Mon, May 11, 6:57 PM

	Unknown Object (File)
	Mon, May 11, 3:12 PM

	Unknown Object (File)
	Sun, May 10, 2:21 PM

View All 25 Files

Subscribers

imp

Details

Reviewers

andrew
asomers

Summary

Commit c8c37141 ("virtio_blk: Use bus_dma for command/ack buffer allocations")
failed to update initialisation of the dedicated dump request structure. This
caused a panic on attempting to dump core to a virtio_blk device.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 71812
Build 68695: arc lint + arc unit

Event Timeline

sarah.walker2_arm.com created this revision.Mar 30 2026, 9:21 AM

Herald added a subscriber: imp. · View Herald TranscriptMar 30 2026, 9:21 AM

sarah.walker2_arm.com requested review of this revision.Mar 30 2026, 9:21 AM

Harbormaster completed remote builds in B71812: Diff 174521.Mar 30 2026, 9:21 AM

sarah.walker2_arm.com added reviewers: andrew, asomers.Mar 30 2026, 9:22 AM

Works for me. Thanks for the fast fix.

This revision is now accepted and ready to land.Mar 30 2026, 1:02 PM

andrew mentioned this in rGf54209510b1b: virtio_blk: Fix initialisation of dump request structure.Apr 9 2026, 2:14 PM

Committed in rGf54209510b1b30b1445792db7d33401f7c7a97d6 with the wrong tag

Revision Contents
Changeset List

Path

Size

sys/

dev/

virtio/

block/

virtio_blk.c

94 lines

Diff 174521

View Options

sys/dev/virtio/block/virtio_blk.c

virtio_blk: Fix initialisation of dump request structureClosedPublicActions