[PATCH v2 1/2] landlock: Fully release unused TSYNC work entries

Günther Noack gnoack at google.com
Tue Feb 17 16:35:31 UTC 2026


On Tue, Feb 17, 2026 at 02:52:46PM +0100, Mickaël Salaün wrote:
> On Tue, Feb 17, 2026 at 01:41:11PM +0100, Günther Noack wrote:
> > On Tue, Feb 17, 2026 at 01:23:39PM +0100, Mickaël Salaün wrote:
> > > +	memset(ctx, 0, sizeof(*ctx));
> > 
> > Minor (and highly optional) remark, this is the same as
> > 
> >   *ctx = (struct tsync_work){};
> 
> What about:
> 
> *ctx = (typeof(*ctx)){};

I find that harder to read, because it is less commonly seen and the typeof() is
an indirection that makes me think as a reader.  But at this point, this is only
a vague opinion and I don't feel strongly about it.  Please submit either one of
these three options :)

—Günther



More information about the Linux-security-module-archive mailing list