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, Oct 2, 3:24 AM

	Unknown Object (File)
	Thu, Oct 1, 2:27 PM

	Unknown Object (File)
	Tue, Sep 29, 3:45 PM

	Unknown Object (File)
	Sat, Sep 26, 7:20 AM

	Unknown Object (File)
	Fri, Sep 25, 8:00 PM

	Unknown Object (File)
	Fri, Sep 25, 2:57 PM

	Unknown Object (File)
	Thu, Sep 24, 3:31 PM

	Unknown Object (File)
	Wed, Sep 23, 4:41 PM

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