Hi Dave,
On Wed, 18 May 2022 10:20:59 -0700 Dave Jiang <dave.jiang@xxxxxxxxx> wrote:
>
> The lock needs to go around both functions, we can move it outside the if().
>
> + mutex_lock(&wq->wq_lock);
> if (wq->state == IDXD_WQ_ENABLED) {
> idxd_wq_disable_cleanup(wq);
> - idxd_wq_device_reset_cleanup(wq);
> wq->state = IDXD_WQ_DISABLED;
> }
> + idxd_wq_device_reset_cleanup(wq);
> + mutex_unlock(&wq->wq_lock);
Thanks for checking. I have made that change to my merge resolution.
--
Cheers,
Stephen Rothwell
Attachment:
pgp_uDxL6RIQn.pgp
Description: OpenPGP digital signature
