From: David Woodhouse <dwmw2@infradead.org>
To: joakim.tjernlund@lumentis.se
Cc: linux-mtd@lists.infradead.org
Subject: Re: CLEANMARKER question
Date: Fri, 04 Jan 2002 10:41:53 +0000 [thread overview]
Message-ID: <9803.1010140913@redhat.com> (raw)
In-Reply-To: <IGEFJKJNHJDCBKALBJLLOEIIFAAA.joakim.tjernlund@lumentis.se>
joakim.tjernlund@lumentis.se said:
> OK, I suspected that much. So after a CLEANMARKER there can be a
> NODETYPE_INODE or a NODETYPE_DIRENT?
Or indeed any other type of node, when new ones get invented - yes.
> Why do you do 2 scan_empty() calls in scan_eraseblock() ?
Consider it loop unrolling.
> Well, sofar I have only identified one improvement. One could add an
> isempty() function in the mtd layer. That would improve scaning for
> empy flash so that you dont have to mtd->read() into a buffer and then
> check the buffer for 0xffffffff. How does that sound?
Sounds ugly, but could be effective. Want to benchmark it to see if it's
really worth it?
If there's a way to safely avoid having to check all the node CRCs on
mount, that would also help.
The most useful thing to do, though, would probably be to implement
checkpointing.
--
dwmw2
next prev parent reply other threads:[~2002-01-04 10:30 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-11-12 12:14 Cache mappings and invalidate Joakim Tjernlund
2001-11-12 17:44 ` Joakim Tjernlund
2001-12-17 13:08 ` Burst read and other improvements Joakim Tjernlund
2002-03-11 8:56 ` compr_zlib.c Joakim Tjernlund
2002-03-19 12:03 ` compr_zlib.c Joakim Tjernlund
2002-03-19 12:24 ` compr_zlib.c David Woodhouse
2002-01-04 8:59 ` CLEANMARKER question Joakim Tjernlund
2002-01-04 9:42 ` David Woodhouse
2002-01-04 10:33 ` Joakim Tjernlund
2002-01-04 10:41 ` David Woodhouse [this message]
2002-01-29 10:33 ` MTD/CFI probe broken? Joakim Tjernlund
2002-01-29 18:09 ` Joakim Tjernlund
2002-02-14 14:05 ` cfi_cmdset0001.c: bug fixes and new features Joakim Tjernlund
2002-02-26 13:42 ` scan.c & ACCURATE Joakim Tjernlund
2002-02-26 15:52 ` David Woodhouse
2002-02-27 7:34 ` Joakim Tjernlund
2002-02-27 8:17 ` David Woodhouse
2002-02-27 12:29 ` Joakim Tjernlund
2002-02-26 15:53 ` David Woodhouse
2002-02-27 7:43 ` Joakim Tjernlund
2002-06-17 9:24 ` point()/unpoint() questions + small cfi_cmdset_0001.c patch Joakim Tjernlund
2002-06-17 9:56 ` David Woodhouse
2002-06-17 13:37 ` David Woodhouse
2002-06-17 15:41 ` Joakim Tjernlund
2002-06-17 15:56 ` David Woodhouse
2002-06-17 16:22 ` Joakim Tjernlund
2002-06-18 14:11 ` David Woodhouse
2002-06-17 9:48 ` [PATCH] scan.c Joakim Tjernlund
2002-06-17 9:54 ` Joakim Tjernlund
2002-06-17 10:15 ` David Woodhouse
2002-06-17 12:16 ` Joakim Tjernlund
2002-06-17 12:45 ` David Woodhouse
2002-06-17 15:12 ` Joakim Tjernlund
2002-06-17 16:00 ` David Woodhouse
2002-06-17 16:51 ` Joakim Tjernlund
2002-06-17 22:59 ` David Woodhouse
-- strict thread matches above, loose matches on Subject: below --
2002-01-05 12:23 CLEANMARKER question Joakim Tjernlund
2002-01-05 12:59 ` David Woodhouse
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=9803.1010140913@redhat.com \
--to=dwmw2@infradead.org \
--cc=joakim.tjernlund@lumentis.se \
--cc=linux-mtd@lists.infradead.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 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.