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
