You forgot to CC Peter and LKML.
Luwei Kang <[email protected]> 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
> -----Original Message-----
> From: Alexander Shishkin <[email protected]>
> Sent: Friday, November 15, 2019 4:38 PM
> To: Kang, Luwei <[email protected]>; [email protected]
> Cc: [email protected]; [email protected]; Kang, Luwei <[email protected]>; [email protected]; linux-
> [email protected]; [email protected]
> 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 <[email protected]> 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