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

[Scott Long]

----- 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?

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.


Scott