On Wed, Jun 1, 2022 at 11:25 AM Alexey Gladkov <legion@xxxxxxxxxx> wrote:
>
> I'm not sure how to get rid of ctl_table since net sysctls are heavily
> dependent on it.
I don't actually think it's worth getting rid of entirely, because
there's just a lot of simple cases where it "JustWorks(tm)" and having
just that table entry describe all the semantics is not wrong at all.
The name may suck, but hey, it's not a big deal. Changing it now would
be more pain than it's worth.
No, I was more thinking that things that already need more
infrastructure than that simple static ctl_table entry might be better
off trying to migrate to your new "proper read op" model, and having
more of that dynamic behavior in the read op.
The whole "create dynamic ctl_table entries on the fly" model works,
but it's kind of ugly.
Anyway, I think all of this is "I think there is more room for cleanup
in this area", and maybe we'll never have enough motivation to
actually do that.
Your patches seem to fix the extant issue with the ipc namespace, and
the truly disgusting parts (although maybe there are other truly
disgusting things hiding - I didn't go look for them).
Linus
