Re: [PATCH] IB/core: temporarily disable create_flow/destroy_flow uverbs

[Yann Droneaud <ydroneaud-RlY5vtjFyJ3QT0dZR+AlfA@public.gmane.org>]

Hi,

Le 10.10.2013 11:30, Or Gerlitz a écrit :
> On 10/10/2013 12:10, Yann Droneaud wrote:
>> create_flow/destroy_flow uverbs are under review and should
>> be considered too experimental at this point.
>> 
>> To not expose to userspace such experimental feature,
>> the data structure and code is temporarily disabled for v3.12.
>> 
>> The feature will be enabled after proper cleanup for v3.13.
> 
> NAK
> 
> The patches were on the list from April and went through review of
> Sean, Jason, Roland, Shawn and others.
> 
> You started your review few months after the point in time we
> submitted them, and after they were accepted.
> 
> We were very happy **and**  responsive for your feedback -- Matan sent
> patch on Sep 22nd to fix few issues you pointed on.
> 

I'm also very happy and looking forward to continue working with you
so I'm very sorry to provide the reviews so late (back in April I became
father for the second time).

I'm also very sorry bring such patch now but I feel I won't be able to
find the needed time to do the patchset rework before the deadline.
I will try to do this before the end of the week (12th october) but no 
promise.

> If you have more point fixes, we're **before** rc5 and 2-3 patches
> that fix things can get it, Matan review your prev
> patchset which contained four patches and acked three of them.
> 
> So you now need to re-prepare a smaller patch set advocate for Roland
> to pick it up.
> 

I'm also afraid that the whole patchset with extended command + public 
API fix
would be too big for v3.12-rc5 ... v3.12.

Regards.

-- 
Yann Droneaud
OPTEYA

--
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