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;
From:   "Kang, Luwei" <luwei.kang@intel.com>
To:     Alexander Shishkin <alexander.shishkin@linux.intel.com>,
        "kvm-pmu@eclists.intel.com" <kvm-pmu@eclists.intel.com>
CC:     "ak@linux.intel.com" <ak@linux.intel.com>,
        "kan.liang@linux.intel.com" <kan.liang@linux.intel.com>,
        "peterz@infradead.org" <peterz@infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH v1 0/4] Enable PEBS when only have PEBS via PT w/o DS
Thread-Topic: [PATCH v1 0/4] Enable PEBS when only have PEBS via PT w/o DS
Thread-Index: AQHVmeoIPfTHl4y/ukGghigk57hp6aeLZhIAgAPdu5A=
Date:   Sun, 17 Nov 2019 11:51:49 +0000
Message-ID: <82D7661F83C1A047AF7DC287873BF1E17383C77D@SHSMSX104.ccr.corp.intel.com>
References: <1573672574-25247-1-git-send-email-luwei.kang@intel.com>
 <87h835edqn.fsf@ashishki-desk.ger.corp.intel.com>
In-Reply-To: <87h835edqn.fsf@ashishki-desk.ger.corp.intel.com>
Accept-Language: en-US
Content-Language: en-US
dlp-product: dlpe-windows
dlp-version: 11.2.0.6
dlp-reaction: no-action
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 8BIT
MIME-Version: 1.0
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


> -----Original Message-----
> From: Alexander Shishkin <alexander.shishkin@linux.intel.com>
> Sent: Friday, November 15, 2019 4:38 PM
> To: Kang, Luwei <luwei.kang@intel.com>; kvm-pmu@eclists.intel.com
> Cc: ak@linux.intel.com; kan.liang@linux.intel.com; Kang, Luwei <luwei.kang@intel.com>; peterz@infradead.org; linux-
> kernel@vger.kernel.org; alexander.shishkin@linux.intel.com
> Subject: Re: [PATCH v1 0/4] Enable PEBS when only have PEBS via PT w/o DS
> 
> You forgot to CC Peter and LKML.
> 
> Luwei Kang <luwei.kang@intel.com> writes:
> 
> > This patchset is purely perf event system changes that to enable the
> > PEBS when the system only supports PEBS via Intel PT w/o DS.
> > Currently, there don't have such hardware which only supports PEBS via
> > PT w/o DS but it is possible in KVM guest. In Tremont Atom platforms,
> > PEBS via PT is the only way to enabled PEBS in KVM guest.
> 
> I don't understand what this says. If PEBS-via-PT is available and DS is not available, what happens and why?

This is an internal review before sending it to community.
In current software implementation, PEBS depends on the HW support of DS feature even if we have PEBS via PT( e.g. in intel_ds_init() function). This patchset just removes the dependency of PEBS and DS because there have a new path PEBS via Intel PT. I think there don't have any problem in host, but PEBS may not work in KVM guest when only have PEBS via PT w/o DS.

Thanks,
Luwei Kang

> 
> Regards,
> --
> Alex