On Monday, August 08, 2005 3:47 PM, James Bottomley wrote: > Eric, > > This attached patch should do DV on both physical devices and the > underlying devices of fusion IM assemblies (providing you apply it on > top of the prior underlying device exposure patch), which, I > believe was > your only outstanding concern with the last generic DV patch. > > There is one slight unsightly piece: the IM device still > attaches to the > transport class however all the parameters it shows actually belong to > the underlying device at that id ... I could do with finding a way of > persuading the SPI transport class not to attach to RAID devices. > > Since this addresses all of LSI's prior concerns, may I now apply it? > Roy/Laura/Roy/Steve, and others - This is a patch to remove our internal dv(domain validation) code and replace with generic dv implemention in the linux kernel. This is a year old push from kernel.org , for the drivers in upstream kernel. James: I've not been able to review this patch, nor the other one you sent. On concern is whether spi transport handle asyn events? Meaning will it do domain validation on RAID1 volume - for a new drive that was hot swapped with a good disk? In the driver look at mptscsih_event_process - this code is handling a aync event from the firmware telling the driver to perform dv on disk that was just added. Pls don't rush this into the kernel untill we have time to review, and/or talk to our internal test teams on test effort, and/or talk to our customers explaning the risk in removing this code. Eric Moore - : 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
