Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1431972

Re: [PATCH] mmc: core: add auto bkops support

From Alex Lemberg <Alex.Lemberg@sandisk.com>
Newsgroups linux.kernel
Subject Re: [PATCH] mmc: core: add auto bkops support
Date 2016-06-27 13:50 +0200
Message-ID <rODd7-86Z-5@gated-at.bofh.it> (permalink)
References (4 earlier) <rMj8S-7Dk-13@gated-at.bofh.it> <rMRkd-4AB-5@gated-at.bofh.it> <rN1MB-2UJ-1@gated-at.bofh.it> <rN5nb-5t9-5@gated-at.bofh.it> <rOAIi-6Df-7@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Hi Ulf and Adrian,


On 6/27/16, 12:08 PM, "Ulf Hansson" <ulf.hansson@linaro.org> wrote:

>[...]
>
>>>> If I am not wrong, in current implementation of runtime suspend,
>>>> the driver stops BKOPS (send HPI) just before sending sleep command,
>>>> see _mmc_suspend(), depends on “MMC_CAP_AGGRESSIVE_PM” flag.
>>>> In this case, the eMMC device will not have enough time to perform internal
>>>> BKOPS in both – Manual and Auto BKOPS configurations.
>>>>
>>>
>>> ye, so it seems a pre-exiting issue before introducing auto bkops?
>>> I think we can push another patch to improve it but not handling
>>> it for this $SUBJECT, does it sound ok to you?
>>
>> Runtime suspend for eMMC has a default auto-suspend delay of 3 seconds
>> (refer mmc_blk_probe()).  Isn't that when auto bkops would happen?
>
>That's correct.
>
>So perhaps we should extend the default auto-suspend delay?

Due to the differences in flash management and technologies,
I suggest to make this value configurable, and not hard coded.
I believe it should be configured per vendor BKOPS needs.
The default value still can be set to 3 seconds, as done today…

>
>The SD case actually have the same issue, as I some background
>operations exists there as well.
>
>[...]
>
>Kind regards
>Uffe

Back to linux.kernel | Previous | NextPrevious in thread | Find similar | Unroll thread


Thread

Re: [PATCH] mmc: core: add auto bkops support Ulf Hansson <ulf.hansson@linaro.org> - 2016-06-27 11:10 +0200
  Re: [PATCH] mmc: core: add auto bkops support Alex Lemberg <Alex.Lemberg@sandisk.com> - 2016-06-27 13:50 +0200

csiph-web