Re: [PATCH] mmc: support BKOPS feature for eMMC

[Per Forlin <per.lkml@gmail.com>]

Jaehoon Chung <jh80.chung <at> samsung.com> writes:

> +++ b/drivers/mmc/core/core.c
> @@ -238,6 +238,50 @@ mmc_start_request(struct mmc_host *host, struct
mmc_request *mrq)
>  	host->ops->request(host, mrq);
>  }
> 
> +/**
> + *	mmc_start_bkops - start BKOPS for supported cards
> + *	@card: MMC card to start BKOPS
> + *
> + *	Start background operations whenever requested.
> + *	when the urgent BKOPS bit is set in a R1 command response
> + *	then background operations should be started immediately.
> +*/
This patch only starts BKOPS if it's urgent or critical. I would be preferable
to run bkops periodically and only when the card is idle to minimize the risk of
reaching URGENT.

The specs says:
-----
Hosts shall still read the full status from the BKOPS_STATUS byte periodically
and start background operations as needed.
-----

I'm thinking of checking BKOPS_STATUS when the card is idle and then run bkops
even if level is only 1 (Operations outstanding – non critical). Would this make
sense?

Regards,
Per