From: Paul Clements <paul.clements@steeleye.com>
To: "Peter T. Breuer" <ptb@lab.it.uc3m.es>
Cc: linux-raid@vger.kernel.org
Subject: Re: [PATCH md 0 of 4] Introduction
Date: Tue, 08 Mar 2005 12:02:17 -0500 [thread overview]
Message-ID: <422DDA99.60608@steeleye.com> (raw)
In-Reply-To: <6jj0g2-c27.ln1@news.it.uc3m.es>
Peter T. Breuer wrote:
> Neil - can you describe for me (us all?) what is meant by
> intentlogging here.
Since I wrote a lot of the code, I guess I'll try...
> Well, I can guess - I suppose the driver marks the bitmap before a write
> (or group of writes) and unmarks it when they have completed
> successfully. Is that it?
Yes. It marks the bitmap before writing (actually queues up the bitmap
and normal writes in bunches for the sake of performance). The code is
actually (loosely) based on your original bitmap (fr1) code.
> If so, how does it manage to mark what it is _going_ to do (without
> psychic powers) on the disk bitmap?
That's actually fairly easy. The pages for the bitmap are locked in
memory, so you just dirty the bits you want (which doesn't actually
incur any I/O) and then when you're about to perform the normal writes,
you flush the dirty bitmap pages to disk.
Once the writes are complete, a thread (we have the raid1d thread doing
this) comes back along and flushes the (now clean) bitmap pages back to
disk. If the pages get dirty again in the meantime (because of more
I/O), we just leave them dirty and don't touch the disk.
> Then resync would only deal with the marked blocks.
Right. It clears the bitmap once things are back in sync.
--
Paul
-
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2005-03-08 17:02 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-03-08 5:50 [PATCH md 0 of 4] Introduction NeilBrown
2005-03-08 5:50 ` [PATCH md 2 of 4] Erroneous sizeof use in raid1 NeilBrown
2005-03-08 5:50 ` [PATCH md 4 of 4] Fix md deadlock due to md thread processing delayed requests NeilBrown
2005-03-08 5:50 ` [PATCH md 3 of 4] Initialise sync_blocks in raid1 resync NeilBrown
2005-03-08 5:50 ` [PATCH md 1 of 4] Fix typo in super_1_sync NeilBrown
2005-03-08 6:10 ` [PATCH md 0 of 4] Introduction Andrew Morton
2005-03-09 3:17 ` Neil Brown
2005-03-09 9:27 ` Mike Tran
2005-03-08 12:49 ` Peter T. Breuer
2005-03-08 17:02 ` Paul Clements [this message]
2005-03-08 19:05 ` Peter T. Breuer
2005-03-09 5:07 ` Neil Brown
2005-03-09 15:37 ` Peter T. Breuer
-- strict thread matches above, loose matches on Subject: below --
2004-11-02 3:37 NeilBrown
2004-09-03 2:20 NeilBrown
2004-08-23 3:10 NeilBrown
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=422DDA99.60608@steeleye.com \
--to=paul.clements@steeleye.com \
--cc=linux-raid@vger.kernel.org \
--cc=ptb@lab.it.uc3m.es \
/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).