On Tue, 12 Jun 2018 10:39:14 +1000 "Tobin C. Harding" <[email protected]> wrote:
> diff --git a/lib/vsprintf.c b/lib/vsprintf.c > index c445f9f28760..6d2479eb7c2b 100644 > --- a/lib/vsprintf.c > +++ b/lib/vsprintf.c > @@ -1651,6 +1651,18 @@ char *device_node_string(char *buf, char *end, struct > device_node *dn, > return widen_string(buf, buf - buf_start, end, spec); > } > > +/* Make pointers available for printing early in the boot sequence. */ > +static int debug_boot_weak_hash __ro_after_init; > +EXPORT_SYMBOL(debug_boot_weak_hash); I was too busy trying to make this use a static_branch, that I forgot to mention this in my review. Why did you add that EXPORT_SYMBOL()? It's useless with a static variable. I think you can nuke that line. -- Steve
