[PATCH v2 01/12] drm/xe/pxp: Initialize PXP structure and KCR reg
John Harrison
john.c.harrison at intel.com
Fri Oct 4 20:29:25 UTC 2024
On 8/16/2024 12:00, Daniele Ceraolo Spurio wrote:
> As the first step towards adding PXP support, hook in the PXP init
> function, allocate the PXP structure and initialize the KCR register to
> allow PXP HWDRM sessions.
>
> Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
> ---
> drivers/gpu/drm/xe/Makefile | 1 +
> .../xe/compat-i915-headers/pxp/intel_pxp.h | 4 +-
> drivers/gpu/drm/xe/regs/xe_pxp_regs.h | 17 +++
> drivers/gpu/drm/xe/xe_device.c | 6 +
> drivers/gpu/drm/xe/xe_device_types.h | 8 +-
> drivers/gpu/drm/xe/xe_pci.c | 2 +
> drivers/gpu/drm/xe/xe_pxp.c | 103 ++++++++++++++++++
> drivers/gpu/drm/xe/xe_pxp.h | 15 +++
> drivers/gpu/drm/xe/xe_pxp_types.h | 28 +++++
> 9 files changed, 180 insertions(+), 4 deletions(-)
> create mode 100644 drivers/gpu/drm/xe/regs/xe_pxp_regs.h
> create mode 100644 drivers/gpu/drm/xe/xe_pxp.c
> create mode 100644 drivers/gpu/drm/xe/xe_pxp.h
> create mode 100644 drivers/gpu/drm/xe/xe_pxp_types.h
>
> diff --git a/drivers/gpu/drm/xe/Makefile b/drivers/gpu/drm/xe/Makefile
> index e11392b5dd3d..9e007e59de83 100644
> --- a/drivers/gpu/drm/xe/Makefile
> +++ b/drivers/gpu/drm/xe/Makefile
> @@ -83,6 +83,7 @@ xe-y += xe_bb.o \
> xe_preempt_fence.o \
> xe_pt.o \
> xe_pt_walk.o \
> + xe_pxp.o \
> xe_query.o \
> xe_range_fence.o \
> xe_reg_sr.o \
> diff --git a/drivers/gpu/drm/xe/compat-i915-headers/pxp/intel_pxp.h b/drivers/gpu/drm/xe/compat-i915-headers/pxp/intel_pxp.h
> index c2c30ece8f77..881680727452 100644
> --- a/drivers/gpu/drm/xe/compat-i915-headers/pxp/intel_pxp.h
> +++ b/drivers/gpu/drm/xe/compat-i915-headers/pxp/intel_pxp.h
> @@ -10,9 +10,9 @@
> #include <linux/types.h>
>
> struct drm_i915_gem_object;
> -struct intel_pxp;
> +struct xe_pxp;
>
> -static inline int intel_pxp_key_check(struct intel_pxp *pxp,
> +static inline int intel_pxp_key_check(struct xe_pxp *pxp,
> struct drm_i915_gem_object *obj,
> bool assign)
> {
> diff --git a/drivers/gpu/drm/xe/regs/xe_pxp_regs.h b/drivers/gpu/drm/xe/regs/xe_pxp_regs.h
> new file mode 100644
> index 000000000000..d67cf210d23d
> --- /dev/null
> +++ b/drivers/gpu/drm/xe/regs/xe_pxp_regs.h
> @@ -0,0 +1,17 @@
> +/* SPDX-License-Identifier: MIT */
> +/*
> + * Copyright(c) 2024, Intel Corporation. All rights reserved.
> + */
> +
> +#ifndef __XE_PXP_REGS_H__
> +#define __XE_PXP_REGS_H__
> +
> +#include "regs/xe_regs.h"
> +
> +/* The following registers are only valid on platforms with a media GT */
> +
> +/* KCR enable/disable control */
> +#define KCR_INIT XE_REG(0x3860f0)
> +#define KCR_INIT_ALLOW_DISPLAY_ME_WRITES REG_BIT(14)
> +
> +#endif /* __XE_PXP_REGS_H__ */
> diff --git a/drivers/gpu/drm/xe/xe_device.c b/drivers/gpu/drm/xe/xe_device.c
> index 206328387150..807a15c49a81 100644
> --- a/drivers/gpu/drm/xe/xe_device.c
> +++ b/drivers/gpu/drm/xe/xe_device.c
> @@ -46,6 +46,7 @@
> #include "xe_pat.h"
> #include "xe_pcode.h"
> #include "xe_pm.h"
> +#include "xe_pxp.h"
> #include "xe_query.h"
> #include "xe_sriov.h"
> #include "xe_tile.h"
> @@ -730,6 +731,11 @@ int xe_device_probe(struct xe_device *xe)
> if (err)
> goto err_fini_oa;
>
> + /* A PXP init failure is not fatal */
> + err = xe_pxp_init(xe);
> + if (err && err != -EOPNOTSUPP)
> + drm_err(&xe->drm, "PXP initialization failed: %pe\n", ERR_PTR(err));
> +
> err = drm_dev_register(&xe->drm, 0);
> if (err)
> goto err_fini_display;
> diff --git a/drivers/gpu/drm/xe/xe_device_types.h b/drivers/gpu/drm/xe/xe_device_types.h
> index 16a24eadd94b..b00a78be3934 100644
> --- a/drivers/gpu/drm/xe/xe_device_types.h
> +++ b/drivers/gpu/drm/xe/xe_device_types.h
> @@ -35,6 +35,7 @@
>
> struct xe_ggtt;
> struct xe_pat_ops;
> +struct xe_pxp;
>
> #define XE_BO_INVALID_OFFSET LONG_MAX
>
> @@ -276,6 +277,8 @@ struct xe_device {
> u8 has_llc:1;
> /** @info.has_mmio_ext: Device has extra MMIO address range */
> u8 has_mmio_ext:1;
> + /** @info.has_pxp: Device has PXP support */
> + u8 has_pxp:1;
> /** @info.has_range_tlb_invalidation: Has range based TLB invalidations */
> u8 has_range_tlb_invalidation:1;
> /** @info.has_sriov: Supports SR-IOV */
> @@ -480,6 +483,9 @@ struct xe_device {
> /** @oa: oa observation subsystem */
> struct xe_oa oa;
>
> + /** @pxp: Encapsulate Protected Xe Path support */
> + struct xe_pxp *pxp;
> +
> /** @needs_flr_on_fini: requests function-reset on fini */
> bool needs_flr_on_fini;
>
> @@ -552,8 +558,6 @@ struct xe_device {
> unsigned int czclk_freq;
> unsigned int fsb_freq, mem_freq, is_ddr3;
> };
> -
> - void *pxp;
> #endif
> };
>
> diff --git a/drivers/gpu/drm/xe/xe_pci.c b/drivers/gpu/drm/xe/xe_pci.c
> index 3c34b032ebf4..d1453ba20dcd 100644
> --- a/drivers/gpu/drm/xe/xe_pci.c
> +++ b/drivers/gpu/drm/xe/xe_pci.c
> @@ -62,6 +62,7 @@ struct xe_device_desc {
> u8 has_heci_cscfi:1;
> u8 has_llc:1;
> u8 has_mmio_ext:1;
> + u8 has_pxp:1;
> u8 has_sriov:1;
> u8 skip_guc_pc:1;
> u8 skip_mtcfg:1;
> @@ -616,6 +617,7 @@ static int xe_info_init_early(struct xe_device *xe,
> xe->info.has_heci_cscfi = desc->has_heci_cscfi;
> xe->info.has_llc = desc->has_llc;
> xe->info.has_mmio_ext = desc->has_mmio_ext;
> + xe->info.has_pxp = desc->has_pxp;
> xe->info.has_sriov = desc->has_sriov;
> xe->info.skip_guc_pc = desc->skip_guc_pc;
> xe->info.skip_mtcfg = desc->skip_mtcfg;
> diff --git a/drivers/gpu/drm/xe/xe_pxp.c b/drivers/gpu/drm/xe/xe_pxp.c
> new file mode 100644
> index 000000000000..f974f74be1d5
> --- /dev/null
> +++ b/drivers/gpu/drm/xe/xe_pxp.c
> @@ -0,0 +1,103 @@
> +// SPDX-License-Identifier: MIT
> +/*
> + * Copyright(c) 2024 Intel Corporation.
> + */
> +
> +#include "xe_pxp.h"
> +
> +#include <drm/drm_managed.h>
> +
> +#include "xe_device_types.h"
> +#include "xe_force_wake.h"
> +#include "xe_gt.h"
> +#include "xe_gt_types.h"
> +#include "xe_mmio.h"
> +#include "xe_pxp_types.h"
> +#include "xe_uc_fw.h"
> +#include "regs/xe_pxp_regs.h"
> +
> +/**
> + * DOC: PXP
> + *
> + * PXP (Protected Xe Path) allows execution and flip to display of protected
> + * (i.e. encrypted) objects. This feature is currently only supported in
> + * integrated parts.
> + */
> +
> +static bool pxp_is_supported(const struct xe_device *xe)
> +{
> + return xe->info.has_pxp && IS_ENABLED(CONFIG_INTEL_MEI_GSC_PROXY);
> +}
> +
> +static int kcr_pxp_set_status(const struct xe_pxp *pxp, bool enable)
> +{
> + u32 val = enable ? _MASKED_BIT_ENABLE(KCR_INIT_ALLOW_DISPLAY_ME_WRITES) :
> + _MASKED_BIT_DISABLE(KCR_INIT_ALLOW_DISPLAY_ME_WRITES);
> + int err;
> +
> + err = xe_force_wake_get(gt_to_fw(pxp->gt), XE_FW_GT);
> + if (err)
> + return err;
> +
> + xe_mmio_write32(pxp->gt, KCR_INIT, val);
> + XE_WARN_ON(xe_force_wake_put(gt_to_fw(pxp->gt), XE_FW_GT));
> +
> + return 0;
> +}
> +
> +static int kcr_pxp_enable(const struct xe_pxp *pxp)
> +{
> + return kcr_pxp_set_status(pxp, true);
> +}
> +
> +/**
> + * xe_pxp_init - initialize PXP support
> + * @xe: the xe_device structure
> + *
> + * Initialize the HW state and allocate the objects required for PXP support.
> + * Note that some of the requirement for PXP support (GSC proxy init, HuC auth)
> + * are performed asynchronously as part of the GSC init. PXP can only be used
> + * after both this function and the async worker have completed.
> + *
> + * Returns -EOPNOTSUPP if PXP is not supported, 0 if PXP initialization is
> + * successful, other errno value if there is an error during the init.
> + */
> +int xe_pxp_init(struct xe_device *xe)
> +{
> + struct xe_gt *gt = xe->tiles[0].media_gt;
> + struct xe_pxp *pxp;
> + int err;
> +
> + if (!pxp_is_supported(xe))
> + return -EOPNOTSUPP;
> +
> + /* we only support PXP on single tile devices with a media GT */
> + if (xe->info.tile_count > 1 || !gt)
> + return -EOPNOTSUPP;
> +
> + /* The GSCCS is required for submissions to the GSC FW */
> + if (!(gt->info.engine_mask & BIT(XE_HW_ENGINE_GSCCS0)))
> + return -EOPNOTSUPP;
> +
> + /* PXP requires both GSC and HuC firmwares to be available */
> + if (!xe_uc_fw_is_loadable(>->uc.gsc.fw) ||
> + !xe_uc_fw_is_loadable(>->uc.huc.fw)) {
> + drm_info(&xe->drm, "skipping PXP init due to missing FW dependencies");
> + return -EOPNOTSUPP;
> + }
> +
> + pxp = drmm_kzalloc(&xe->drm, sizeof(struct xe_pxp), GFP_KERNEL);
> + if (!pxp)
> + return -ENOMEM;
> +
> + pxp->xe = xe;
> + pxp->gt = gt;
> +
> + err = kcr_pxp_enable(pxp);
> + if (err)
> + return err;
Won't this leak the pxp object? DRM itself will clean it up eventually
on module unload but it should really be freed up here explicitly?
> +
> + xe->pxp = pxp;
> +
> + return 0;
> +}
> diff --git a/drivers/gpu/drm/xe/xe_pxp.h b/drivers/gpu/drm/xe/xe_pxp.h
> new file mode 100644
> index 000000000000..79c951667f13
> --- /dev/null
> +++ b/drivers/gpu/drm/xe/xe_pxp.h
> @@ -0,0 +1,15 @@
> +/* SPDX-License-Identifier: MIT */
> +/*
> + * Copyright(c) 2024, Intel Corporation. All rights reserved.
> + */
> +
> +#ifndef __XE_PXP_H__
> +#define __XE_PXP_H__
> +
> +#include <linux/types.h>
Is this actually needed? The only types in use are the locally defined
struct and 'int'.
> +
> +struct xe_device;
> +
> +int xe_pxp_init(struct xe_device *xe);
> +
> +#endif /* __XE_PXP_H__ */
> diff --git a/drivers/gpu/drm/xe/xe_pxp_types.h b/drivers/gpu/drm/xe/xe_pxp_types.h
> new file mode 100644
> index 000000000000..3a141021972a
> --- /dev/null
> +++ b/drivers/gpu/drm/xe/xe_pxp_types.h
> @@ -0,0 +1,28 @@
> +/* SPDX-License-Identifier: MIT */
> +/*
> + * Copyright(c) 2024, Intel Corporation. All rights reserved.
> + */
> +
> +#ifndef __XE_PXP_TYPES_H__
> +#define __XE_PXP_TYPES_H__
> +
> +#include <linux/types.h>
As above.
John.
> +
> +struct xe_device;
> +struct xe_gt;
> +
> +/**
> + * struct xe_pxp - pxp state
> + */
> +struct xe_pxp {
> + /** @xe: Backpoiner to the xe_device struct */
> + struct xe_device *xe;
> +
> + /**
> + * @gt: pointer to the gt that owns the submission-side of PXP
> + * (VDBOX, KCR and GSC)
> + */
> + struct xe_gt *gt;
> +};
> +
> +#endif /* __XE_PXP_TYPES_H__ */
More information about the Intel-xe
mailing list