> From: Michael Kelley (LINUX) <mikelley@xxxxxxxxxxxxx> > Sent: Wednesday, March 29, 2023 9:42 AM > ... > From: Dexuan Cui <decui@xxxxxxxxxxxxx> Sent: Monday, March 27, 2023 9:51 > PM > > > > [snip] > > > @@ -3945,20 +3962,26 @@ static int hv_pci_resume(struct hv_device > *hdev) > > if (ret) > > goto out; > > > > + mutex_lock(&hbus->state_lock); > > + > > ret = hv_pci_enter_d0(hdev); > > if (ret) > > goto out; > > Shouldn't this be goto release_state_lock? Hmm, somehow I missed this. Will fix this in v2.