[Accel-config] Re: [PATCH 1/1] accel-config: Fix idxd modprobe issue with idxd_mdev for user test

Good patch. I aware this issue as well.

Thanks,
-Xudong

> -----Original Message-----
> From: Tony Zhu <tony.zhu(a)intel.com>
> Sent: Thursday, March 4, 2021 5:12 AM
> To: accel-config(a)lists.01.org
> Cc: Zhu, Tony <tony.zhu(a)intel.com>
> Subject: [Accel-config] [PATCH 1/1] accel-config: Fix idxd modprobe issue
> with idxd_mdev for user test
> 
> User test will clean the idxd module in the memory moving tests. As
> idxd_mdev is supported and it depends on idxd, should change the clean as
> idxd_mdev. To improve the robust and compatibility to old kernel, we add
> the condition to check if the module is loaded and keep idxd reset.
> 
> Signed-off-by: Tony Zhu <tony.zhu(a)intel.com>
> ---
>  test/common | 14 +++++++++++---
>  1 file changed, 11 insertions(+), 3 deletions(-)
> 
> diff --git a/test/common b/test/common
> index a60bec2..d4041e3 100644
> --- a/test/common
> +++ b/test/common
> @@ -99,8 +99,16 @@ check_prereq()
>  #
>  _cleanup()
>  {
> -	modprobe -r idxd
> -	modprobe idxd
> +	lsmod | grep -q "idxd_mdev" && {
> +		modprobe -r idxd_mdev
> +		sleep 1
> +		modprobe idxd_mdev
> +	}
> +	lsmod | grep -q "idxd" && {
> +		modprobe -r idxd
> +		sleep 1
> +		modprobe idxd
> +	}
>  }
> 
>  # json2var
> @@ -158,4 +166,4 @@ wq_mode2name()
>  		echo "UNKNOWN"
>  		;;
>  	esac
> -}
> \ No newline at end of file
> +}
> --
> 2.27.0
> _______________________________________________
> Accel-config mailing list -- accel-config(a)lists.01.org To unsubscribe send an
> email to accel-config-leave(a)lists.01.org