-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 01/30/2015 04:46 PM, Dave Hansen wrote: > On 01/30/2015 09:49 AM, r...@redhat.com wrote: >> @@ -440,8 +440,9 @@ static inline fpu_switch_t >> switch_fpu_prepare(struct task_struct *old, struct ta >> new->thread.fpu_counter > 5); if (__thread_has_fpu(old)) { if >> (!__save_init_fpu(old)) - cpu = ~0; - >> old->thread.fpu.last_cpu >> = cpu; + task_disable_lazy_fpu_restore(old); + >> else + >> old->thread.fpu.last_cpu = cpu; > > What is the 'else' doing here? It seems a bit disconnected from > the other parts of the patch.
The assignment was already there before, but we would set cpu to ~0 if __save_init_fpu failed. Now there are different branches for failure and success, with the success branch having the same assignment that was there before. - -- All rights reversed -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQEcBAEBAgAGBQJUy/w1AAoJEM553pKExN6D/fsH/RjpQE67P368cvHl6QKa0RUo TDF0iEl4XqLT9wlYPsr+NgVJG0rFK36iVrGtKVV/uziNmk+1H43WZUt/u8tsTC1k gYptwOeBpGVyktmwdperBejTm3q4k9WLYASOvU90S2fMO5IavCpafkM5CJwojFie 0fquULSkgmyxXIJrNTNYznA2YYZaiHcKY8i05H9W0iLYbYQseQhDKl7sZ5uWcfd2 ziWmWpMKSgD0AaC1PbMm8k4kKHkbaKlabR9AyOd+9Nk5voU/pPcJOm4QgQru2ZhR 2wRdVA+MMwoglsPygV3I2OPqJN95o0JAgSTJk5DXfPuiMYoAe2+KWy5pa0ERvMo= =Fmf0 -----END PGP SIGNATURE----- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/