Re: [PATCH v3 1/5] scsi: ufs: atomic update for clkgating_enable

2020-10-26 Thread Can Guo
On 2020-10-27 03:48, Jaegeuk Kim wrote: On 10/26, Can Guo wrote: On 2020-10-26 14:13, Jaegeuk Kim wrote: > On 10/26, Can Guo wrote: > > On 2020-10-24 23:06, Jaegeuk Kim wrote: > > > From: Jaegeuk Kim > > > > > > When giving a stress test which enables/disables clkgating, we hit > > > device >

Re: [PATCH v3 1/5] scsi: ufs: atomic update for clkgating_enable

2020-10-26 Thread Jaegeuk Kim
On 10/26, Can Guo wrote: > On 2020-10-26 14:13, Jaegeuk Kim wrote: > > On 10/26, Can Guo wrote: > > > On 2020-10-24 23:06, Jaegeuk Kim wrote: > > > > From: Jaegeuk Kim > > > > > > > > When giving a stress test which enables/disables clkgating, we hit > > > > device > > > > timeout sometimes. This

Re: [PATCH v3 1/5] scsi: ufs: atomic update for clkgating_enable

2020-10-26 Thread Can Guo
On 2020-10-26 14:13, Jaegeuk Kim wrote: On 10/26, Can Guo wrote: On 2020-10-24 23:06, Jaegeuk Kim wrote: > From: Jaegeuk Kim > > When giving a stress test which enables/disables clkgating, we hit > device > timeout sometimes. This patch avoids subtle racy condition to address > it. > > If we

Re: [PATCH v3 1/5] scsi: ufs: atomic update for clkgating_enable

2020-10-26 Thread Jaegeuk Kim
On 10/26, Can Guo wrote: > On 2020-10-24 23:06, Jaegeuk Kim wrote: > > From: Jaegeuk Kim > > > > When giving a stress test which enables/disables clkgating, we hit > > device > > timeout sometimes. This patch avoids subtle racy condition to address > > it. > > > > If we use __ufshcd_release(),

Re: [PATCH v3 1/5] scsi: ufs: atomic update for clkgating_enable

2020-10-25 Thread Can Guo
On 2020-10-24 23:06, Jaegeuk Kim wrote: From: Jaegeuk Kim When giving a stress test which enables/disables clkgating, we hit device timeout sometimes. This patch avoids subtle racy condition to address it. If we use __ufshcd_release(), I've seen that gate_work can be called in parallel

[PATCH v3 1/5] scsi: ufs: atomic update for clkgating_enable

2020-10-24 Thread Jaegeuk Kim
From: Jaegeuk Kim When giving a stress test which enables/disables clkgating, we hit device timeout sometimes. This patch avoids subtle racy condition to address it. If we use __ufshcd_release(), I've seen that gate_work can be called in parallel with ungate_work, which results in UFS timeout