public inbox for glibc-bugs@sourceware.org help / color / mirror / Atom feed
From: "rguenther at suse dot de" <sourceware-bugzilla@sourceware.org> To: glibc-bugs@sourceware.org Subject: [Bug string/27457] vzeroupper use in AVX2 multiarch string functions cause HTM aborts Date: Sat, 27 Feb 2021 07:34:27 +0000 [thread overview] Message-ID: <bug-27457-131-lze04Et83u@http.sourceware.org/bugzilla/> (raw) In-Reply-To: <bug-27457-131@http.sourceware.org/bugzilla/> https://sourceware.org/bugzilla/show_bug.cgi?id=27457 --- Comment #4 from rguenther at suse dot de --- On February 27, 2021 3:39:50 AM GMT+01:00, "hjl.tools at gmail dot com" <sourceware-bugzilla@sourceware.org> wrote: >https://sourceware.org/bugzilla/show_bug.cgi?id=27457 > >H.J. Lu <hjl.tools at gmail dot com> changed: > > What |Removed |Added >---------------------------------------------------------------------------- > Target Milestone|--- |2.34 > >--- Comment #3 from H.J. Lu <hjl.tools at gmail dot com> --- >(In reply to Richard Biener from comment #2) >> More correct, the wcscmp path ends here with higher %ymm regs used >> >> diff --git a/sysdeps/x86_64/multiarch/strcmp-avx2.S >> b/sysdeps/x86_64/multiarch/strcmp-avx2.S >> index ee82fa3e19..bd3b6243e2 100644 >> --- a/sysdeps/x86_64/multiarch/strcmp-avx2.S >> +++ b/sysdeps/x86_64/multiarch/strcmp-avx2.S >> @@ -122,13 +122,16 @@ L(wcscmp_return): >> negl %eax >> orl $1, %eax >> L(return): >> + VZEROUPPER >> + ret >> # else >> movzbl (%rdi, %rdx), %eax >> movzbl (%rsi, %rdx), %edx >> subl %edx, %eax >> -# endif >> - VZEROUPPER >> + vpxor %ymm0, %ymm0, %ymm0 >> + vpxor %ymm1, %ymm1, %ymm1 >> ret >> +# endif >> >> .p2align 4 >> L(return_vec_size): > >These won't remove AVX-SSE transition penalty. I am re-implementing >all AVX string/memory functions with YMM16-YMM31, which don't need >VZEROUPPER. It should still avoid any false dependences. Ymm16 to ymm31 are only available with AVX512, that will make the AVX2 strong functions unusable on non-avx512 hardware. Are you introducing another set of functions then? My current work is on users/hjl/pr27457/evex branch at > >https://gitlab.com/x86-glibc/glibc/-/commits/users/hjl/pr27457/evex -- You are receiving this mail because: You are on the CC list for the bug.
next prev parent reply other threads:[~2021-02-27 7:34 UTC|newest] Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-02-22 12:40 [Bug string/27457] New: " rguenth at gcc dot gnu.org 2021-02-22 12:40 ` [Bug string/27457] " rguenth at gcc dot gnu.org 2021-02-22 14:50 ` matz at suse dot de 2021-02-22 15:00 ` rguenth at gcc dot gnu.org 2021-02-22 15:26 ` hjl.tools at gmail dot com 2021-02-22 15:26 ` hjl.tools at gmail dot com 2021-02-22 18:45 ` fweimer at redhat dot com 2021-02-23 9:44 ` roman.dementiev at intel dot com 2021-02-27 2:39 ` hjl.tools at gmail dot com 2021-02-27 7:34 ` rguenther at suse dot de [this message] 2021-02-28 14:53 ` hjl.tools at gmail dot com 2021-03-01 11:32 ` fweimer at redhat dot com 2021-03-01 12:24 ` mliska at suse dot cz 2021-03-01 12:47 ` rguenther at suse dot de 2021-03-01 13:13 ` roman.dementiev at intel dot com 2021-03-01 13:19 ` fweimer at redhat dot com 2021-03-01 13:21 ` hjl.tools at gmail dot com 2021-03-01 13:24 ` hjl.tools at gmail dot com 2021-03-01 13:27 ` hjl.tools at gmail dot com 2021-03-01 13:29 ` hjl.tools at gmail dot com 2021-03-01 13:44 ` rguenth at gcc dot gnu.org 2021-03-01 14:05 ` hjl.tools at gmail dot com 2021-03-01 14:14 ` rguenther at suse dot de 2021-03-01 14:25 ` rguenth at gcc dot gnu.org 2021-03-01 14:37 ` hjl.tools at gmail dot com 2021-03-01 14:47 ` hjl.tools at gmail dot com 2021-03-01 14:49 ` rguenth at gcc dot gnu.org 2021-03-01 14:53 ` rguenth at gcc dot gnu.org 2021-03-01 15:19 ` hjl.tools at gmail dot com 2021-03-01 23:39 ` hjl.tools at gmail dot com 2021-03-05 16:54 ` hjl.tools at gmail dot com 2021-03-11 10:42 ` rguenth at gcc dot gnu.org 2021-03-16 13:53 ` rguenth at gcc dot gnu.org 2021-03-16 14:12 ` hjl.tools at gmail dot com 2021-03-29 23:00 ` hjl.tools at gmail dot com 2022-01-27 20:21 ` cvs-commit at gcc dot gnu.org 2022-01-27 20:23 ` cvs-commit at gcc dot gnu.org 2022-01-27 20:47 ` cvs-commit at gcc dot gnu.org 2022-01-27 20:47 ` cvs-commit at gcc dot gnu.org 2022-01-27 20:48 ` cvs-commit at gcc dot gnu.org 2022-01-27 22:41 ` cvs-commit at gcc dot gnu.org 2022-01-28 2:24 ` hjl.tools at gmail dot com
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=bug-27457-131-lze04Et83u@http.sourceware.org/bugzilla/ \ --to=sourceware-bugzilla@sourceware.org \ --cc=glibc-bugs@sourceware.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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).