[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 4/9] target-i386: Add infrastructure for repo
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] [PATCH v3 4/9] target-i386: Add infrastructure for reporting TPR MMIO accesses |
Date: |
Thu, 16 Feb 2012 16:29:07 +0100 |
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 |
On 2012-02-16 16:21, Avi Kivity wrote:
> On 02/14/2012 05:13 PM, Jan Kiszka wrote:
>> Note that KVM without in-kernel irqchip will report the address after
>> the instruction that triggered a write access. In contrast, read
>> accesses will return the precise information.
>>
>
> Well this is wierd. We could retro-doc one or the other behaviour, but
> this-on-read-but-that-on-write is just too strange.
>
> The documented way of dealing with this is to queue a signal and reenter
> the guest. kvm will perform anything it needs to complete the
> instruction (perhaps issuing more mmio, say if someone used movsd to
> read the APIC) and then exit on the signal. By then rip will point
> exactly after the instruction.
Hmm, true. And can trivially be changed (I'm injecting the event after
instruction completion). Will role out a new version.
Jan
--
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux
- [Qemu-devel] [PATCH v3 0/9] uq/master: TPR access optimization for Windows guests, Jan Kiszka, 2012/02/14
- [Qemu-devel] [PATCH v3 2/9] Remove useless casts from cpu iterators, Jan Kiszka, 2012/02/14
- [Qemu-devel] [PATCH v3 5/9] kvmvapic: Add option ROM, Jan Kiszka, 2012/02/14
- [Qemu-devel] [PATCH v3 8/9] optionsrom: Reserve space for checksum, Jan Kiszka, 2012/02/14
- [Qemu-devel] [PATCH v3 1/9] kvm: Set cpu_single_env only once, Jan Kiszka, 2012/02/14
- [Qemu-devel] [PATCH v3 6/9] kvmvapic: Introduce TPR access optimization for Windows guests, Jan Kiszka, 2012/02/14
- [Qemu-devel] [PATCH v3 4/9] target-i386: Add infrastructure for reporting TPR MMIO accesses, Jan Kiszka, 2012/02/14
- [Qemu-devel] [PATCH v3 7/9] kvmvapic: Simplify mp/up_set_tpr, Jan Kiszka, 2012/02/14
- [Qemu-devel] [PATCH v3 9/9] kvmvapic: Use optionrom helpers, Jan Kiszka, 2012/02/14
- [Qemu-devel] [PATCH v3 3/9] Allow to use pause_all_vcpus from VCPU context, Jan Kiszka, 2012/02/14