> -----Original Message-----
> From: Simon Horman <[email protected]>
> Sent: Friday, March 20, 2026 7:05 PM
> To: Loktionov, Aleksandr <[email protected]>
> Cc: [email protected]; Nguyen, Anthony L
> <[email protected]>; [email protected]; Wieczerzycka,
> Katarzyna <[email protected]>
> Subject: Re: [PATCH] ice: add missing reset of the mac header
>
> On Fri, Mar 20, 2026 at 06:05:18AM +0100, Aleksandr Loktionov wrote:
> > From: Katarzyna Wieczerzycka <[email protected]>
> >
> > By default skb->mac_header is not set, so reset prevents access to
> an
> > invalid pointer.
> >
> > Call skb_reset_mac_header() before accessing the mac header from
> skb.
> >
> > Signed-off-by: Katarzyna Wieczerzycka
> > <[email protected]>
> > Signed-off-by: Aleksandr Loktionov <[email protected]>
>
> Hi Katarzyna and Aleksandr,
>
> I am curious:
>
> Is this a bug? If so then it should probably have a fixes tag and a
> bit more of a description around how this can happen.
>
> If it is not a bug, then is this defensive? And if so, is it really
> necessary?
>
> ...
Withdrawing this patch. The bug justification and repro are not
strong enough to meet the bar for upstream. Will drop for now.
With the best regards
Alex