On Mon, Jul 8, 2024 at 5:07 PM Sami Tolvanen <[email protected]> wrote: > > On Mon, Jul 8, 2024 at 2:33 PM Luis Chamberlain <[email protected]> wrote: > > > > Looking at this again its not to me why Masahiro Yamada's suggestion on > > that old patch series to just increase the length and put long symbols > > names into its own section [0] could not be embraced with a new kconfig > > option, so new kernels and new userspace could support it: > > > > https://lore.kernel.org/lkml/CAK7LNATsuszFR7JB5ZkqVS1W=hWr9=e7btf+mvgj+nxt3az...@mail.gmail.com/ > > Matt, was there a reason we didn't move forward with Masahiro's > proposal? It sounds reasonable to me, but I might be missing some > background here.
I liked Masahiro's proposal, and implemented it [1], but in the first version of the patch, folks complained [2] that I was touching code that was hard to reason about, and that I should clean it up. In the second version of the patch, I cleaned it up, and was told the cleanup patch was too big and needed to be broken down [3]. If we want to revive that, it appears there's one or two PPC userspace tool issues I need to address first, but I could do so, and attempt to break the cleanup patch into smaller pieces to be more reviewable. I had abandoned it because even if completed, without the new work Sami did on extracting a hash from debug symbols, it would not have been accepted as a way to enable MODVERSIONS. When we generated this new approach, I suggested we do hashes to avoid modifying the module loading code given the previous reaction. [1]: https://lore.kernel.org/all/[email protected]/ [2]: https://lore.kernel.org/all/zvznh%[email protected]/ [3]: https://lore.kernel.org/all/2023111818-agent-verdict-99a5@gregkh/
