Differential D43891

acpi: Don't assume a resource is reserved in acpi_delete_resource
ClosedPublic
Actions

Authored by jhb on Feb 14 2024, 6:55 PM.

Tags

None

Referenced Files

	F173431259: D43891.id134363.diff
	Fri, Sep 25, 9:49 PM

	F173371357: D43891.id134363.diff
	Fri, Sep 25, 12:44 PM

	Unknown Object (File)
	Thu, Sep 24, 8:19 PM

	Unknown Object (File)
	Mon, Sep 21, 1:12 PM

	Unknown Object (File)
	Sun, Sep 20, 2:59 PM

	Unknown Object (File)
	Sun, Sep 6, 8:43 AM

	Unknown Object (File)
	Sat, Sep 5, 10:13 AM

	Unknown Object (File)
	Sat, Sep 5, 9:27 AM

Subscribers

Details

Reviewers

kevans
andrew
imp

Commits

rG7204a656fcfa: acpi: Don't assume a resource is reserved in acpi_delete_resource
rGcfb8cc9c5fa9: acpi: Don't assume a resource is reserved in acpi_delete_resource
rGe05436d577de: acpi: Don't assume a resource is reserved in acpi_delete_resource

Summary

This fixes a panic if a driver uses bus_set_resource to add a resource
that fails to reserve and then deletes the resource via
bus_delete_resource.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb created this revision.Feb 14 2024, 6:55 PM

Herald added a subscriber: imp. · View Herald TranscriptFeb 14 2024, 6:55 PM

jhb requested review of this revision.Feb 14 2024, 6:55 PM

jhb added a parent revision: D43890: pci_host_generic: Include the bridge's device name in rman descriptions.

Harbormaster completed remote builds in B56010: Diff 134327.Feb 14 2024, 6:55 PM

jhb added a child revision: D43892: acpi: Allow drivers to use bus_set_resource for more resources.Feb 14 2024, 6:55 PM

imp accepted this revision.Feb 14 2024, 7:10 PM

This revision is now accepted and ready to land.Feb 14 2024, 7:10 PM

I'm surprised this hasn't bitten sooner

In D43891#1001335, @imp wrote:

I'm surprised this hasn't bitten sooner

Very few things use bus_delete_resource. I only hit this when I ran into the ram0 problem and my changes to pci_host_generic caused a bridge to fail to probe when it couldn't allocate its window,

Closed by commit rGe05436d577de: acpi: Don't assume a resource is reserved in acpi_delete_resource (authored by jhb). · Explain WhyFeb 14 2024, 10:53 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rGe05436d577de: acpi: Don't assume a resource is reserved in acpi_delete_resource.

jhb added a commit: rGcfb8cc9c5fa9: acpi: Don't assume a resource is reserved in acpi_delete_resource.Apr 8 2024, 7:06 PM

jhb added a commit: rG7204a656fcfa: acpi: Don't assume a resource is reserved in acpi_delete_resource.Apr 8 2024, 8:26 PM

Revision Contents
Changeset List

Path

Size

sys/

dev/

acpica/

3 lines

Diff 134363

sys/dev/acpica/acpi.c

Loading...