Re: [Intel-gfx] [PATCH v4] drm/i915/pxp: Add drm_dbgs for critical PXP events.

2023-10-12 Thread Balasubrawmanian, Vivaik
On 9/19/2023 11:04 AM, Alan Previn wrote: Debugging PXP issues can't even begin without understanding precedding sequence of important events. Add drm_dbg into the most important PXP events. v3 : - move gt_dbg to after mutex block in function i915_gsc_proxy_component_bind. (Vivaik)

Re: [Intel-gfx] [Patch v1] drm/i915: Add uAPI to query micro-controller FW version

2023-09-27 Thread Balasubrawmanian, Vivaik
On 9/27/2023 2:20 AM, Tvrtko Ursulin wrote: On 27/09/2023 05:14, Balasubrawmanian, Vivaik wrote: Due to a bug in GuC firmware, Mesa can't enable by default the usage of compute engines in DG2 and newer. A new GuC firmware fixed the issue but until now there was no way for Mesa to know

[Intel-gfx] [Patch v1] drm/i915: Add uAPI to query micro-controller FW version

2023-09-26 Thread Balasubrawmanian, Vivaik
Due to a bug in GuC firmware, Mesa can't enable by default the usage of compute engines in DG2 and newer. A new GuC firmware fixed the issue but until now there was no way for Mesa to know if KMD was running with the fixed GuC version or not, so this uAPI is required. It may be expanded in

Re: [Intel-gfx] [PATCH v4 2/3] drm/i915/pxp/mtl: Update pxp-firmware packet size

2023-09-15 Thread Balasubrawmanian, Vivaik
On 9/14/2023 3:28 PM, Balasubrawmanian, Vivaik wrote: On 9/6/2023 5:15 PM, Alan Previn wrote: Update the GSC-fw input/output HECI packet size to match updated internal fw specs. Signed-off-by: Alan Previn ---   drivers/gpu/drm/i915/pxp/intel_pxp_cmd_interface_43.h | 4 ++--   1 file changed, 2

Re: [Intel-gfx] [PATCH v2 1/1] drm/i915/pxp: Add drm_dbgs for critical PXP events.

2023-09-15 Thread Balasubrawmanian, Vivaik
On 9/14/2023 3:03 PM, Balasubrawmanian, Vivaik wrote: On 9/13/2023 3:57 PM, Alan Previn wrote: Debugging PXP issues can't even begin without understanding precedding sequence of events. Add drm_dbg into the most important PXP events.   v2 : - remove __func__ since drm_dbg covers that (Jani

Re: [Intel-gfx] [PATCH v4 2/3] drm/i915/pxp/mtl: Update pxp-firmware packet size

2023-09-15 Thread Balasubrawmanian, Vivaik
On 9/14/2023 3:28 PM, Balasubrawmanian, Vivaik wrote: On 9/6/2023 5:15 PM, Alan Previn wrote: Update the GSC-fw input/output HECI packet size to match updated internal fw specs. Signed-off-by: Alan Previn ---   drivers/gpu/drm/i915/pxp/intel_pxp_cmd_interface_43.h | 4 ++--   1 file changed, 2

Re: [Intel-gfx] [PATCH v5 3/3] drm/i915/lrc: User PXP contexts requires runalone bit in lrc

2023-09-15 Thread Balasubrawmanian, Vivaik
On 9/14/2023 3:51 PM, Balasubrawmanian, Vivaik wrote: On 9/9/2023 3:38 PM, Alan Previn wrote: On Meteorlake onwards, HW specs require that all user contexts that run on render or compute engines and require PXP must enforce run-alone bit in lrc. Add this enforcement for protected contexts

Re: [Intel-gfx] [PATCH v3] drm/i915/pxp: Optimize GET_PARAM:PXP_STATUS

2023-07-05 Thread Balasubrawmanian, Vivaik
bool platform_cfg_is_bad; + /** * @kcr_base: base mmio offset for the KCR engine which is different on legacy platforms * vs newer platforms where the KCR is inside the media-tile. base-commit: 6f8963ce33be65c67e53b16fa18325e12ab76861 Reviewed-by: Balasubrawmanian, Vivaik

Re: [Intel-gfx] [PATCH] drm/i915/pxp: Optimize GET_PARAM:PXP_STATUS

2023-06-20 Thread Balasubrawmanian, Vivaik
On 6/1/2023 12:45 PM, Alan Previn wrote: After recent discussions with Mesa folks, it was requested that we optimize i915's GET_PARAM for the PXP_STATUS without changing the UAPI spec. This patch adds this additional optimizations: - If any PXP initializatoin flow failed, then ensure that