From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752024AbaLRDRE (ORCPT ); Wed, 17 Dec 2014 22:17:04 -0500 Received: from mga01.intel.com ([192.55.52.88]:26154 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751628AbaLRDRB convert rfc822-to-8bit (ORCPT ); Wed, 17 Dec 2014 22:17:01 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.07,598,1413270000"; d="scan'208";a="639422142" From: "Wu, Feng" To: Paolo Bonzini , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , "x86@kernel.org" , Gleb Natapov , "dwmw2@infradead.org" , "joro@8bytes.org" , Alex Williamson CC: "iommu@lists.linux-foundation.org" , "linux-kernel@vger.kernel.org" , KVM list , Eric Auger , "Wu, Feng" Subject: RE: [v3 24/26] KVM: Update Posted-Interrupts Descriptor when vCPU is blocked Thread-Topic: [v3 24/26] KVM: Update Posted-Interrupts Descriptor when vCPU is blocked Thread-Index: AQHQGhyWLl8wct5qTkqk5aJ8G13nUpyUpILQ Date: Thu, 18 Dec 2014 03:16:53 +0000 Message-ID: References: <1418397300-10870-1-git-send-email-feng.wu@intel.com> <1418397300-10870-25-git-send-email-feng.wu@intel.com> <5491B8DD.5020000@redhat.com> In-Reply-To: <5491B8DD.5020000@redhat.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > -----Original Message----- > From: kvm-owner@vger.kernel.org [mailto:kvm-owner@vger.kernel.org] On > Behalf Of Paolo Bonzini > Sent: Thursday, December 18, 2014 1:10 AM > To: Wu, Feng; Thomas Gleixner; Ingo Molnar; H. Peter Anvin; x86@kernel.org; > Gleb Natapov; Paolo Bonzini; dwmw2@infradead.org; joro@8bytes.org; Alex > Williamson > Cc: iommu@lists.linux-foundation.org; linux-kernel@vger.kernel.org; KVM list; > Eric Auger > Subject: Re: [v3 24/26] KVM: Update Posted-Interrupts Descriptor when vCPU > is blocked > > > > On 12/12/2014 16:14, Feng Wu wrote: > > This patch updates the Posted-Interrupts Descriptor when vCPU > > is blocked. > > > > pre-block: > > - Add the vCPU to the blocked per-CPU list > > - Clear 'SN' > > Should SN be already clear (and NV set to POSTED_INTR_VECTOR)? I think the SN bit should be clear here, Adding it here is just to make sure SN is clear when vCPU is blocked, so it can receive wakeup notification event later. > Can it > happen that you go from sched-out to blocked without doing a sched-in first? > I cannot imagine this scenario, can you please be more specific? Thanks a lot! > In fact, if this is possible, what happens if vcpu->preempted && > vcpu->blocked? In fact, vcpu->preempted && vcpu->blocked happens sometimes, but I think there is no issues. Please refer to the following case: kvm_vcpu_block() -> vcpu->blocked = true; -> prepare_to_wait(&vcpu->wq, &wait, TASK_INTERRUPTIBLE); before schedule() is called, this vcpu is woken up by another guy, so the state of the vcpu associated thread is changed to TASK_RUNNING, then preemption happens after interrupts or the following schedule() is hit, this will call kvm_sched_out(), in which current->state == TASK_RUNNING and vcpu->preempted is set to true. So now vcpu->preempted and vcpu->blocked are both true. In vmx_vcpu_put(), we will check vcpu->preempted first, so the vCPU will not be blocked, and the vcpu->blocked will be set the false in vmx_vcpu_load(). But maybe I need do a little change to the vmx_vcpu_load() like below: /* * Delete the vCPU from the related wakeup queue * if we are resuming from blocked state */ if (vcpu->blocked) { vcpu->blocked = false; + /* if wakeup_cpu == -1, the vcpu is currently not blocked on any + pCPU, don't need dequeue here */ + if (vcpu->wakeup_cpu != -1) { spin_lock_irqsave(&per_cpu(blocked_vcpu_on_cpu_lock, vcpu->wakeup_cpu), flags); list_del(&vcpu->blocked_vcpu_list); spin_unlock_irqrestore(&per_cpu(blocked_vcpu_on_cpu_lock, vcpu->wakeup_cpu), flags); vcpu->wakeup_cpu = -1; + } } Any ideas about this? Thanks a lot! Thanks, Feng -> schedule(); > > > - Set 'NV' to POSTED_INTR_WAKEUP_VECTOR > > > > post-block: > > - Remove the vCPU from the per-CPU list > > Paolo > > > Signed-off-by: Feng Wu > -- > To unsubscribe from this list: send the line "unsubscribe kvm" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html