All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christian Loehle <CLoehle@hyperstone.com>
To: Richard Weinberger <richard@nod.at>
Cc: Biju Das <biju.das.jz@bp.renesas.com>,
	linux-mmc <linux-mmc@vger.kernel.org>,
	linux-renesas-soc <linux-renesas-soc@vger.kernel.org>,
	wsa+renesas <wsa+renesas@sang-engineering.com>,
	ulf hansson <ulf.hansson@linaro.org>
Subject: RE: Poor write performance to boot area using rcar-gen3-sdhi
Date: Fri, 21 Apr 2023 16:23:25 +0000	[thread overview]
Message-ID: <398e244a4d3746f1bd7b4c32d1ea70e3@hyperstone.com> (raw)
In-Reply-To: <1273293952.226564.1682003880265.JavaMail.zimbra@nod.at>



-----Original Message-----
From: Richard Weinberger <richard@nod.at> 
Sent: Thursday, April 20, 2023 5:18 PM
To: Christian Loehle <CLoehle@hyperstone.com>
Cc: Biju Das <biju.das.jz@bp.renesas.com>; linux-mmc <linux-mmc@vger.kernel.org>; linux-renesas-soc <linux-renesas-soc@vger.kernel.org>; wsa+renesas <wsa+renesas@sang-engineering.com>; ulf hansson <ulf.hansson@linaro.org>
Subject: Re: Poor write performance to boot area using rcar-gen3-sdhi
> 
> I did further tracing and on Linux I see clearly that tmio_mmc_irq() always much later when writing to the boot area.
> 
> Another idea, since u-boot does not really support interrupts it polls various status bits to detect completion of commands and data transfers.
> Crazy idea: Maybe u-boot writes are faster than they should because it polls not always the right bits?
> 
> ...or Linux is too conservative and it matters for the boot area.
> 
> Thanks,
> //richard
> 

If u-boot does not wait until the card no longer signals busy, I would expect data loss.
There are probably some more differences if you really want to go down that rabbit hole,
like different bus modes for uboot and linux.
Anyway I don't really know how I could help you out here, if you could provide a trace showing the busy times,
that would be a smoking gun for the eMMC vendor.

Regards,
Christian

Hyperstone GmbH | Reichenaustr. 39a  | 78467 Konstanz
Managing Director: Dr. Jan Peter Berns.
Commercial register of local courts: Freiburg HRB381782

  reply	other threads:[~2023-04-21 16:23 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-04-17 20:08 Poor write performance to boot area using rcar-gen3-sdhi Richard Weinberger
2023-04-18  6:09 ` Biju Das
2023-04-18  6:16   ` Richard Weinberger
2023-04-18  7:48     ` Christian Loehle
2023-04-18  8:11       ` Richard Weinberger
2023-04-18  8:36         ` Christian Loehle
2023-04-18 11:58           ` Richard Weinberger
2023-04-20 15:18             ` Richard Weinberger
2023-04-21 16:23               ` Christian Loehle [this message]
2023-05-09 11:28                 ` Richard Weinberger
2023-05-11  9:46                   ` Christian Loehle
2023-05-11 10:35                     ` Richard Weinberger
2023-04-18  8:12       ` Biju Das

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=398e244a4d3746f1bd7b4c32d1ea70e3@hyperstone.com \
    --to=cloehle@hyperstone.com \
    --cc=biju.das.jz@bp.renesas.com \
    --cc=linux-mmc@vger.kernel.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=richard@nod.at \
    --cc=ulf.hansson@linaro.org \
    --cc=wsa+renesas@sang-engineering.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.