From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752189AbcAFQs7 (ORCPT ); Wed, 6 Jan 2016 11:48:59 -0500 Received: from mx142.netapp.com ([216.240.21.19]:11932 "EHLO mx142.netapp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751341AbcAFQs5 (ORCPT ); Wed, 6 Jan 2016 11:48:57 -0500 X-IronPort-AV: E=Sophos;i="5.20,529,1444719600"; d="scan'208";a="86734303" Subject: Re: linux-next: manual merge of the rdma tree with the nfsd tree To: Chuck Lever , Or Gerlitz References: <20151231133022.0b3ecab9@canb.auug.org.au> <20160104124450.7ba21c87@canb.auug.org.au> <5689DEA0.1020803@redhat.com> <20160104193616.GB30315@fieldses.org> <24AC8112-95A2-4D7B-8048-3D7D50A78B90@oracle.com> <20160106121529.GA21780@lst.de> <90E07B04-AAFD-45E9-85EA-44C659191180@oracle.com> <568D28D2.4090507@mellanox.com> <3B1D726B-5225-40D5-8C94-CA87EF198D89@oracle.com> <568D3838.7010606@mellanox.com> <2118F38A-846F-4690-A6E8-0BA7A437D567@oracle.com> CC: Christoph Hellwig , "J. Bruce Fields" , Doug Ledford , Stephen Rothwell , , LKML Kernel , Ira Weiny From: Anna Schumaker Message-ID: <568D4571.7030201@Netapp.com> Date: Wed, 6 Jan 2016 11:48:49 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.0 MIME-Version: 1.0 In-Reply-To: <2118F38A-846F-4690-A6E8-0BA7A437D567@oracle.com> Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/06/2016 11:33 AM, Chuck Lever wrote: > >> On Jan 6, 2016, at 10:52 AM, Or Gerlitz wrote: >> >> On 1/6/2016 5:20 PM, Chuck Lever wrote: >>>> Chuck, >>>>> >>>>> Lets be concrete... anything wrong with patch [1]? >>> Yes. It is missing Acked-by: lines from the maintainers of >>> those files. >>> >>> All changes to files under net/sunrpc need an Ack from one >>> of the maintainers listed in MAINTAINERS for that directory, >>> if the changes are going through another maintainer's tree. >>> >>> I have been personally asked to remind folks that the >>> nfs-sunrpc maintainers do not read linux-rdma, so they >>> must be contacted directly (and cc: linux-nfs) as part of >>> proposing finished patches in that area. >> >> I did that!! >> >> I copied you and Anna on the patch [1]. > > The marc.info archive does not preserve To: or Cc: > fields on e-mail. I had to look in my own inbox to > confirm who the recipients were. > > Anna was the only NFS maintainer listed, and > linux-nfs was not included on that e-mail at all. > Did Anna ever reply with an Acked-by ? I don't see > that in either the archive or my own inbox. I'm not sure if I ever replied to that one. This patch was part of an alternative to Christoph's deviceattr cleanup, right? The NFS client bits looked okay to me, but I held off on replying to see which version was accepted. Feel free to add my: Acked-by: Anna Schumaker Anna > > Bruce needs to Ack the svc_rdma_transport.c hunks. > > >> Again, lets be concrete, this very small cleanup was picked and merged, anything there >> need to be fixed? > > I have rebased on Doug's current for-4.5, and will > be testing today before submitting my for-4.5 patches > again. Your changes are mechanical, so I don't expect > to encounter a technical problem. > > However, I would be more comfortable if the missing > Acks were fixed in Doug's tree (with Anna and Bruce's > permission, of course) before Linus pulls. > > >> Or. >> >> [1] http://marc.info/?l=linux-rdma&m=145042924110411&w=2 >> >> >>> Unfortunately I have not been able to review every patch >>> that has come by on linux-rdma in the past 9 months to >>> ensure the eyes are dotted and tees crossed. More than >>> a few commits in the tree are missing the proper tags. >>> >>> >>>> [1] commit e3e45b1 "xprtrdma: Avoid calling ib_query_device" >>>> >>>> in git://git.kernel.org/pub/scm/linux/kernel/git/dledford/rdma.git k.o/for-4.5 >>>> >>>> http://git.kernel.org/cgit/linux/kernel/git/dledford/rdma.git/commit/?h=k.o/for-4.5&id=e3e45b1b43988b99007a9908ca0ba738b3fbd0ff >>>> >> > > -- > Chuck Lever > > > >