[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 10/21] memory: make section size a 128-bit integ
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 10/21] memory: make section size a 128-bit integer |
Date: |
Fri, 31 May 2013 09:12:47 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130514 Thunderbird/17.0.6 |
Il 31/05/2013 08:56, Alexey Kardashevskiy ha scritto:
> On 05/31/2013 07:16 AM, Paolo Bonzini wrote:
>> So far, the size of all regions passed to listeners could fit in 64 bits,
>> because artificial regions (containers and aliases) are eliminated by
>> the memory core, leaving only device regions which have reasonable sizes
>>
>> An IOMMU however cannot be eliminated by the memory core, and may have
>> an artificial size, hence we may need 65 bits to represent its size.
>>
>> Signed-off-by: Paolo Bonzini <address@hidden>
>
> [...]
>
>> diff --git a/kvm-all.c b/kvm-all.c
>> index 8222729..86c9af3 100644
>> --- a/kvm-all.c
>> +++ b/kvm-all.c
>> @@ -316,7 +316,7 @@ static void kvm_log_start(MemoryListener *listener,
>> int r;
>>
>> r = kvm_dirty_pages_log_change(section->offset_within_address_space,
>> - section->size, true);
>> + int128_get64(section->size), true);
>> if (r < 0) {
>> abort();
>> }
>> @@ -328,7 +328,7 @@ static void kvm_log_stop(MemoryListener *listener,
>> int r;
>>
>> r = kvm_dirty_pages_log_change(section->offset_within_address_space,
>> - section->size, false);
>> + int128_get64(section->size), false);
>> if (r < 0) {
>> abort();
>> }
>> @@ -366,7 +366,8 @@ static int
>> kvm_get_dirty_pages_log_range(MemoryRegionSection *section,
>> unsigned int i, j;
>> unsigned long page_number, c;
>> hwaddr addr, addr1;
>> - unsigned int len = ((section->size / getpagesize()) + HOST_LONG_BITS -
>> 1) / HOST_LONG_BITS;
>> + unsigned int pages = int128_get64(section->size) / getpagesize();
>> + unsigned int len = (pages + HOST_LONG_BITS - 1) / HOST_LONG_BITS;
>> unsigned long hpratio = getpagesize() / TARGET_PAGE_SIZE;
>>
>> /*
>> @@ -409,7 +410,7 @@ static int
>> kvm_physical_sync_dirty_bitmap(MemoryRegionSection *section)
>> KVMSlot *mem;
>> int ret = 0;
>> hwaddr start_addr = section->offset_within_address_space;
>> - hwaddr end_addr = start_addr + section->size;
>> + hwaddr end_addr = start_addr + int128_get64(section->size);
>>
>> d.dirty_bitmap = NULL;
>> while (start_addr < end_addr) {
>> @@ -619,7 +620,7 @@ static void kvm_set_phys_mem(MemoryRegionSection
>> *section, bool add)
>> MemoryRegion *mr = section->mr;
>> bool log_dirty = memory_region_is_logging(mr);
>> hwaddr start_addr = section->offset_within_address_space;
>> - ram_addr_t size = section->size;
>> + ram_addr_t size = int128_get64(section->size);
>> void *ram = NULL;
>> unsigned delta;
>>
>
> Tried to replay part2 and part3 on qemu.org/master (I assume part1 is
> already there), part2 played well, part3 failed. What did I do wrong? Thanks.
Needs a rebase, it seems. I'll push to the iommu branch later today and
repost this patch.
Paolo
- Re: [Qemu-devel] [PATCH 05/21] exec: Implement subpage_read/write via address_space_rw, (continued)
- [Qemu-devel] [PATCH 07/21] Revert "memory: limit sections in the radix tree to the actual address space size", Paolo Bonzini, 2013/05/30
- [Qemu-devel] [PATCH 06/21] exec: return MemoryRegion from address_space_translate, Paolo Bonzini, 2013/05/30
- [Qemu-devel] [PATCH 08/21] Revert "s390x: reduce TARGET_PHYS_ADDR_SPACE_BITS to 62", Paolo Bonzini, 2013/05/30
- [Qemu-devel] [PATCH 09/21] exec: reorganize mem_add to match Int128 version, Paolo Bonzini, 2013/05/30
- [Qemu-devel] [PATCH 10/21] memory: make section size a 128-bit integer, Paolo Bonzini, 2013/05/30
- [Qemu-devel] [PATCH 11/21] memory: iommu support, Paolo Bonzini, 2013/05/30
- [Qemu-devel] [PATCH 12/21] memory: Add iommu map/unmap notifiers, Paolo Bonzini, 2013/05/30
- [Qemu-devel] [PATCH 13/21] vfio: abort if an emulated iommu is used, Paolo Bonzini, 2013/05/30
- [Qemu-devel] [PATCH 15/21] spapr: make IOMMU translation go through IOMMUTLBEntry, Paolo Bonzini, 2013/05/30
- [Qemu-devel] [PATCH 14/21] spapr: convert TCE API to use an opaque type, Paolo Bonzini, 2013/05/30
- [Qemu-devel] [PATCH 16/21] spapr: use memory core for iommu support, Paolo Bonzini, 2013/05/30