From: Niraj Kulkarni <kulkarniniraj14@gmail.com>
To: Amir Goldstein <amir73il@gmail.com>
Cc: Ted Ts'o <tytso@mit.edu>, Ding Dinghua <dingdinghua85@gmail.com>,
Yongqiang Yang <xiaoqiangnk@gmail.com>,
linux-ext4@vger.kernel.org
Subject: Re: Need of revoke mechanism in JBD
Date: Sat, 30 Apr 2011 07:36:49 +0530 [thread overview]
Message-ID: <4DBB6EB9.40100@gmail.com> (raw)
In-Reply-To: <BANLkTin35yOg+ndy3i4vTT7RHxtFso+HCA@mail.gmail.com>
On Saturday 30 April 2011 01:15 AM, Amir Goldstein wrote:
> On Tue, Apr 26, 2011 at 3:25 PM, Ted Ts'o<tytso@mit.edu> wrote:
>> On Tue, Apr 26, 2011 at 05:23:21PM +0800, Ding Dinghua wrote:
>>> I think it's not only a performance issue but more important, a
>>> correctness issue.
>>> Revoke table is used for preventing the wrong replay of journal which
>>> cause data corruption:
>>> If block A has been journalled its modification, committed to journal
>>> and hasn't been checkpointed,
>>> and in later transactions block A is freed and reused for data in
>>> no-journalled-data mode, then If
>>> we don't have revoke table which recording the releasing event, replay
>>> of journal will overwrite the new data,
>>> which causing data corruption.
>> Yes, this is correct. It should be covered fairly well in Stephen
>> Tweedie's, "Journaling the ext2fs file system" paper, which you can
>> find at:
>>
>> https://ext4.wiki.kernel.org/index.php/Publications
> Actually, the original paper has no mention of revoke records.
> I went out to look for useful documentation on journal forget/revoke
> and came back empty handed as well.
>
>> if you'd like more details.
>>
>> Hope this helps!
>>
>> - Ted
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>>
Yes, I tried some other papers too, but no use. Anyway I've figured out
that for my change, I dont need any kind of journalling related
facilities, so I am going to bypass it completely.
Niraj
next prev parent reply other threads:[~2011-04-30 2:06 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-26 8:29 Need of revoke mechanism in JBD Niraj Kulkarni
2011-04-26 8:56 ` Yongqiang Yang
2011-04-26 9:23 ` Ding Dinghua
2011-04-26 10:47 ` Niraj Kulkarni
2011-04-26 10:57 ` Amir Goldstein
2011-04-26 17:27 ` Andreas Dilger
2011-04-27 0:52 ` Ding Dinghua
2011-04-26 12:25 ` Ted Ts'o
2011-04-29 19:45 ` Amir Goldstein
2011-04-30 2:06 ` Niraj Kulkarni [this message]
2011-05-01 22:28 ` Ted Ts'o
2011-05-02 10:42 ` Amir Goldstein
2011-05-02 14:43 ` Ted Ts'o
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=4DBB6EB9.40100@gmail.com \
--to=kulkarniniraj14@gmail.com \
--cc=amir73il@gmail.com \
--cc=dingdinghua85@gmail.com \
--cc=linux-ext4@vger.kernel.org \
--cc=tytso@mit.edu \
--cc=xiaoqiangnk@gmail.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.