[U-Boot] [PATCH] dlmalloc: Ignore gcc4.4 compiler warnings
Kumar Gala
galak at kernel.crashing.org
Wed Jul 29 07:55:01 CEST 2009
On Jul 29, 2009, at 12:47 AM, Wolfgang Denk wrote:
> Dear Scott,
>
> In message <20090728225244.GA8187 at b07421-ec1.am.freescale.net> you
> wrote:
>>
>> The patch title is bad -- it's not disabling warnings, it's
>> disabling an
>> aspect of C99 that the code is incompatible with (and which is pretty
>> questionable in the first place with such low level code). Note
>> that in
>> Linux, this is disabled for the entire kernel.
>
> I know. But Linux is bigger than U-Boot, and I think we should be able
> to fix the few isolated places that throw such warnings.
>
>> As things stand, GCC may do bad things with that code. With this
>> patch,
>> it may not (at least not this particular sort of bad thing). Those
>> bad
>> things are not limited to the places where it warns -- those are
>> just the
>> violations it detected.
>
> Agreed, and that's why I want to see this fixed.
>
>> Do you have an alternative malloc implementation in mind that is
>> designed
>> to work with strict aliasing, or a suggested fix to the current one?
>
> I did not look into this yet - there was some discussion about a
> malloc replacement, but it faded away without visible result.
>
> I cannot do everything myself, but I can oppose changes that are IMO
> to the worse.
Do we have any ideas what type of performance we're looking for from
malloc?
- k
More information about the U-Boot
mailing list