Hi,

> -----Original Message-----
> From: [email protected] [mailto:[email protected]]
> Sent: Wednesday, September 19, 2018 9:23 AM
> To: Pattan, Reshma <[email protected]>
> Cc: [email protected]; Bao-Long Tran <[email protected]>;
> [email protected]
> Subject: [PATCH] latency: clear mbuf timestamp after latency calculation
> 
> The timestamp of a mbuf should be cleared after that mbuf was used for
> latency calculation, otherwise future packets which reuse the same mbuf
> would inherit that previous timestamp. The latencystats library looks for
> mbuf with non-zero timestamp, thus incorrectly inherited value would result
> in incorrect latency measurement.
> 
> Cc: [email protected]
> 
> Signed-off-by: Bao-Long Tran <[email protected]>

You need to add the Fixes line just before CC:  in the commit message. 

Original commit that introduced the bug was  5cd3cac9ed. So fixes should  be  
added like below
Fixes: 5cd3cac9ed ("latency: added new library for latency stats").  

You can send v2 with fixes line and my ack. Other than that 

Acked-by: Reshma Pattan <[email protected]>

Reply via email to