From: Chris Mason <chris.mason@oracle.com>
To: Gregory Maxwell <gmaxwell@gmail.com>
Cc: Jim Faulkner <jfaulkne@ccs.neu.edu>,
Josef Bacik <josef@redhat.com>,
linux-btrfs@vger.kernel.org
Subject: Re: worse than expected compression ratios with -o compress
Date: Thu, 21 Jan 2010 15:07:50 -0500 [thread overview]
Message-ID: <20100121200750.GC23006@think> (raw)
In-Reply-To: <e692861c1001211204u7b16efe9tf7ee5703054caf76@mail.gmail.com>
On Thu, Jan 21, 2010 at 03:04:56PM -0500, Gregory Maxwell wrote:
> On Thu, Jan 21, 2010 at 1:16 PM, Jim Faulkner <jfaulkne@ccs.neu.edu> =
wrote:
> >
> > On Wed, 20 Jan 2010, Chris Mason wrote:
> >
> >> Please let me know if this improves your ratios
> >
> > It most certainly does! =C2=A0It also greatly reduced the time requ=
ired to copy
> > the data to my (not very fast) disk. =C2=A0All my testing was done =
on 2.6.32.4.
> > The line numbers in your patch were a little off for 2.6.32.4, but =
I did
> > manage to apply it cleanly. =C2=A0Here's the results of my testing:
> [snip]
> > I'd be very happy to see the -o compress-force option in the mainli=
ne kernel
> > someday!
>=20
>=20
> Sweet. But I think a force mount option is an unreasonably blunt tool=
=2E
>=20
> I think two things would be nice:
>=20
> (1) Fix the compression decision, I think this example suggests that
> something is broken. (I'd noticed poorer than expected compression on
> my laptop, but I'd chalked it up to the 64k blocks=E2=80=A6 now I'm n=
ot so
> confident)
The current code assumes that files have consistent data in them. This
is very true for the average data set, but it'll be horribly wrong for
something like a database file.
>=20
> (2) An IOCTL for compression control. Userspace knows best, some
> files ought to have a different compression policy.
Yes, we'll get #2 added.
-chris
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" =
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:[~2010-01-21 20:07 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-16 16:16 worse than expected compression ratios with -o compress Jim Faulkner
2010-01-17 14:34 ` Sander
2010-01-18 14:46 ` Jim Faulkner
2010-01-18 16:06 ` Jim Faulkner
2010-01-18 14:12 ` Josef Bacik
2010-01-18 21:29 ` Chris Mason
2010-01-18 22:11 ` Jim Faulkner
2010-01-20 16:30 ` Chris Mason
2010-01-21 18:16 ` Jim Faulkner
2010-01-21 20:04 ` Gregory Maxwell
2010-01-21 20:07 ` Chris Mason [this message]
2010-01-21 20:05 ` Chris Mason
2010-01-21 22:38 ` Jim Faulkner
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=20100121200750.GC23006@think \
--to=chris.mason@oracle.com \
--cc=gmaxwell@gmail.com \
--cc=jfaulkne@ccs.neu.edu \
--cc=josef@redhat.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 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.