[OpenBIOS] [PATCH 3/5] On ppc64, asm/sigcontext.h includes asm/elf.h

[Laurent Vivier]

On 25/10/2015 14:14, Laurent Vivier wrote:
>
> On 25/10/2015 12:36, Mark Cave-Ayland wrote:
>> On 24/10/15 13:53, Segher Boessenkool wrote:
>>
>>> On Sat, Oct 24, 2015 at 11:16:59AM +0200, Laurent Vivier wrote:
>>>> asm/sigcontext.h:
>>>>
>>>>     #ifdef __powerpc64__
>>>>     #include <asm/elf.h>
>>>>     #endif
>>>>
>>>> to avoid to include here target/include/asm/elf.h, hardcode the path
>>>> to /usr/include/asm/elf.h in it.
>>> That only works for native builds on powerpc64-linux.  Is that all
>>> that is wanted here?
>> Hmmm that does seem a bit unusual. Is there a better solution for this?
>> Perhaps changing the layout of the OpenBIOS headers to avoid include
>> confusion?
> Yes, I agree with you.
>
> There are two solutions here:
> 1- normally, "#include <...>" searches firstly in system headers, then
> in "-I" headers, I can seek why it is not the case in the bootstrap case,
> 2- otherwise, we can change the layout, it was what I tried to do first,
> but this implies a lot of changes.
>
> I'm going to try to find a solution using the first one, feel free to
> include the other patches from the series if they are ok for you.
OK, I've a cleaner solution that can replace patches 3/5 and 4/5.
I'll resend the series once I'll have tested it on an x86_64 host.

Laurent