From: "Steve Wise" <swise@opengridcomputing.com>
To: "'Christoph Hellwig'" <hch@infradead.org>
Cc: "'Sagi Grimberg'" <sagig@mellanox.com>,
"'Doug Ledford'" <dledford@redhat.com>,
<linux-rdma@vger.kernel.org>, <linux-nfs@vger.kernel.org>,
<target-devel@vger.kernel.org>
Subject: RE: [PATCH for-4.3 11/15] iw_cxgb4: Support ib_alloc_mr verb
Date: Fri, 7 Aug 2015 11:19:59 -0500 [thread overview]
Message-ID: <003a01d0d12c$e84b4e80$b8e1eb80$@opengridcomputing.com> (raw)
In-Reply-To: <20150807151318.GA13777@infradead.org>
> -----Original Message-----
> From: linux-rdma-owner@vger.kernel.org [mailto:linux-rdma-owner@vger.kernel.org] On Behalf Of Christoph Hellwig
> Sent: Friday, August 07, 2015 10:13 AM
> To: Steve Wise
> Cc: 'Sagi Grimberg'; 'Doug Ledford'; linux-rdma@vger.kernel.org; linux-nfs@vger.kernel.org; target-devel@vger.kernel.org
> Subject: Re: [PATCH for-4.3 11/15] iw_cxgb4: Support ib_alloc_mr verb
>
> On Fri, Aug 07, 2015 at 10:06:26AM -0500, Steve Wise wrote:
> > If it is too much of a pain to alter this patch, then I'll just
> > submit the NFSRDMA fix and live with the bisect issue...
>
> Doug's tree is still to be rebased. So please submit your NFS
> fix now as ask Doug to merge it before Sagi's series in the final
> tree.
My new NFS fix needs to land before these two:
af78181 cxgb3: Support ib_alloc_mr verb
b7e06cd iw_cxgb4: Support ib_alloc_mr verb
But it will cause the following patch, which is after the above two, to need rework because it hits the same lines:
e20684a xprtrdma, svcrdma: Convert to ib_alloc_mr
I guess I'll post two patches, the NFS fix that preceeds af78181/ b7e06cd, and a reworked patch to replace e20684a.
Is that the way to go in your opinion?
Steve.
WARNING: multiple messages have this Message-ID (diff)
From: "Steve Wise" <swise-7bPotxP6k4+P2YhJcF5u+vpXobYPEAuW@public.gmane.org>
To: 'Christoph Hellwig' <hch-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>
Cc: 'Sagi Grimberg' <sagig-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
'Doug Ledford' <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
target-devel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: RE: [PATCH for-4.3 11/15] iw_cxgb4: Support ib_alloc_mr verb
Date: Fri, 7 Aug 2015 11:19:59 -0500 [thread overview]
Message-ID: <003a01d0d12c$e84b4e80$b8e1eb80$@opengridcomputing.com> (raw)
In-Reply-To: <20150807151318.GA13777-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>
> -----Original Message-----
> From: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org [mailto:linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org] On Behalf Of Christoph Hellwig
> Sent: Friday, August 07, 2015 10:13 AM
> To: Steve Wise
> Cc: 'Sagi Grimberg'; 'Doug Ledford'; linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; target-devel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> Subject: Re: [PATCH for-4.3 11/15] iw_cxgb4: Support ib_alloc_mr verb
>
> On Fri, Aug 07, 2015 at 10:06:26AM -0500, Steve Wise wrote:
> > If it is too much of a pain to alter this patch, then I'll just
> > submit the NFSRDMA fix and live with the bisect issue...
>
> Doug's tree is still to be rebased. So please submit your NFS
> fix now as ask Doug to merge it before Sagi's series in the final
> tree.
My new NFS fix needs to land before these two:
af78181 cxgb3: Support ib_alloc_mr verb
b7e06cd iw_cxgb4: Support ib_alloc_mr verb
But it will cause the following patch, which is after the above two, to need rework because it hits the same lines:
e20684a xprtrdma, svcrdma: Convert to ib_alloc_mr
I guess I'll post two patches, the NFS fix that preceeds af78181/ b7e06cd, and a reworked patch to replace e20684a.
Is that the way to go in your opinion?
Steve.
--
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
next prev parent reply other threads:[~2015-08-07 16:19 UTC|newest]
Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-30 7:32 [PATCH for-4.3 00/15] Modify MR allocation API Sagi Grimberg
2015-07-30 7:32 ` Sagi Grimberg
2015-07-30 7:32 ` [PATCH for-4.3 01/15] IB/core: Get rid of redundant verb ib_destroy_mr Sagi Grimberg
2015-07-30 7:32 ` Sagi Grimberg
2015-07-30 7:32 ` [PATCH for-4.3 02/15] IB: Modify ib_create_mr API Sagi Grimberg
2015-07-30 14:50 ` Steve Wise
2015-07-30 14:50 ` Steve Wise
2015-07-30 7:32 ` [PATCH for-4.3 03/15] IB/iser: Convert to ib_alloc_mr Sagi Grimberg
2015-07-30 7:32 ` Sagi Grimberg
2015-07-30 7:32 ` [PATCH for-4.3 04/15] iser-target: " Sagi Grimberg
2015-07-30 7:32 ` [PATCH for-4.3 05/15] IB/srp: " Sagi Grimberg
2015-07-31 22:41 ` Bart Van Assche
2015-07-31 22:41 ` Bart Van Assche
2015-07-30 7:32 ` [PATCH for-4.3 06/15] xprtrdma, svcrdma: " Sagi Grimberg
2015-07-30 7:32 ` Sagi Grimberg
2015-08-06 14:22 ` Anna Schumaker
2015-08-06 14:22 ` Anna Schumaker
2015-07-30 7:32 ` [PATCH for-4.3 07/15] RDS: " Sagi Grimberg
2015-07-30 7:32 ` [PATCH for-4.3 08/15] mlx5: Drop mlx5_ib_alloc_fast_reg_mr Sagi Grimberg
2015-07-30 7:32 ` Sagi Grimberg
2015-07-30 7:32 ` [PATCH for-4.3 09/15] mlx4: Support ib_alloc_mr verb Sagi Grimberg
2015-07-30 7:32 ` [PATCH for-4.3 10/15] ocrdma: " Sagi Grimberg
2015-07-30 7:32 ` [PATCH for-4.3 11/15] iw_cxgb4: " Sagi Grimberg
2015-08-07 15:06 ` Steve Wise
2015-08-07 15:06 ` Steve Wise
2015-08-07 15:13 ` Christoph Hellwig
2015-08-07 15:13 ` Christoph Hellwig
2015-08-07 16:19 ` Steve Wise [this message]
2015-08-07 16:19 ` Steve Wise
2015-08-07 16:26 ` Steve Wise
2015-08-07 16:26 ` 'Christoph Hellwig'
2015-08-07 16:26 ` 'Christoph Hellwig'
2015-08-07 16:29 ` Steve Wise
2015-08-07 16:29 ` Steve Wise
2015-08-07 16:31 ` 'Christoph Hellwig'
2015-08-07 16:31 ` 'Christoph Hellwig'
2015-07-30 7:32 ` [PATCH for-4.3 12/15] cxgb3: " Sagi Grimberg
2015-07-30 7:32 ` [PATCH for-4.3 13/15] nes: " Sagi Grimberg
2015-07-30 7:32 ` [PATCH for-4.3 14/15] qib: " Sagi Grimberg
2015-07-30 7:32 ` Sagi Grimberg
2015-07-31 11:41 ` Marciniszyn, Mike
2015-07-31 11:41 ` Marciniszyn, Mike
2015-07-31 12:52 ` Doug Ledford
2015-07-31 12:52 ` Doug Ledford
2015-08-07 14:47 ` Marciniszyn, Mike
2015-07-30 7:32 ` [PATCH for-4.3 15/15] IB/core: Drop ib_alloc_fast_reg_mr Sagi Grimberg
2015-07-30 13:53 ` [PATCH for-4.3 00/15] Modify MR allocation API Christoph Hellwig
2015-07-30 13:53 ` Christoph Hellwig
2015-07-30 14:56 ` Steve Wise
2015-07-30 16:42 ` Doug Ledford
2015-07-30 16:42 ` Doug Ledford
2015-07-30 17:22 ` Jason Gunthorpe
2015-07-30 17:22 ` Jason Gunthorpe
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='003a01d0d12c$e84b4e80$b8e1eb80$@opengridcomputing.com' \
--to=swise@opengridcomputing.com \
--cc=dledford@redhat.com \
--cc=hch@infradead.org \
--cc=linux-nfs@vger.kernel.org \
--cc=linux-rdma@vger.kernel.org \
--cc=sagig@mellanox.com \
--cc=target-devel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.