[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20180628153642.GB8521@thunk.org>
Date: Thu, 28 Jun 2018 11:36:42 -0400
From: "Theodore Y. Ts'o" <tytso@....edu>
To: Lukas Czerner <lczerner@...hat.com>
Cc: linux-ext4@...r.kernel.org, adilger@...ger.ca
Subject: Re: test f_extent_htree fails on 1.44.3-rc1
On Thu, Jun 28, 2018 at 08:55:34AM +0200, Lukas Czerner wrote:
>
> Understandable. I didn't really like the idea of adding separate options
> for that so I changed the -d parameter.
Adding a new extended options doesn't bother me that much, since a
global default to supress all xattr copying is probably something that
a number of folks would find generically useful.
> I'm not the target user either, but the -d options scheme I proposed
> is not really in conflict with supporting external config file as well.
It's not, but if we had an external config file, then the -d options
scheme would be duplicated functionality, and the advantage of using
the profile parsing code is we don't need to implement a new set of
config parsing code, which is its own extra bit of complexity and
opportunity for Coverity/gcc -Wall nits. :-)
- Ted
Powered by blists - more mailing lists