Re: [PATCH net v2] ice: Fix race during aux device (un)plugging

From: Ivan Vecera
Date: Thu Apr 21 2022 - 02:09:03 EST


On Thu, 21 Apr 2022 08:12:58 +0300
Leon Romanovsky <leon@xxxxxxxxxx> wrote:

> > static struct iidc_auxiliary_drv *ice_get_auxiliary_drv(struct ice_pf *pf)
> > {
> > struct auxiliary_device *adev;
> >
> > + BUG_ON(!mutex_is_locked(&pf->adev_mutex));
>
> Please don't add BUG_ON() in driver code.
>
> I think that you are looking for lockdep_assert_held(&pf->adev_mutex).

Will fix.

I.