Re: [PATCH] [PATCH v3] Fetch the OVMF repository from specific git mirror and enable it

[Attilio Rao <attilio.rao@citrix.com>]

[-- Attachment #1.1: Type: text/plain, Size: 1583 bytes --]

On 19/03/12 16:36, Ian Jackson wrote:
> Attilio Rao writes ("Re: [Xen-devel] [PATCH] [PATCH v3] Fetch the OVMF repository from specific git mirror and enable it"):
>    
>> I think I see the issue here. Are you running with i486 as an arch?
>> This is a deficiency of the OVMF support, because it has some wrappers
>> to build appropriately the firmware which are OS and architectures
>> specific and it does support Linux-i686 and Linux-x86_64 but not Linux-i486.
>> I'm not sure about what we should do here, possible options include:
>> - Verify that, on all i486,i586,etc. we can eventually use i686 and
>> unify the script to do that
>>      
> Surely it's just wanting to know roughly what architecture to build
> for.  The difference between i686 and i486 isn't relevant here.
> (I hope.  If it is then we have other problems because this is tested
> at build-time and might be run on a different kernel to the one it's
> built on.)
>
> So I think this is the right thing to do.
>
>    

Here is my take on OVMF for what is needed:
http://xenbits.xen.org/people/attilio/ovmf-xen-arches.patch 
<http://xenbits.xen.org/people/attilio/ovmf-xen-arches.patch>

This should at least fix tianocore to build on all i*86 Linux flavours.
Unfortunately I don't have a NetBSD machine to test it out, so if 
Christoph and/or Roger can apply this patch to ovmf and try the original 
patch on this thread (and verify NetBSD can build ok) I'll propose this 
patch upstream for TIANOCORE, otherwise I will just send out a patch 
with OVMF disabled until NetBSD is not tested.

Thanks,
Attilio

[-- Attachment #1.2: Type: text/html, Size: 2096 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel