From: Matthew Garrett <mjg59@srcf.ucam.org>
To: "Éric Piel" <E.A.B.Piel@tudelft.nl>
Cc: linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org
Subject: Re: [PATCH, resend] relatime: Let relatime update atime at least once per day
Date: Sun, 28 Dec 2008 21:38:52 +0000 [thread overview]
Message-ID: <20081228213852.GA20439@srcf.ucam.org> (raw)
In-Reply-To: <4957EEA7.3050101@tudelft.nl>
On Sun, Dec 28, 2008 at 10:24:55PM +0100, Éric Piel wrote:
> Of course, by construction, there is nothing relying on the current
> relatime semantics. The problem is that whenever you are making relatime
> closer to the atime behaviour, you are also getting closer to the
> original drawbacks of atime (one read generates one write).
If your io is sufficiently limited that one write per file per day is a
significant problem, then I think there's a more serious underlying
problem.
> Actually, you are changing relatime from a boolean condition (maximum
> one additional write per write) to a atime with a coarse grain (maximum
> one additional write per day). Today you found a use case that needs a
> precision of one day. Tomorrow, someone else will find a use case that
> needs a precision of one hour. So maybe what is actually needed is a
> third option, a "grainatime" option where you can change the precision
> of the atime.
The answer to "The current two options are suboptimal in almost all
cases" is very rarely "Add a new option". If boot is touching too many
files, it should stop touching as many files. If you have crap disks,
get better disks. If your life is spent tuning fs parameters for 1%
performance gains, find a better job.
--
Matthew Garrett | mjg59@srcf.ucam.org
next prev parent reply other threads:[~2008-12-28 21:39 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-12-28 15:29 [PATCH, resend] relatime: Let relatime update atime at least once per day Matthew Garrett
2008-12-28 18:35 ` Jesper Juhl
2008-12-28 20:19 ` Matthew Garrett
2008-12-28 19:04 ` Éric Piel
2008-12-28 19:59 ` Matthew Garrett
2008-12-28 21:24 ` Éric Piel
2008-12-28 21:36 ` Matthew Wilcox
2008-12-28 21:38 ` Matthew Garrett [this message]
2009-01-08 12:29 ` Peter Moulder
2009-01-08 13:32 ` Matthew Garrett
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=20081228213852.GA20439@srcf.ucam.org \
--to=mjg59@srcf.ucam.org \
--cc=E.A.B.Piel@tudelft.nl \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox