Commit 374a246e authored by Subhash Jadavani's avatar Subhash Jadavani Committed by Christoph Hellwig
Browse files

ufs: tune bkops while power managment events 



Add capability to control the auto bkops during suspend.
If host explicitly enables the auto bkops (background operation) on device
then only device would perform the bkops on its own. If auto bkops is not
enabled explicitly and if the device reaches to state where it must do
background operation, device would raise the urgent bkops exception event
to host and then host will enable the auto bkops on device. This patch
adds the option to choose whether auto bkops should be enabled during
runtime suspend or not. Since we don't want to keep the device active to
perform the non critical bkops, host will enable urgent bkops only.

Keep auto-bkops enabled after resume if urgent bkops needed.
If device bkops status shows that its in critical need of executing
background operations, host should allow the device to continue doing
background operations.

Signed-off-by: default avatarSubhash Jadavani <subhashj@codeaurora.org>
Signed-off-by: default avatarDolev Raviv <draviv@codeaurora.org>
Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
parent 856b3483

drivers/scsi/ufs/ufshcd.c

+18 −8
Original line number Original line Diff line number Diff line
@@ -4884,13 +4884,19 @@ static int ufshcd_suspend(struct ufs_hba *hba, enum ufs_pm_op pm_op) 
	}

	}





	if (ufshcd_is_runtime_pm(pm_op)) {

	if (ufshcd_is_runtime_pm(pm_op)) {

		if (ufshcd_can_autobkops_during_suspend(hba)) {

			/*

			/*

			 * The device is idle with no requests in the queue,

			 * The device is idle with no requests in the queue,

		 * allow background operations if needed.

			 * allow background operations if bkops status shows

			 * that performance might be impacted.

			 */

			 */

		ret = ufshcd_bkops_ctrl(hba, BKOPS_STATUS_NON_CRITICAL);

			ret = ufshcd_urgent_bkops(hba);

			if (ret)

			if (ret)

				goto enable_gating;

				goto enable_gating;

		} else {

			/* make sure that auto bkops is disabled */

			ufshcd_disable_auto_bkops(hba);

		}

	}

	}





	if ((req_dev_pwr_mode != hba->curr_dev_pwr_mode) &&

	if ((req_dev_pwr_mode != hba->curr_dev_pwr_mode) &&
@@ -5038,7 +5044,11 @@ static int ufshcd_resume(struct ufs_hba *hba, enum ufs_pm_op pm_op) 
			goto set_old_link_state;

			goto set_old_link_state;

	}

	}





	ufshcd_disable_auto_bkops(hba);

	/*

	 * If BKOPs operations are urgently needed at this moment then

	 * keep auto-bkops enabled or else disable it.

	 */

	ufshcd_urgent_bkops(hba);

	hba->clk_gating.is_suspended = false;

	hba->clk_gating.is_suspended = false;





	if (ufshcd_is_clkscaling_enabled(hba))

	if (ufshcd_is_clkscaling_enabled(hba))

drivers/scsi/ufs/ufshcd.h

+7 −0
Original line number Original line Diff line number Diff line
@@ -469,6 +469,8 @@ struct ufs_hba { 
#define UFSHCD_CAP_HIBERN8_WITH_CLK_GATING (1 << 1)

#define UFSHCD_CAP_HIBERN8_WITH_CLK_GATING (1 << 1)

	/* Allow dynamic clk scaling */

	/* Allow dynamic clk scaling */

#define UFSHCD_CAP_CLK_SCALING	(1 << 2)

#define UFSHCD_CAP_CLK_SCALING	(1 << 2)

	/* Allow auto bkops to enabled during runtime suspend */

#define UFSHCD_CAP_AUTO_BKOPS_SUSPEND (1 << 3)





	struct devfreq *devfreq;

	struct devfreq *devfreq;

	struct ufs_clk_scaling clk_scaling;

	struct ufs_clk_scaling clk_scaling;
@@ -487,6 +489,11 @@ static inline int ufshcd_is_clkscaling_enabled(struct ufs_hba *hba) 
{

{

	return hba->caps & UFSHCD_CAP_CLK_SCALING;

	return hba->caps & UFSHCD_CAP_CLK_SCALING;

}

}

static inline bool ufshcd_can_autobkops_during_suspend(struct ufs_hba *hba)

{

	return hba->caps & UFSHCD_CAP_AUTO_BKOPS_SUSPEND;

}



#define ufshcd_writel(hba, val, reg)	\

#define ufshcd_writel(hba, val, reg)	\

	writel((val), (hba)->mmio_base + (reg))

	writel((val), (hba)->mmio_base + (reg))

#define ufshcd_readl(hba, reg)	\

#define ufshcd_readl(hba, reg)	\

CRA Git | Maintained and supported by SUSTech CRA and CCSE