- Subject: Re: [PATCH v3] introduce sys_syncfs to sync a single file system
- From: Jonathan Nieder <jrnieder@xxxxxxxxx>
- Date: Fri, 11 Mar 2011 20:10:01 -0600
- Cc: Arnd Bergmann <arnd@xxxxxxxx>, Sage Weil <sage@xxxxxxxxxxxx>, linux-fsdevel@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, "Aneesh Kumar K. V" <aneesh.kumar@xxxxxxxxxxxxxxxxxx>, akpm@xxxxxxxxxxxxxxxxxxxx, linux-api@xxxxxxxxxxxxxxx, mtk.manpages@xxxxxxxxx, viro@xxxxxxxxxxxxxxxxxx, hch@xxxxxx, l@xxxxxxxxx
- In-reply-to: <9446ab1a2315c0d2476c30f8315a0503.squirrel@xxxxxxxxxxxxxxxxxxxx>
- References: <Pine.LNX.4.64.1102171035220.13904@xxxxxxxxxxxxxxxxxx> <Pine.LNX.4.64.1103101125150.4190@xxxxxxxxxxxxxxxxxx> <edfa4cf081249734807e582c14253fca.squirrel@xxxxxxxxxxxxxxxxxxxx> <201103111255.44979.arnd@xxxxxxxx> <ffafd3ccb3fd1be7c40348c9cee84aef.squirrel@xxxxxxxxxxxxxxxxxxxx> <20110311235607.GB15853@elie> <9446ab1a2315c0d2476c30f8315a0503.squirrel@xxxxxxxxxxxxxxxxxxxx>
- User-agent: Mutt/1.5.21 (2010-09-15)
Indan Zupancic wrote:
> I'm not pushing for any official convention, just what seems good taste.
In cases like this, conventions (consistency and best practices) are
very important.
> Less code added, less bloat. Architecture independent, no need to update
> all system call tables everywhere (all archs, libc versions and strace).
> Two files changed, instead of 7 (which only hooks up x86).
Thanks for explaining. Those do seem like good reasons to use a ioctl
instead of a new syscall.
> In this case it's just a performance improvement over sync(2). It doesn't
> add a new feature. Main argument given for the performance problem seems
> to be "NFS can be slow". Anything else?
Huh? It is not just the speed of the sync --- unnecessary writeback
will cause wear on your thumbdrive, eat up your laptop battery, and
kill I/O performance in other tasks running at the same time.
I'm afraid I don't understand what you're saying here at all. Would
you say that fsync is superfluous, too?
Jonathan
--
To unsubscribe from this list: send the line "unsubscribe linux-api" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[Home]
[Linux USB Devel]
[Video for Linux]
[Linux Audio Users]
[Photo]
[Yosemite News]
[Yosemite Photos]
[Free Online Dating]
[Linux Kernel]
[Linux SCSI]
[XFree86]