linux-btrfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Duncan <1i5t5.duncan@cox.net>
To: linux-btrfs@vger.kernel.org
Subject: Re: What to do about snapshot-aware defrag
Date: Sun, 1 Jun 2014 01:52:08 +0000 (UTC)	[thread overview]
Message-ID: <pan$86a1$bc0de339$ebe33469$156d19e6@cox.net> (raw)
In-Reply-To: 538A6B15.3080609@swiftspirit.co.za

Brendan Hide posted on Sun, 01 Jun 2014 01:51:49 +0200 as excerpted:

>> Will option one mean that we always need to mount with noatime or
>> read-only to allow snapshot defragging to do anything?
> 
> That is a very good question. I very rarely have mounts without noatime
> - and usually only because I hadn't thought of it.

Heh, a couple months ago I got tired of having to add noatime to all my 
standard mounts, and decided to patch my kernel to noatime (instead of 
relatime) by default.  I can if I need to still use relatime or 
strictatime, but if they aren't listed, noatime gets added by default, 
now. =:^)

The only remaining problem is that it's not a full default, as noatime 
still shows up in /proc/self/mounts, but avoiding that would have 
complexified the patch, and not being an actual coder (and DEFINITELY not 
a kernel coder), I decided I best leave well enough alone, patching only 
what I had to to avoid having relatime if I omitted the parameter, and 
only what I was reasonably sure I could do without screwing things up, 
based on my limited sysadmin level reading of the sources.

It would sure be nice if there were a kernel config option for that 
default, which would naturally still default to relatime, which is I'm 
sure where most of the distros not already patching it to noatime would 
leave it, as well.

Meanwhile, agreed, a good question that does indeed remain! =:^)

-- 
Duncan - List replies preferred.   No HTML msgs.
"Every nonfree program has a lord, a master --
and if you use the program, he is your master."  Richard Stallman


  reply	other threads:[~2014-06-01  1:52 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-30 20:43 What to do about snapshot-aware defrag Josef Bacik
2014-05-30 22:00 ` Martin
2014-05-31 23:51   ` Brendan Hide
2014-06-01  1:52     ` Duncan [this message]
2014-06-02  3:07     ` Mitch Harder
2014-06-02 13:19       ` Josef Bacik
2014-06-02 13:22   ` Josef Bacik
2014-06-03 23:54     ` Martin
2014-06-04  9:19       ` Erkki Seppala
2014-06-04 13:15         ` Martin
2014-06-04 19:33           ` Chris Murphy

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='pan$86a1$bc0de339$ebe33469$156d19e6@cox.net' \
    --to=1i5t5.duncan@cox.net \
    --cc=linux-btrfs@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;
as well as URLs for NNTP newsgroup(s).