From: Paul Millar <paul.millar@desy.de>
To: Chris Mason <chris.mason@oracle.com>
Cc: linux-btrfs@vger.kernel.org
Subject: Re: A couple of questions
Date: Mon, 31 May 2010 20:06:45 +0200 [thread overview]
Message-ID: <201005312006.47321.paul.millar@desy.de> (raw)
In-Reply-To: <20100527160044.GD3835@think>
Hi Chris,
On Thursday 27 May 2010 18:00:44 Chris Mason wrote:
> I'd suggest that you look at T10 DIF and DIX, which are targeted at
> exactly this kind of thing. We're looking at integrating dif/dix into
> btrfs at some point.
I've been keeping half-an-eye on T10's work in ensuring end-to-end integrity.
That you guys are planning to integrate dif/dix support is certainly welcome
news!
In my use-case (a file-server that receives a new file from a remote client),
I believe that, to ensure end-to-end integrity, the server software would
have to push the client-supplied checksum into the FS when writing a new file.
(I believe there's some T10 slides somewhere that show this use-case) -- or
(equivalently) the server software obtains the FS checksum for the file and
matches it against the client-supplied value.
I'm deliberately taking the simplest case when the client has chosen the same
checksum algorithm as the FS uses. In reality, this may not be the case, but
we can probably cope with that.
My concern is that, if the server-software doesn't push the client-provided
checksum then the FS checksum (plus T-10 DIF/DIX) would not provide a rigorous
assurance that the bytes are the same. Without this assurance, corruption
could still occur; for example, within the server's memory.
> We haven't looked at adler32. crc32c was chosen because it is supported
> in hardware by recent intel CPUs.
OK, fair enough :)
Cheers,
Paul.
next prev parent reply other threads:[~2010-05-31 18:06 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-05-27 13:39 A couple of questions Paul Millar
2010-05-27 14:56 ` Hubert Kario
2010-05-31 17:59 ` Paul Millar
2010-06-02 16:19 ` Hubert Kario
2010-05-27 16:00 ` Chris Mason
2010-05-31 18:06 ` Paul Millar [this message]
2010-05-31 20:33 ` Mike Fedyk
2010-06-02 11:56 ` Paul Millar
2010-06-01 13:39 ` Martin K. Petersen
2010-06-02 13:40 ` Paul Millar
2010-06-04 1:17 ` Martin K. Petersen
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=201005312006.47321.paul.millar@desy.de \
--to=paul.millar@desy.de \
--cc=chris.mason@oracle.com \
--cc=linux-btrfs@vger.kernel.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 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).