Differential D49270

mmc: Use bus_topo_lock and taskqueue_bus while adding/removing child devices
ClosedPublic
Actions

Authored by jhb on Thu, Mar 6, 10:07 PM.

Tags

None

Referenced Files

	F112606552: D49270.diff
	Thu, Mar 20, 10:52 AM

	Unknown Object (File)
	Tue, Mar 11, 4:57 AM

	Unknown Object (File)
	Mon, Mar 10, 7:40 PM

	Unknown Object (File)
	Sat, Mar 8, 12:49 PM

	Unknown Object (File)
	Sat, Mar 8, 7:22 AM

	Unknown Object (File)
	Sat, Mar 8, 6:37 AM

	Unknown Object (File)
	Fri, Mar 7, 2:50 PM

	Unknown Object (File)
	Fri, Mar 7, 2:32 PM

View All 9 Files

Subscribers

None

Details

Reviewers

imp
andrew
manu

Commits

rGc0bed9bd0bda: mmc: Use bus_topo_lock and taskqueue_bus while adding/removing child devices

Summary

Some drivers held regular mutexes across some new-bus calls, instead
depend on bus_topo_lock to protect the relevant softc members. This
also fixes the bus_topo_lock to be explicit in these drivers rather
than relying on the implicit Giant from taskqueue_swi_giant, and it
avoids calling sleepable routines like device_probe_and_attach from an
swi context.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb created this revision.Thu, Mar 6, 10:07 PM

Herald added a reviewer: andrew. · View Herald TranscriptThu, Mar 6, 10:07 PM

Herald added a reviewer: manu. · View Herald Transcript

jhb requested review of this revision.Thu, Mar 6, 10:07 PM

Harbormaster completed remote builds in B62806: Diff 151982.Thu, Mar 6, 10:07 PM

jhb added a parent revision: D49269: pcib: Use taskqueue_bus for hot-plug events instead of a private taskqueue.Thu, Mar 6, 10:07 PM

jhb added a child revision: D49271: sdhci: Use bus_topo_lock and taskqueue_bus for hotplug events.

This revision was not accepted when it landed; it landed in state Needs Review.Mon, Mar 10, 5:36 PM

Closed by commit rGc0bed9bd0bda: mmc: Use bus_topo_lock and taskqueue_bus while adding/removing child devices (authored by jhb). · Explain Why

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rGc0bed9bd0bda: mmc: Use bus_topo_lock and taskqueue_bus while adding/removing child devices.

Revision Contents
Changeset List

Path

Size

sys/

arm/

allwinner/

11 lines

dev/

mmc/

host/

22 lines

mmc_fdt_helpers.c

10 lines

rtsx/

18 lines

Diff 152093

sys/arm/allwinner/aw_mmc.c

Loading...

sys/dev/mmc/host/dwmmc.c

Loading...

sys/dev/mmc/mmc_fdt_helpers.c

Loading...

sys/dev/rtsx/rtsx.c

Loading...