From: Janne Grunau <j@jannau.net>
To: Kevin Greenan <kmgreen2@gmail.com>
Cc: Loic Dachary <loic@dachary.org>,
Ceph Development <ceph-devel@vger.kernel.org>,
Ethan Miller <elm@cs.ucsc.edu>
Subject: Re: ARM NEON optimisations for gf-complete/jerasure/ceph-erasure
Date: Thu, 18 Sep 2014 12:11:28 +0200 [thread overview]
Message-ID: <20140918101128.GV2591@jannau.net> (raw)
In-Reply-To: <CA+AFVBg1oKix1U=qYdLBQ+j4MPYek-npz5UkkFh+dtR_UADUxw@mail.gmail.com>
Hi Kevin,
On 2014-09-16 11:25:12 -0700, Kevin Greenan wrote:
>
> I feel that separating the arch-specific implementations out and have a
> default 'generic' implementation would be a huge improvement. Note that
> gf-complete was in active development for some time before including the
> SIMD code. In hindsight, we should have done this separation back in 2012,
> but had some time pressure due to a paper deadline and limited time
> available to the contributors.
>
> Also, I agree w.r.t. the preprocessor stuff. Going with SIMD/NOSIMD is
> fine by me.
I'll rename than and start implementing neon optimized function in their
own files.
> Also, there should be very little "SIMD" work with jerasure, as gf-complete
> is the Galois field backend, so I would not worry too much about that.
I noticed, I have hooked my neon code already locally in ceph with
touching jerasure.
> That covers "clean-up" work. We can discuss the best way to choose the
> underlying implementation (looks like we have a bunch of options) as this
> work is completed.
>
> With this in mind, what work were you planning to do? I can try to free up
> cycles to help, but that may not happen for a few weeks.
Primarily NEON optimisations for gf-complete/ceph. Shouldn't take more
than a few days though.
> One last thing... If you do have code you want to push upstream, please
> submit a pull request(s) to our main bitbucket repo.
>
> Make sense?
yes, thanks.
Janne
next prev parent reply other threads:[~2014-09-18 10:11 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-04 14:42 ARM NEON optimisations for gf-complete/jerasure/ceph-erasure Janne Grunau
2014-09-04 15:21 ` Loic Dachary
[not found] ` <CA+AFVBg1oKix1U=qYdLBQ+j4MPYek-npz5UkkFh+dtR_UADUxw@mail.gmail.com>
2014-09-18 10:11 ` Janne Grunau [this message]
2014-10-10 14:01 ` Janne Grunau
2014-09-04 15:57 ` Loic Dachary
2014-09-05 0:27 ` Ethan L. Miller
2014-09-05 7:51 ` Janne Grunau
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=20140918101128.GV2591@jannau.net \
--to=j@jannau.net \
--cc=ceph-devel@vger.kernel.org \
--cc=elm@cs.ucsc.edu \
--cc=kmgreen2@gmail.com \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox