parisc: pass through '\t' to early (iodc) console
authorKyle McMartin <kyle@mcmartin.ca>
Wed, 4 Aug 2010 00:38:08 +0000 (20:38 -0400)
committerLinus Torvalds <torvalds@linux-foundation.org>
Wed, 4 Aug 2010 18:17:10 +0000 (11:17 -0700)
The firmware handles '\t' internally, so stop trying to emulate it
(which, incidentally, had a bug in it.)

Fixes a really weird hang at bootup in rcu_bootup_announce, which,
as far as I can tell, is the first printk in the core kernel to use
a tab as the first character.

Cc: stable@kernel.org
Signed-off-by: Kyle McMartin <kyle@parisc-linux.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
arch/parisc/kernel/firmware.c

index 4c247e02d9b1b0e0655e47ebc15470801b4d9881..df971fa0c32f1fcbf411437a8e61ce07be5e57af 100644 (file)
@@ -1123,7 +1123,6 @@ static char __attribute__((aligned(64))) iodc_dbuf[4096];
  */
 int pdc_iodc_print(const unsigned char *str, unsigned count)
 {
-       static int posx;        /* for simple TAB-Simulation... */
        unsigned int i;
        unsigned long flags;
 
@@ -1133,19 +1132,12 @@ int pdc_iodc_print(const unsigned char *str, unsigned count)
                        iodc_dbuf[i+0] = '\r';
                        iodc_dbuf[i+1] = '\n';
                        i += 2;
-                       posx = 0;
                        goto print;
-               case '\t':
-                       while (posx & 7) {
-                               iodc_dbuf[i] = ' ';
-                               i++, posx++;
-                       }
-                       break;
                case '\b':      /* BS */
-                       posx -= 2;
+                       i--; /* overwrite last */
                default:
                        iodc_dbuf[i] = str[i];
-                       i++, posx++;
+                       i++;
                        break;
                }
        }