Re: [PATCH 00/88] pnfs block layout driver

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

 



On 06/09/2011 07:16 PM, Boaz Harrosh wrote:
> On 06/09/2011 03:15 PM, Jim Rees wrote:
>> Boaz Harrosh wrote:
>>
>>   Who is going to SQUASH all the SQUASHMEs and re think the all patch
>>   separation again. To something that makes a more logical progression
>>   and easier on the review. The way it is now I'm not able to review,
>>   sorry, I got lost trying to understand which is which.
>>
>> I'm open to suggestions and happy to do the work.  I agree that 88 patches
>> is nearly indigestable.  However I note that Benny seems to have pulled in
>> the entire set so I'm not sure how to proceed at this point.  Also this code
>> was in Benny's 2.6.38 and only got dropped when the 3.0 merge came along, so
>> most of it's already been under review for a year or more.
> 
> Lets start by squashing all the SQUASHMEs into their proper place, that will
> get you down to 49. You might notice that it could get hard and it will be
> easier to actually take the complete code and re-divide it to patches, a fresh.
> crafted more less on the old division strategy.
> 
> It's what Fred did at the final files submission. And it is what I did more
> or less with the objects final submission. But you might find that for you
> it is easier to rebase and squash the patches together after you re-order
> them. It's your call and you will not know before you experiment a little.
> I can show you some techniques I use on both these paths.
> (Let's meet in B next week)
> 
> Then you should do a second pass to see that each patch is compile-able and
> makes sense and maybe also reorder the introduction of the generic parts
> close to where they are used in the pachset progression. Again like we did
> for files and objects.
> 
> Thanks
> Boaz

Ho and I forgot, once you do all that we might want to revisit all these patches
that have HACK in their title. I guess Fred can help with some of these

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


[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux