On Thu, 2023-07-20 at 14:52 -0700, Ceraolo Spurio, Daniele wrote:
>
> On 7/20/2023 2:40 PM, Alan Previn wrote:
> > On MTL, if the GSC Proxy init flows haven't completed, submissions to the
> > GSC engine will fail. Those init flows are dependent on the mei's
> > gsc_proxy component that is loaded in parallel with i915 and a
> > worker that could potentially start after i915 driver init is done.
> >
> > That said, all subsytems that access the GSC engine today does check
> > for such init flow completion before using the GSC engine. However,
> > selftests currently don't wait on anything before starting.
> >
> > To fix this, add a waiter function at the start of __run_selftests
> > that waits for gsc-proxy init flows to complete. Selftests shouldn't
> > care if the proxy-init failed as that should be flagged elsewhere.
> >
> > Difference from prior versions:
> > v6: - Add a helper that returns something more than a boolean
> > so we selftest can stop waiting if proxy-init hadn't
> > completed but failed (Daniele).
alan:snip
>
> > +int intel_gsc_uc_fw_proxy_get_status(struct intel_gsc_uc *gsc)
> > +{
> > + if (!(IS_ENABLED(CONFIG_INTEL_MEI_GSC_PROXY)))
> > + return -ENODEV;
> > + if (!intel_uc_fw_is_loadable(&gsc->fw))
> > + return -ENODEV;
> > + if (__intel_uc_fw_status(&gsc->fw) == INTEL_UC_FIRMWARE_LOAD_FAIL)
>
> You're missing the change to move the FW status to LOAD_FAIL if the
> proxy fails to initialize. Or are you expecting
> https://patchwork.freedesktop.org/series/118723/, which included that
> change, to be merged first?
>
> Daniele
alan: as per our offline sync, I'll respin this one and move it away from the
other patch (since this is more critical) and we can respin the other after
this is done so we get a smooth merge. Also, as i move that "change fw status
to fail" from that PXP patch to this patch, I'll fix that issue where i missed
the 2nd failure point in the proxy init flow.
Thanks for your help. :)
