Re: [PATCH v5 2/2] ethdev: support xstats reset telemetry command

2023-07-03 Thread fengchengwen
Hi Thomas and Morten, On 2023/7/3 21:44, Morten Brørup wrote: >> From: Thomas Monjalon [mailto:tho...@monjalon.net] >> Sent: Monday, 3 July 2023 09.20 >> >> 03/07/2023 05:58, fengchengwen: >>> >>> On 2023/2/20 21:05, Thomas Monjalon wrote: 17/02/2023 10:44, fengchengwen: > On 2023/2/16 20

RE: [PATCH v5 2/2] ethdev: support xstats reset telemetry command

2023-07-03 Thread Morten Brørup
> From: Thomas Monjalon [mailto:tho...@monjalon.net] > Sent: Monday, 3 July 2023 09.20 > > 03/07/2023 05:58, fengchengwen: > > > > On 2023/2/20 21:05, Thomas Monjalon wrote: > > > 17/02/2023 10:44, fengchengwen: > > >> On 2023/2/16 20:54, Bruce Richardson wrote: > > >>> On Thu, Feb 16, 2023 at 08:

Re: [PATCH v5 2/2] ethdev: support xstats reset telemetry command

2023-07-03 Thread Thomas Monjalon
03/07/2023 05:58, fengchengwen: > > On 2023/2/20 21:05, Thomas Monjalon wrote: > > 17/02/2023 10:44, fengchengwen: > >> On 2023/2/16 20:54, Bruce Richardson wrote: > >>> On Thu, Feb 16, 2023 at 08:42:34PM +0800, fengchengwen wrote: > On 2023/2/16 20:06, Ferruh Yigit wrote: > > On 2/16/202

Re: [PATCH v5 2/2] ethdev: support xstats reset telemetry command

2023-07-02 Thread fengchengwen
On 2023/2/20 21:05, Thomas Monjalon wrote: > 17/02/2023 10:44, fengchengwen: >> On 2023/2/16 20:54, Bruce Richardson wrote: >>> On Thu, Feb 16, 2023 at 08:42:34PM +0800, fengchengwen wrote: On 2023/2/16 20:06, Ferruh Yigit wrote: > On 2/16/2023 11:53 AM, fengchengwen wrote: >> On 20

Re: [PATCH v5 2/2] ethdev: support xstats reset telemetry command

2023-02-20 Thread Thomas Monjalon
17/02/2023 10:44, fengchengwen: > On 2023/2/16 20:54, Bruce Richardson wrote: > > On Thu, Feb 16, 2023 at 08:42:34PM +0800, fengchengwen wrote: > >> On 2023/2/16 20:06, Ferruh Yigit wrote: > >>> On 2/16/2023 11:53 AM, fengchengwen wrote: > On 2023/2/15 11:19, Dongdong Liu wrote: > > Hi Che

Re: [PATCH v5 2/2] ethdev: support xstats reset telemetry command

2023-02-17 Thread fengchengwen
On 2023/2/16 20:54, Bruce Richardson wrote: > On Thu, Feb 16, 2023 at 08:42:34PM +0800, fengchengwen wrote: >> On 2023/2/16 20:06, Ferruh Yigit wrote: >>> On 2/16/2023 11:53 AM, fengchengwen wrote: On 2023/2/15 11:19, Dongdong Liu wrote: > Hi Chengwen > > On 2023/2/9 10:32, Chengwe

Re: [PATCH v5 2/2] ethdev: support xstats reset telemetry command

2023-02-16 Thread Bruce Richardson
On Thu, Feb 16, 2023 at 12:54:20PM +, Bruce Richardson wrote: > On Thu, Feb 16, 2023 at 08:42:34PM +0800, fengchengwen wrote: > > On 2023/2/16 20:06, Ferruh Yigit wrote: > > > On 2/16/2023 11:53 AM, fengchengwen wrote: > > >> On 2023/2/15 11:19, Dongdong Liu wrote: > > >>> Hi Chengwen > > >>> >

Re: [PATCH v5 2/2] ethdev: support xstats reset telemetry command

2023-02-16 Thread Bruce Richardson
On Thu, Feb 16, 2023 at 08:42:34PM +0800, fengchengwen wrote: > On 2023/2/16 20:06, Ferruh Yigit wrote: > > On 2/16/2023 11:53 AM, fengchengwen wrote: > >> On 2023/2/15 11:19, Dongdong Liu wrote: > >>> Hi Chengwen > >>> > >>> On 2023/2/9 10:32, Chengwen Feng wrote: > The xstats reset is useful

Re: [PATCH v5 2/2] ethdev: support xstats reset telemetry command

2023-02-16 Thread fengchengwen
On 2023/2/16 20:06, Ferruh Yigit wrote: > On 2/16/2023 11:53 AM, fengchengwen wrote: >> On 2023/2/15 11:19, Dongdong Liu wrote: >>> Hi Chengwen >>> >>> On 2023/2/9 10:32, Chengwen Feng wrote: The xstats reset is useful for debugging, so add it to the ethdev telemetry command lists. >

Re: [PATCH v5 2/2] ethdev: support xstats reset telemetry command

2023-02-16 Thread Ferruh Yigit
On 2/16/2023 11:53 AM, fengchengwen wrote: > On 2023/2/15 11:19, Dongdong Liu wrote: >> Hi Chengwen >> >> On 2023/2/9 10:32, Chengwen Feng wrote: >>> The xstats reset is useful for debugging, so add it to the ethdev >>> telemetry command lists. >>> >>> Signed-off-by: Chengwen Feng >> This patch lo

Re: [PATCH v5 2/2] ethdev: support xstats reset telemetry command

2023-02-16 Thread fengchengwen
On 2023/2/15 11:19, Dongdong Liu wrote: > Hi Chengwen > > On 2023/2/9 10:32, Chengwen Feng wrote: >> The xstats reset is useful for debugging, so add it to the ethdev >> telemetry command lists. >> >> Signed-off-by: Chengwen Feng > This patch looks good, so > Reviewed-by: Dongdong Liu > > A min

Re: [PATCH v5 2/2] ethdev: support xstats reset telemetry command

2023-02-14 Thread Dongdong Liu
Hi Chengwen On 2023/2/9 10:32, Chengwen Feng wrote: The xstats reset is useful for debugging, so add it to the ethdev telemetry command lists. Signed-off-by: Chengwen Feng This patch looks good, so Reviewed-by: Dongdong Liu A minior question Do we need to support stats reset ? Thanks, Dong