From: "Arkadiusz Miśkiewicz" <arekm@maven.pl>
To: Christoph Hellwig <hch@infradead.org>
Cc: xfs@oss.sgi.com
Subject: Re: [PATCH 0/7] better dquot caching
Date: Sat, 11 Feb 2012 21:13:15 +0100 [thread overview]
Message-ID: <201202112113.15683.arekm@maven.pl> (raw)
In-Reply-To: <20120201135719.202171828@bombadil.infradead.org>
On Wednesday 01 of February 2012, Christoph Hellwig wrote:
> This series improves handling of large number of dquots. It replaced
> the direct recycling of dquots from the freelist with a shrinker, removes
> the upper bound of dquots, and uses per-filesystem structures for all
> quota state, including switching from a hash to a radix-tree for lookups.
>
> For repeated lookups of dquots out of a large pool I see improvements
> betwen 50% and 500% compared to the previous code. All these tests
> have been performed with Q_XQUOTASYNC already disabled as it would
> change the result to much for both the old and new code.
Thanks for this improvement!
On 3.0.17 my script that executes ~130k of Q_XQUOTAGET lookups does it's job
in 1m35s-1m45s and most of that is sys time.
3.3.0git + quota patchsets does the same thing in 23s and when cache gets hot
it goes down to 9s-10s but sys time is only ~1s.
Note that I'm using xfs_quota patched to avoid doing XQUOTASYNC, so that time
is mostly XQUOTAGET also on 3.0.17 kernel.
Of course the best test would be to test 3.3 without and with quota patchsets
but I don't think there were any other improvements between 3.0 and 3.3 that
could get such speed improvement for tons of XQUOTAGET calls.
ps. my 3.3.0git kernel had this patchset and "[PATCH 0/3] include reservations
in quota reportin" included
3.0.17
real 1m34.970s
user 0m4.026s
sys 1m25.081s
real 1m45.365s
user 0m4.106s
sys 1m35.380s
real 1m45.835s
user 0m3.936s
sys 1m35.854s
real 1m45.608s
user 0m4.023s
sys 1m34.181s
real 1m36.207s
user 0m3.986s
sys 1m26.068s
real 1m45.969s
user 0m4.016s
sys 1m35.827s
real 1m35.608s
user 0m3.836s
sys 1m25.534s
real 1m45.882s
user 0m3.986s
sys 1m35.700s
real 1m45.913s
user 0m3.876s
sys 1m36.064s
real 1m45.702s
user 0m4.010s
sys 1m35.937s
3.3git + quota patchsets
real 0m23.811s
user 0m3.760s
sys 0m1.373s
real 0m9.712s
user 0m3.740s
sys 0m1.047s
real 0m9.276s
user 0m3.713s
sys 0m1.113s
real 0m9.880s
user 0m3.870s
sys 0m0.937s
real 0m11.841s
user 0m3.740s
sys 0m1.077s
real 0m10.086s
user 0m3.760s
sys 0m1.057s
real 0m10.170s
user 0m3.790s
sys 0m1.063s
real 0m9.457s
user 0m3.686s
sys 0m1.100s
real 0m10.150s
user 0m3.756s
sys 0m1.073s
real 0m9.331s
user 0m3.810s
sys 0m1.007s
--
Arkadiusz Miśkiewicz PLD/Linux Team
arekm / maven.pl http://ftp.pld-linux.org/
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
prev parent reply other threads:[~2012-02-11 20:13 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-01 13:57 [PATCH 0/7] better dquot caching Christoph Hellwig
2012-02-01 13:57 ` [PATCH 1/7] xfs: use a normal shrinker for the dquot freelist Christoph Hellwig
2012-02-09 22:03 ` Ben Myers
2012-02-09 22:56 ` Christoph Hellwig
2012-02-09 23:13 ` Ben Myers
2012-02-10 1:52 ` Dave Chinner
2012-02-10 16:48 ` Ben Myers
2012-02-10 1:49 ` Dave Chinner
2012-02-10 16:56 ` Ben Myers
2012-02-01 13:57 ` [PATCH 2/7] xfs: per-filesystem dquot LRU lists Christoph Hellwig
2012-02-01 13:57 ` [PATCH 3/7] xfs: use per-filesystem radix trees for dquot lookup Christoph Hellwig
2012-02-01 13:57 ` [PATCH 4/7] xfs: remove the per-filesystem list of dquots Christoph Hellwig
2012-02-01 13:57 ` [PATCH 5/7] xfs: use per-cpu data for the quota statistics Christoph Hellwig
2012-02-01 13:57 ` [PATCH 6/7] xfs: user per-cpu stats for the total dquot numbers Christoph Hellwig
2012-02-01 13:57 ` [PATCH 7/7] xfs: remove the globalk xfs_Gqm structure Christoph Hellwig
2012-02-07 7:30 ` [PATCH 0/7] better dquot caching Ben Myers
2012-02-07 13:24 ` Christoph Hellwig
2012-02-07 15:23 ` Ben Myers
2012-02-11 20:13 ` Arkadiusz Miśkiewicz [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=201202112113.15683.arekm@maven.pl \
--to=arekm@maven.pl \
--cc=hch@infradead.org \
--cc=xfs@oss.sgi.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