lkml.org 
[lkml]   [2013]   [Nov]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 04/11] vfs: add renameat2 syscall
From
On Thu, Nov 21, 2013 at 5:10 AM, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> Hi all,
>
> On Wed, 20 Nov 2013 14:01:45 +0100 Miklos Szeredi <miklos@szeredi.hu> wrote:
>>
>> From: Miklos Szeredi <mszeredi@suse.cz>
>>
>> Add new renameat2 syscall, which is the same as renameat with an added
>> flags argument.
>>
>> Pass flags to vfs_rename() and to i_op->rename() as well.
>>
>> All filesystems check flags and return -EOPNOTSUPP for unsupported flags.
>
> Can we please consider doing this slightly slower (and avoiding
> conflicts/missed new uses etc) by creating a new i_op callback and only
> implementing this when necessary?
>
> From past experience, I can just about guarantee that the patch as it is
> will cause conflicts (for me in linux-next if noone else) and a new use
> case will turn up before (or during) the next merge window.

I understand, but it makes little sense to have two i_op's with almost
exactly the same functionality, so at some point they are bound to be
merged, resulting in the same pain.

So Linus, how about applying the patches 1-4 at the end of this merge
window? These are simple with no functional changes. And then the
meat of the patchset can go into the next merge window without fear of
conflicts.

Thanks,
Miklos


\
 
 \ /
  Last update: 2013-11-21 14:21    [W:0.129 / U:0.024 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site