Am 14.08.2011 um 06:29 schrieb Anthony Liguori <[email protected]>:
> On 08/13/2011 10:05 PM, Avi Kivity wrote:
>> On 08/12/2011 06:48 AM, Anthony Liguori wrote:
>>>> target_phys_addr_t offset_in_region;
>>>> AddrRange addr;
>>>> uint8_t dirty_log_mask;
>>>> + bool readable;
>>> @@ -125,6 +125,7 @@ struct FlatRange {
>>>
>>> In a follow up, it might be good to add a comment explaining that this
>>> whole readable thing is not just an optimization, but a hard
>>> requirement for KVM in order to be able to execute code from ROM.
>>
>> This has nothing to do with kvm (in fact, I think we cannot support it
>> under kvm with current interfaces). It's there to support devices that
>> sometimes act as RAM and sometimes as mmio.
>
> That is not a functional behavior but rather an optimization. Functionally
> speaking, there is absolutely no different between "acting as RAM" and
> "acting as mmio".
>
> But you cannot remove the optimization because of the aforementioned
> limitation in KVM.
>
> Maybe it's just me but I find this to be a very subtle detail so a comment
> would be helpful :-)
Well, TCG also can't execute from MMIO, right? So it's at least not KVM
specific - TCG also gets the speedup from mapping roms as ram.
Alex
>
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
