diff options
| author | H.J. Lu <hjl.tools@gmail.com> | 2021-01-12 05:15:49 -0800 |
|---|---|---|
| committer | H.J. Lu <hjl.tools@gmail.com> | 2021-01-12 07:01:34 -0800 |
| commit | 8493ba72b1a391e30b2518e82df4dc2e67e1b948 (patch) | |
| tree | e646d6fb1f3838b5ff4cd23c80e5fe1a6f8efd32 | |
| parent | a3c78954ee0009e558b7e6b9eb05208814099e21 (diff) | |
| download | glibc-8493ba72b1a391e30b2518e82df4dc2e67e1b948.tar.gz glibc-8493ba72b1a391e30b2518e82df4dc2e67e1b948.tar.xz glibc-8493ba72b1a391e30b2518e82df4dc2e67e1b948.zip | |
x86-64: Avoid rep movsb with short distance [BZ #27130]
When copying with "rep movsb", if the distance between source and destination is N*4GB + [1..63] with N >= 0, performance may be very slow. This patch updates memmove-vec-unaligned-erms.S for AVX and AVX512 versions with the distance in RCX: cmpl $63, %ecx // Don't use "rep movsb" if ECX <= 63 jbe L(Don't use rep movsb") Use "rep movsb" Benchtests data with bench-memcpy, bench-memcpy-large, bench-memcpy-random and bench-memcpy-walk on Skylake, Ice Lake and Tiger Lake show that its performance impact is within noise range as "rep movsb" is only used for data size >= 4KB. (cherry picked from commit 3ec5d83d2a237d39e7fd6ef7a0bc8ac4c171a4a5)
| -rw-r--r-- | NEWS | 1 | ||||
| -rw-r--r-- | sysdeps/x86_64/multiarch/memmove-vec-unaligned-erms.S | 21 |
2 files changed, 22 insertions, 0 deletions
diff --git a/NEWS b/NEWS index 24733f343a..58e7079f89 100644 --- a/NEWS +++ b/NEWS @@ -41,6 +41,7 @@ The following bugs are resolved with this release: [25232] No const correctness for strchr et al. for Clang++ [25933] Off by one error in __strncmp_avx2 [25976] nss_compat: internal_end*ent may clobber errno, hiding ERANGE + [27130] "rep movsb" performance issue Security related changes: diff --git a/sysdeps/x86_64/multiarch/memmove-vec-unaligned-erms.S b/sysdeps/x86_64/multiarch/memmove-vec-unaligned-erms.S index 2e9d86bd33..364cdde62b 100644 --- a/sysdeps/x86_64/multiarch/memmove-vec-unaligned-erms.S +++ b/sysdeps/x86_64/multiarch/memmove-vec-unaligned-erms.S @@ -67,6 +67,13 @@ # define REP_MOVSB_THRESHOLD (2048 * (VEC_SIZE / 16)) #endif +/* Avoid short distance rep movsb only with non-SSE vector. */ +#ifndef AVOID_SHORT_DISTANCE_REP_MOVSB +# define AVOID_SHORT_DISTANCE_REP_MOVSB (VEC_SIZE > 16) +#else +# define AVOID_SHORT_DISTANCE_REP_MOVSB 0 +#endif + #ifndef PREFETCH # define PREFETCH(addr) prefetcht0 addr #endif @@ -257,7 +264,21 @@ L(movsb): # error Unsupported REP_MOVSB_THRESHOLD and VEC_SIZE! # endif jb L(more_8x_vec_backward) +# if AVOID_SHORT_DISTANCE_REP_MOVSB + movq %rdi, %rcx + subq %rsi, %rcx + jmp 2f +# endif 1: +# if AVOID_SHORT_DISTANCE_REP_MOVSB + movq %rsi, %rcx + subq %rdi, %rcx +2: +/* Avoid "rep movsb" if RCX, the distance between source and destination, + is N*4GB + [1..63] with N >= 0. */ + cmpl $63, %ecx + jbe L(more_2x_vec) /* Avoid "rep movsb" if ECX <= 63. */ +# endif mov %RDX_LP, %RCX_LP rep movsb L(nop): |