From: Chris Mason <clm@fb.com>
To: Nick Krause <xerofoify@gmail.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-btrfs@vger.kernel.org SYSTEM list:BTRFS FILE"
<linux-btrfs@vger.kernel.org>
Subject: Re: Work Queue for btrfs compression writes
Date: Mon, 4 Aug 2014 09:29:44 -0400 [thread overview]
Message-ID: <53DF8AC8.3010708@fb.com> (raw)
In-Reply-To: <CAPDOMVjJHbCwDVyKQ+Vuse7G76FLSLq69Ja4kmopDbjtEb_Zkw@mail.gmail.com>
On 07/29/2014 11:54 PM, Nick Krause wrote:
> Hey Guys ,
> I am new to reading and writing kernel code.I got interested in
> writing code for btrfs as it seems to
> need more work then other file systems and this seems other then
> drivers, a good use of time on my part.
> I interested in helping improving the compression of btrfs by using a
> set of threads using work queues like XFS
> or reads and keeping the page cache after reading compressed blocks as
> these seem to be a great way to improve
> on compression performance mostly with large partitions of compressed
> data. I am not asking you to write the code
> for me but as I am new a little guidance and help would be greatly
> appreciated as this seems like too much work for just a newbie.
[ Back from vacation ]
Reading through the thread, I don't see anyone mentioning that btrfs
already funnels most compression through helper threads in the kernel
workqueues.
There is also an ordering component that submits the compressed bios to
disk (for writes) in the same order they were created. This lets us
scatter compression across N cpus, but not introduce seeks if they make
progress at different rates.
-chris
prev parent reply other threads:[~2014-08-04 13:29 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-30 3:54 Work Queue for btrfs compression writes Nick Krause
2014-07-30 9:38 ` Hugo Mills
2014-07-30 14:13 ` Theodore Ts'o
2014-07-30 14:36 ` Peter Hurley
2014-07-31 11:35 ` Theodore Ts'o
2014-07-30 15:36 ` ashford
2014-07-30 17:19 ` Nick Krause
2014-07-30 18:31 ` ashford
2014-07-30 19:40 ` Nick Krause
2014-08-04 13:29 ` Chris Mason [this message]
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=53DF8AC8.3010708@fb.com \
--to=clm@fb.com \
--cc=linux-btrfs@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=xerofoify@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 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).