Re: adding support for above 2giga to kvm (include patchs)

[Ryan Harper <ryanh-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>]

* Ryan Harper <ryanh-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org> [2007-08-14 15:44]:
> * Izik Eidus <izike-atKUWr5tajBWk0Htik3J/w@public.gmane.org> [2007-08-14 15:00]:
> > i am sorry, i wrote you message and it get messed.
> > 
> > anyway you remember in the old patch the:
> > +    if (ram_size > 0xf0000000)
> > >      val = 640; /* base memory in K */
> > i bet this is the problem :)
> 
> I removed that now and I didn't help =(
 
So, I don't think passing the structure as a pointer works.  The 16-bit
compiler bcc has a number of quirks.  I don't pretend to understand what
the issue is here, but, changing the signature of set_e820_range() to:

void set_e820_range(ES, DI, start, end, extra_start, extra_end, type)
     Bit16u ES;
     Bit16u DI;
     Bit32u start;
     Bit32u end;
     Bit16u extra_start;
     Bit8u  extra_end;
     Bit16u type;
{

and the callers accordingly, fixes the issue with memtest iso.  I can
still boot guests with more than 4G, and memtest sees the correct
amount, above and below 4G.

-- 
Ryan Harper
Software Engineer; Linux Technology Center
IBM Corp., Austin, Tx
(512) 838-9253   T/L: 678-9253
ryanh-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/