----- Original Message -----
> From: John Baldwin <j...@freebsd.org>
> To: Alexander Motin <m...@freebsd.org>
> Cc: src-committ...@freebsd.org; svn-src-all@freebsd.org; 
> svn-src-h...@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

_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to