linux-kernel - Re: [PATCH] modpost: fix modules on ia64 - use @fptr() on exported function symbols

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] [day] [month] [year] [list]

Message-ID: <20091123232520.GA18976@ldl.fc.hp.com>
Date:	Mon, 23 Nov 2009 16:25:20 -0700
From:	Alex Chiang <achiang@...com>
To:	Alan Jenkins <alan-jenkins@...fmail.co.uk>
Cc:	Rusty Russell <rusty@...tcorp.com.au>,
	Tony Luck <tony.luck@...il.com>, James.Bottomley@...e.de,
	linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] modpost: fix modules on ia64 - use @fptr() on exported
	function symbols

* Alan Jenkins <alan-jenkins@...fmail.co.uk>:
> On function descriptor architectures (powerpc and ia64), we need to export
> the address of the function descriptor and not the actual function.
> The C compiler naturally does this for us.  However we now export vmlinux
> symbols using the assembler, in order to sort the exports.
>
> This is not a problem on powerpc.  On powerpc, the function descriptor
> is simply "printk" (the action function entry point is ".printk").
> But ia64 assembly needs special handling.  The function descriptor for
> printk(), for example, is "@fptr(printk)".
>
> I have tested that modpost now generates the desired assembly code when
> run against an IA64 module.  I have not tested with an actual IA64
> vmlinux, assembler, compiler or processor.  I assume that the IA64
> assembler defines __ia64__, just as the i386 assembler defines __i386__.
>
> Reported-by: Alex Chiang <achiang@...com>

I applied this patch to a tree that had 20ba8ef1de7e77e4c
(kbuild: sort the list of symbols exported by the kernel
(__ksymtab)) as a HEAD.

Without the patch, I get a panic. With the patch, boot succeeds.

I also applied this patch on top of 9d5bf6fad2 (Add linux-next
specific files for 20091123), and verified that boot also
succeeds.

So for ia64, you can add my:

Tested-by: Alex Chiang <achiang@...com>

However, I've cc'ed James Bottomley who is currently testing
parisc. parsic is another function descriptor architecture, and
due to your very specific #ifdef __ia64__ in the patch, it
clearly won't fix parisc (if indeed it is broken).

James mentioned something about needing an fptr arch symbol, but
I'll let him expand on his idea.

Thanks,
/ac
--
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/