[S390] Inline assembly cleanup.

Major cleanup of all s390 inline assemblies. They now have a common
coding style. Quite a few have been shortened, mainly by using register
asm variables. Use of the EX_TABLE macro helps  as well. The atomic ops,
bit ops and locking inlines new use the Q-constraint if a newer gcc
is used.  That results in slightly better code.

Thanks to Christian Borntraeger for proof reading the changes.

Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>

1 files changed, 14 insertions, 40 deletions

diff --git a/drivers/s390/block/xpram.c b/drivers/s390/block/xpram.c
index cab2c736683a..a04d9120cef0 100644
--- a/drivers/s390/block/xpram.c
+++ b/drivers/s390/block/xpram.c
@@ -89,28 +89,15 @@ MODULE_LICENSE("GPL");
  */
 static int xpram_page_in (unsigned long page_addr, unsigned int xpage_index)
 {
-        int cc;
+        int cc = 2;     /* return unused cc 2 if pgin traps */
 
-        __asm__ __volatile__ (
-                "   lhi   %0,2\n"  /* return unused cc 2 if pgin traps */
-                "   .insn rre,0xb22e0000,%1,%2\n"  /* pgin %1,%2 */
-                "0: ipm   %0\n"
-                "   srl   %0,28\n"
+        asm volatile(
+                "       .insn   rre,0xb22e0000,%1,%2\n"  /* pgin %1,%2 */
+                "0:     ipm     %0\n"
+                "       srl     %0,28\n"
                 "1:\n"
-#ifndef CONFIG_64BIT
-                ".section __ex_table,\"a\"\n"
-                "   .align 4\n"
-                "   .long  0b,1b\n"
-                ".previous"
-#else
-                ".section __ex_table,\"a\"\n"
-                "   .align 8\n"
-                "   .quad 0b,1b\n"
-                ".previous"
-#endif
-                : "=&d" (cc) 
-                : "a" (__pa(page_addr)), "a" (xpage_index) 
-                : "cc" );
+                EX_TABLE(0b,1b)
+                : "+d" (cc) : "a" (__pa(page_addr)), "d" (xpage_index) : "cc");
         if (cc == 3)
                 return -ENXIO;
         if (cc == 2) {
@@ -137,28 +124,15 @@ static int xpram_page_in (unsigned long page_addr, unsigned int xpage_index)
  */
 static long xpram_page_out (unsigned long page_addr, unsigned int xpage_index)
 {
-        int cc;
+        int cc = 2;     /* return unused cc 2 if pgin traps */
 
-        __asm__ __volatile__ (
-                "   lhi   %0,2\n"  /* return unused cc 2 if pgout traps */
-                "   .insn rre,0xb22f0000,%1,%2\n"  /* pgout %1,%2 */
-                "0: ipm   %0\n"
-                "   srl   %0,28\n"
+        asm volatile(
+                "       .insn   rre,0xb22f0000,%1,%2\n"  /* pgout %1,%2 */
+                "0:     ipm     %0\n"
+                "       srl     %0,28\n"
                 "1:\n"
-#ifndef CONFIG_64BIT
-                ".section __ex_table,\"a\"\n"
-                "   .align 4\n"
-                "   .long  0b,1b\n"
-                ".previous"
-#else
-                ".section __ex_table,\"a\"\n"
-                "   .align 8\n"
-                "   .quad 0b,1b\n"
-                ".previous"
-#endif
-                : "=&d" (cc) 
-                : "a" (__pa(page_addr)), "a" (xpage_index) 
-                : "cc" );
+                EX_TABLE(0b,1b)
+                : "+d" (cc) : "a" (__pa(page_addr)), "d" (xpage_index) : "cc");
         if (cc == 3)
                 return -ENXIO;
         if (cc == 2) {