Adrian Hunter wrote:
On 31/01/12 15:00, Ulf Hansson wrote:
Adrian Hunter wrote:
On 19/01/12 18:39, Ulf Hansson wrote:
These patches is based upon the patch recently pushed to the mmc mailing
list:
mmc: core: Force a "detect" to handle non-properly removed cards
According to Adrian Hunters comment about adding a CAP2 flag to enable
this feature has been done.
Patch 2 depends on patch 1; but patch 1 can also be discussed separately.
There should only be 1 patch otherwise you are breaking bisectablility for
people not setting MMC_CAP2_DETECT_ON_ERR
The first patch "Prevent I/O as soon as possible..." does only make sure
that I/O is prevented as soon as possible but still within the timeout for
the scheduled detect work. Once the detect work (mmc_rescan) has run the
"detect_change" flag is preventing any further I/O errors from directly
trying to detect a card removal.
The second patch introduces the MMC_CAP2_DETECT_ON_ERR, as you suggested.
For drivers not using MMC_CAP2_DETECT_ON_ERR, the first patch introduces a
change in behaviour and the second patch removes it again. Anyone doing a
git bisect that lands between the patches will see that change in behaviour.
There is no reason for that - just make it one patch.
The first patch is trying to "Prevent I/O as soon as possible at card
removal". It changes some behavior for how we handle card removal, yes.
A change I believe you would like to have no matter if the second patch
is accepted or not.
The second patch, takes the card removal handling to the next level. If
MMC_CAP2_DETECT_ON_ERR is used, card removal will be checked for every
I/O error until the card has been detected to be removed. It does not
remove the behavior for the first patch it only makes it possible to
enable the option of checking for card removal at I/O errors as well.
Ulf Hansson (2):
mmc: core: Prevent I/O as soon as possible at card removal
mmc: core: Detect card removal on I/O error
drivers/mmc/core/core.c | 19 ++++++++++++++++---
include/linux/mmc/host.h | 1 +
2 files changed, 17 insertions(+), 3 deletions(-)
Br
Ulf Hansson
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
