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

	F156087348: D56156.diff
	Sun, May 10, 2:21 PM

	F156085895: D56156.diff
	Sun, May 10, 2:14 PM

	Unknown Object (File)
	Tue, May 5, 8:16 AM

	Unknown Object (File)
	Sun, Apr 19, 4:55 PM

	Unknown Object (File)
	Sat, Apr 18, 12:15 AM

	Unknown Object (File)
	Fri, Apr 17, 11:43 AM

	Unknown Object (File)
	Wed, Apr 15, 3:32 AM

	Unknown Object (File)
	Wed, Apr 15, 3:06 AM

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