* [PATCH] x86/boot: Fix cmdline_find_option() prototype visibility
[not found] ` <20190130172019.GD21221@zn.tnic>
@ 2019-02-05 13:13 ` Borislav Petkov
2019-02-06 10:55 ` [tip:x86/boot] " tip-bot for Borislav Petkov
0 siblings, 1 reply; 2+ messages in thread
From: Borislav Petkov @ 2019-02-05 13:13 UTC (permalink / raw)
To: kbuild test robot; +Cc: Chao Fan, x86-ml, lkml
On Wed, Jan 30, 2019 at 06:20:19PM +0100, Borislav Petkov wrote:
> On Thu, Jan 31, 2019 at 01:12:16AM +0800, kbuild test robot wrote:
> > tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/bp/bp.git tip-x86-boot
> > branch HEAD: 593ab2163e72a9697814dbca7247928a2bbb8b49 x86/boot/KASLR: Limit KASLR to extract the kernel in immovable memory only
> >
> > Regressions in current branch:
> >
> > arch/x86/boot/compressed/acpi.c:37:8: warning: implicit declaration of function 'cmdline_find_option' [-Wimplicit-function-declaration]
> >
> > Error ids grouped by kconfigs:
> >
> > recent_errors
> > └── x86_64-randconfig-s4-01290430
>
> I got a report this morning about x86_64-randconfig-g1-01290904 which I
> already fixed.
>
> Which config is that x86_64-randconfig-s4-01290430?
Ok, I was able to trigger it. The below should fix it:
---
From: Borislav Petkov <bp@suse.de>
Subject: [PATCH] x86/boot: Fix cmdline_find_option() prototype visibility
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
ac09c5f43cf6 ("x86/boot: Build the command line parsing code unconditionally")
enabled building the command line parsing code unconditionally but it
forgot to remove the respected ifdeffery around the prototypes in the
misc.h header, leading to
arch/x86/boot/compressed/acpi.c: In function ‘get_acpi_rsdp’:
arch/x86/boot/compressed/acpi.c:37:8: warning: implicit declaration of function \
‘cmdline_find_option’ [-Wimplicit-function-declaration]
ret = cmdline_find_option("acpi_rsdp", val, MAX_ADDR_LEN);
^~~~~~~~~~~~~~~~~~~
for configs where neither CONFIG_EARLY_PRINTK nor CONFIG_RANDOMIZE_BASE
was defined.
Drop the ifdeffery in the header too.
Fixes: ac09c5f43cf6 ("x86/boot: Build the command line parsing code unconditionally")
Reported-by: kbuild test robot <lkp@intel.com>
Signed-off-by: Borislav Petkov <bp@suse.de>
Cc: Chao Fan <fanc.fnst@cn.fujitsu.com>
Cc: x86@kernel.org
Link: https://lkml.kernel.org/r/5c51daf0.83pQEkvDZILqoSYW%lkp@intel.com
---
arch/x86/boot/compressed/misc.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/x86/boot/compressed/misc.h b/arch/x86/boot/compressed/misc.h
index 19e8231a6fd6..5b8d51996cdd 100644
--- a/arch/x86/boot/compressed/misc.h
+++ b/arch/x86/boot/compressed/misc.h
@@ -66,11 +66,9 @@ static inline void debug_puthex(const char *s)
#endif
-#if CONFIG_EARLY_PRINTK || CONFIG_RANDOMIZE_BASE
/* cmdline.c */
int cmdline_find_option(const char *option, char *buffer, int bufsize);
int cmdline_find_option_bool(const char *option);
-#endif
struct mem_vector {
unsigned long long start;
--
2.19.1
--
Regards/Gruss,
Boris.
Good mailing practices for 400: avoid top-posting and trim the reply.
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [tip:x86/boot] x86/boot: Fix cmdline_find_option() prototype visibility
2019-02-05 13:13 ` [PATCH] x86/boot: Fix cmdline_find_option() prototype visibility Borislav Petkov
@ 2019-02-06 10:55 ` tip-bot for Borislav Petkov
0 siblings, 0 replies; 2+ messages in thread
From: tip-bot for Borislav Petkov @ 2019-02-06 10:55 UTC (permalink / raw)
To: linux-tip-commits; +Cc: hpa, tglx, mingo, bp, fanc.fnst, linux-kernel, lkp
Commit-ID: 82f9ed3a93307089242ff8a5c694e82c8c93f522
Gitweb: https://git.kernel.org/tip/82f9ed3a93307089242ff8a5c694e82c8c93f522
Author: Borislav Petkov <bp@suse.de>
AuthorDate: Tue, 5 Feb 2019 12:05:45 +0100
Committer: Borislav Petkov <bp@suse.de>
CommitDate: Wed, 6 Feb 2019 11:41:21 +0100
x86/boot: Fix cmdline_find_option() prototype visibility
ac09c5f43cf6 ("x86/boot: Build the command line parsing code unconditionally")
enabled building the command line parsing code unconditionally but it
forgot to remove the respective ifdeffery around the prototypes in the
misc.h header, leading to
arch/x86/boot/compressed/acpi.c: In function ‘get_acpi_rsdp’:
arch/x86/boot/compressed/acpi.c:37:8: warning: implicit declaration of function \
‘cmdline_find_option’ [-Wimplicit-function-declaration]
ret = cmdline_find_option("acpi_rsdp", val, MAX_ADDR_LEN);
^~~~~~~~~~~~~~~~~~~
for configs where neither CONFIG_EARLY_PRINTK nor CONFIG_RANDOMIZE_BASE
was defined.
Drop the ifdeffery in the header too.
Fixes: ac09c5f43cf6 ("x86/boot: Build the command line parsing code unconditionally")
Reported-by: kbuild test robot <lkp@intel.com>
Signed-off-by: Borislav Petkov <bp@suse.de>
Cc: Chao Fan <fanc.fnst@cn.fujitsu.com>
Cc: x86@kernel.org
Link: https://lkml.kernel.org/r/5c51daf0.83pQEkvDZILqoSYW%lkp@intel.com
Link: https://lkml.kernel.org/r/20190205131352.GA27396@zn.tnic
---
arch/x86/boot/compressed/misc.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/x86/boot/compressed/misc.h b/arch/x86/boot/compressed/misc.h
index 19e8231a6fd6..5b8d51996cdd 100644
--- a/arch/x86/boot/compressed/misc.h
+++ b/arch/x86/boot/compressed/misc.h
@@ -66,11 +66,9 @@ static inline void debug_puthex(const char *s)
#endif
-#if CONFIG_EARLY_PRINTK || CONFIG_RANDOMIZE_BASE
/* cmdline.c */
int cmdline_find_option(const char *option, char *buffer, int bufsize);
int cmdline_find_option_bool(const char *option);
-#endif
struct mem_vector {
unsigned long long start;
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-02-06 10:56 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <5c51daf0.83pQEkvDZILqoSYW%lkp@intel.com>
[not found] ` <20190130172019.GD21221@zn.tnic>
2019-02-05 13:13 ` [PATCH] x86/boot: Fix cmdline_find_option() prototype visibility Borislav Petkov
2019-02-06 10:55 ` [tip:x86/boot] " tip-bot for Borislav Petkov
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.