[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 17/22] kvm: Move irqchip event processing out of
From: |
Jan Kiszka |
Subject: |
[Qemu-devel] Re: [PATCH 17/22] kvm: Move irqchip event processing out of inner loop |
Date: |
Mon, 31 Jan 2011 12:36:54 +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 2011-01-31 11:08, Avi Kivity wrote:
> On 01/27/2011 03:10 PM, Jan Kiszka wrote:
>> Align with qemu-kvm and prepare for IO exit fix: There is no need to run
>> kvm_arch_process_irqchip_events in the inner VCPU loop. Any state change
>> this service processes will first cause an exit from kvm_cpu_exec
>> anyway. And we will have to reenter the kernel on IO exits
>> unconditionally, something that the current logic prevents.
>>
>> Signed-off-by: Jan Kiszka<address@hidden>
>> ---
>> kvm-all.c | 11 ++++++-----
>> 1 files changed, 6 insertions(+), 5 deletions(-)
>>
>> diff --git a/kvm-all.c b/kvm-all.c
>> index 5bfa8c0..46ecc1c 100644
>> --- a/kvm-all.c
>> +++ b/kvm-all.c
>> @@ -892,6 +892,12 @@ int kvm_cpu_exec(CPUState *env)
>>
>> DPRINTF("kvm_cpu_exec()\n");
>>
>> + if (kvm_arch_process_irqchip_events(env)) {
>> + env->exit_request = 0;
>> + env->exception_index = EXCP_HLT;
>> + return 0;
>> + }
>> +
>> do {
>> #ifndef CONFIG_IOTHREAD
>> if (env->exit_request) {
>> @@ -901,11 +907,6 @@ int kvm_cpu_exec(CPUState *env)
>> }
>
> We check for ->exit_request here
>
>> #endif
>>
>> - if (kvm_arch_process_irqchip_events(env)) {
>> - ret = 0;
>> - break;
>> - }
>> -
>
> But this checks for ->interrupt_request. What ensures that we exit when
> ->interrupt_request is set?
Good question, need to check again. But if that turns out to be an
issue, qemu-kvm would be broken as well. I'm just aligning the code here.
Jan
--
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux
- [Qemu-devel] Re: [PATCH 01/22] Prevent abortion on multiple VCPU kicks, (continued)
[Qemu-devel] [PATCH 09/22] Refactor signal setup functions in cpus.c, Jan Kiszka, 2011/01/27
[Qemu-devel] [PATCH 12/22] kvm: Call qemu_kvm_eat_signals also under !CONFIG_IOTHREAD, Jan Kiszka, 2011/01/27
[Qemu-devel] [PATCH 20/22] kvm: Leave kvm_cpu_exec directly after KVM_EXIT_SHUTDOWN, Jan Kiszka, 2011/01/27
[Qemu-devel] [PATCH 22/22] Fix a few coding style violations in cpus.c, Jan Kiszka, 2011/01/27
[Qemu-devel] [PATCH 17/22] kvm: Move irqchip event processing out of inner loop, Jan Kiszka, 2011/01/27
- [Qemu-devel] Re: [PATCH 17/22] kvm: Move irqchip event processing out of inner loop, Avi Kivity, 2011/01/31
- [Qemu-devel] Re: [PATCH 17/22] kvm: Move irqchip event processing out of inner loop,
Jan Kiszka <=
- [Qemu-devel] Re: [PATCH 17/22] kvm: Move irqchip event processing out of inner loop, Jan Kiszka, 2011/01/31
- [Qemu-devel] Re: [PATCH 17/22] kvm: Move irqchip event processing out of inner loop, Jan Kiszka, 2011/01/31
- [Qemu-devel] Re: [PATCH 17/22] kvm: Move irqchip event processing out of inner loop, Gleb Natapov, 2011/01/31
- [Qemu-devel] Re: [PATCH 17/22] kvm: Move irqchip event processing out of inner loop, Jan Kiszka, 2011/01/31
- [Qemu-devel] Re: [PATCH 17/22] kvm: Move irqchip event processing out of inner loop, Jan Kiszka, 2011/01/31
- [Qemu-devel] Re: [PATCH 17/22] kvm: Move irqchip event processing out of inner loop, Gleb Natapov, 2011/01/31
- [Qemu-devel] Re: [PATCH 17/22] kvm: Move irqchip event processing out of inner loop, Jan Kiszka, 2011/01/31
- [Qemu-devel] Re: [PATCH 17/22] kvm: Move irqchip event processing out of inner loop, Gleb Natapov, 2011/01/31
- [Qemu-devel] [PATCH v2 17&18/22] kvm: Unconditionally reenter kernel after IO exits, Jan Kiszka, 2011/01/31
[Qemu-devel] [PATCH 07/22] kvm: Handle kvm_init_vcpu errors, Jan Kiszka, 2011/01/27