[Accel-config] Re: [PATCH 1/1] accel-config: Fix idxd_mdev modprobe on specific silicon stepping

I have reworked this patch and sent for review
https://lists.01.org/hyperkitty/list/accel-config(a)lists.01.org/thread/MEIPOQZLTTV4PCBVRLFFMYL3G3CIIASH/

On Sun, Apr 04, 2021 at 01:29:23PM, Ramesh Thomas wrote:
> This will not work because now idxd_uacce also depends on idxd module.
> Also I find the module dependency order between idxd_uacce and idxd_mdev
> is not fixed. You may need to do something like -
> 
> 1. Check and unload idxd_mdev. (This can fail depending on module order)
> 2. Check and unload idxd_uacce.
> 3. Check and unload idxd_mdev again. (Will succeed for any module order)
> 4. Check and unload idxd
> 5. Load idxd
> 
> sleep 1 may be required between steps
> 
> -Ramesh
> 
> On Fri, Apr 02, 2021 at 10:33:09AM +0800, Tony Zhu <tony.zhu(a)intel.com>
> wrote:
> > When reset the module, on specific silicon steppings, need the delay
> > to make sure devices and wqs are ready. Otherwise, it will fail to
> > get the available wqs.
> >
> > Signed-off-by: Tony Zhu <tony.zhu(a)intel.com>
> > ---
> >  test/common | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/test/common b/test/common
> > index 4bd4ed3..625d347 100644
> > --- a/test/common
> > +++ b/test/common
> > @@ -108,6 +108,7 @@ _cleanup()
> >  		sleep 1
> >  	}
> >  	modprobe idxd
> > +	sleep 1
> >  }
> >
> >  # json2var
> > --
> > 2.17.1