linux-kernel - Re: [PATCH] firmware: fix build errors in paged buffer handling code

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <s5hzhl68es7.wl-tiwai@suse.de>
Date:   Mon, 22 Jul 2019 13:55:20 +0200
From:   Takashi Iwai <tiwai@...e.de>
To:     Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc:     Luis Chamberlain <mcgrof@...nel.org>,
        "Rafael J . Wysocki" <rafael@...nel.org>,
        Mauro Rossi <issor.oruam@...il.com>,
        Chih-Wei Huang <cwhuang@...roid-x86.org>,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] firmware: fix build errors in paged buffer handling code

On Mon, 22 Jul 2019 07:55:36 +0200,
Takashi Iwai wrote:
> 
> From: Mauro Rossi <issor.oruam@...il.com>
> 
> fw_{grow,map}_paged_buf() need to be defined as static inline
> when CONFIG_FW_LOADER_PAGED_BUF is not enabled,
> infact fw_free_paged_buf() is also defined as static inline
> when CONFIG_FW_LOADER_PAGED_BUF is not enabled.
> 
> Fixes the following mutiple definition building errors for Android kernel:
> 
> drivers/base/firmware_loader/fallback_efi.o: In function `fw_grow_paged_buf':
> fallback_efi.c:(.text+0x0): multiple definition of `fw_grow_paged_buf'
> drivers/base/firmware_loader/main.o:(.text+0x73b): first defined here
> drivers/base/firmware_loader/fallback_efi.o: In function `fw_map_paged_buf':
> fallback_efi.c:(.text+0xf): multiple definition of `fw_map_paged_buf'
> drivers/base/firmware_loader/main.o:(.text+0x74a): first defined here
> 
> [ slightly corrected the patch description -- tiwai ]
> 
> Fixes: 5342e7093ff2 ("firmware: Factor out the paged buffer handling code")
> Fixes: 82fd7a8142a1 ("firmware: Add support for loading compressed files")
> Signed-off-by: Mauro Rossi <issor.oruam@...il.com>
> Signed-off-by: Takashi Iwai <tiwai@...e.de>

Please discard this one.  It's missing the inline, as Mauro already
suggested in another mail.

Will resubmit the revised one.  Sorry for inconvenience.


Takashi