[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 6/7] Set slots more carefully
From: |
Jan Kiszka |
Subject: |
[Qemu-devel] Re: [PATCH 6/7] Set slots more carefully |
Date: |
Fri, 17 Jul 2009 15:48:10 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
Alexander Graf wrote:
> KVM only supports slot sizes of PAGE_SIZE granilarity. On PPC the OS
> sets the framebuffer to some odd size though, causing the current code
> to simply abort().
>
> So let's bet graceful here. We can just allocate memory sizes that are of
> PAGE_SIZE granularity and everything that exceeds that comes in as MMIO and
> gets handled too - just slower.
>
> Signed-off-by: Alexander Graf <address@hidden>
> ---
> kvm-all.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/kvm-all.c b/kvm-all.c
> index 961fa32..60b76cf 100644
> --- a/kvm-all.c
> +++ b/kvm-all.c
> @@ -134,7 +134,7 @@ static int kvm_set_user_memory_region(KVMState *s,
> KVMSlot *slot)
>
> mem.slot = slot->slot;
> mem.guest_phys_addr = slot->start_addr;
> - mem.memory_size = slot->memory_size;
> + mem.memory_size = slot->memory_size & ~(TARGET_PAGE_SIZE - 1);
TARGET_PAGE_MASK? And I bet you want to round up here...
Does the caller use the odd size consistently (i.e. also for dirty log
enable/disable)? Otherwise we may run into troubles while looking up
that slot as it will be registered in user space via its odd end
address. Not sure yet, but maybe it's better to round up at the call
sites of kvm_set_user_memory_region.
Jan
--
Siemens AG, Corporate Technology, CT SE 2
Corporate Competence Center Embedded Linux
- Re: [Qemu-devel] [PATCH 7/7] Fake dirty loggin when it's not there, (continued)
- Re: [Qemu-devel] [PATCH 7/7] Fake dirty loggin when it's not there, Stefano Stabellini, 2009/07/17
- Re: [Qemu-devel] [PATCH 7/7] Fake dirty loggin when it's not there, Alexander Graf, 2009/07/17
- Re: [Qemu-devel] [PATCH 7/7] Fake dirty loggin when it's not there, Alexander Graf, 2009/07/17
- Re: [Qemu-devel] [PATCH 7/7] Fake dirty loggin when it's not there, Stefano Stabellini, 2009/07/17
- Re: [Qemu-devel] [PATCH 7/7] Fake dirty loggin when it's not there, Alexander Graf, 2009/07/17
- Re: [Qemu-devel] [PATCH 7/7] Fake dirty loggin when it's not there, Stefano Stabellini, 2009/07/17
- Re: [Qemu-devel] [PATCH 7/7] Fake dirty loggin when it's not there, Alexander Graf, 2009/07/17
- [Qemu-devel] Re: [PATCH 7/7] Fake dirty loggin when it's not there, Jan Kiszka, 2009/07/17
- [Qemu-devel] Re: [PATCH 7/7] Fake dirty loggin when it's not there, Alexander Graf, 2009/07/17
- [Qemu-devel] Re: [PATCH 7/7] Fake dirty loggin when it's not there, Jan Kiszka, 2009/07/17
- [Qemu-devel] Re: [PATCH 6/7] Set slots more carefully,
Jan Kiszka <=
- [Qemu-devel] Re: [PATCH 6/7] Set slots more carefully, Alexander Graf, 2009/07/17
- [Qemu-devel] Re: [PATCH 6/7] Set slots more carefully, Jan Kiszka, 2009/07/17
- [Qemu-devel] Re: [PATCH 6/7] Set slots more carefully, Alexander Graf, 2009/07/17
- [Qemu-devel] Re: [PATCH 6/7] Set slots more carefully, Jan Kiszka, 2009/07/17
- Re: [Qemu-devel] [PATCH 6/7] Set slots more carefully, Anthony Liguori, 2009/07/21
- Re: [Qemu-devel] [PATCH 3/7] Add mp_state to PPC CPU struct, Anthony Liguori, 2009/07/22
- Re: [Qemu-devel] [PATCH 3/7] Add mp_state to PPC CPU struct, Hollis Blanchard, 2009/07/22
- Re: [Qemu-devel] [PATCH 3/7] Add mp_state to PPC CPU struct, Anthony Liguori, 2009/07/22
- Re: [Qemu-devel] [PATCH 3/7] Add mp_state to PPC CPU struct, Hollis Blanchard, 2009/07/22
- [Qemu-devel] Re: [PATCH 3/7] Add mp_state to PPC CPU struct, Jan Kiszka, 2009/07/22