From: David Woodhouse <dwmw2@infradead.org>
To: "Curtis, Allen" <Allen.Curtis@Thales-IFS.com>
Cc: jffs-dev@axis.com
Subject: Re: jffs2_scan_eraseblock() - errors
Date: Thu, 01 Aug 2002 11:44:54 +0100 [thread overview]
Message-ID: <30325.1028198694@redhat.com> (raw)
In-Reply-To: <FA06AA2C99BCD511951200005A99441001E3ECD8@irvexch1.sextantifs.com>
(Moved to JFFS list).
Allen.Curtis@Thales-IFS.com said:
> > real 47.258s
> With the CRC patch:
> real 36.7s
OK, that's respectable. I just need to finish the other part of the change
-- to actually check the CRCs later on, rather than just leaving them
unchecked. Make sure you take the patch back out of your tree now :)
We may get a little bit more of a speedup when we do that too, as we'll also
stop building up the fragment lists for every inode on mount, but in fact
that code wasn't showing up very high on the profile last time I looked.
> PS: The version of scan.c we are using is 1.57. The patch note was for
> 1.51.2.3.
Doesn't matter; it's basically the same in both branches.
I suspect that if you profile it now you'll find the most time is taken in
your flash map driver's copy_from routine or memcpy.
I need to implement the XIP scheme which will also allow JFFS2 to directly
access the flash through a kind of pageable ioremap (which handles the times
when the flash is in a mode other than read mode correctly) instead of
having to read into a buffer and work from that. We get to use caches and
burst reads from flash at that point too.
--
dwmw2
next prev parent reply other threads:[~2002-08-01 10:45 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-07-31 23:02 jffs2_scan_eraseblock() - errors Curtis, Allen
2002-08-01 10:44 ` David Woodhouse [this message]
2002-08-01 20:33 ` write suspend and/or erase suspend Joakim Tjernlund
2002-08-01 23:31 ` David Woodhouse
2002-08-02 9:27 ` Joakim Tjernlund
2002-08-02 9:42 ` David Woodhouse
-- strict thread matches above, loose matches on Subject: below --
2002-07-31 1:15 jffs2_scan_eraseblock() - errors Curtis, Allen
2002-07-31 3:07 ` David Woodhouse
2002-07-31 0:38 Curtis, Allen
2002-07-31 0:48 ` David Woodhouse
2002-07-30 23:59 Curtis, Allen
2002-07-31 0:15 ` David Woodhouse
2002-07-31 0:18 ` David Woodhouse
2002-07-30 23:30 Curtis, Allen
2002-07-30 23:42 ` David Woodhouse
2002-07-30 18:11 Curtis, Allen
2002-07-30 22:21 ` David Woodhouse
2002-07-31 10:34 ` Jörn Engel
2002-07-31 11:45 ` David Woodhouse
2002-07-31 11:57 ` Jörn Engel
2002-07-31 11:59 ` David Woodhouse
2002-07-31 12:16 ` Jörn Engel
2002-07-31 12:17 ` David Woodhouse
2002-07-31 13:07 ` Jörn Engel
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=30325.1028198694@redhat.com \
--to=dwmw2@infradead.org \
--cc=Allen.Curtis@Thales-IFS.com \
--cc=jffs-dev@axis.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox