linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: [PATCH v1 0/4] Enable PEBS when only have PEBS via PT w/o DS
       [not found] <1573672574-25247-1-git-send-email-luwei.kang@intel.com>
@ 2019-11-15  8:38 ` Alexander Shishkin
  2019-11-17 11:51   ` Kang, Luwei
  0 siblings, 1 reply; 2+ messages in thread
From: Alexander Shishkin @ 2019-11-15  8:38 UTC (permalink / raw)
  To: Luwei Kang, kvm-pmu
  Cc: ak, kan.liang, Luwei Kang, peterz, linux-kernel, alexander.shishkin

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?

Regards,
--
Alex

^ permalink raw reply	[flat|nested] 2+ messages in thread

* RE: [PATCH v1 0/4] Enable PEBS when only have PEBS via PT w/o DS
  2019-11-15  8:38 ` [PATCH v1 0/4] Enable PEBS when only have PEBS via PT w/o DS Alexander Shishkin
@ 2019-11-17 11:51   ` Kang, Luwei
  0 siblings, 0 replies; 2+ messages in thread
From: Kang, Luwei @ 2019-11-17 11:51 UTC (permalink / raw)
  To: Alexander Shishkin, kvm-pmu; +Cc: ak, kan.liang, peterz, linux-kernel



> -----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

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-11-17 11:51 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <1573672574-25247-1-git-send-email-luwei.kang@intel.com>
2019-11-15  8:38 ` [PATCH v1 0/4] Enable PEBS when only have PEBS via PT w/o DS Alexander Shishkin
2019-11-17 11:51   ` Kang, Luwei

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).