On Tue, 2024-11-19 at 08:46 -0800, Randy Dunlap wrote: > Hi-- > > On 11/19/24 2:49 AM, Roberto Sassu wrote: > > +/** > > + * struct parser - Structure to store a function pointer to parse digest list > > + * @list: Linked list > > + * @owner: Kernel module owning the parser > > + * @name: Parser name (must match the format in the digest list file name) > > + * @func: Function pointer for parsing > > + * > > + * This structure stores a function pointer to parse a digest list. > > + */ > > +struct parser { > > + struct list_head list; > > + struct module *owner; > > + const char name[NAME_MAX + 1]; > > + parser_func func; > > +}; > > I would make the struct name not so generic -- maybe digest_parser ... Hi sure, thanks for the suggestion! Roberto