xfs
[Top] [All Lists]

Re: [REVIEW] Don't make lazy counters default for mkfs

To: "Josef 'Jeff' Sipek" <jeffpc@xxxxxxxxxxxxxx>
Subject: Re: [REVIEW] Don't make lazy counters default for mkfs
From: Eric Sandeen <sandeen@xxxxxxxxxxx>
Date: Sun, 02 Mar 2008 22:19:16 -0600
Cc: Mark Goodwin <markgw@xxxxxxx>, Timothy Shimmin <tes@xxxxxxx>, nscott@xxxxxxxxxx, Russell Cattelan <cattelan@xxxxxxxxxxx>, Barry Naujok <bnaujok@xxxxxxx>, "xfs@xxxxxxxxxxx" <xfs@xxxxxxxxxxx>
In-reply-to: <20080303041409.GC13879@xxxxxxxxxxxxxx>
References: <op.t67mtawg3jf8g2@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <1204166101.13569.102.camel@xxxxxxxxxxxxxxxxx> <47C87775.2010007@xxxxxxxxxxx> <47C89137.3070805@xxxxxxxxxxx> <47C89303.7070902@xxxxxxxxxxx> <1204500895.10190.3.camel@xxxxxxxxxxxxxxxxx> <47CB434B.4040005@xxxxxxx> <47CB4696.1030304@xxxxxxx> <20080303011559.GB13879@xxxxxxxxxxxxxx> <47CB7702.5080905@xxxxxxxxxxx> <20080303041409.GC13879@xxxxxxxxxxxxxx>
Sender: xfs-bounce@xxxxxxxxxxx
User-agent: Thunderbird 2.0.0.12 (Macintosh/20080213)
Josef 'Jeff' Sipek wrote:
> On Sun, Mar 02, 2008 at 09:56:50PM -0600, Eric Sandeen wrote:
>> Josef 'Jeff' Sipek wrote:
>>> On Mon, Mar 03, 2008 at 11:30:14AM +1100, Mark Goodwin wrote:
>>> ...
>>>> Maybe I'm missing something, but if we export all the feature bits,
>>>> both new and old, then (a) an old mkfs will continue to ignore them,
>>>> and (b) future versions of mkfs will have all the information needed,
>>>> but will need t be smart about how that information is used.
>>> IMHO:
>>>
>>> 1) mkfs should make a filesystem, the defaults should be conservative (say
>>>    using features that have been around >1 year)
>> I suppose I have to agree, unfortunately that means most competetive
>> benchmarks will be using sub-optimal mkfs's, but...
>  
> Benchmarks that use default mkfs options on xfs, but non-default on other
> fs?

most benchmarks I see tune the heck out of "the home team" and leave the
rest ;)

> If you want, have a simple printf in mkfs that tells the user that he's not
> using the latest and greatest features (e.g., lazy-count); that should be
> enough to make it obvious that there're better options than the default.

eh, nobody reads that stuff :)

-Eric


<Prev in Thread] Current Thread [Next in Thread>