From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leon Romanovsky Subject: Re: [PULL REQUEST] Please pull rdma.git Date: Fri, 18 Mar 2016 19:37:48 +0200 Message-ID: <20160318173748.GL25216@leon.nu> References: <56EAE9D6.2030908@redhat.com> Reply-To: leon-2ukJVAZIZ/Y@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Linus Torvalds Cc: Doug Ledford , RDMA mailing list , Matan Barak , Or Gerlitz , Leon Romanovsky List-Id: linux-rdma@vger.kernel.org On Fri, Mar 18, 2016 at 09:52:16AM -0700, Linus Torvalds wrote: > On Thu, Mar 17, 2016 at 10:31 AM, Doug Ledford wrote: > > > > There is the one merge issue between the net tree and this tree that was > > simple and obvious to fix up. I've had a phone call with Mellanox about > > how to avoid this in the future (the result of that conversation was > > that when they have a pre-req patch that is going through Dave's tree, > > but the dependent patches are going through my tree, then they should > > submit the pre-req patch to me as well, we can carry identical patches > > in both trees, and when you merge the duplicate patches will just drop > > one and then there will be no merge issue). > > That will end up causing just more problems if there are then other > patches on top in either tree. > > It would be much better to have the shared pre-requisite not as a > _patch_, but as a separate branch. So that you actually share the > commits, not just the diff. > > Other subsystems have been able to do that. Linus, I'm not sure that my understanding is correct and will be glad if you can point us to the examples of such branches, so we will use it as soon as possible. Thanks. > > > drivers/infiniband/hw/mlx5/srq.c | 41 +- > .. > > 91 files changed, 3647 insertions(+), 1995 deletions(-) > > create mode 100644 drivers/infiniband/hw/mlx5/gsi.c > > My diffstat doesn't match yours, and it's that "srq.c" file in > particular that I have no changes to, and see no changes in your pull > request. > > Other than that it looks normal, but it makes me wonder how that diff > was generated. > > Linus > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html