On 21/07/2025 12.40, Vlastimil Babka wrote: > On 7/15/25 20:58, Daniel Gomez wrote: >> On 15/07/2025 10.43, Vlastimil Babka wrote: >>> Christoph suggested that the explicit _GPL_ can be dropped from the >>> module namespace export macro, as it's intended for in-tree modules >>> only. It would be possible to restrict it technically, but it was >>> pointed out [2] that some cases of using an out-of-tree build of an >>> in-tree module with the same name are legitimate. But in that case those >>> also have to be GPL anyway so it's unnecessary to spell it out in the >>> macro name. >>> >>> Link: https://lore.kernel.org/all/[email protected]/ [1] >>> Link: >>> https://lore.kernel.org/all/cak7lnatrkzhwjgpojcnvdiaodnp%[email protected]/ >>> [2] >>> Suggested-by: Christoph Hellwig <[email protected]> >>> Reviewed-by: Shivank Garg <[email protected]> >>> Acked-by: David Hildenbrand <[email protected]> >>> Acked-by: Nicolas Schier <[email protected]> >>> Reviewed-by: Daniel Gomez <[email protected]> >>> Reviewed-by: Christian Brauner <[email protected]> >>> Signed-off-by: Vlastimil Babka <[email protected]> >>> --- >>> Daniel, please clarify if you'll take this via module tree or Christian >>> can take it via vfs tree? >> >> Patch 707f853d7fa3 ("module: Provide EXPORT_SYMBOL_GPL_FOR_MODULES() helper") >> from Peter was merged through Masahiro in v6.16-rc1. Since this is a related >> fix/rename/cleanup, it'd make sense for it to go through his kbuild tree as >> well. Masahiro, please let me know if you'd prefer otherwise. If not, I'll >> queue >> it up in the modules tree. > > Maybe with no reply, you can queue it then?
+ Jiri, Stephen and Greg, added to the To: list. EXPORT_SYMBOL_GPL_FOR_MODULES macro was merged [1] through Masahiro's pull request in v6.16-rc1. This patch from Vlastimil renames the macro to EXPORT_SYMBOL_FOR_MODULES. This means Jiri's patch b20d6576cdb3 "serial: 8250: export RSA functions" will need to be updated accordingly. I'd like like to know how you prefer to proceed, since it was requested to have this merged as a fix before Linus releases a new kernel with the former name. Link: https://lore.kernel.org/all/cak7lnaqunzxohr+vmzlf8kqxyrtlx-z2g2vzqujmndrt9tz...@mail.gmail.com/ [1] Masahiro, just a heads-up that I plan to merge this through the linux-modules tree unless you advise otherwise.
