[PATCH i-g-t 1/2] lib/igt_kmod: fix check to confirm correct driver unload
Kamil Konieczny
kamil.konieczny at linux.intel.com
Thu Dec 19 09:42:42 UTC 2024
Hi Daniele,
On 2024-12-18 at 15:38:29 -0800, Daniele Ceraolo Spurio wrote:
> The driver variable is a string, so we need to pass it in as a variable
> for the driver name instead of stringifying it.
>
> Fixes: e5edf7ea7 ("lib/igt_kmod: add support for Xe driver")
> Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
> Cc: Mauro Carvalho Chehab <mchehab at kernel.org>
> Cc: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> ---
> lib/igt_kmod.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/igt_kmod.c b/lib/igt_kmod.c
> index d58867d7e..a8b0339b9 100644
> --- a/lib/igt_kmod.c
> +++ b/lib/igt_kmod.c
> @@ -731,7 +731,7 @@ igt_intel_driver_unload(const char *driver)
> igt_kmod_unload("drm_kms_helper");
> igt_kmod_unload("drm");
>
> - if (igt_kmod_is_loaded("driver")) {
> + if (igt_kmod_is_loaded(driver)) {
Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> igt_warn("%s.ko still loaded!\n", driver);
> return -EBUSY;
> }
> --
> 2.43.0
>
More information about the igt-dev
mailing list