]> git.dujemihanovic.xyz Git - u-boot.git/commit
dm: Add dm_remove_devices_active() for ordered device removal
authorJanne Grunau <j@jannau.net>
Sat, 23 Nov 2024 21:44:05 +0000 (22:44 +0100)
committerTom Rini <trini@konsulko.com>
Sun, 24 Nov 2024 21:41:28 +0000 (15:41 -0600)
commitdabaa4ae32062cb3f3d995e5c63e6cef54ad079b
treef6cbd5ae28f1e3d6db7e83ef3ff724d055ddf8f6
parent544a76bac3393045e86a56cc5dfe2477e437c59b
dm: Add dm_remove_devices_active() for ordered device removal

This replaces dm_remove_devices_flags() calls in all boot
implementations to ensure non vital devices are consistently removed
first. All boot implementation except arch/arm/lib/bootm.c currently
just call dm_remove_devices_flags(DM_REMOVE_ACTIVE_ALL). This can result
in crashes when dependencies between devices exists. The driver model's
design document describes DM_FLAG_VITAL as "indicates that the device is
'vital' to the operation of other devices". Device removal at boot
should follow this.

Instead of adding dm_remove_devices_flags() with (DM_REMOVE_ACTIVE_ALL |
DM_REMOVE_NON_VITAL) everywhere add dm_remove_devices_active() which
does this.

Fixes a NULL pointer deref in the apple dart IOMMU driver during EFI
boot. The xhci-pci (driver which depends on the IOMMU to work) removes
its mapping on removal. This explodes when the IOMMU device was removed
first.

dm_remove_devices_flags() is kept since it is used for testing of
device_remove() calls in dm.

Signed-off-by: Janne Grunau <j@jannau.net>
arch/arm/lib/bootm.c
arch/riscv/lib/bootm.c
arch/x86/lib/bootm.c
drivers/core/root.c
include/dm/root.h
lib/efi_loader/efi_boottime.c
test/dm/core.c