kern.ipc.sockbuf limits: anyone mind if I commit this?

Alfred Perlstein alfred at freebsd.org
Wed Nov 11 00:24:36 UTC 2015



On 11/10/15 3:13 PM, Adrian Chadd wrote:
> hiya,
>
> there's a PR with a patch:
>
> https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=204438
>
> https://github.com/sparrc/freebsd/commit/157f90c55d1d54d33f41c6f7517de1a9c5f5e229
>
> Does anyone know why setting the limits isn't as simple as this patch?
>
> Does anyone mind if I just commit this?
>
Don't mind too heavily, however the old behavior is bad and confusing 
however at least it stops you, however the new behavior will be odd and 
incorrect without warning.

More succinctly: Silently "accepting" but actually changing the value 
passed in seems wrong.

It would seem the reason for the calculation is to actually limit the 
number of bytes of mbufs (not just data) to the max value?  Is that true?

Maybe it makes sense to export sb_max_adj via sysctl and allow setting 
of it instead?  Having silent clipping seems worse than an error.

-Alfred


More information about the freebsd-net mailing list