From: Oskar Andero <oskar.andero@sonymobile.com>
To: Ulf Hansson <ulf.hansson@linaro.org>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
linux-mmc <linux-mmc@vger.kernel.org>,
Chris Ball <cjb@laptop.org>,
"Svensson, Lars 1" <Lars1.Svensson@sonymobile.com>
Subject: Re: [PATCH 1/1] MMC: Detect execution mode errors after r/w command
Date: Thu, 7 Nov 2013 09:36:57 +0100 [thread overview]
Message-ID: <20131107083657.GA2558@caracas.corpusers.net> (raw)
In-Reply-To: <CAPDyKFrYvw0RjscTZbq+4DFoV_xiYAH=KcseC=T4bT5+RPUrGw@mail.gmail.com>
Hi Ulf,
On 18:50 Tue 22 Oct , Ulf Hansson wrote:
> > And this is after the patch has been applied:
> > KB reclen write rewrite read reread
> > 51200 4 251 990 3280 3244
> > 51200 8 460 1545 4460 4463
> > 51200 16 878 2633 7023 7028
> > 51200 32 1380 4394 9802 9832
> > 51200 64 2457 6216 12314 12314
> > 51200 128 3667 7894 14087 14088
> > 51200 256 6422 5916 15085 15086
> > 51200 512 5536 10994 12571 15659
> > 51200 1024 9112 9499 16203 16205
> > 51200 2048 10197 10502 16363 16368
> > 51200 4096 10524 10238 8850 16309
> > 51200 8192 9615 10456 16528 16529
> > 51200 16384 10553 10428 16803 16803
>
> Hi Oskar,
>
> The numbers were not that impressing from the beginning, so that could
> be why you don't see any impact.
> What kind of card are you using, eMMC/SD? In what speed mode are the
> card operating in?
The test was run on an Beagleboard with sdcard. I will try to find another board with
eMMC support and rerun the tests.
> >> > + /*
> >> > + * Try to get card status which indicates the card state after
> >> > + * command execution. If the first attempt fails, we can't be
> >> > + * sure the returned status is for the r/w command.
> >> > + */
> >> > + for (retries = 2; retries >= 0; retries--) {
> >> > + err = get_card_status(card, &status, 0);
> >> > + if (!err)
> >> > + break;
> >> > +
> >> > + status_valid = false;
> >> > + pr_err("%s: error %d sending status command, %sing\n",
> >> > + req->rq_disk->disk_name, err,
> >> > + retries ? "retry" : "abort");
> >> > + }
> >>
> >> Do we have to issue a CMD13 (get_card_status), even if we are using
> >> the open-ended transmission sequence? In other words, could we make
> >> use of the response from CMD12 (stop transmission) instead?
> >
> > That's probably a good idea. Do you know of a way to check if CMD12 has been
> > sent or not?
>
> Have a look for "mmc_host_cmd23", which gets translated into
> "MMC_BLK_CMD23" for the mmc block layer. This will give you some hints
> of were to look.
Thanks! We will revise the patch according to your comments.
-Oskar
prev parent reply other threads:[~2013-11-07 8:37 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-10 13:28 [PATCH 1/1] MMC: Detect execution mode errors after r/w command Oskar Andero
2013-10-16 15:01 ` Ulf Hansson
2013-10-22 11:21 ` Oskar Andero
2013-10-22 16:50 ` Ulf Hansson
2013-11-07 8:36 ` Oskar Andero [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20131107083657.GA2558@caracas.corpusers.net \
--to=oskar.andero@sonymobile.com \
--cc=Lars1.Svensson@sonymobile.com \
--cc=cjb@laptop.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mmc@vger.kernel.org \
--cc=ulf.hansson@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).