From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753764AbcESCCb (ORCPT ); Wed, 18 May 2016 22:02:31 -0400 Received: from mail-io0-f194.google.com ([209.85.223.194]:35534 "EHLO mail-io0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751660AbcESCC3 (ORCPT ); Wed, 18 May 2016 22:02:29 -0400 MIME-Version: 1.0 In-Reply-To: <20160519111702.752921f8@canb.auug.org.au> References: <20160519111702.752921f8@canb.auug.org.au> From: Steve French Date: Wed, 18 May 2016 21:02:09 -0500 Message-ID: Subject: Re: linux-next: manual merge of the vfs tree with Linus' tree To: Stephen Rothwell Cc: Al Viro , linux-next@vger.kernel.org, LKML , Andreas Gruenbacher , "linux-cifs@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org As noted below, it is easier to remove the obsolete comments and I just pushed the following trivial patch to cifs-2.6.git for-next to do that. https://git.samba.org/?p=sfrench/cifs-2.6.git;a=commit;h=fea17ae8ac1c8c44b2fd1c02ae2e15b847d327d1 On Wed, May 18, 2016 at 8:17 PM, Stephen Rothwell wrote: > Hi Al, > > Today's linux-next merge of the vfs tree got a conflict in: > > fs/cifs/cifsfs.c > > between commit: > > a9ae008f407b ("cifs: Switch to generic xattr handlers") > > from Linus' tree and commit: > > 51085a1f913a ("cifs: use C99 syntax for inode_operations initializer") > > from the vfs tree. > > I fixed it up (see below) and can carry the fix as necessary. This > is now fixed as far as linux-next is concerned, but any non trivial > conflicts should be mentioned to your upstream maintainer when your tree > is submitted for merging. You may also want to consider cooperating > with the maintainer of the conflicting tree to minimise any particularly > complex conflicts. > > Al: the c99 fixup is not really necessary as the line is commented out > and you didn't fix the one in the line above ... a better fix might be > to delete both those lines? > -- > Cheers, > Stephen Rothwell > > diff --cc fs/cifs/cifsfs.c > index 67f622df0858,586d4eadd49e..000000000000 > --- a/fs/cifs/cifsfs.c > +++ b/fs/cifs/cifsfs.c > @@@ -917,11 -919,13 +917,11 @@@ const struct inode_operations cifs_syml > .permission = cifs_permission, > /* BB add the following two eventually */ > /* revalidate: cifs_revalidate, > - setattr: cifs_notify_change, *//* BB do we need notify change */ > + .setattr = cifs_notify_change, *//* BB do we need notify change */ > -#ifdef CONFIG_CIFS_XATTR > - .setxattr = cifs_setxattr, > - .getxattr = cifs_getxattr, > + .setxattr = generic_setxattr, > + .getxattr = generic_getxattr, > .listxattr = cifs_listxattr, > - .removexattr = cifs_removexattr, > -#endif > + .removexattr = generic_removexattr, > }; > > static int cifs_clone_file_range(struct file *src_file, loff_t off, -- Thanks, Steve