[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 07/22] memory: add address_space_translate
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 07/22] memory: add address_space_translate |
Date: |
Thu, 20 Jun 2013 16:53:31 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130514 Thunderbird/17.0.6 |
Il 20/06/2013 16:43, Peter Maydell ha scritto:
>>> >> There are other places in memory.c which do an int128_get64()
>>> >> on mr->size, which also look suspicious...
>> >
>> > They are all on I/O regions so they are safe
> Not entirely sure I understand this. There's no particular
> reason I can't create a 2^64 sized I/O memory region
> and put it in an address space, is there?
I think there are problems in the core if you do that (probably part of
it is fixed now). Still, in cases like this:
memory_region_add_coalescing(mr, 0, int128_get64(mr->size));
the API simply doesn't support it.
Paolo