svn commit: r234603 - head/sys/geom/raid

[John Baldwin]

On Friday, April 27, 2012 1:27:23 pm Scott Long wrote:
> 
> ----- Original Message -----
> > From: John Baldwin <jhb at freebsd.org>
> > To: Alexander Motin <mav at freebsd.org>
> > Cc: src-committers at freebsd.org; svn-src-all at freebsd.org; svn-src-head at freebsd.org
> > Sent: Friday, April 27, 2012 5:45 AM
> > Subject: Re: svn commit: r234603 - head/sys/geom/raid
> > 
> > On Monday, April 23, 2012 9:04:03 am Alexander Motin wrote:
> >>  Author: mav
> >>  Date: Mon Apr 23 13:04:02 2012
> >>  New Revision: 234603
> >>  URL: http://svn.freebsd.org/changeset/base/234603
> >> 
> >>  Log:
> >>    Add names for all primary RAID levels defined by DDF 2.0 specification.
> >> 
> >>  Modified:
> >>    head/sys/geom/raid/g_raid.c
> >>    head/sys/geom/raid/g_raid.h
> >>    head/sys/geom/raid/tr_raid1.c
> >>    head/sys/geom/raid/tr_raid1e.c
> > 
> > We should probably add a separate header to hold DDF constants.  graid isn't
> > the only place that uses them (e.g. mfi(4) uses it to describe volumes, so
> > mfiutil(8) has its own DDF constants as well in mfiutil.h).
> > 
> > -- 
> 
> You mean src/sys/dev/ata/ata-raid-ddf.h?

Yes.

> That said, I trust DDF to be neither universal nor standard, and it's probably a futile micro-optimization to try too hard at this.  At the very 
least, leave MFI alone.

Hmm, LSI claims that MFI uses the constant values (but not necessarily the
structures from DDF).  Certainly the primary RAID type in an mfi(4) volume
uses the same constants as both g_raid.h and ata-raid-ddf.h.

-- 
John Baldwin