releases/3.4.103/arch-sparc-math-emu-math_32.c-drop-stray-break-operator.patch

   1 From foo@baz Thu Aug  7 22:33:35 PDT 2014
   2 From: Andrey Utkin <andrey.krieger.utkin@gmail.com>
   3 Date: Mon, 4 Aug 2014 23:47:41 +0300
   4 Subject: arch/sparc/math-emu/math_32.c: drop stray break operator
   5
   6 From: Andrey Utkin <andrey.krieger.utkin@gmail.com>
   7
   8 [ Upstream commit 093758e3daede29cb4ce6aedb111becf9d4bfc57 ]
   9
  10 This commit is a guesswork, but it seems to make sense to drop this
  11 break, as otherwise the following line is never executed and becomes
  12 dead code. And that following line actually saves the result of
  13 local calculation by the pointer given in function argument. So the
  14 proposed change makes sense if this code in the whole makes sense (but I
  15 am unable to analyze it in the whole).
  16
  17 Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=81641
  18 Reported-by: David Binderman <dcb314@hotmail.com>
  19 Signed-off-by: Andrey Utkin <andrey.krieger.utkin@gmail.com>
  20 Signed-off-by: David S. Miller <davem@davemloft.net>
  21 Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
  22 ---
  23  arch/sparc/math-emu/math_32.c |    2 +-
  24  1 file changed, 1 insertion(+), 1 deletion(-)
  25
  26 --- a/arch/sparc/math-emu/math_32.c
  27 +++ b/arch/sparc/math-emu/math_32.c
  28 @@ -499,7 +499,7 @@ static int do_one_mathemu(u32 insn, unsi
  29                 case 0: fsr = *pfsr;
  30                         if (IR == -1) IR = 2;
  31                         /* fcc is always fcc0 */
  32 -                       fsr &= ~0xc00; fsr |= (IR << 10); break;
  33 +                       fsr &= ~0xc00; fsr |= (IR << 10);
  34                         *pfsr = fsr;
  35                         break;
  36                 case 1: rd->s = IR; break;