Re: [PATCH] net/ice: fix overflow on storing device args

[Bruce Richardson <bruce.richardson@intel.com>]

On Fri, Sep 25, 2026 at 11:36:51AM +0200, Burakov, Anatoly wrote:
> On 9/14/2026 5:30 PM, Bruce Richardson wrote:
> > Boolean device args to ice driver are sometimes stored as an int and
> > sometimes as a uint8_t. The parse bool function stores them as an int,
> > leading to overflow when storing a uint8_t value.
> > 
> > Fix all these by standardizing on "bool" type, both for the arg values
> > in the structure and inside the parsing function.
> > 
> > Bugzilla ID: 2037
> > Fixes: 603beeb970b5 ("net/ice: add safe mode devarg")
> > Cc: stable@dpdk.org
> > 
> > Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> > ---
> > Note: this fix is superceded by the kvargs numeric args patchset [1].
> > However, that patchset relies on new kvargs features unlikely to be
> > backported, so this standalone fix is useful for backporting or if the
> > whole other set doesn't make the 26.11 release.
> > 
> > [1] https://patches.dpdk.org/project/dpdk/list/?series=39247
> > ---
> 
> 
> Acked-by: Anatoly Burakov <anatoly.burakov@intel.com>
>
Applied to dpdk-next-net-intel

/Bruce