Re: [PATCH 3/3] drm/xe/pf: Add the definition of VF resfix abort event

["K V P, Satyanarayana" <satyanarayana.k.v.p@intel.com>]

On 24-Jun-26 3:26 PM, Michał Winiarski wrote:
> The VF RESFIX_ABORT event is similar to FIXUP_DONE event, in that
> there's no handling required from PF perspective.
> Add the definition to avoid the "unrecognized event" message.
>
> Signed-off-by: Michał Winiarski <michal.winiarski@intel.com>
> ---
>   drivers/gpu/drm/xe/abi/guc_actions_sriov_abi.h | 2 ++
>   drivers/gpu/drm/xe/xe_gt_sriov_pf_control.c    | 2 ++
>   2 files changed, 4 insertions(+)
>
> diff --git a/drivers/gpu/drm/xe/abi/guc_actions_sriov_abi.h b/drivers/gpu/drm/xe/abi/guc_actions_sriov_abi.h
> index 8665b8c5b7336..59c52b7c21ab6 100644
> --- a/drivers/gpu/drm/xe/abi/guc_actions_sriov_abi.h
> +++ b/drivers/gpu/drm/xe/abi/guc_actions_sriov_abi.h
> @@ -230,6 +230,7 @@
>    *  |   |       |   - _`GUC_PF_NOTIFY_VF_PAUSE_DONE` = 3                       |
>    *  |   |       |   - _`GUC_PF_NOTIFY_VF_FIXUP_DONE` = 4                       |
>    *  |   |       |   - _`GUC_PF_NOTIFY_VF_PAUSE_ABORTED` = 5                    |
> + *  |   |       |   - _`GUC_PF_NOTIFY_VF_RESFIX_ABORTED` = 6                   |
>    *  +---+-------+--------------------------------------------------------------+
>    */
>   #define GUC_ACTION_GUC2PF_VF_STATE_NOTIFY		0x5106u
> @@ -244,6 +245,7 @@
>   #define   GUC_PF_NOTIFY_VF_PAUSE_DONE			3u
>   #define   GUC_PF_NOTIFY_VF_FIXUP_DONE			4u
>   #define   GUC_PF_NOTIFY_VF_PAUSE_ABORTED		5u
> +#define   GUC_PF_NOTIFY_VF_RESFIX_ABORTED		6u
>   
>   /**
>    * DOC: VF2GUC_MATCH_VERSION
> diff --git a/drivers/gpu/drm/xe/xe_gt_sriov_pf_control.c b/drivers/gpu/drm/xe/xe_gt_sriov_pf_control.c
> index 82957c596b683..2762fff78f113 100644
> --- a/drivers/gpu/drm/xe/xe_gt_sriov_pf_control.c
> +++ b/drivers/gpu/drm/xe/xe_gt_sriov_pf_control.c
> @@ -1986,6 +1986,8 @@ static int pf_handle_vf_event(struct xe_gt *gt, u32 vfid, u32 eventid)
>   	case GUC_PF_NOTIFY_VF_PAUSE_ABORTED:
>   		pf_handle_vf_pause_aborted(gt, vfid);
>   		break;
> +	case GUC_PF_NOTIFY_VF_RESFIX_ABORTED:
> +		break;
Better to log a message if the event is not handled.
>   	default:
>   		xe_gt_sriov_notice(gt, "unrecognized VF%u event %#x\n", vfid, eventid);
>   	}