Re: [patch] x64, fpu: fix possible FPU leakage in error conditions

From: Linus Torvalds
Date: Thu Jul 24 2008 - 15:02:59 EST




On Thu, 24 Jul 2008, Suresh Siddha wrote:
>
> If we don't do init_fpu() + restore from the sane init state, process has
> to be killed, in the paranoid failing scenario of math_state_restore()

Umm. I'm still not seeing why the right answer is not just to do "stts +
math_used = 0".

And the specific case of math_state_restore(), that will also fix the
problem - next time around.

As far as I can tell, your patch causes serious problems in case
init_fpu() fails. Which it can do, afaik.

Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/