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