Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-18 Thread Asutosh Das (asd)
On 3/18/2021 12:16 PM, Adrian Hunter wrote: On 18/03/21 7:58 pm, Asutosh Das (asd) wrote: On 3/18/2021 10:54 AM, Rafael J. Wysocki wrote: On Thu, Mar 18, 2021 at 6:33 PM Asutosh Das (asd) wrote: On 3/18/2021 7:00 AM, Rafael J. Wysocki wrote: On Wed, Mar 17, 2021 at 7:37 AM Adrian Hunter

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-18 Thread Adrian Hunter
On 18/03/21 7:58 pm, Asutosh Das (asd) wrote: > On 3/18/2021 10:54 AM, Rafael J. Wysocki wrote: >> On Thu, Mar 18, 2021 at 6:33 PM Asutosh Das (asd) >> wrote: >>> >>> On 3/18/2021 7:00 AM, Rafael J. Wysocki wrote: On Wed, Mar 17, 2021 at 7:37 AM Adrian Hunter wrote: > > On

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-18 Thread Asutosh Das (asd)
On 3/18/2021 10:54 AM, Rafael J. Wysocki wrote: On Thu, Mar 18, 2021 at 6:33 PM Asutosh Das (asd) wrote: On 3/18/2021 7:00 AM, Rafael J. Wysocki wrote: On Wed, Mar 17, 2021 at 7:37 AM Adrian Hunter wrote: On 16/03/21 10:35 pm, Asutosh Das (asd) wrote: On 3/16/2021 12:48 AM, Adrian Hunter

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-18 Thread Rafael J. Wysocki
On Thu, Mar 18, 2021 at 6:33 PM Asutosh Das (asd) wrote: > > On 3/18/2021 7:00 AM, Rafael J. Wysocki wrote: > > On Wed, Mar 17, 2021 at 7:37 AM Adrian Hunter > > wrote: > >> > >> On 16/03/21 10:35 pm, Asutosh Das (asd) wrote: > >>> On 3/16/2021 12:48 AM, Adrian Hunter wrote: > On 16/03/21

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-18 Thread Asutosh Das (asd)
On 3/18/2021 7:00 AM, Rafael J. Wysocki wrote: On Wed, Mar 17, 2021 at 7:37 AM Adrian Hunter wrote: On 16/03/21 10:35 pm, Asutosh Das (asd) wrote: On 3/16/2021 12:48 AM, Adrian Hunter wrote: On 16/03/21 12:22 am, Asutosh Das (asd) wrote: On 3/14/2021 1:11 AM, Adrian Hunter wrote: On

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-18 Thread Rafael J. Wysocki
On Wed, Mar 17, 2021 at 7:37 AM Adrian Hunter wrote: > > On 16/03/21 10:35 pm, Asutosh Das (asd) wrote: > > On 3/16/2021 12:48 AM, Adrian Hunter wrote: > >> On 16/03/21 12:22 am, Asutosh Das (asd) wrote: > >>> On 3/14/2021 1:11 AM, Adrian Hunter wrote: > On 10/03/21 5:04 am, Asutosh Das

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-17 Thread Adrian Hunter
On 16/03/21 10:35 pm, Asutosh Das (asd) wrote: > On 3/16/2021 12:48 AM, Adrian Hunter wrote: >> On 16/03/21 12:22 am, Asutosh Das (asd) wrote: >>> On 3/14/2021 1:11 AM, Adrian Hunter wrote: On 10/03/21 5:04 am, Asutosh Das (asd) wrote: > On 3/9/2021 7:56 AM, Asutosh Das (asd) wrote:

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-16 Thread Asutosh Das (asd)
On 3/16/2021 12:48 AM, Adrian Hunter wrote: On 16/03/21 12:22 am, Asutosh Das (asd) wrote: On 3/14/2021 1:11 AM, Adrian Hunter wrote: On 10/03/21 5:04 am, Asutosh Das (asd) wrote: On 3/9/2021 7:56 AM, Asutosh Das (asd) wrote: On 3/8/2021 9:17 AM, Rafael J. Wysocki wrote: On Mon, Mar 8, 2021

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-16 Thread Adrian Hunter
On 16/03/21 12:22 am, Asutosh Das (asd) wrote: > On 3/14/2021 1:11 AM, Adrian Hunter wrote: >> On 10/03/21 5:04 am, Asutosh Das (asd) wrote: >>> On 3/9/2021 7:56 AM, Asutosh Das (asd) wrote: On 3/8/2021 9:17 AM, Rafael J. Wysocki wrote: > On Mon, Mar 8, 2021 at 5:21 PM Rafael J. Wysocki

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-15 Thread Asutosh Das (asd)
On 3/14/2021 1:11 AM, Adrian Hunter wrote: On 10/03/21 5:04 am, Asutosh Das (asd) wrote: On 3/9/2021 7:56 AM, Asutosh Das (asd) wrote: On 3/8/2021 9:17 AM, Rafael J. Wysocki wrote: On Mon, Mar 8, 2021 at 5:21 PM Rafael J. Wysocki wrote: On Sat, Mar 6, 2021 at 5:17 PM Alan Stern wrote:

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-14 Thread Adrian Hunter
On 10/03/21 5:04 am, Asutosh Das (asd) wrote: > On 3/9/2021 7:56 AM, Asutosh Das (asd) wrote: >> On 3/8/2021 9:17 AM, Rafael J. Wysocki wrote: >>> On Mon, Mar 8, 2021 at 5:21 PM Rafael J. Wysocki wrote: On Sat, Mar 6, 2021 at 5:17 PM Alan Stern wrote: > > On Fri, Mar 05,

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-10 Thread Asutosh Das (asd)
On 3/10/2021 8:27 AM, Alan Stern wrote: On Tue, Mar 09, 2021 at 08:04:53PM -0800, Asutosh Das (asd) wrote: On 3/9/2021 7:14 PM, Alan Stern wrote: On Tue, Mar 09, 2021 at 07:04:34PM -0800, Asutosh Das (asd) wrote: Hello I & Can (thanks CanG) debugged this further: Looks like this issue can

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-10 Thread Alan Stern
On Tue, Mar 09, 2021 at 08:04:53PM -0800, Asutosh Das (asd) wrote: > On 3/9/2021 7:14 PM, Alan Stern wrote: > > On Tue, Mar 09, 2021 at 07:04:34PM -0800, Asutosh Das (asd) wrote: > > > Hello > > > I & Can (thanks CanG) debugged this further: > > > > > > Looks like this issue can occur if the sd

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-09 Thread Asutosh Das (asd)
On 3/9/2021 7:14 PM, Alan Stern wrote: On Tue, Mar 09, 2021 at 07:04:34PM -0800, Asutosh Das (asd) wrote: Hello I & Can (thanks CanG) debugged this further: Looks like this issue can occur if the sd probe is asynchronous. Essentially, the sd_probe() is done asynchronously and

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-09 Thread Alan Stern
On Tue, Mar 09, 2021 at 07:04:34PM -0800, Asutosh Das (asd) wrote: > Hello > I & Can (thanks CanG) debugged this further: > > Looks like this issue can occur if the sd probe is asynchronous. > > Essentially, the sd_probe() is done asynchronously and driver_probe_device() > invokes

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-09 Thread Asutosh Das (asd)
On 3/9/2021 7:56 AM, Asutosh Das (asd) wrote: On 3/8/2021 9:17 AM, Rafael J. Wysocki wrote: On Mon, Mar 8, 2021 at 5:21 PM Rafael J. Wysocki wrote: On Sat, Mar 6, 2021 at 5:17 PM Alan Stern wrote: On Fri, Mar 05, 2021 at 06:54:24PM -0800, Asutosh Das (asd) wrote: Now during my testing

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-09 Thread Asutosh Das (asd)
On 3/8/2021 9:17 AM, Rafael J. Wysocki wrote: On Mon, Mar 8, 2021 at 5:21 PM Rafael J. Wysocki wrote: On Sat, Mar 6, 2021 at 5:17 PM Alan Stern wrote: On Fri, Mar 05, 2021 at 06:54:24PM -0800, Asutosh Das (asd) wrote: Now during my testing I see a weird issue sometimes (1 in 7). Scenario

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-08 Thread Rafael J. Wysocki
On Mon, Mar 8, 2021 at 5:21 PM Rafael J. Wysocki wrote: > > On Sat, Mar 6, 2021 at 5:17 PM Alan Stern wrote: > > > > On Fri, Mar 05, 2021 at 06:54:24PM -0800, Asutosh Das (asd) wrote: > > > > > Now during my testing I see a weird issue sometimes (1 in 7). > > > Scenario - bootups > > > > > >

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-08 Thread Rafael J. Wysocki
On Sat, Mar 6, 2021 at 5:17 PM Alan Stern wrote: > > On Fri, Mar 05, 2021 at 06:54:24PM -0800, Asutosh Das (asd) wrote: > > > Now during my testing I see a weird issue sometimes (1 in 7). > > Scenario - bootups > > > > Issue: > > The supplier 'ufs_device_wlun 0:0:0:49488' goes into runtime

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-06 Thread Alan Stern
On Fri, Mar 05, 2021 at 06:54:24PM -0800, Asutosh Das (asd) wrote: > Now during my testing I see a weird issue sometimes (1 in 7). > Scenario - bootups > > Issue: > The supplier 'ufs_device_wlun 0:0:0:49488' goes into runtime suspend even > when one/more of its consumers are in RPM_ACTIVE state.

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-05 Thread Asutosh Das (asd)
On 3/4/2021 7:35 AM, Adrian Hunter wrote: On 3/03/21 12:52 am, Asutosh Das wrote: During runtime-suspend of ufs host, the scsi devices are already suspended and so are the queues associated with them. But the ufs host sends SSU to wlun during its runtime-suspend. During the process

Re: [PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-04 Thread Adrian Hunter
On 3/03/21 12:52 am, Asutosh Das wrote: > During runtime-suspend of ufs host, the scsi devices are > already suspended and so are the queues associated with them. > But the ufs host sends SSU to wlun during its runtime-suspend. > During the process blk_queue_enter checks if the queue is not in >

[PATCH v10 1/2] scsi: ufs: Enable power management for wlun

2021-03-03 Thread Asutosh Das
During runtime-suspend of ufs host, the scsi devices are already suspended and so are the queues associated with them. But the ufs host sends SSU to wlun during its runtime-suspend. During the process blk_queue_enter checks if the queue is not in suspended state. If so, it waits for the queue to