Hi,

> -----Original Message-----
> From: Maxime Coquelin <[email protected]>
> Sent: Thursday, September 16, 2021 7:34 PM
> To: Yigit, Ferruh <[email protected]>; Hu, Jiayu <[email protected]>;
> [email protected]
> Cc: Xia, Chenbo <[email protected]>; [email protected]; Jiang,
> Cheng1 <[email protected]>
> Subject: Re: [dpdk-dev] [PATCH] vhost: remove copy threshold for async vhost
> 
> Hi Ferruh,
> 
> On 9/16/21 13:15, Ferruh Yigit wrote:
> > On 8/20/2021 1:44 PM, Jiayu Hu wrote:
> >> Copy threshold is introduced in async vhost data path to select
> >> the appropriate copy engine to do copies for higher efficiency.
> >> However, it may cause packets out-of-order, and it also causes
> >> data path performance unpredictable.
> >>
> >
> > Just checking, if the patch is fixing data performance issue, should it be
> > backported?
> 
> This is experimental code that is not used in real product for now, and
> will likely heavily change given OVS rejected this design.
> 
> So I would personally not bother backporting patches for it.

+1. Based on the experimental state. Let's save the effort of backporting.

Thanks,
Chenbo

> Jiayu, Chenbo, do you think otherwise?
> 
> Regards,
> Maxime
> 
> >> Therefore, this patch removes copy threshold support in async vhost
> >> data path.
> >>
> >> Signed-off-by: Jiayu Hu <[email protected]>
> >> Signed-off-by: Cheng Jiang <[email protected]>
> > <...>
> >
> >

Reply via email to