• Leo Yan's avatar
    perf auxtrace: Drop legacy __sync functions · 1fc7e593
    Leo Yan authored
    The main purpose for using __sync built-in functions is to support
    compat mode for 32-bit perf with 64-bit kernel.  But using these
    built-in functions might cause potential issues.
    
    __sync functions originally support Intel Itanium processoer [1] but it
    cannot promise to support all 32-bit archs.  Now these functions have
    become the legacy functions.
    
    Considering __sync functions cannot really fix the 64-bit value
    atomicity on 32-bit archs, thus this patch drops __sync functions.
    
    Credits to Peter for detailed analysis.
    
    [1] https://gcc.gnu.org/onlinedocs/gcc/_005f_005fsync-Builtins.html#g_t_005f_005fsync-BuiltinsSuggested-by: default avatarPeter Zijlstra <peterz@infradead.org>
    Signed-off-by: default avatarLeo Yan <leo.yan@linaro.org>
    Cc: Adrian Hunter <adrian.hunter@intel.com>
    Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
    Cc: Alexei Starovoitov <ast@kernel.org>
    Cc: Andi Kleen <ak@linux.intel.com>
    Cc: Andrii Nakryiko <andriin@fb.com>
    Cc: Borislav Petkov <bp@alien8.de>
    Cc: Daniel Díaz <daniel.diaz@linaro.org>
    Cc: Frank Ch. Eigler <fche@redhat.com>
    Cc: H. Peter Anvin <hpa@zytor.com>
    Cc: Jiri Olsa <jolsa@redhat.com>
    Cc: Mark Rutland <mark.rutland@arm.com>
    Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
    Cc: Michael Petlan <mpetlan@redhat.com>
    Cc: Mike Leach <mike.leach@linaro.org>
    Cc: Namhyung Kim <namhyung@kernel.org>
    Cc: Peter Zijlstra <peterz@infradead.org>
    Cc: Sedat Dilek <sedat.dilek@gmail.com>
    Cc: Song Liu <songliubraving@fb.com>
    Cc: Suzuki Poulouse <suzuki.poulose@arm.com>
    Cc: Thomas Gleixner <tglx@linutronix.de>
    Cc: coresight@lists.linaro.org
    Cc: x86@kernel.org
    Link: http://lore.kernel.org/lkml/20210809111407.596077-7-leo.yan@linaro.orgSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
    1fc7e593
auxtrace.h 27.5 KB