Subject: Re: [PATCH net v3 0/7] net/sched: Fix packet loops in mirred and netem

From: Stephen Hemminger <[email-masked]>
Date: Thu, 2 Apr 2026 13:16:18 -0700

On Thu, 26 Mar 2026 11:00:59 -0700
Stephen Hemminger <[email-masked]> wrote:

> This a minor revision of Jamal's series that
> fixes packet loops caused by mirred ingress redirects
> and netem duplication in stacked qdisc trees.
> 
> The core idea is a 2-bit per-skb tc_depth counter that travels with
> the packet.  The existing per-CPU mirred nest tracking loses state
> when a packet is deferred through the backlog or moves between CPUs
> via XPS/RPS.  A per-skb field covers both cases.
> 
> Patch 1 adds the tc_depth field in a padding hole in sk_buff.
> Patches 2-3 revert the check_netem_in_tree() fix and its tests,
> which broke legitimate multi-netem configurations.
> Patch 4 uses tc_depth to stop netem duplicate recursion.
> Patch 5 uses tc_depth to catch mirred ingress redirect loops.
> Patches 6-7 add mirred and netem test cases.
> 
> Thanks to Jamal and Victor for fixing this.
> 
> There are additional netem bug fixes in the pipeline but those
> are held back until this series lands.

Why is this marked Changes Requested.
The only yellow in patchwork is AI review warning that the patch
didn't address my comments. Which is a clear case of AI confusion.

I updated comments and commit methods to my satisfaction.
Sorry if that didn't satisfy it.