[Bugs] [Bug 1140845] Current implementation depends on Intel's SSE2 extensions

bugzilla at redhat.com bugzilla at redhat.com
Fri Sep 12 09:56:53 UTC 2014


https://bugzilla.redhat.com/show_bug.cgi?id=1140845



--- Comment #2 from Anand Avati <aavati at redhat.com> ---
COMMIT: http://review.gluster.org/8701 committed in release-3.6 by Vijay Bellur
(vbellur at redhat.com) 
------
commit 449c046bf6b039b2ba86be34677056686f7bc5ce
Author: Xavier Hernandez <xhernandez at datalab.es>
Date:   Mon Aug 4 20:50:31 2014 +0200

    ec: Removed SSE2 dependency

    This patch implements the Galois Field multiplications using pure C
    code without any assembler support. This makes the ec xlator portable
    to other architectures.

    In the future it will be possible to use an optimized implementation
    of the multiplications using architecture dependent facilities (it
    will be automatically detected and configured). To allow bricks with
    different machine word sizes to be able to work seamlessly in the
    same volume, the minimum fragment length to be stored in any brick
    has been fixed to 512 bytes. Otherwise, different implementations
    will corrupt the data (SSE2 used 128 bytes, while new implementation
    would have used 64).

    This patch also removes the '-msse2' option added on patch
    http://review.gluster.org/8396/

    This is a backport of http://review.gluster.org/8413/

    Change-Id: Iaf6e4ef3dcfda6c68f48f16ca46fc4fb61a215f4
    BUG: 1140845
    Signed-off-by: Xavier Hernandez <xhernandez at datalab.es>
    Reviewed-on: http://review.gluster.org/8701
    Tested-by: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Vijay Bellur <vbellur at redhat.com>

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=IHOFdeShXk&a=cc_unsubscribe


More information about the Bugs mailing list