linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: David Woodhouse <dwmw2@infradead.org>
To: Qu Wenruo <quwenruo.btrfs@gmx.com>, Lukas Straub <lukasstraub2@web.de>
Cc: Linux FS Devel <linux-fsdevel@vger.kernel.org>,
	"linux-block@vger.kernel.org" <linux-block@vger.kernel.org>,
	"dm-devel@redhat.com" <dm-devel@redhat.com>,
	linux-raid@vger.kernel.org
Subject: Re: [dm-devel] Proper way to test RAID456?
Date: Sun, 09 Jan 2022 10:04:05 +0000	[thread overview]
Message-ID: <e209bfe191442846f66d790321f2db672edfb8ca.camel@infradead.org> (raw)
In-Reply-To: <5ffc44f1-7e82-bc85-fbb1-a4f89711ae8f@gmx.com>

[-- Attachment #1: Type: text/plain, Size: 1069 bytes --]

On Sun, 2022-01-09 at 07:55 +0800, Qu Wenruo wrote:
> On 2022/1/9 04:29, Lukas Straub wrote:
> > But there is a even simpler solution for btrfs: It could just not touch
> > stripes that already contain data.
> 
> That would waste a lot of space, if the fs is fragemented.
> 
> Or we have to write into data stripes when free space is low.
> 
> That's why I'm trying to implement a PPL-like journal for btrfs RAID56.

PPL writes the P/Q of the unmodified chunks from the stripe, doesn't
it?

An alternative in a true file system which can do its own block
allocation is to just calculate the P/Q of the final stripe after it's
been modified, and write those (and) the updated data out to newly-
allocated blocks instead of overwriting the original.

Then the final step is to free the original data blocks and P/Q.

This means that your RAID stripes no longer have a fixed topology; you
need metadata to be able to *find* the component data and P/Q chunks...
it ends up being non-trivial, but it has attractive properties if we
can work it out.

[-- Attachment #2: smime.p7s --]
[-- Type: application/pkcs7-signature, Size: 5965 bytes --]

  reply	other threads:[~2022-01-09 10:04 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-07  2:30 Proper way to test RAID456? Qu Wenruo
2022-01-08 19:52 ` [dm-devel] " Lukas Straub
2022-01-08 20:29   ` Lukas Straub
2022-01-08 23:55     ` Qu Wenruo
2022-01-09 10:04       ` David Woodhouse [this message]
2022-01-09 12:13         ` Qu Wenruo
2022-01-12 16:56           ` Lukas Straub
2022-01-13  1:30             ` Qu Wenruo

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=e209bfe191442846f66d790321f2db672edfb8ca.camel@infradead.org \
    --to=dwmw2@infradead.org \
    --cc=dm-devel@redhat.com \
    --cc=linux-block@vger.kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-raid@vger.kernel.org \
    --cc=lukasstraub2@web.de \
    --cc=quwenruo.btrfs@gmx.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;
as well as URLs for NNTP newsgroup(s).