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 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.