linux-kernel - Re: [PATCH 05/10] kbuild: sort the list of symbols exported by the kernel (_

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:	Wed, 25 Nov 2009 09:15:18 +0000
From:	Alan Jenkins <alan-jenkins@...fmail.co.uk>
To:	James Bottomley <James.Bottomley@...senPartnership.com>
CC:	Rusty Russell <rusty@...tcorp.com.au>,
	Alex Chiang <achiang@...com>, Tony Luck <tony.luck@...il.com>,
	Sam Ravnborg <sam@...nborg.org>,
	Mike Frysinger <vapier.adi@...il.com>, greg@...ah.com,
	linux-kbuild@...r.kernel.org, carmelo73@...il.com,
	linux-kernel@...r.kernel.org, kyle@...artin.ca, deller@....de,
	jejb@...isc-linux.org,
	Benjamin Herrenschmidt <benh@...nel.crashing.org>,
	paulus@...ba.org
Subject: Re: [PATCH 05/10] kbuild: sort the list of symbols exported by the
 kernel (__ksymtab)

James Bottomley wrote:
> On Tue, 2009-11-24 at 09:28 +0000, Alan Jenkins wrote:
>   
>> James Bottomley wrote:
>>     
>>> On Tue, 2009-11-24 at 11:27 +1030, Rusty Russell wrote:
>>>   
>>>       
>>>> On Tue, 24 Nov 2009 06:23:20 am Alex Chiang wrote:
>>>>     
>>>>         
>>>>> Hi Alan, Rusty,
>>>>>       
>>>>>           
>>>> Hi Alex,
>>>>
>>>>     
>>>>         
>>>>> In the meantime, while Alan is deciding the proper way to fix
>>>>> this, would it be possible to drop the offending patch series
>>>>> from linux-next?
>>>>>       
>>>>>           
>>>> Done.  That takes the pressure off Alan, and makes sure he has time to get
>>>> it right.
>>>>     
>>>>         
>>> That probably suits us on parisc too.  I just checked out our build in
>>> linux-next: we don't pass __modpost ... it looks like we have all the
>>> module symbols undefined.  Will investigate more.
>>>
>>> James
>>>   
>>>       
>> I think parisc wants P'printk where ia64 uses @fptr(printk).
>>
>> It may also need ".import printk,code" or similar.
>>     
>
> I think if you have to make modpost architecture specific, there's
> something a bit wrong in the patch series.
>
> I can confirm that reverting this particular patch allows the parisc
> build to work again.  It still won't boot because module symbols aren't
> resolved.
>
> James
>   

Yes, the series as a whole relies on that patch.  Rusty pulled the 
series from linux-next (thanks rusty!).

I'm working on alternatives now.  I can't promise to avoid architecture 
specific code, but if it's necessary then I understand I have to arrange 
to test it myself :-).  Sorry for the inconvenience caused by 
"arch-independent assembly code" that wasn't.

Alan
--
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/