[lkml]   [2010]   [Feb]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 09/10] x86-32: use SSE for atomic64_read/set if available
    Luca Barbieri <> writes:

    > This patch uses SSE movlps to perform 64-bit atomic reads and writes.

    You seem to have forgotten to add benchmark results that show this is
    actually worth while? And is there really any user on 32bit
    that needs 64bit atomic_t?

    I have to agree with Peter on it being a bad idea very likely
    to mess with FPU state like this.

    I'm also suspicious of your use of global register variables.
    This means they won't be saved on entry/exit of the functions.
    Does that really work?

    It's also a not widely used gcc feature and those are always
    somewhat risky.

    -- -- Speaking for myself only.

     \ /
      Last update: 2010-02-18 09:25    [W:0.020 / U:8.552 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site