> > +static void exynos_ufs_dbg_register_dump(struct ufs_hba *hba) { > > + struct exynos_ufs *ufs = ufshcd_get_variant(hba); > > + unsigned long flags; > > + > > + spin_lock_irqsave(&ufs->dbg_lock, flags); > > + if (ufs->under_dump == 0) > If you would use test_and_set_bit it would save you both under_dump and > dbg_lock ? Great. Got it. Thanks. Kiwoong Kim