On Fri, Jan 16, 2026 at 1:17 PM Ilya Maximets <[email protected]> wrote:
>
> On 1/16/26 6:12 PM, [email protected] wrote:
> > From: Numan Siddique <[email protected]>
> >
> > The netdev-offload-tc module uses a flow's recirc_id (a uint32_t)
> > as the TC flower chain ID during offloading.  However, the kernel's
> > TC implementation reserves the upper 4 bits of the chain index for
> > extended action opcodes.  If a recirc_id exceeds (2^28 - 1), the
> > kernel rejects the flow offload with EINVAL.  This patch caps the
> > maximum recirc_id to (2^28 - 1) to prevent these failures.
> >
> > This limit is sufficiently high that it is unlikely to impact the
> > number of active datapath flows in practice.
> >
> > Reported-at: 
> > https://mail.openvswitch.org/pipermail/ovs-dev/2025-November/427485.html
> > Suggested-by: Ilya Maximets <[email protected]>
> > Suggested-by: Eelco Chaudron <[email protected]>
> > Acked-by: Eelco Chaudron <[email protected]>
> > Signed-off-by: Numan Siddique <[email protected]>
> > ---
> >
> > v3 -> v4
> > -------
> >   - Rebased (no conflicts).
> >   - Added Eelco's Ack.
> >
> > v2 -> v3
> > -------
> >   - Addressed review comments from v2.
> >
> >  ofproto/ofproto-dpif-rid.c | 11 ++++++++---
> >  1 file changed, 8 insertions(+), 3 deletions(-)
>
>
> Thanks, Numan and Eelco!  Applied and backported down to 3.3.

Thanks Ilya and Eelco.

Numan

>
> Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to