Received: by 2002:a25:31c3:0:0:0:0:0 with SMTP id x186csp393906ybx; Tue, 29 Oct 2019 21:07:50 -0700 (PDT) X-Google-Smtp-Source: APXvYqzxxCUpBk5aZFiWrScCnFiLCcUenjuv3YFTOijc2ziaHbJ88/3e7m1P3Xezh6SOiSA4WbZd X-Received: by 2002:a50:f288:: with SMTP id f8mr29209564edm.43.1572408470606; Tue, 29 Oct 2019 21:07:50 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1572408470; cv=none; d=google.com; s=arc-20160816; b=PH2TW9WoZV40CI0gPit0xWp7nzN5X6RFrUAmjUksFb2leDuh7ckwyAla1KPdxq8Qz3 QKWfh6uS+obkSM3nnGs5zmhEBud65ke+98lISm8djKiwudUVDrzvD58c2gpXfNdoaUss YRt6M2UpiXO1Xo+u99bdZ7DT31Q2MXMyf54au3hZ75NweTd2ODGvll4Mdt5GcZMr/OCG Lu+WXx+oTGFZ6sy7Rw6bcDnXOblutbt23pPIx6iHL//wiHsG8HHf1A7jpvpAhsRMNo5J AszZRPs8nkZrbbzgf19MjSlzqIT4fpZevEwPk7zlYpdlPcSIS00coEpAQJzyyvc7M/rW 5ltQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:content-transfer-encoding :dlp-reaction:dlp-version:dlp-product:content-language :accept-language:in-reply-to:references:message-id:date:thread-index :thread-topic:subject:cc:to:from; bh=LHNJb20vpMoszC+g89eeBiZhXTEx6tEN/9Axrin6RFw=; b=eaGPr3vUVrR+FeBSFxsbp5wmJK5Mz1gS0xeIbyPLotqQTCmumYg1YlgtpYoJc4hgWc ZlNYgRCFRRKEonKK4l5Q0w9WzCTMgww6B8anOyOWDmVnzHiFTvkpB4Sg35gH1L3s9MnO 5SBhhRAv7Uq8syal50qRIAZ/m2sG9q2fu5+z+PoFrmNpO+7OXq9h0o83NZxyK2V5HJgD fTwhhOkwvOkYXSCdK7Bx7E+3D8p+WnyZnLMasbfz9+U5pH9sto1JbTeZx0YMQB6jHWzq MlKnCtBF6jjKZ/yoxZTksEZ8bXePhQAStYpDRiBk5z/cxepwhbD4OKSBHWWLp/6U4VrL vVEA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id p15si496776ejo.212.2019.10.29.21.07.27; Tue, 29 Oct 2019 21:07:50 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727071AbfJ3EGq convert rfc822-to-8bit (ORCPT + 99 others); Wed, 30 Oct 2019 00:06:46 -0400 Received: from mga05.intel.com ([192.55.52.43]:56188 "EHLO mga05.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727006AbfJ3EGp (ORCPT ); Wed, 30 Oct 2019 00:06:45 -0400 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga105.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 29 Oct 2019 21:06:45 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.68,246,1569308400"; d="scan'208";a="205679945" Received: from fmsmsx104.amr.corp.intel.com ([10.18.124.202]) by FMSMGA003.fm.intel.com with ESMTP; 29 Oct 2019 21:06:45 -0700 Received: from fmsmsx151.amr.corp.intel.com (10.18.125.4) by fmsmsx104.amr.corp.intel.com (10.18.124.202) with Microsoft SMTP Server (TLS) id 14.3.439.0; Tue, 29 Oct 2019 21:06:44 -0700 Received: from shsmsx107.ccr.corp.intel.com (10.239.4.96) by FMSMSX151.amr.corp.intel.com (10.18.125.4) with Microsoft SMTP Server (TLS) id 14.3.439.0; Tue, 29 Oct 2019 21:06:44 -0700 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.127]) by SHSMSX107.ccr.corp.intel.com ([169.254.9.63]) with mapi id 14.03.0439.000; Wed, 30 Oct 2019 12:06:42 +0800 From: "Kang, Luwei" To: Peter Zijlstra CC: "kvm@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "pbonzini@redhat.com" , "rkrcmar@redhat.com" , "Christopherson, Sean J" , "vkuznets@redhat.com" , "wanpengli@tencent.com" , "jmattson@google.com" , "joro@8bytes.org" , "tglx@linutronix.de" , "mingo@redhat.com" , "bp@alien8.de" , "hpa@zytor.com" , "x86@kernel.org" , "ak@linux.intel.com" , "thomas.lendacky@amd.com" , "acme@kernel.org" , "mark.rutland@arm.com" , "alexander.shishkin@linux.intel.com" , "jolsa@redhat.com" , "namhyung@kernel.org" Subject: RE: [PATCH v1 4/8] KVM: x86: Aviod clear the PEBS counter when PEBS enabled in guest Thread-Topic: [PATCH v1 4/8] KVM: x86: Aviod clear the PEBS counter when PEBS enabled in guest Thread-Index: AQHVjLd4hZIWJ8IukEOg3V+5ZZvUU6dxMmCAgAE4tMA= Date: Wed, 30 Oct 2019 04:06:42 +0000 Message-ID: <82D7661F83C1A047AF7DC287873BF1E173835B24@SHSMSX104.ccr.corp.intel.com> References: <1572217877-26484-1-git-send-email-luwei.kang@intel.com> <1572217877-26484-5-git-send-email-luwei.kang@intel.com> <20191029145553.GL4097@hirez.programming.kicks-ass.net> In-Reply-To: <20191029145553.GL4097@hirez.programming.kicks-ass.net> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ctpclassification: CTP_NT x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiMDAzMWRiNDgtOWQwOC00YzAxLThhNDktM2Q3MTJjMDlkOTAxIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiOFZESXpSTkRXSmJNT1NjR3dkWlwvSHRWTElYWk1xWHhwN1wvYWpTaE5aSFk2bTB0WThsZmNpRUJ2Unk3ZGNiaHhBIn0= dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action 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 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > > This patch introduce a parameter that avoid clear the PEBS event > > counter while running in the guest. The performance counter which use > > for PEBS event can be enabled through VM-entry when PEBS is enabled in > > guest by PEBS output to Intel PT. > > Please write coherent Changelogs. I have no clue what you're trying to say. > > Also, maybe this attrocious coding style is accepted in KVM, but I'm not having it. Pretty much all your linebreaks and subsequent > indenting is against style. Sorry. I mean the performance counter for PEBS event must be disabled before VM-entry at present. After PEBS enabled in guest by PEBS via PT, we don't need to disable the PEBS counters. We be corrected in next version. Thanks, Luwei Kang