Re: [igt-dev] [PATCH i-g-t 2/2] lib/igt_kmod: Unload mei modules before unloading i915

["Dixit, Ashutosh" <ashutosh.dixit@intel.com>]

On Wed, 30 Mar 2022 11:32:59 -0700, Daniele Ceraolo Spurio wrote:
>
> mei_gsc binds to an aux device exposed by i915, so it depends on it and
> need to be removed before we can unload i915.
> On platforms with a GSC, the mei components (pxp, hdcp) depend on
> mei_gsc, so those need to be unloaded first.

Reviewed-by: Ashutosh Dixit <ashutosh.dixit@intel.com>

> Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
> Cc: Alexander Usyskin <alexander.usyskin@intel.com>
> ---
>  lib/igt_kmod.c | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/lib/igt_kmod.c b/lib/igt_kmod.c
> index d2ac8a56..28ffe515 100644
> --- a/lib/igt_kmod.c
> +++ b/lib/igt_kmod.c
> @@ -400,6 +400,10 @@ int __igt_i915_driver_unload(const char **who)
>	const char *aux[] = {
>		/* gen5: ips uses symbol_get() so only a soft module dependency */
>		"intel_ips",
> +		/* mei_gsc uses an i915 aux dev and the other mei mods depend on it */
> +		"mei_pxp",
> +		"mei_hdcp",
> +		"mei_gsc",
>		NULL,
>	};
>
> --
> 2.25.1
>