Date: Fri, 27 Apr 2012 10:27:23 -0700 (PDT) From: Scott Long <scott4long@yahoo.com> To: John Baldwin <jhb@freebsd.org>, Alexander Motin <mav@freebsd.org> Cc: "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "src-committers@freebsd.org" <src-committers@freebsd.org> Subject: Re: svn commit: r234603 - head/sys/geom/raid Message-ID: <1335547643.283.YahooMailNeo@web45713.mail.sp1.yahoo.com> In-Reply-To: <201204270745.16844.jhb@freebsd.org> References: <201204231304.q3ND43Yg098748@svn.freebsd.org> <201204270745.16844.jhb@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
=0A=0A=0A=0A----- Original Message -----=0A> From: John Baldwin <jhb@freebs= d.org>=0A> To: Alexander Motin <mav@freebsd.org>=0A> Cc: src-committers@fre= ebsd.org; svn-src-all@freebsd.org; svn-src-head@freebsd.org=0A> Sent: Frida= y, April 27, 2012 5:45 AM=0A> Subject: Re: svn commit: r234603 - head/sys/g= eom/raid=0A> =0A> On Monday, April 23, 2012 9:04:03 am Alexander Motin wrot= e:=0A>> Author: mav=0A>> Date: Mon Apr 23 13:04:02 2012=0A>> New Revisio= n: 234603=0A>> URL: http://svn.freebsd.org/changeset/base/234603=0A>> =0A>= > Log:=0A>> =A0 Add names for all primary RAID levels defined by DDF 2.0 = specification.=0A>> =0A>> Modified:=0A>> =A0 head/sys/geom/raid/g_raid.c= =0A>> =A0 head/sys/geom/raid/g_raid.h=0A>> =A0 head/sys/geom/raid/tr_raid= 1.c=0A>> =A0 head/sys/geom/raid/tr_raid1e.c=0A> =0A> We should probably ad= d a separate header to hold DDF constants.=A0 graid isn't=0A> the only plac= e that uses them (e.g. mfi(4) uses it to describe volumes, so=0A> mfiutil(8= ) has its own DDF constants as well in mfiutil.h).=0A> =0A> --=A0=0A=0AYou = mean src/sys/dev/ata/ata-raid-ddf.h?=0A=0AThat said, I trust DDF to be neit= her universal nor standard, and it's probably a futile micro-optimization t= o try too hard at this. =A0At the very least, leave MFI alone.=0A=0A=0AScot= t=0A
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1335547643.283.YahooMailNeo>