[0/12] More Driver Model Locking Changes

[Patrick Mochel]

Here is the next round of driver model locking changes. These build off of
the previous set of changes, including the klist patch. They eradicate all
of the uses of the subsystems' rwsem in the driver core.

It does include the fix posted earlier that happened when removing the
driver.

A summary is listed below. The patches follow.

Thanks,


	Pat


You may pull from

	bk://kernel.bkbits.net:/home/mochel/linux-2.6-core

Which will update the following files:

 drivers/base/bus.c        |   12 -------
 drivers/base/core.c       |   51 ++++++++----------------------
 drivers/base/dd.c         |   77 +++++++++++++++++++++++++---------------------
 drivers/base/driver.c     |    3 -
 drivers/scsi/scsi_sysfs.c |   14 +++++---
 drivers/usb/core/usb.c    |    4 +-
 include/linux/device.h    |   13 +------
 include/linux/klist.h     |    2 +
 lib/klist.c               |   21 +++++++++++-
 9 files changed, 92 insertions(+), 105 deletions(-)

through these ChangeSets:

<mochel@xxxxxxxxxxxxxxxxxx> (05/03/24 1.2250)
   [driver core] Fix up bogus comment.
   Signed-off-by: Patrick Mochel <mochel@xxxxxxxxxxxxxxxxxx>

<mochel@xxxxxxxxxxxxxxxxxx> (05/03/24 1.2249)
   [driver core] Use a klist for device child lists.

   - Use klist iterator in device_for_each_child(), making it safe to use for
     removing devices.
   - Remove unused list_to_dev() function.
   - Kills all usage of devices_subsys.rwsem.


   Signed-off-by: Patrick Mochel <mochel@xxxxxxxxxxxxxxxxxx>

<mochel@xxxxxxxxxxxxxxxxxx> (05/03/24 1.2248)
   [scsi] Use device_for_each_child() to unregister devices in scsi_remove_target().


   Signed-off-by: Patrick Mochel <mochel@xxxxxxxxxxxxxxxxxx>

<mochel@xxxxxxxxxxxxxxxxxx> (05/03/24 1.2247)
   [klist] Don't reference NULL klist pointer in klist_remove().


   Signed-off-by: Patrick Mochel <mochel@xxxxxxxxxxxxxxxxxx>

<mochel@xxxxxxxxxxxxxxxxxx> (05/03/24 1.2246)
   [driver core] Call klist_del() instead of klist_remove().

   - Can't wait on removing the current item in the list (the positive refcount *because*
     we are using it causes it to deadlock).


   Signed-off-by: Patrick Mochel <mochel@xxxxxxxxxxxxxxxxxx>

<mochel@xxxxxxxxxxxxxxxxxx> (05/03/24 1.2245)
   [driver core] Remove struct device::driver_list.


   Signed-off-by: Patrick Mochel <mochel@xxxxxxxxxxxxxxxxxx>

<mochel@xxxxxxxxxxxxxxxxxx> (05/03/24 1.2244)
   [driver core] Remove struct device::bus_list.


   Signed-off-by: Patrick Mochel <mochel@xxxxxxxxxxxxxxxxxx>

<mochel@xxxxxxxxxxxxxxxxxx> (05/03/24 1.2243)
   [driver core] Fix up bus code and remove use of rwsem.

   - Don't add devices to bus's embedded kset, since it's not used by anyone anymore.
   - Don't need to take the bus rwsem when calling {device,driver}_attach(), since
     those functions use the klists and the klists' spinlocks.


   Signed-off-by: Patrick Mochel <mochel@xxxxxxxxxxxxxxxxxx>

<mochel@xxxxxxxxxxxxxxxxxx> (05/03/24 1.2242)
   [usb] Fix up USB to use klist_node_attached() instead of list_empty() on lists that will go away.


   Signed-off-by: Patrick Mochel <mochel@xxxxxxxxxxxxxxxxxx>

<mochel@xxxxxxxxxxxxxxxxxx> (05/03/24 1.2241)
   [klist] add klist_node_attached() to determine if a node is on a list or not.


   Signed-off-by: Patrick Mochel <mochel@xxxxxxxxxxxxxxxxxx>

<mochel@xxxxxxxxxxxxxxxxxx> (05/03/24 1.2240)
   [driver core] Use bus_for_each_{dev,drv} for driver binding.

   - Now possible, since the lists are locked using the klist lock and not the
     global rwsem.


   Signed-off-by: Patrick Mochel <mochel@xxxxxxxxxxxxxxxxxx>

<mochel@xxxxxxxxxxxxxxxxxx> (05/03/24 1.2239)
   [driver core] Remove the unused device_find().


   Signed-off-by: Patrick Mochel <mochel@xxxxxxxxxxxxxxxxxx>

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/