Re: [PATCH v3 06/16] perf/core: Rework guest callbacks to prepare for static_call support

From: Boris Ostrovsky
Date: Wed Sep 22 2021 - 14:33:33 EST



On 9/21/21 8:05 PM, Sean Christopherson wrote:
> From: Like Xu <like.xu@xxxxxxxxxxxxxxx>
>
> To prepare for using static_calls to optimize perf's guest callbacks,
> replace ->is_in_guest and ->is_user_mode with a new multiplexed hook
> ->state, tweak ->handle_intel_pt_intr to play nice with being called when
> there is no active guest, and drop "guest" from ->is_in_guest.
>
> Return '0' from ->state and ->handle_intel_pt_intr to indicate "not in
> guest" so that DEFINE_STATIC_CALL_RET0 can be used to define the static
> calls, i.e. no callback == !guest.
>
> Suggested-by: Peter Zijlstra (Intel) <peterz@xxxxxxxxxxxxx>
> Originally-by: Peter Zijlstra (Intel) <peterz@xxxxxxxxxxxxx>
> Signed-off-by: Like Xu <like.xu@xxxxxxxxxxxxxxx>
> Signed-off-by: Zhu Lingshan <lingshan.zhu@xxxxxxxxx>
> [sean: extracted from static_call patch, fixed get_ip() bug, wrote changelog]
> Signed-off-by: Sean Christopherson <seanjc@xxxxxxxxxx>


For Xen bits


Reviewed-by: Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>