Re: [RFC][PATCH V6] btrfs: allocation_hint mode

[Goffredo Baroncelli <kreijack@libero.it>]

On 2/10/21 5:04 PM, Josef Bacik wrote:
> On 2/1/21 4:28 PM, Goffredo Baroncelli wrote:
>>
>> Hi all,
>>
>> the previous V5 serie was called "btrfs: preferred_metadata: preferred device
>> for metadata".
>>
> 
> A few general points up front, first I'd highly recommend reading our patch submission guidelines
> 
> https://github.com/btrfs/btrfs-workflow/blob/master/patch-submission.md
> 
> specifically the 'Git config options' section, as it tells you how to apply our git hooks to your local repo.  This will check your patches for all the automatic formatting things we'll complain about, that way you don't have to get bogged down in those style of comments in the review.  For example as soon as I started applying your patches I was getting a ton of whitespace warnings, these are better caught before sending them along.
> 
ok
> Also try to develop on Dave's misc-next branch.  I realize this is a moving target, so I'm fine with massaging patches so I can review, but again everything needed massaging.
> 
ok
> And finally for a new feature we're going to need an xfstest or two in order to merge them.  I realize we're still working out the details, but the further you get into this it would be good to go ahead and have a test that validates everything.  Thanks,

Definitely I have to do it. Unfortunately the last time I tried I found complex to setup it. Do you have some link for an "xfstest beginner" ?
> 
> Josef


-- 
gpg @keyserver.linux.it: Goffredo Baroncelli <kreijackATinwind.it>
Key fingerprint BBF5 1610 0B64 DAC6 5F7D  17B2 0EDA 9B37 8B82 E0B5