Hi Chandra,

based on your latest patchset there are some issues which
should be fixed:

- When SYSFS_DEPRECATED is not set also scsi_target and
  scsi_host devices will be attached to the scsi bus.
  So we have to check with 'scsi_is_sdev_device' before
  processing the notifier call
- Newer EMC flarecode also provide for a 'VRAID' device
  model; this should be added to the device map
- The name should be 'emc', not 'emc_clariion', for
  integration with existing multipath-tools
- It's spelled 'detached' :-)

Patch attached.

Some issues which I'd like to see eventually:

- We should have a way of adding/modifying the internal
  device map. Currently we have to recompile the module
  every time a new device definition is found.
- The ->activate function should be able to be triggered
  manually via sysfs.

James, seeing that the scsi_dh pointer is already present
in the sdev structure, would you accept a patch which adds
scsi_dh attributes to drivers/scsi/scsi_sysfs.c?
Having them encapsulated into drivers/scsi/device_handler
somewhere is really painful due to the asynchronous nature
of this whole thing ...

Apart from this: Great work!
I'll check the other device handlers, too; already noticed
that the device map for HP is wrong :-)

Cheers,

hannes
-- 
Dr. Hannes Reinecke		      zSeries & Storage
hare@suse.de			      +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Markus Rex, HRB 16746 (AG Nürnberg)