Re: [PATCH] ice/base: fix gcc 12 warning stringop-overflow

2022-06-23 Thread David Marchand
On Mon, Jun 20, 2022 at 7:06 AM Wu, WenxuanX wrote: > > > > diff --git a/drivers/net/ice/base/ice_protocol_type.h > > > > b/drivers/net/ice/base/ice_protocol_type.h > > > > index 0e6e5990be..cfe3b62630 100644 > > > > --- a/drivers/net/ice/base/ice_protocol_type.h > > > > +++ b/drivers/net/ice/base

RE: [PATCH] ice/base: fix gcc 12 warning stringop-overflow

2022-06-19 Thread Wu, WenxuanX
> -Original Message- > From: Wu, WenxuanX > Sent: 2022年6月20日 9:39 > To: Zhang, Qi Z ; Yang, Qiming > ; dev@dpdk.org; tho...@monjalon.net > Cc: step...@networkplumber.org; sta...@dpdk.org > Subject: RE: [PATCH] ice/base: fix gcc 12 warning stringop-overflow > &g

RE: [PATCH] ice/base: fix gcc 12 warning stringop-overflow

2022-06-19 Thread Wu, WenxuanX
- > From: Zhang, Qi Z > Sent: 2022年6月19日 20:35 > To: Wu, WenxuanX ; Yang, Qiming > ; dev@dpdk.org; tho...@monjalon.net > Cc: step...@networkplumber.org; sta...@dpdk.org > Subject: RE: [PATCH] ice/base: fix gcc 12 warning stringop-overflow > > > > > -Original

RE: [PATCH] ice/base: fix gcc 12 warning stringop-overflow

2022-06-19 Thread Zhang, Qi Z
> -Original Message- > From: Wu, WenxuanX > Sent: Thursday, June 16, 2022 6:33 PM > To: Yang, Qiming ; Zhang, Qi Z > ; dev@dpdk.org; tho...@monjalon.net > Cc: step...@networkplumber.org; Wu, WenxuanX ; > sta...@dpdk.org > Subject: [PATCH] ice/base: fix gcc 12 w

Re: [PATCH] ice/base: fix gcc 12 warning stringop-overflow

2022-06-16 Thread David Marchand
On Fri, Jun 17, 2022 at 3:29 AM Wu, WenxuanX wrote: > > Thanks, do I need to send a new version to resolve formatting issues? You are not a new contributor, please pay attention to such details. To answer your question, I don't think there is a need for a new version unless someone has a better

RE: [PATCH] ice/base: fix gcc 12 warning stringop-overflow

2022-06-16 Thread Wu, WenxuanX
bject: Re: [PATCH] ice/base: fix gcc 12 warning stringop-overflow > > Title should start with net/ice/base:. > > On Thu, Jun 16, 2022 at 12:35 PM wrote: > > > > From: Wenxuan Wu > > > > Gcc with -O2 flag, would retrieve the value in one time. > >

Re: [PATCH] ice/base: fix gcc 12 warning stringop-overflow

2022-06-16 Thread David Marchand
Title should start with net/ice/base:. On Thu, Jun 16, 2022 at 12:35 PM wrote: > > From: Wenxuan Wu > > Gcc with -O2 flag, would retrieve the value in one time. > This patch changed the type of fv_idx in struct ice_recp_grp_entry to > align with its callers which is also u8 type. > > When u8 idx

[PATCH] ice/base: fix gcc 12 warning stringop-overflow

2022-06-16 Thread wenxuanx . wu
From: Wenxuan Wu Gcc with -O2 flag, would retrieve the value in one time. This patch changed the type of fv_idx in struct ice_recp_grp_entry to align with its callers which is also u8 type. When u8 idx[5] = a value u16 index[4], gcc12 would give this warning, because it is not big enough to stor