[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20070301145256.3e999932.akpm@linux-foundation.org>
Date: Thu, 1 Mar 2007 14:52:56 -0800
From: Andrew Morton <akpm@...ux-foundation.org>
To: nscott@...nex.com
Cc: "Amit K. Arora" <aarora@...ux.vnet.ibm.com>,
linux-fsdevel@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-ext4@...r.kernel.org, suparna@...ibm.com, cmm@...ibm.com,
alex@...sterfs.com, suzuki@...ibm.com,
Ulrich Drepper <drepper@...hat.com>
Subject: Re: [RFC] Heads up on sys_fallocate()
On Fri, 02 Mar 2007 09:40:54 +1100
Nathan Scott <nscott@...nex.com> wrote:
> On Thu, 2007-03-01 at 14:25 -0800, Andrew Morton wrote:
> > On Fri, 2 Mar 2007 00:04:45 +0530
> > "Amit K. Arora" <aarora@...ux.vnet.ibm.com> wrote:
> >
> > > This is to give a heads up on few patches that we will be soon coming up
> > > with. These patches implement a new system call sys_fallocate() and a
> > > new inode operation "fallocate", for persistent preallocation. The new
> > > system call, as Andrew suggested, will look like:
> > >
> > > asmlinkage long sys_fallocate(int fd, loff_t offset, loff_t len);
> > ...
> >
> > I'd agree with Eric on the "command" flag extension.
>
> Seems like a separate syscall would be better, "command" sounds
> a bit ioctl like, especially if that command is passed into the
> filesystems..
>
madvise, fadvise, lseek, etc seem to work OK.
I get repeatedly traumatised by patch rejects whenever a new syscall gets
added, so I'm biased.
The advantage of a command flag is that we can add new modes in the future
without causing lots of churn, waiting for arch maintainers to catch up,
potentially adding new compat code, etc.
Rename it to "mode"? ;)
I'm inclined to merge this patch nice and early, so the syscall number is
stabilised. Otherwise the people who are working on out-of-tree code (ie:
ext4) will have to keep playing catchup.
-
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists