[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] hvmloader: fix SeaBIOS interface



>>> On 23.08.13 at 14:46, Ian Campbell <Ian.Campbell@xxxxxxxxxx> wrote:
> On Fri, 2013-08-23 at 13:42 +0100, Jan Beulich wrote:
>> The SeaBIOS ROM image may validly exceed 128k in size, it's only our
>> interface code that so far assumed that it wouldn't. Remove that
>> restriction by setting the base address depending on image size.
>> 
>> Add a check to HVM loader so that too big images won't result in silent
>> guest failure anymore.
>> 
>> Uncomment the intended build-time size check for rombios, moving it
>> into a function so that it would actually compile.
>> 
>> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
> 
> Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
>  
>> -    .bios_address = SEABIOS_PHYSICAL_ADDRESS,
>> +    .bios_address = 0x100000 - sizeof(seabios),
> 
> Do we rely on the seabios build to pad this, or are arbitrary sizes
> aloud? (Doesn't invalidate my Ack, just curious)

Since the padding happens at the front, we don't really care.

Jan


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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.