On Thu, Jun 11, 2015 at 8:55 PM, Jason Gunthorpe
<[email protected]> wrote:
> On Thu, Jun 11, 2015 at 04:35:20PM +0300, Or Gerlitz wrote:
>> From: Matan Barak <[email protected]>
>>
>> Add a new ib_cq_init_attr structure which contains the
>> previous cqe (minimum number of CQ entries) and comp_vector
>> (completion vector) in addition to a new flags field.
>> All vendors' create_cq callbacks are changed in order
>> to work with the new API.
>>
>> This commit does not change any functionality.
>>
>> Signed-off-by: Matan Barak <[email protected]>
>> Signed-off-by: Or Gerlitz <[email protected]>
>
> You need to collect any Reviwed-By's etc people provide to earlier
> series, and include them when spining a new revision.  For instance

Right, I agree, but our official WW here has ended and the merge
window is likely to open Monday, I hope this time Doug can do that and
pick the patches today so they can pass linux-next built tests and
such in his tree

> http://permalink.gmane.org/gmane.linux.drivers.rdma/26129
>
> I think I had provided a Reviewed-By for some of these patches too.

You only provided your reviewed by to patch #1 and said once we do
the last fix to #2 it can go there too.

> It isn't reasonable to expect Doug to do that.

next time I'll make sure not to forget it, so

Add Jason's Reviewed-By to patches #1/#2

Reviewed-By: Devesh Sharma <[email protected]> to patch #2
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to