On Wed, 6 May 2026 11:00:20 +0200 Antonio Quartulli wrote:
> > ovpn_decrypt_post() {
> > ...
> > if (unlikely(ret < 0))
> > goto drop;
> > ...
> > drop:
> > if (unlikely(skb))
> > dev_dstats_rx_dropped(peer->ovpn->dev);
> > ...
> > }
> >
> > Since dev_dstats_rx_dropped() updates the same per-CPU dstats structure
> > without disabling bottom halves, could it still be vulnerable to softirq
> > preemption and stat corruption?
> >
>
> Actually we were already looking into this.
> However, since this needs a separate analysis, I wanted to get this
> fixed in a follow up patch.
>
> Would it be ok to pull this PR as is, so we don't hold back the
> outstanding fixes?
>
> Then we will address the issue highlighted by Sashiko in a new patch.
> The problem is similar, but may need to be fixed differently.
Ugh, fair. The way the AI formatted the output made me think
it's an error path in the same function, which would make sense
to address with a single patch.. Pulling now.
