Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 29 Mar 2020 17:30:09 +0000 (UTC)
From:      Ian Lepore <ian@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r359423 - head/sys/arm/include
Message-ID:  <202003291730.02THU9V3081753@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: ian
Date: Sun Mar 29 17:30:08 2020
New Revision: 359423
URL: https://svnweb.freebsd.org/changeset/base/359423

Log:
  Add a missing suffix that was causing a whole word to get loaded instead
  of the proper 8 or 16 bits when the macro was expanded for those sizes.
  
  Fixes a hang in the armv7 kernel.
  
  Submitted by:	Thomas Skibo
  Pointy hat:	ian@

Modified:
  head/sys/arm/include/atomic-v6.h

Modified: head/sys/arm/include/atomic-v6.h
==============================================================================
--- head/sys/arm/include/atomic-v6.h	Sun Mar 29 15:43:00 2020	(r359422)
+++ head/sys/arm/include/atomic-v6.h	Sun Mar 29 17:30:08 2020	(r359423)
@@ -196,7 +196,7 @@ ATOMIC_ACQ_REL_LONG(clear)
                                                               \
 	__asm __volatile(                                     \
 	    "1: ldrex" SUF "   %[tmp], [%[ptr]]          \n"  \
-	    "   ldr            %[ret], [%[oldv]]         \n"  \
+	    "   ldr" SUF "     %[ret], [%[oldv]]         \n"  \
 	    "   teq            %[tmp], %[ret]            \n"  \
 	    "   ittee          ne                        \n"  \
 	    "   str" SUF "ne   %[tmp], [%[oldv]]         \n"  \



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202003291730.02THU9V3081753>