Yay for PCI config space oddities. You make a good point about recovery too. Well, scratch this patch then. As far as distinguishing the "new unexpected raidset" vs the "known raidset" case, perhaps we could have a dmraidtab that would list all the array names that we know about, and some option that says "mount the raidsets listed in dmraidtab", sort of like what mount -a does? --D On Tue, 2006-03-07 at 18:54 -0500, Peter Jones wrote: > On Tue, 2006-03-07 at 15:00 -0800, Darrick J. Wong wrote: > > Hi all (and Heinz in particular), > > > > The attached patch against dmraid v1.0.0rc10 removes an unneeded > > comment, supplements an existing one, and most importantly, and causes > > is_asr to reject a disk if it can prove that the PCI controller attached > > to the disk is not a categorized as a RAID controller (this > > characteristic of the PCI class code is true for all the hostraid > > adapters in my lab). > > It's not true for all of the ones on my desk... some show themselves as > IDE, some as SCSI, and some as SATA. > > > This part of the patch helps us to avoid the situation where you can > > take a disk out of a HostRAID array, plug it in to a different system, > > and have dmraid -ay go all haywire when it can't find the rest of the > > disks. It is my intent, however, that if is_asr will not reject a disk > > if it can't make this positive identification; this patch therefore > > should not affect Heinz's dm-based fakefakeraid testing rig. :) > > It also screws up the recovery case -- moving a set of disks from one > machine to another just to read them. > > We need to be treating "discovery of new, unexpected raidsets" > differently from "discovery and assembly of raidsets we know about", but > really that's at a higher level than what the "dmraid" utility is doing. > It just scans; it doesn't care why. For this sort of elimination, you > care about why. >
Attachment:
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Ataraid-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/ataraid-list
