Re: [PATCH net v2] net/sched: cake: reject overhead values that underflow length

[Samuel Moelius]

On Sat, Jun 13, 2026 at 5:26 PM Jakub Kicinski <kuba@xxxxxxxxxx> wrote:
>
> On Tue,  9 Jun 2026 23:29:36 +0000 Samuel Moelius wrote:
> > +static const struct netlink_range_validation_signed cake_overhead_range = {
> > +     .min = -64,
> > +     .max = 256,
>
> Both Sashiko's complain - these values are neither safe nor sufficient.
>
> How was the -64 chosen? It looks suspiciously close the min ethernet
> frame length.

That's how it was chosen. But as you've shown, it was not a good choice.

I would like to submit a revised patch that does the check on the
datapath, as you suggested. Should I send that patch? Or would you
prefer to wait for a response from Toke?