Re: [PATCH 10/13] IB/srp: use the new CQ API

From: Doug Ledford
Date: Mon Dec 14 2015 - 11:26:16 EST


On 12/13/2015 05:26 AM, Sagi Grimberg wrote:
>
>> Allright. How do you want to proceed? The current rdma-cq branch
>> has all kinds of dependencies, but I've also prepared a new rdma-cq.2
>> branch that could go straight on top of your current queue:
>>
>> http://git.infradead.org/users/hch/rdma.git/shortlog/refs/heads/rdma-cq.2
>>
>> If you're ready to start the 4.5 tree I can send those out as a patch
>> series.
>
> Will this get on top of iser-remote-inv? Or should I resend atop of this?

I'm going through my inbox right now. I expect somewhere in there Or
will make his case for why he doesn't like Christoph's patch to get rid
of the attr struct. I'll listen, and if I'm not convinced, I'll take
that patchset first and this one second. (I reviewed the patchset
already....aside from the fact that I *like* having the attr struct
elements in an organized sub-struct, it's fine and it definitely
improves on all of those query calls).

--
Doug Ledford <dledford@xxxxxxxxxx>
GPG KeyID: 0E572FDD


Attachment: signature.asc
Description: OpenPGP digital signature