This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] Enhance tst-xmmymm.sh to detect zmm register usage in ld.so
- From: "Carlos O'Donell" <carlos at redhat dot com>
- To: Siddhesh Poyarekar <siddhesh at redhat dot com>, libc-alpha at sourceware dot org
- Date: Thu, 11 Sep 2014 15:39:16 -0400
- Subject: Re: [PATCH] Enhance tst-xmmymm.sh to detect zmm register usage in ld.so
- Authentication-results: sourceware.org; auth=none
- References: <20140911173002 dot GJ14885 at spoyarek dot pnq dot redhat dot com>
On 09/11/2014 01:30 PM, Siddhesh Poyarekar wrote:
> Hi,
>
> 2d63a517e4084ec80403cd9f278690fa8b676cc4 added support to save and
> restore zmm register in the dynamic linker, but did not enhance
> test-xmmymm.sh to detect accidental usage of these registers. The
> patch below adds that check.
>
> Tested on x86_64 by injecting an instruction into dl-runtime.os.
>
> Siddhesh
>
> * sysdeps/x86/tst-xmmymm.sh: Check for zmm register usage.
>
> diff --git a/sysdeps/x86/tst-xmmymm.sh b/sysdeps/x86/tst-xmmymm.sh
> index 69ddb58..fe37328 100755
> --- a/sysdeps/x86/tst-xmmymm.sh
> +++ b/sysdeps/x86/tst-xmmymm.sh
> @@ -1,5 +1,5 @@
> #! /bin/bash
> -# Make sure no code in ld.so uses xmm/ymm registers on x86-64.
> +# Make sure no code in ld.so uses xmm/ymm/zmm registers on x86-64.
> # Copyright (C) 2009-2014 Free Software Foundation, Inc.
> # This file is part of the GNU C Library.
>
> @@ -80,12 +80,12 @@ echo "object files needed: $tocheck"
> cp /dev/null "$tmp"
> for f in $tocheck; do
> $OBJDUMP -d "$objpfx"../*/"$f" |
> - awk 'BEGIN { last="" } /^[[:xdigit:]]* <[_[:alnum:]]*>:$/ { fct=substr($2, 2, length($2)-3) } /,%[xy]mm[[:digit:]]*$/ { if (last != fct) { print fct; last=fct} }' |
> + awk 'BEGIN { last="" } /^[[:xdigit:]]* <[_[:alnum:]]*>:$/ { fct=substr($2, 2, length($2)-3) } /,%[xyz]mm[[:digit:]]*$/ { if (last != fct) { print fct; last=fct} }' |
> while read fct; do
> if test "$fct" = "_dl_runtime_profile" -o "$fct" = "_dl_x86_64_restore_sse"; then
> continue;
> fi
> - echo "function $fct in $f modifies xmm/ymm" >> "$tmp"
> + echo "function $fct in $f modifies xmm/ymm/zmm" >> "$tmp"
> result=1
> done
> done
Please rename to tst-zmmxmmymm.sh and commit, otherwise it still
looks like it only checks for xmmymm, and that's misleading.
Cheers,
Carlos.