From: "Matt W. Benjamin" <matt@cohortfs.com>
To: Loic Dachary <loic@dachary.org>
Cc: Casey Bodley <casey@cohortfs.com>,
ceph-devel <ceph-devel@vger.kernel.org>
Subject: Re: ceph performance call: buffer
Date: Thu, 16 Oct 2014 14:42:27 -0400 (EDT) [thread overview]
Message-ID: <1251350300.123.1413484947475.JavaMail.root@thunderbeast.private.linuxbox.com> (raw)
In-Reply-To: <54400C8C.8050006@dachary.org>
Ack, very interested in that :).
Matt
----- "Loic Dachary" <loic@dachary.org> wrote:
> Hi Again,
>
> On 16/10/2014 11:16, Loic Dachary wrote:
> > Hi Matt,
> >
> > You will probably be interested in
>
> (I have to find which sequence of keys send out the mail by
> accident... :-)
>
> https://github.com/ceph/ceph/pull/2595
>
> and
>
> https://github.com/dachary/ceph/commit/734e555c640a3b5e9bbf0329bd524cffaca61c4f
>
> in particular. It does not do anything performance wise but it's going
> to be merged in the not-to-distant future.
>
> Cheers
>
> >
> > On 16/10/2014 11:00, Matt W. Benjamin wrote:
> >> Hi Folks,
> >>
> >> Per yesterday's Ceph performance call, we've pushed 2 wip branches
> for review and feedback to
> https://github.com/linuxbox2/linuxbox-ceph.git .
> >>
> >> 1. xio-v084-inline reorganizes ceph::buffer as a namespace, and
> (nearly) all of
> >> buffer::hash, buffer::ptr and buffer::list as inline
> >>
> >> 2. xio-v084-inline-flat restores some raw encapsulation lost in the
> prior change,
> >> then refactors buffer::raw to remove virtual methods
> >>
> >> We've made some effort to measure the impact of the changes, but
> don't have
> >> reliable data yet. (It's possible that our initial impression of
> improvement from the inlining change was not accurate, but more likely
> we need better workload tests.)
> >>
> >> In addition, we've been working on an experiment to refactor
> buffer::ptr and buffer::raw to use boost::intrusive::list, to unify
> the interfaces of the two types to permit interning buffer::raw
> directly on the containing list when there is no sharing, and a slab
> allocator for both types. We plan to push a wip branch with these
> changes to when they've stabilized a bit more and/or we have real perf
> numbers.
> >>
> >> Thanks,
> >>
> >> Matt
> >>
> >
>
> --
> Loïc Dachary, Artisan Logiciel Libre
--
Matt Benjamin
CohortFS, LLC.
206 South Fifth Ave. Suite 150
Ann Arbor, MI 48104
http://cohortfs.com
tel. 734-761-4689
fax. 734-769-8938
cel. 734-216-5309
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
prev parent reply other threads:[~2014-10-16 18:43 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-16 18:00 ceph performance call: buffer Matt W. Benjamin
2014-10-16 18:16 ` Loic Dachary
2014-10-16 18:21 ` Loic Dachary
2014-10-16 18:42 ` Matt W. Benjamin [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=1251350300.123.1413484947475.JavaMail.root@thunderbeast.private.linuxbox.com \
--to=matt@cohortfs.com \
--cc=casey@cohortfs.com \
--cc=ceph-devel@vger.kernel.org \
--cc=loic@dachary.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.