about summary refs log tree commit diff
diff options
context:
space:
mode:
authorH.J. Lu <hongjiu.lu@intel.com>2009-11-20 08:30:20 -0800
committerUlrich Drepper <drepper@redhat.com>2009-11-20 08:30:20 -0800
commit66221417957963c1d2df8d79f43573e328df2275 (patch)
treefb61d07e4512b1aa04c27affdd5b03f3f2aacfa1
parentdf1934daa7788ca5f19d82efb02cdf9b9c495b1a (diff)
downloadglibc-66221417957963c1d2df8d79f43573e328df2275.tar.gz
glibc-66221417957963c1d2df8d79f43573e328df2275.tar.xz
glibc-66221417957963c1d2df8d79f43573e328df2275.zip
Properly recover from shorter read.
-rw-r--r--ChangeLog6
-rw-r--r--sysdeps/ia64/memchr.S14
2 files changed, 17 insertions, 3 deletions
diff --git a/ChangeLog b/ChangeLog
index a60bfcc8e6..c99965f6e4 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2009-11-18  H.J. Lu  <hongjiu.lu@intel.com>
+
+	[BZ #10162]
+	* sysdeps/ia64/memchr.S: Don't use a simple loop on data shorter
+	than software pipeline.  Properly recover from shorter read.
+
 2009-11-19  Ulrich Drepper  <drepper@redhat.com>
 
 	[BZ #10958]
diff --git a/sysdeps/ia64/memchr.S b/sysdeps/ia64/memchr.S
index e721e1714f..cdd71ca5a5 100644
--- a/sysdeps/ia64/memchr.S
+++ b/sysdeps/ia64/memchr.S
@@ -71,8 +71,7 @@ ENTRY(__memchr)
 	cmp.ne	p7, p0 = r0, r0		// clear p7
 	extr.u	chr = in1, 0, 8		// chr = (unsigned char) in1
 	mov	len = in2
-	// use a simple loop on data shorter than software pipeline
-	cmp.gtu	p6, p0 = ((MEMLAT + 1) * 8), in2
+	cmp.gtu	p6, p0 = 16, in2	// use a simple loop for short
 (p6)	br.cond.spnt .srchfew ;;	// searches
 	sub	loopcnt = 8, tmp	// loopcnt = 8 - tmp
 	cmp.eq	p6, p0 = tmp, r0
@@ -127,7 +126,16 @@ ENTRY(__memchr)
 	br.ret.sptk.many b0
 
 .recovery:
-	adds	ret0 = -((MEMLAT + 1) * 8), ret0;;
+#if MEMLAT != 6
+# error "MEMLAT must be 6!"
+#endif
+(p[MEMLAT-6])	add	ret0 = -8, ret0;;
+(p[MEMLAT-5])	add	ret0 = -8, ret0;;
+(p[MEMLAT-4])	add	ret0 = -8, ret0;;
+(p[MEMLAT-3])	add	ret0 = -8, ret0;;
+(p[MEMLAT-2])	add	ret0 = -8, ret0;;
+(p[MEMLAT-1])	add	ret0 = -8, ret0;;
+(p[MEMLAT])	add	ret0 = -8, ret0;;
 (p[MEMLAT+1])	add	ret0 = -8, ret0;;
 (p[MEMLAT+2])	add	ret0 = -8, ret0;;
 .l4: