> -----Original Message----- > From: Max Gurtovoy <maxg@xxxxxxxxxxxx> > Sent: Monday, June 4, 2018 8:52 AM > To: Christoph Hellwig <hch@xxxxxx>; Sagi Grimberg <sagi@xxxxxxxxxxx> > Cc: Steve Wise <swise@xxxxxxxxxxxxxxxxxxxxx>; 'Ruhl, Michael J' > <michael.j.ruhl@xxxxxxxxx>; axboe@xxxxxxxxx; 'Busch, Keith' > <keith.busch@xxxxxxxxx>; linux-nvme@xxxxxxxxxxxxxxxxxxx; > parav@xxxxxxxxxxxx; linux-rdma@xxxxxxxxxxxxxxx > Subject: Re: [PATCH v3 1/3] nvme-rdma: correctly check for target keyed sgl > support > > > > On 6/4/2018 3:11 PM, Christoph Hellwig wrote: > > On Mon, Jun 04, 2018 at 03:01:43PM +0300, Sagi Grimberg wrote: > >> > >>> He's referring to patch 1 and 2, which are the host side. No page > allocations. > >> > >> I'm good with 1 & 2, > >> > >> Christoph, you can add my > >> > >> Reviewed-by: Sagi Grimberg <sagi@xxxxxxxxxxx> > > > > We've missed the merge window now, so we can just wait for a proper > > resend from Steve I think. > > > > There are still issue that I'm trying to help Steve with their debug so > let's wait with the merge until we figure them out. I would like review on my new nvmet-rdma changes to avoid > 0 order page allocations though. Perhaps I'll resend the series and add the RFC tag (or WIP?) with verbiage saying don't merge yet. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html