Re: [PATCH] hostraid dmraid plugin should check for hostraid controller
"Darrick J. Wong" <[email protected]>
| Newsgroups | gmane.linux.ataraid,gmane.linux.kernel.device-mapper.devel |
|---|---|
| Message-ID | <[email protected]> |
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. > _______________________________________________ Ataraid-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/ataraid-list
signature.asc
(application/pgp-signature, 191 B)
-----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.2.1 (GNU/Linux) iD8DBQBEDiLWa6vRYYgWQuURAoBTAKCkUjy8lIQwryHtlGWKTqiJYpf6fwCgiaGr u8ExNx3SfPgkmTlSiXuNncE= =B5uQ -----END PGP SIGNATURE-----