All of lore.kernel.org
 help / color / mirror / Atom feed
* [RFC] scsi host sysfs support [3/4]
@ 2003-04-14 18:41 Mike Anderson
  0 siblings, 0 replies; 2+ messages in thread
From: Mike Anderson @ 2003-04-14 18:41 UTC (permalink / raw)
  To: linux-hotplug

-andmike
--
Michael Anderson
andmike@us.ibm.com


DESC
This patch adds a scsi subsystem to the sysfs tree.
EDESC


 drivers/scsi/scsi_sysfs.c |    3 +++
 1 files changed, 3 insertions(+)

diff -puN drivers/scsi/scsi_sysfs.c~sysfs-scsi-subsys drivers/scsi/scsi_sysfs.c
--- sysfs-bleed-2.5/drivers/scsi/scsi_sysfs.c~sysfs-scsi-subsys	Mon Apr 14 10:11:35 2003
+++ sysfs-bleed-2.5-andmike/drivers/scsi/scsi_sysfs.c	Mon Apr 14 10:11:35 2003
@@ -107,10 +107,12 @@ static struct bus_type scsi_bus_type = {
         .match		= scsi_bus_match,
 };
 
+static decl_subsys(scsi, NULL);
 
 int scsi_sysfs_register(void)
 {
 	bus_register(&scsi_bus_type);
+	subsystem_register(&scsi_subsys);
 	devclass_register(&shost_devclass);
 
 	return 0;
@@ -119,6 +121,7 @@ int scsi_sysfs_register(void)
 void scsi_sysfs_unregister(void)
 {
 	devclass_unregister(&shost_devclass);
+	subsystem_unregister(&scsi_subsys);
 	bus_unregister(&scsi_bus_type);
 }
 

_


-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel

^ permalink raw reply	[flat|nested] 2+ messages in thread
* [RFC] scsi host sysfs support [0/4]
@ 2003-04-14 18:37 Mike Anderson
  2003-04-14 18:39 ` [RFC] scsi host sysfs support [1/4] Mike Anderson
  0 siblings, 1 reply; 2+ messages in thread
From: Mike Anderson @ 2003-04-14 18:37 UTC (permalink / raw)
  To: linux-scsi; +Cc: mochel, linux-hotplug-devel, mdharm-scsi

This patch set is against 2.5.67 current. It will not apply to scsi-miscs-2.5
as the decl_subsys macro has added a third argument.

This patch set changes the sysfs support for scsi hosts.
- Removes the previous support of shost_devclass device class.

- Modifies scsi_debug to support the new order of object cleanup and also
  changes the structure of sdebug_host_info to help in object cleanup.

- Creates a scsi subsystem. This is rooted at the top of the sysfs tree.

- Creates a scsi_host subsystem under the scsi subsystem. Modified the
  cleanup in scsi remove host. Switched to a kobject embedded in
  scsi_host. If a driver has not be converted to the driver model or does
  not use the scsi_set_device call it will still show up in the scsi_host
  subsystem, but will not have a symlink to the parent.

The usage of driver model classes for scsi hosts was removed because many
of our drivers have not been converted to the driver model so this gave an
inconsistent model to count on for cleanup and migration of some the
procfs support. I tried adding some fake structures to use classes, but it
failed to cleanup properly do to asymmetric cleanup interfaces in the core
code and also did not seem like the correct direction to go.

This change allows for host cleanup when sysfs references drop to zero
(there still needs be work done on scsi mid get / put calls).

I have tested the insmod / rmmod cycle with scsi_debug and booted with the
ips and aic drivers.

Current Known Issues:
	- Current users of scsi_remove_host will need to have there
	  cleanup reordered to support the restoration of the call to the
	  Scsi_Host_Template release.
	- Callers of scsi_set_device should be migrated off this interface
	  if possible.
	- Need better name for symlink than adapter.
	
Example tree:

[root@elm3a49 sys]# tree -d scsi
scsi
`-- scsi_host
    |-- host0
    |   |-- 0:0:0:0
    |   |   |-- 0:0:0:0:gen
    |   |   `-- block -> ../../../../block/sda
    |   |-- 0:0:15:0
    |   |   `-- 0:0:15:0:gen
    |   |-- 0:0:1:0
    |   |   |-- 0:0:1:0:gen
    |   |   `-- block -> ../../../../block/sdb
    |   |-- 0:1:15:0
    |   |   `-- 0:1:15:0:gen
    |   `-- adapter -> ../../../devices/pci0/00:09.0
    |-- host1
    |   `-- adapter -> ../../../devices/pci1/01:03.0
    |-- host3
    |   |-- 3:0:0:0
    |   |   |-- 3:0:0:0:gen
    |   |   `-- block -> ../../../../block/sdc
    |   `-- adapter -> ../../../devices/pseudo_0/adapter0
    `-- host4
        |-- 4:0:0:0
        |   |-- 4:0:0:0:gen
        |   `-- block -> ../../../../block/sdd
        |-- 4:0:4:0
        |   |-- 4:0:4:0:gen
        |   |-- 4:0:4:0:mt
        |   |-- 4:0:4:0:mta
        |   |-- 4:0:4:0:mtan
        |   |-- 4:0:4:0:mtl
        |   |-- 4:0:4:0:mtln
        |   |-- 4:0:4:0:mtm
        |   |-- 4:0:4:0:mtmn
        |   `-- 4:0:4:0:mtn
        |-- 4:0:5:0
        |   |-- 4:0:5:0:gen
        |   `-- block -> ../../../../block/sr0
        `-- adapter -> ../../../devices/pci1/01:03.1

-andmike
--
Michael Anderson
andmike@us.ibm.com


^ permalink raw reply	[flat|nested] 2+ messages in thread
end of thread, other threads:[~2003-04-14 18:41 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2003-04-14 18:41 [RFC] scsi host sysfs support [3/4] Mike Anderson
  -- strict thread matches above, loose matches on Subject: below --
2003-04-14 18:37 [RFC] scsi host sysfs support [0/4] Mike Anderson
2003-04-14 18:39 ` [RFC] scsi host sysfs support [1/4] Mike Anderson
2003-04-14 18:40   ` [RFC] scsi host sysfs support [2/4] Mike Anderson
2003-04-14 18:41     ` [RFC] scsi host sysfs support [3/4] Mike Anderson

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.