On 2026-01-23 10:36, Petr Pavlu wrote: > On 1/23/26 12:46 AM, Jim Cromie wrote: > > "modprobe foo" currently does strcmp on the name, this can be improved. > > > > So this commit: > > > > 1. adds name_crc to struct module > > 2. modpost.c computes the value and > > 3. outputs it for "modinfo foo" to see/use. > > > > 4. adds hotpath to find_module_all() > > this uses name_crc to do quick "name-check" > > falls back to strcmp only to guard against collisions. > > > > This should significantly reduce modprobe workload, and shorten module > > load-time. > > > > Since it alters struct module, its binary incompatible. This means: > > > > 1. RFC for its wide "blast radius". > > 2. suitable for major version bump *only* > > > > 3. it opens door for further struct module reorg, to: > > a. segregate fields by "temperature" > > b. pack out paholes. > > c. improve cache locality (by reordering coldest on bottom) > > name should be cold now. > > bikeshedding is appropriate here. > > > > NB: this isn't a substitute for CONFIG_MODULE_SIG. > > It reimplements crc_le(), doesn't reuse kernel's version. > > > > CC: Luis Chamberlain <[email protected]> > > CC: Petr Pavlu <[email protected]> > > CC: Daniel Gomez <[email protected]> > > CC: Sami Tolvanen <[email protected]> > > CC: Aaron Tomlin <[email protected]> > > CC: [email protected] > > > > Signed-off-by: Jim Cromie <[email protected]> > > > > '#' will be ignored, and an empty message aborts the commit. > > This patch looks as if it were generated by AI. If so, please avoid > sending such changes. Otherwise, the commit description should explain
FYI, this is a process already documented. You can check out what maintainers expect from contributions and possible guidelines: https://lore.kernel.org/all/[email protected]/
