Re: [kvm-devel] [PATCH] fix -kernel option

2007-12-10 Thread Dor Laor
Glauber de Oliveira Costa wrote: Dor Laor wrote: > Glauber de Oliveira Costa wrote: >> >> Currently, the -kernel option is not working. >> >> Reason is, because we're registering chunks for regions 0-0xa and >> 0x10-ram_size, the phys_ram_addr + PA is broken. >> The real fix should be to

Re: [kvm-devel] [PATCH] fix -kernel option

2007-12-10 Thread Glauber de Oliveira Costa
Dor Laor wrote: > Glauber de Oliveira Costa wrote: >> >> Currently, the -kernel option is not working. >> >> Reason is, because we're registering chunks for regions 0-0xa and >> 0x10-ram_size, the phys_ram_addr + PA is broken. >> The real fix should be to rewrite all the load_linux() code t

Re: [kvm-devel] [PATCH] fix -kernel option

2007-12-09 Thread Uri Lublin
Glauber de Oliveira Costa wrote: > Currently, the -kernel option is not working. > > Reason is, because we're registering chunks for regions 0-0xa and > 0x10-ram_size, the phys_ram_addr + PA is broken. > The real fix should be to rewrite all the load_linux() code to not rely > on this, but

Re: [kvm-devel] [PATCH] fix -kernel option

2007-12-09 Thread Izik Eidus
Dor Laor wrote: > Glauber de Oliveira Costa wrote: >> >> Currently, the -kernel option is not working. >> >> Reason is, because we're registering chunks for regions 0-0xa and >> 0x10-ram_size, the phys_ram_addr + PA is broken. >> The real fix should be to rewrite all the load_linux() code t

Re: [kvm-devel] [PATCH] fix -kernel option

2007-12-09 Thread Dor Laor
Glauber de Oliveira Costa wrote: Currently, the -kernel option is not working. Reason is, because we're registering chunks for regions 0-0xa and 0x10-ram_size, the phys_ram_addr + PA is broken. The real fix should be to rewrite all the load_linux() code to not rely on this, but meanwhil

Re: [kvm-devel] [PATCH] fix -kernel option

2007-12-09 Thread Avi Kivity
Anthony Liguori wrote: > Glauber de Oliveira Costa wrote: > >> Currently, the -kernel option is not working. >> >> Reason is, because we're registering chunks for regions 0-0xa and >> 0x10-ram_size, the phys_ram_addr + PA is broken. >> >> > > I think this patch is a reasonable wo

Re: [kvm-devel] [PATCH] fix -kernel option

2007-12-07 Thread Luca Tettamanti
On Dec 7, 2007 7:38 PM, Anthony Liguori <[EMAIL PROTECTED]> wrote: > Izik Eidus wrote: > > Anthony Liguori wrote: > >> Glauber de Oliveira Costa wrote: > >> > >>> Currently, the -kernel option is not working. > >>> > >>> Reason is, because we're registering chunks for regions 0-0xa and > >>> 0x

Re: [kvm-devel] [PATCH] fix -kernel option

2007-12-07 Thread Izik Eidus
Anthony Liguori wrote: > Izik Eidus wrote: >> Anthony Liguori wrote: >>> Glauber de Oliveira Costa wrote: >>> Currently, the -kernel option is not working. Reason is, because we're registering chunks for regions 0-0xa and 0x10-ram_size, the phys_ram_addr + PA is broken

Re: [kvm-devel] [PATCH] fix -kernel option

2007-12-07 Thread Anthony Liguori
Izik Eidus wrote: > Anthony Liguori wrote: >> Glauber de Oliveira Costa wrote: >> >>> Currently, the -kernel option is not working. >>> >>> Reason is, because we're registering chunks for regions 0-0xa and >>> 0x10-ram_size, the phys_ram_addr + PA is broken. >>> >> >> I think this

Re: [kvm-devel] [PATCH] fix -kernel option

2007-12-07 Thread Izik Eidus
Glauber de Oliveira Costa wrote: > Currently, the -kernel option is not working. > > Reason is, because we're registering chunks for regions 0-0xa and > 0x10-ram_size, the phys_ram_addr + PA is broken. > The real fix should be to rewrite all the load_linux() code to not rely > on this, but

Re: [kvm-devel] [PATCH] fix -kernel option

2007-12-07 Thread Izik Eidus
Anthony Liguori wrote: > Glauber de Oliveira Costa wrote: > >> Currently, the -kernel option is not working. >> >> Reason is, because we're registering chunks for regions 0-0xa and >> 0x10-ram_size, the phys_ram_addr + PA is broken. >> >> > > I think this patch is a reasonable wo

Re: [kvm-devel] [PATCH] fix -kernel option

2007-12-07 Thread Anthony Liguori
Glauber de Oliveira Costa wrote: > Currently, the -kernel option is not working. > > Reason is, because we're registering chunks for regions 0-0xa and > 0x10-ram_size, the phys_ram_addr + PA is broken. > I think this patch is a reasonable work-around but the long term solution is proba

[kvm-devel] [PATCH] fix -kernel option

2007-12-07 Thread Glauber de Oliveira Costa
Currently, the -kernel option is not working. Reason is, because we're registering chunks for regions 0-0xa and 0x10-ram_size, the phys_ram_addr + PA is broken. The real fix should be to rewrite all the load_linux() code to not rely on this, but meanwhile, filling in the gap up to 0xc