[coreboot] [PATCH] fix 'AMD Fam10 code breaks with gcc 4.5.0'
Myles Watson
mylesgw at gmail.com
Thu Sep 2 23:42:50 CEST 2010
> Root cause: After function STOP_CAR_AND_CPU disables cache as
> ram, the cache as ram stack can no longer be used. Called
> functions must be inlined to avoid stack usage. Also, the
> compiler must keep local variables register based and not
> allocated them from the stack. With gcc 4.5.0, some functions
> declared as inline are not being inlined. This patch forces
> these functions to always be inlined by adding the qualifier
> __attribute__((always_inline)) to their declaration.
Should we do something like
#define INLINE __attribute__((always_inline))
and replace all (or most) inline directives with INLINE? There are
probably other places where the code depends on inline working.
Thanks,
Myles
More information about the coreboot
mailing list