Re: [PATCH net 1/2] net/sched: hhf: cap hh_flows_limit at change time
From: Simon Horman <horms@kernel.org>
Date: 2026-09-14 09:14:40
Also in:
stable
On Sun, Sep 13, 2026 at 06:34:15AM -0400, Jamal Hadi Salim wrote:
On Sat, Sep 12, 2026 at 4:36 PM [off-list ref] wrote:quoted
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider: Critical: 0 · High: 0 · Medium: 1 · Low: 0 - [Medium] net/sched/sch_hhf.c: the new `[TCA_HHF_HH_FLOWS_LIMIT] = NLA_POLICY_MAX(NLA_U32, 2 * HH_FLOWS_CNT)` policy entry (line 530) is…These findings by sashiko-nipa are accurate but as in the previous codel thread they concern changelog and selftest-robustness nits. IOW, there is _no code defect_. The -ERANGE behavior change on add path and the changelog overstatement are worth a one-line fix if/when the thread is re-rev'd, but IMO are not a revision blocker
Reviewed-by: Simon Horman <horms@kernel.org>