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)
	Fri, Jul 10, 11:14 PM

	Unknown Object (File)
	Wed, Jul 8, 11:02 AM

	Unknown Object (File)
	Mon, Jul 6, 6:22 AM

	Unknown Object (File)
	Sun, Jul 5, 8:57 PM

	Unknown Object (File)
	Sun, Jul 5, 6:43 PM

	Unknown Object (File)
	Sat, Jul 4, 6:48 AM

	Unknown Object (File)
	Mon, Jun 15, 6:08 AM

	Unknown Object (File)
	Jun 7 2026, 4:29 PM

View All 33 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