Re: [Qemu-devel] [PULL] Urgent memory fix for kvm with unaligned memory slots

2012-03-02 Thread Anthony Liguori
On 03/01/2012 07:40 AM, Avi Kivity wrote: The memory core may generate RAM memory regions that are not page aligned, but the kvm code is not prepared to handle them well and will abort under certain conditions. This patch fixes the problem. Please pull from:

Re: [Qemu-devel] [PULL] Urgent memory fix for kvm with unaligned memory slots

2012-03-01 Thread Eric Blake
On 03/01/2012 10:03 AM, Avi Kivity wrote: -ram = memory_region_get_ram_ptr(mr) + section-offset_within_region; +ram = memory_region_get_ram_ptr(mr) + section-offset_within_region + delta; Am I crazy, or does this look wrong? Could be both. Why do you thing it is wrong? Line

Re: [Qemu-devel] [PULL] Urgent memory fix for kvm with unaligned memory slots

2012-03-01 Thread Bobby Powers
On Thu, Mar 1, 2012 at 12:08 PM, Eric Blake ebl...@redhat.com wrote: On 03/01/2012 10:03 AM, Avi Kivity wrote: -    ram = memory_region_get_ram_ptr(mr) + section-offset_within_region; +    ram = memory_region_get_ram_ptr(mr) + section-offset_within_region + delta; Am I crazy, or does this

Re: [Qemu-devel] [PULL] Urgent memory fix for kvm with unaligned memory slots

2012-03-01 Thread Avi Kivity
On 03/01/2012 07:08 PM, Eric Blake wrote: On 03/01/2012 10:03 AM, Avi Kivity wrote: -ram = memory_region_get_ram_ptr(mr) + section-offset_within_region; +ram = memory_region_get_ram_ptr(mr) + section-offset_within_region + delta; Am I crazy, or does this look wrong? Could