From: Jamie Lokier <jamie@shareable.org>
To: Trent Piepho <tpiepho@freescale.com>
Cc: Arnaud Mouiche <arnaud.mouiche@thomson.net>,
Andre Puschman <andre.puschmann@imms.de>,
MTD mailing list <linux-mtd@lists.infradead.org>
Subject: Re: flash read performance
Date: Fri, 7 Nov 2008 05:57:08 +0000 [thread overview]
Message-ID: <20081107055708.GG17593@shareable.org> (raw)
In-Reply-To: <Pine.LNX.4.64.0811062121130.3864@t2.domain.actdsltmp>
Trent Piepho wrote:
> On Fri, 7 Nov 2008, Jamie Lokier wrote:
> > Based on Andre's observation, I will soon try enabling cache for my
> > NOR, and see if it makes a difference to cold-cache read performance.
> > I don't expect it, but it's worth a try.
>
> It possible it could help by efficiently doing the reads back-to-back with
> no wasted cycles between them. I think it's necessary if you want to
> benefit from page mode, but that's something I haven't tried yet.
>
> > You might also find the write operation to be unreliable, if the
> > caching mode is write-back rather than write-through.
>
> Proper use of "sync" instructions, or whatever the arch uses to insure that
> writel() is strictly ordered should fix that.
I think that won't work because strongly ordered writes don't
translate directly to bus transactions when the region is mapped with
ioremap_cached(). The flash wants exact bus transactions.
-- Jamie
next prev parent reply other threads:[~2008-11-07 5:57 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-10-28 10:14 flash read performance Andre Puschmann
2008-10-29 11:42 ` Josh Boyer
2008-10-29 12:03 ` Jamie Lokier
2008-10-29 15:52 ` Andre Puschmann
2008-10-30 8:33 ` Arnaud Mouiche
2008-10-30 9:52 ` Andre Puschmann
2008-10-30 10:06 ` Arnaud Mouiche
2008-11-03 14:23 ` Andre Puschmann
2008-11-04 8:30 ` Andre Puschmann
2008-11-04 11:42 ` Jamie Lokier
2008-11-04 14:31 ` Andre Puschman
2008-11-07 2:41 ` Trent Piepho
2008-11-07 4:02 ` Jamie Lokier
2008-11-07 5:36 ` Trent Piepho
2008-11-07 5:57 ` Jamie Lokier [this message]
2008-11-07 9:47 ` Andre Puschmann
2008-11-08 5:28 ` Trent Piepho
2008-11-11 13:28 ` Andre Puschmann
2008-11-15 2:02 ` Trent Piepho
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=20081107055708.GG17593@shareable.org \
--to=jamie@shareable.org \
--cc=andre.puschmann@imms.de \
--cc=arnaud.mouiche@thomson.net \
--cc=linux-mtd@lists.infradead.org \
--cc=tpiepho@freescale.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.