- Subject: Re: [PATCH v2] lib/decompress_unxz.c: removing all memory helper functions
- From: "H. Peter Anvin" <hpa@xxxxxxxxx>
- Date: Tue, 12 Jun 2012 10:48:02 -0700
- Cc: Lasse Collin <lasse.collin@xxxxxxxxxxx>, T Makphaibulchoke <tmac@xxxxxx>, Russell King - ARM Linux <linux@xxxxxxxxxxxxxxxx>, schwidefsky@xxxxxxxxxx, heiko.carstens@xxxxxxxxxx, linux390@xxxxxxxxxx, lethal@xxxxxxxxxxxx, tglx@xxxxxxxxxxxxx, mingo@xxxxxxxxxx, x86@xxxxxxxxxx, kaloz@xxxxxxxxxxx, jj@xxxxxxxxxxxxx, matt.fleming@xxxxxxxxx, linux-arm-kernel@xxxxxxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-s390@xxxxxxxxxxxxxxx, linux-sh@xxxxxxxxxxxxxxx
- In-reply-to: <alpine.LFD.2.02.1206121336060.23555@xanadu.home>
- References: <1339470307-28223-1-git-send-email-tmac@hp.com> <20120612191804.4211462a@tukaani.org> <alpine.LFD.2.02.1206121316190.23555@xanadu.home> <4FD77CAF.1020402@zytor.com> <alpine.LFD.2.02.1206121336060.23555@xanadu.home>
- User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:12.0) Gecko/20120430 Thunderbird/12.0.1
On 06/12/2012 10:44 AM, Nicolas Pitre wrote:
> On Tue, 12 Jun 2012, H. Peter Anvin wrote:
>
>> On 06/12/2012 10:26 AM, Nicolas Pitre wrote:
>>>
>>> Instead of copying, maybe this would be much better to make the content
>>> of lib/string.c usable in a pre-boot environment.
>>>
>>
>> Not sure about that... a lot of it is #ifndef __HAVE_ARCH_* and you
>> *really* want those optimizations on some arches.
>
> Sure. What about:
>
> lib/boot/mem.c: lib/string.c
> sed 's/^#ifndef __HAVE_ARCH_/#ifndef __HAVE_PREBOOT_ARCH_/' < $< > $@
>
> This is over simplified but you get the idea. Maybe lib/string.c itself
> should be split into several files to help with a finer grained
> selection.
>
The point is that I think we pretty much end up with a null usable set
in the end.
-hpa
--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[Linux OMAP]
[Linux USB Devel]
[Video for Linux]
[Linux Audio Users]
[Photo]
[Yosemite News]
[Yosemite Photos]
[Free Online Dating]
[Linux Kernel]
[Linux SCSI]
[XFree86]