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