Re: [PATCH 2/2] seq_file: Require explicit initialization of struct seq_operations
From: Greg KH
Date: Thu Oct 15 2026 - 03:11:50 EST
On Wed, Oct 14, 2026 at 10:57:57PM -0700, Kees Cook wrote:
>
>
> On October 14, 2026 10:46:02 PM PDT, Miguel Ojeda <miguel.ojeda.sandonis@xxxxxxxxx> wrote:
> >On Thu, Oct 15, 2026 at 7:21 AM Greg KH <greg@xxxxxxxxx> wrote:
> >>
> >> Anyway, I don't know. I like this feature, and maybe it's going to be
> >> used so infrequently it's not a big deal, but feels clunky.
> >
> >Perhaps we could call it something shorter, like `__explicit_init` or
> >similar -- I don't see that substring (even without the double
> >underscores) used anywhere.
__must_init might be smaller?
Naming is hard :)
> That's nice, yeah. And it's not the first time we've changed the "spelling" of an attribute.
>
> Does anyone object to that? I can replace it with that in my tree...
No objection from me to make it shorter, and yes, we can always change
it going forward if someone comes up with a better term.
thanks,
greg k-h