On 3/18/26 7:51 AM, Krzysztof Kozlowski wrote:
On 17/03/2026 15:00, Fernando Fernandez Mancera wrote:
Maintaining a modular IPv6 stack offers image size savings for specific
setups, this benefit is outweighed by the architectural burden it
imposes on the subsystems on implementation and maintenance. Therefore,
drop it.
Change CONFIG_IPV6 from tristate to bool. Remove all Kconfig
dependencies across the tree that explicitly checked for IPV6=m. In
addition, remove MODULE_DESCRIPTION(), MODULE_ALIAS(), MODULE_AUTHOR()
and MODULE_LICENSE().
This is also replacing module_init() by device_initcall(). It is not
possible to use fs_initcall() as IPv4 does because that creates a race
condition on IPv6 addrconf.
Finally, modify the default configs from CONFIG_IPV6=m to CONFIG_IPV6=y
except for m68k as according to the bloat-o-meter the image is
increasing by 330KB~ and that isn't acceptable. Instead, disable IPv6 on
this architecture by default. This is aligned with m68k RAM requirements
and recommendations [1].
[1] http://www.linux-m68k.org/faq/ram.html
Signed-off-by: Fernando Fernandez Mancera <[email protected]>
Tested-by: Ricardo B. Marlière <[email protected]>
That's a Kconfig/defconfig only patch, so build system. You cannot test
it in a meaning of testing code. Building code is not testing.
I do not agree. This isn't a Kconfig/defconfig only patch. It is taking
down some module logic like changing module_init() to device_initcall().
Indeed, on v1, this patch was introducing a regression as it was using
fs_initcall().
Acked-by: Krzysztof Kozlowski <[email protected]>
You removed important parts of Ack. It was not provided like that.
Sorry about that. I will make sure to include the # arm64 on the next
revision.
Thanks,
Fernando.
