Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932589AbbHDArJ (ORCPT ); Mon, 3 Aug 2015 20:47:09 -0400 Received: from mga02.intel.com ([134.134.136.20]:15771 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932218AbbHDArH convert rfc822-to-8bit (ORCPT ); Mon, 3 Aug 2015 20:47:07 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.15,605,1432623600"; d="scan'208";a="777146021" From: "Zhang, Yang Z" To: Paolo Bonzini , "linux-kernel@vger.kernel.org" , "kvm@vger.kernel.org" CC: "alex.williamson@redhat.com" , "srutherford@intel.com" , "Gudimetla, Giridhar Kumar" Subject: RE: [PATCH 1/2] KVM: x86: set TMR when the interrupt is accepted Thread-Topic: [PATCH 1/2] KVM: x86: set TMR when the interrupt is accepted Thread-Index: AQHQygPaQ9RT5A1SM0ONvbyHVLL0wp30qLewgAALZQCABN7OgP//2pKAgACM57D//6FOAIABaYBA Date: Tue, 4 Aug 2015 00:46:10 +0000 Message-ID: References: <1438177055-26764-1-git-send-email-pbonzini@redhat.com> <1438177055-26764-2-git-send-email-pbonzini@redhat.com> <55BB2B62.1030605@redhat.com> <55BF21E1.5030100@redhat.com> <55BF48A4.5030409@redhat.com> In-Reply-To: <55BF48A4.5030409@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 Content-Length: 1877 Lines: 51 Paolo Bonzini wrote on 2015-08-03: > > > On 03/08/2015 12:23, Zhang, Yang Z wrote: >>> In any case, the TMR behavior introduced by the APICv patches is >>> completely different from the hardware behavior, so it has to be fixed. >> >> But any real problem with it? > > It is a problem for split irqchip, where the EOI exit bitmap can be > inferred from the IOAPIC routes but the TMR cannot. The hardware > behavior on the other hand can be implemented purely within the LAPIC. So updating the TMR within LAPIC is the only solution to handle it? > >>> The alternative is to inject level-triggered interrupts >>> synchronously, without using posted interrupts. >>> >>> I'll write some testcases to understand the functioning of TMR in >>> the virtual-APIC page, but the manual seems clear to me. >> >> Currently, no existing hardware will use TMR and will not cause any >> problem.(That's the reason why we leave it in Xen).But we don't know >> whether future hardware will use it or not(SDM always keeps changing >> :)). > > But that would be covered by a different execution control (for > backwards compatibility). We'll get there when such a feature is introduced. Yes, we can leave it in future. But one concern is that it may hard to handle it at that time if someone also develops feature which rely on it (like current patch to split irqchip). > >> And per 24.11.4's description, the perfect solution is don't modify >> it. btw, IIRC, only TMR doesn't follow the rule. All other VMCS >> accesses are issued in right VMCS context. > > Yes, that's correct. It's just the TMR. > > Paolo Best regards, Yang -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/