linux-kernel - Re: Linux 2.6.37-rc3 (fixmap build error)

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]

Date:	Mon, 22 Nov 2010 08:20:10 -0800
From:	Randy Dunlap <randy.dunlap@...cle.com>
To:	Andrew Morton <akpm@...ux-foundation.org>
CC:	Linus Torvalds <torvalds@...ux-foundation.org>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: Linux 2.6.37-rc3 (fixmap build error)

On 11/21/10 21:44, Andrew Morton wrote:
> On Sun, 21 Nov 2010 20:33:07 -0800 Linus Torvalds <torvalds@...ux-foundation.org> wrote:
> 
>> On Sun, Nov 21, 2010 at 6:16 PM, Randy Dunlap <randy.dunlap@...cle.com> wrote:
>>>
>>> I thought that I saw a patch for this one, but I can't find it now.
>>>
>>> on i386:
>>> arch/x86/built-in.o: In function `fix_to_virt':
>>> /usr/builds/linux-2.6.37-rc3/arch/x86/include/asm/fixmap.h:207: undefined reference to `__this_fixmap_does_not_exist'
>>
>> Hmm. Does it help to mark __set_fixmap_offset() as "__always_inline"
>> rather than just "inline"?
> 
> yup.

yep.  Acked-by: Randy Dunlap <randy.dunlap@...cle.com>


> 
> Subject: arch/x86/include/asm/fixmap.h: mark __set_fixmap_offset as __always_inline
> From: Andrew Morton <akpm@...ux-foundation.org>
> 
> When compiling arch/x86/kernel/early_printk_mrst.c with i386 allmodconfig,
> gcc-4.1.0 generates an out-of-line copy of __set_fixmap_offset() which
> contains a reference to __this_fixmap_does_not_exist which the compiler cannot
> elide.
> 
> Marking __set_fixmap_offset() as __always_inline prevents this.
> 
> Cc: Thomas Gleixner <tglx@...utronix.de>
> Cc: Ingo Molnar <mingo@...hat.com>
> Cc: "H. Peter Anvin" <hpa@...or.com>
> Cc: Feng Tang <feng.tang@...el.com>
> Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
> ---
> 
>  arch/x86/include/asm/fixmap.h |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff -puN arch/x86/kernel/early_printk_mrst.c~arch-x86-include-asm-fixmaph-mark-__set_fixmap_offset-as-__always_inline arch/x86/kernel/early_printk_mrst.c
> diff -puN arch/x86/include/asm/fixmap.h~arch-x86-include-asm-fixmaph-mark-__set_fixmap_offset-as-__always_inline arch/x86/include/asm/fixmap.h
> --- a/arch/x86/include/asm/fixmap.h~arch-x86-include-asm-fixmaph-mark-__set_fixmap_offset-as-__always_inline
> +++ a/arch/x86/include/asm/fixmap.h
> @@ -220,8 +220,8 @@ static inline unsigned long virt_to_fix(
>  }
>  
>  /* Return an pointer with offset calculated */
> -static inline unsigned long __set_fixmap_offset(enum fixed_addresses idx,
> -				phys_addr_t phys, pgprot_t flags)
> +static __always_inline unsigned long
> +__set_fixmap_offset(enum fixed_addresses idx, phys_addr_t phys, pgprot_t flags)
>  {
>  	__set_fixmap(idx, phys, flags);
>  	return fix_to_virt(idx) + (phys & (PAGE_SIZE - 1));
> _
> 


-- 
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
--
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/