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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 6 Oct 2015 12:14:43 -0500 From: Rob Herring <robh@...nel.org> To: Daniel Walker <danielwa@...co.com> Cc: xe-kernel@...ernal.cisco.com, Frank Rowand <frowand.list@...il.com>, Grant Likely <grant.likely@...aro.org>, Daniel Walker <dwalker@...o99.com>, "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org> Subject: Re: [PATCH-RFC 6/7] drivers: of: ifdef out cmdline section On Tue, Oct 6, 2015 at 10:47 AM, Daniel Walker <danielwa@...co.com> wrote: > It looks like there's some seepage of cmdline stuff into > the generic device tree code. This conflicts with the > generic cmdline implementation so I remove it in the case > when that's enabled. Nice series in general. I've had passing desire to do this every time I run into the command line code. The DT handling of the command line is generic across architectures. The current design is working around that the kernel command line code is not that way. I think we can take this a bit further by making the generic DT code add the command line string directly rather than relying on the arch to do that. Then we can remove all command line handling from the arch code. I would also look at whether we can make boot_command_line static rather than directly accessed. We might have to leave it public for now, but could treat it as static for generic cmdline case. Rob > Cc: xe-kernel@...ernal.cisco.com > Cc: Daniel Walker <dwalker@...o99.com> > Signed-off-by: Daniel Walker <danielwa@...co.com> > --- > drivers/of/fdt.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/of/fdt.c b/drivers/of/fdt.c > index 6e82bc42..80f1e48 100644 > --- a/drivers/of/fdt.c > +++ b/drivers/of/fdt.c > @@ -953,7 +953,7 @@ int __init early_init_dt_scan_chosen(unsigned long node, const char *uname, > * managed to set the command line, unless CONFIG_CMDLINE_FORCE > * is set in which case we override whatever was found earlier. > */ > -#ifdef CONFIG_CMDLINE > +#if defined(CONFIG_CMDLINE) && !defined(CONFIG_GENERIC_CMDLINE) > #ifndef CONFIG_CMDLINE_FORCE > if (!((char *)data)[0]) > #endif > -- > 2.1.4 > -- 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