Files
Mark Langsdorf e6badad186 driver core: have match() callback in struct bus_type take a const *
JIRA: https://issues.redhat.com/browse/RHEL-81536
Omitted-fix: b57d5ffc3ab507d0e19fc8b90b19c76af43fb790
Omitted-fix: abfceba0a7a246ac082bf569807738ff7416f59f
Omitted-fix: cdd1fa91a6b8c7cd93b3abf9f3ef05b8ce741b61
Omitted-fix: c8477bb0a8e7f6b2e47952b403c5cb67a6929e55
Omitted-fix: c94cf023dce30d5f28323a9f28ee2912f248a68f
all 5 of these commits apply to architectures that aren't supported
Omitted-fix: 91314e7dfd83345b8b820b782b2511c9c32866cd
apparently applied under another hash idea as part of the DRM merge

Conflicts:
	drivers/firmware/google/coreboot_table.c - minor context
differences
	drivers/platform/x86/wmi.c - the #defines did not apply,
but the intent was to create the wmi_driver as a const using
container_of_const, so I made those changes inline to the
previous declaration
	drivers/spi/spi.c - minor context differences
caused by a partial backport of this commit via
commit 9989f0ea4b

commit d69d804845985c29ab5be5a4b3b1f4787893daf8
Author: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Date: Wed, 03 Jul 2024 15:16:54 +0000

In the match() callback, the struct device_driver * should not be
changed, so change the function callback to be a const *.  This is one
step of many towards making the driver core safe to have struct
device_driver in read-only memory.

Because the match() callback is in all busses, all busses are modified
to handle this properly.  This does entail switching some container_of()
calls to container_of_const() to properly handle the constant *.

For some busses, like PCI and USB and HV, the const * is cast away in
the match callback as those busses do want to modify those structures at
this point in time (they have a local lock in the driver structure.)
That will have to be changed in the future if they wish to have their
struct device * in read-only-memory.

Cc: Rafael J. Wysocki <rafael@kernel.org>
Reviewed-by: Alex Elder <elder@kernel.org>
Acked-by: Sumit Garg <sumit.garg@linaro.org>
Link: https://lore.kernel.org/r/2024070136-wrongdoer-busily-01e8@gregkh
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Mark Langsdorf <mlangsdo@redhat.com>
2025-12-03 15:35:12 -05:00
..