From: Lukas Herbolt <lukas@herbolt.com>
To: Eric Sandeen <sandeen@sandeen.net>
Cc: Carlos Maiolino <cem@kernel.org>,
djwong@kernel.org, aalbersh@kernel.org,
linux-xfs@vger.kernel.org
Subject: Re: [RFC PATCH 1/1] xfsprogs: mkfs.xfs add default configuration file.
Date: Wed, 20 May 2026 09:17:16 +0200 [thread overview]
Message-ID: <ab057e365f5d2a50fd4ef1385c7f7e11@herbolt.com> (raw)
In-Reply-To: <fddc8573-8461-475f-9826-ee465c48c969@sandeen.net>
On 2026-05-14 18:05, Eric Sandeen wrote:
> On 5/14/26 10:21 AM, Carlos Maiolino wrote:
>> On Thu, May 14, 2026 at 04:37:17PM +0200, Lukas Herbolt wrote:
>>> Various users may prefer different default values. Having a default
>>> config
>>> file will allow them to utilize it without the need specifying
>>> configuration
>>> file on command line.
>>>
>> The idea seems reasonable, to have a default file to load if it's
>> found.
>> I just particularly don't like the idea of shipping/installing a
>> 'default'
>> config file.
>> I think we could ship an "example" one, but leave to distributions to
>> decide
>> what to do. If they would install a default config or not. And
>> maintain
>> the default config file.
>> LTS configs are easy to maintain because we know no new features will
>> be
>> ported to LTS. Providing a default config file from the mainline risks
>> getting a lot of user complains if we in the future decide to change
>> the
>> 'defaults' of the default config file.
>
> Thanks for starting this discussion Lukas.
>
> I'm sure others will have Deeper Thoughts but I more or less agree with
> Carlos here - a default config file to be loaded /if it exists/ but
> maybe
> not existing by default - because we already have defaults hardcoded
> into
> the binary itself - might be a more maintainable path for upstream.
> Otherwise I see new tests appearing which are like "make sure the
> dfault
> config we ship matches the defaults built into the binary itself" which
> seems like unneeded complexity.
>
> (i.e. the os-specific config files could be renamed or symlinked to
> that special default config file path, for example?)
>
> I could maybe see generating an example defaults config file which
> matches
> the built-in defaults at build time? Not sure if that would be useful
> or helpful (or complex).
>
> Thanks,
> -Eric
Noted, I think we can be shipping mkfs.xfs.conf.example with some of the
defaults options set and keep it with the lts configs
--
-lhe
next prev parent reply other threads:[~2026-05-20 7:17 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-14 14:37 [RFC PATCH 0/1] Add default config file for mkfs.xfs Lukas Herbolt
2026-05-14 14:37 ` [RFC PATCH 1/1] xfsprogs: mkfs.xfs add default configuration file Lukas Herbolt
2026-05-14 15:21 ` Carlos Maiolino
2026-05-14 16:05 ` Eric Sandeen
2026-05-14 16:29 ` Darrick J. Wong
2026-05-14 22:27 ` Dave Chinner
2026-05-20 7:17 ` Lukas Herbolt [this message]
2026-05-14 16:27 ` Darrick J. Wong
2026-05-14 23:35 ` Dave Chinner
2026-05-18 14:07 ` Lukas Herbolt
2026-05-14 15:05 ` [RFC PATCH 0/1] Add default config file for mkfs.xfs Darrick J. Wong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ab057e365f5d2a50fd4ef1385c7f7e11@herbolt.com \
--to=lukas@herbolt.com \
--cc=aalbersh@kernel.org \
--cc=cem@kernel.org \
--cc=djwong@kernel.org \
--cc=linux-xfs@vger.kernel.org \
--cc=sandeen@sandeen.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.