On Mon, Jun 02, 2025 at 12:17:54PM -0700, Ackerley Tng wrote: > +struct inode *alloc_anon_secure_inode(struct super_block *s, const char *name) > +{ > + return anon_inode_make_secure_inode(s, name, NULL, true); > +} > +EXPORT_SYMBOL_GPL(alloc_anon_secure_inode); What is "secure" about this inode? A kerneldoc explaining that would probably help. > +extern struct inode *alloc_anon_secure_inode(struct super_block *, const char *); No need for the extern here. Spelling out the parameter names in protypes is nice, though. (and fix the long line while you're at it).