> -----Original Message-----
> From: Thomas Monjalon <[email protected]>
> Sent: Wednesday, March 24, 2021 1:16 PM
> To: Kathleen Capella <[email protected]>
> Cc: [email protected]; nd <[email protected]>; Honnappa Nagarahalli
> <[email protected]>
> Subject: Re: [dpdk-dev] [PATCH] examples/l3fwd: fix TX burst queue drain edge
> case
> 
> 23/02/2021 19:23, Kathleen Capella:
> > Initialize prev_tsc to cur_tsc. This avoids running the TX queue drain
> > in the first iteration of the packet processing loop.
> 
> Is it really a fix? What was broken?
> Isn't it an optimization?

It could be considered an optimization since effectively it prevents entering 
the if statement on the
first iteration of the loop. I will remove the "fixes" line and reword the 
subject line.
> 
> > Fixes: af75078fece3 ("first public release")
> 
> If it's really a fix, we should probably backport it, so
> Cc: [email protected]

[Kathleen Capella] 
It is not really a significant problem to be fixed in previous releases so it 
does not need to be backported.
> 
> > Signed-off-by: Kathleen Capella <[email protected]>
> > Reviewed-by: Honnappa Nagarahalli <[email protected]>
> 
> 

Reply via email to