On Fri, Jul 15, 2011 at 02:41:55PM -0700, Dan Ehrenberg wrote: > If the stripe width was set to 1, then this patch will ignore > that stripe width and ext4 will act as if the stripe width > were 0 with respect to optimizing allocations. > > Signed-off-by: Dan Ehrenberg <dehrenberg@xxxxxxxxxx> Applied to the ext4 tree. I did make one formatting change. Please don't have blank lines between the if and else clauses, like this: if (sbi->s_stripe && sbi->s_stripe <= sbi->s_blocks_per_group) ret = sbi->s_stripe; else if (stripe_width <= sbi->s_blocks_per_group) ret = stripe_width; else if (stride <= sbi->s_blocks_per_group) ret = stride; it wastes vertical whitespace and makes the control flow harder to follow. Eliminate the blank lines, and it's easier to read, I think. if (sbi->s_stripe && sbi->s_stripe <= sbi->s_blocks_per_group) ret = sbi->s_stripe; else if (stripe_width <= sbi->s_blocks_per_group) ret = stripe_width; else if (stride <= sbi->s_blocks_per_group) ret = stride; - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
