Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Enhance tst-xmmymm.sh to detect zmm register usage in ld.so (BZ #16194)
2d63a51 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.

The script has also been renamed to tst-ld-sse-use.sh.  To see the
minimal changes, run `git show -M`.

	[BZ #16194]
	* sysdeps/x86/tst-xmmymm.sh: Rename file to...
	* sysdeps/x86/tst-ld-sse-use.sh: ... this.  Check for zmm
	register usage.
	* sysdeps/x86/Makefile: Adjust.
  • Loading branch information
Siddhesh Poyarekar committed Sep 12, 2014
1 parent 8c0ab91 commit 5379aeb
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 6 deletions.
8 changes: 8 additions & 0 deletions ChangeLog
@@ -1,3 +1,11 @@
2014-09-12 Siddhesh Poyarekar <siddhesh@redhat.com>

[BZ #16194]
* sysdeps/x86/tst-xmmymm.sh: Rename file to...
* sysdeps/x86/tst-ld-sse-use.sh: ... this. Check for zmm
register usage.
* sysdeps/x86/Makefile: Adjust.

2014-09-11 Kostya Serebryany <konstantin.s.serebryany@gmail.com>
Roland McGrath <roland@hack.frob.com>

Expand Down
4 changes: 2 additions & 2 deletions sysdeps/x86/Makefile
Expand Up @@ -2,8 +2,8 @@ ifeq ($(subdir),elf)
CFLAGS-.os += $(if $(filter $(@F),$(patsubst %,%.os,$(all-rtld-routines))),\
-mno-sse -mno-mmx)

tests-special += $(objpfx)tst-xmmymm.out
$(objpfx)tst-xmmymm.out: ../sysdeps/x86/tst-xmmymm.sh $(objpfx)ld.so
tests-special += $(objpfx)tst-ld-sse-use.out
$(objpfx)tst-ld-sse-use.out: ../sysdeps/x86/tst-ld-sse-use.sh $(objpfx)ld.so
@echo "Checking ld.so for SSE register use. This will take a few seconds..."
$(SHELL) $< $(objpfx) '$(NM)' '$(OBJDUMP)' '$(READELF)' > $@; \
$(evaluate-test)
Expand Down
8 changes: 4 additions & 4 deletions sysdeps/x86/tst-xmmymm.sh → sysdeps/x86/tst-ld-sse-use.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.

Expand All @@ -24,7 +24,7 @@ NM="$2"
OBJDUMP="$3"
READELF="$4"

tmp=$(mktemp ${objpfx}tst-xmmymm.XXXXXX)
tmp=$(mktemp ${objpfx}tst-ld-sse-use.XXXXXX)
trap 'rm -f "$tmp"' 1 2 3 15

# List of object files we have to test
Expand Down Expand Up @@ -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
Expand Down

0 comments on commit 5379aeb

Please sign in to comment.