Re: [PATCH 4 of 4] DM RAID: Add support for MD's RAID10 "far" and "offset" algorithms

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Jan 7, 2013, at 12:05 AM, NeilBrown wrote:

> On Mon, 17 Dec 2012 12:41:28 -0600 Jonathan Brassow <jbrassow@xxxxxxxxxx>
> wrote:
> 
> 
>> --- linux-upstream.orig/drivers/md/dm-raid.c
>> +++ linux-upstream/drivers/md/dm-raid.c
>> @@ -91,15 +91,44 @@ static struct raid_type {
>> 	{"raid6_nc", "RAID6 (N continue)",		2, 4, 6, ALGORITHM_ROTATING_N_CONTINUE}
>> };
>> 
>> +static char *raid10_md_layout_to_format(int layout)
>> +{
>> +	/*
>> +	 * Bit 16 and 17 stand for "offset" and "dev_stride"
>> +	 * Refer to MD's raid10.c for details
>> +	 */
> 
> I don't think "bit 17" is "dev_stride" any more..  "use_far_sets" ??

yes, s/dev_stride/use_far_sets/.

Thanks for catching that.   I think there was a change made to dm-raid.c recently (an arg was removed from the mapping function) that causes this patch to not apply cleanly.  I correct the above and resend a fresh patch for this.

 brassow

--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux RAID Wiki]     [ATA RAID]     [Linux SCSI Target Infrastructure]     [Linux Block]     [Linux IDE]     [Linux SCSI]     [Linux Hams]     [Device Mapper]     [Device Mapper Cryptographics]     [Kernel]     [Linux Admin]     [Linux Net]     [GFS]     [RPM]     [git]     [Yosemite Forum]


  Powered by Linux