[LinuxBIOS] [PATCH] LAR compression in xip not supported

ron minnich rminnich at gmail.com
Wed Sep 5 19:21:53 CEST 2007


On 9/5/07, Alex Beregszaszi <alex at rtfs.hu> wrote:

> LAR doesn't has a field for telling that an entry will be XIP or not.
> Whether we run it in place or move it is only our _runtime_ decision,
> thus this check is needed.
>

So here is a good point. Why not adopt the existing convention that a
load address of 0xffffffff means "XIP"?

> I think such an extra check wouldn't hurt (maybe if you care about
> space, you could merge some checks to print only one message).

We should do all possible checks at LAR time. Any runtime check that
fails means we had a failure in how we built the lar.

ron




More information about the coreboot mailing list