>> >> Immediate consumers (sd, ssd, cmdk, and dad target drivers) are all >> in ON. There are no contracts needed at this time. I am seeking >> patch binding. > >There are two other drivers that use ddi_devid_register: >sata and vdc (ldom's virtual disk client) - should they be listed here ? >
It wasn't apparent to me that vdc needs to provide this property for its encapsulated devid. This is either an existing bug or there is no requirement. I'll follow up on this. And with Prasad's work put back now, yes sata is now a devid registrant. >What happens if a driver uses ddi_devid_register but the corresponding >driver.conf is not updated to have this flag ? > If the cache were to become damaged or the disk moved, the cache auto-rebuild mechanism would not include trying to configure that driver to re-discover the disk. -jg
