On Fri, 2006-02-10 at 14:10 +0100, Christoph Hellwig wrote: > > Here is the patch for review that implements slave_alloc to block direct > > access to the Physical Disks. > > By using no_uld_attach and clearing the scsi device writeable flag the disks will be > > still accessible even if it belongs to a RAID volume. > > I would prefer to hide the Physical Disks to block any kind of access. > > > > I will send a separate patch to monitor drive insertion/removal to update internally > > the physical disk list. > > Far too much bloat for what you're trying to do. If you want the same > behaviour as the initial megaraid_sas patch just take my patch and > replace the setting of no_uld_attach with return -ENXIO in > megasas_slave_alloc or move all that to slave_alloc if you you really > care. > > But I'm strongly against even that, physical disks should be available > through /dev/sg. I'd prefer to expose the underlying disks but refuse ULD binding so they only show up in /dev/sg. I have to say that this is extremely useful for external applications (like clustered ones) who want to be able to identify shared resources. Since only root has access to /dev/sg, using it wrongly is strongly caveat emptor. James - : send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
