| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20140513162731.GD16837@atomide.com> Date: Tue, 13 May 2014 09:27:33 -0700 From: Tony Lindgren <tony@...mide.com> To: Arnd Bergmann <arnd@...db.de> Cc: linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org, Kishon Vijay Abraham I <kishon@...com>, Felipe Balbi <balbi@...com>, Greg KH <gregkh@...uxfoundation.org> Subject: Re: [PATCH] bus/omap_l3: avoid sync initcall for modules * Arnd Bergmann <arnd@...db.de> [140513 08:13]: > On Monday 12 May 2014 15:20:07 Tony Lindgren wrote: > > > > How about instead let's add this into include/linux/module.h > > like we have for other initcalls: I mean include/linux/init.h naturally.. > > #define postcore_initcall(fn) module_init(fn) > > > > There may be others missing too from there. BTW, this driver > > might be few of the drivers that actually might need to be > > initialized early for debugging to catch bad access from > > other drivers. > > There is a comment above those defines saying: > > /* Don't use these in loadable modules, but some people do... */ > > so I'm not sure if adding more would be appreciated. > > Arnd Well "some" in that comment is probably an understatement :) $ git grep _initcall drivers/ | wc -l 737 I did not look what percentage has tristate in the Kconfig, but that might keep some people from making loadable drivers. Greg, is the following patch something you may want to consider picking up? Regards, Tony 8< --------------------------- From: Tony Lindgren <tony@...mide.com> Date: Tue, 13 May 2014 09:05:26 -0700 Subject: [PATCH] init.h: Update initcall_sync variants to fix build errors We are getting randconfig build errors on device drivers with tristate Kconfig option if they are using custom initcall levels. Rather than add ifdeffery into the drivers, let's add the missing initcall_sync variants. As the comment in init.h has kept people from updating the list of initcalls that can be just module_init when the driver is loaded as a loadable module, let's also update the comment a bit to describe valid use cases custom initcall levels. While most drivers should nowadays use just regular module_init because of the deferred probe, we do have quite a few custom initcall levels left that we cannot remove until tested properly. There are also still few valid cases where a custom initcall level might make sense that I'm aware of. For example a bus snooping driver can provide information about invalid bus access and is handy loader early when built in. But there's no hard dependency to have it necessarily built in and a loadable module is a valid option. Another example is a driver implementing a Linux framework like pinctrl framework. That driver may be needed early on some platforms because of legacy reasons, while it can be just a regular module_init on most platforms. Signed-off-by: Tony Lindgren <tony@...mide.com> --- a/include/linux/init.h +++ b/include/linux/init.h @@ -297,16 +297,28 @@ void __init parse_early_options(char *cmdline); #else /* MODULE */ -/* Don't use these in loadable modules, but some people do... */ +/* + * In most cases loadable modules do not need custom + * initcall levels. There are still some valid cases where + * a driver may be needed early if built in, and does not + * matter when built as a loadable module. Like bus + * snooping debug drivers. + */ #define early_initcall(fn) module_init(fn) #define core_initcall(fn) module_init(fn) +#define core_initcall_sync(fn) module_init(fn) #define postcore_initcall(fn) module_init(fn) +#define postcore_initcall_sync(fn) module_init(fn) #define arch_initcall(fn) module_init(fn) #define subsys_initcall(fn) module_init(fn) +#define subsys_initcall_sync(fn) module_init(fn) #define fs_initcall(fn) module_init(fn) +#define fs_initcall_sync(fn) module_init(fn) #define rootfs_initcall(fn) module_init(fn) #define device_initcall(fn) module_init(fn) +#define device_initcall_sync(fn) module_init(fn) #define late_initcall(fn) module_init(fn) +#define late_initcall_sync(fn) module_init(fn) #define console_initcall(fn) module_init(fn) #define security_initcall(fn) module_init(fn) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists