Remove useless second time checking of fsave argument in save_i387_ia32()
routine.  It's possible the compiler is doing the same but that is much
better to remove the dead code explicitly.
Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
        if (fsave)
                return 0;
        err |= __put_user(tsk->thread.i387.fxsave.swd, &buf->status);
-       if (fsave)
-               return err ? -1 : 1;
        err |= __put_user(X86_FXSR_MAGIC, &buf->magic);
        err |= __copy_to_user(&buf->_fxsr_env[0], &tsk->thread.i387.fxsave,
                              sizeof(struct i387_fxsave_struct));