Re: cvs commit: src/sys/amd64/conf GENERIC src/sys/i386/conf GENERIC src/sys/ia64/conf GENERIC src/sys/pc98/conf GENERIC src/sys/powerpc/conf GENERIC src/sys/sparc64/conf GENERIC src/sys/sun4v/conf GENERIC

From

Dag-Erling Smørgrav <des@des.no>

Date

25 Feb 2007 14:54:22

Subject

Re: cvs commit: src/sys/amd64/conf GENERIC src/sys/i386/conf GENERIC src/sys/ia64/conf GENERIC src/sys/pc98/conf GENERIC src/sys/powerpc/conf GENERIC src/sys/sparc64/conf GENERIC src/sys/sun4v/conf GENERIC

Message-ID

86k5y6p9t2.fsf@dwp.des.no

References to

Brooks Davis (cvs-src) , Brooks Davis

Replies

Brooks Davis, Brooks Davis (cvs-src)

Referenced by

Brooks Davis, Brooks Davis (cvs-src) , Dag-Erling Smørgrav (cvs-src) , Dag-Erling Smørgrav, mjacob@freebsd.org, mjacob@freebsd.org (cvs-src) , Brooks Davis (cvs-src) , Brooks Davis, Pawel Jakub Dawidek (cvs-src) , Pawel Jakub Dawidek, Tom Rhodes, Tom Rhodes (cvs-src)

[ Hide this part ]

Brooks Davis <brooks@FreeBSD.org> writes:
>   Log:
>   Include GEOM_LABEL in GENERIC.  It's very useful and not well publicized
>   enough.

Bad idea.  GEOM_LABEL can lead to filesystem corruption when used in
conjunction with mirrored disks.  You mount /dev/ufs/root thinking it
corresponds to /dev/ar0s1a, when in fact it corresponds to /dev/ad0s1a
(in most cases, if ad0 is a component of RAID 1 array ar0, ad0s1a will
be tasted before ar0s1a, and GEOM_LABEL does not handle multiple
identically labeled providers gracefully).

DES
--
Dag-Erling Smrgrav - des@des.no