> > > Subject: [PATCH v3 0/2] Vector support for event crypto adapter
> > >
> > > Introduce ability to aggregate crypto operations processed by event
> > > crypto adapter into single event containing rte_event_vector whose
> > > event type is RTE_EVENT_TYPE_CRYPTODEV_VECTOR.
> > >
> > > * Changes since v1
> > > - Add missing newline/space issues
> > > - Add missing error messages
> > > - Remove double check of conf input parameter
> > >
> > > * Changes since v2
> > > - Rebase on top of dpdk-next-eventdev branch
> > > - Remove already merged patch
> > >
> >
> > Series Acked-by: Akhil Goyal <gak...@marvell.com>
> >
> > Since there is no response and we are approaching RC1 deadline patches are
> > applied to dpdk-next-crypto  with below changes.
> Hi @Akhil Goyal,
> 
> Did not we discuss in the call, and I committed to acked the patch by 
> Saturday?
> You cant call that as no response!

I mentioned in the call that we have RC1 deadline, ack/nack shall be provided 
as soon as possible.
It was agreed that it will come by Friday. I can not spend all of my weekend on 
a particular series.
There are other series also.
The patches have been there for about 2 months(V1 was shared on August 4).
As per DPDK guidelines, I could merge a patch if there is no response for more 
than 3 weeks even without asking anybody.
I asked out to you just to be courteous on Sep 22. And even after that your 
responses were pretty slow.
You should give ample time for rework to happen before the deadline.

>From next time onwards, I will not ask for review. You should be proactive 
>within 3 weeks.
> >
> > Updated release notes and removed deprecation notice.
> > Added a build fix because of removal of a macro
> RTE_FUNC_PTR_OR_ERR_RET.
> >
> > Thanks.

Reply via email to