From: Linus Torvalds <torvalds@linux-foundation.org>
To: Suresh Siddha <sbsiddha@gmail.com>
Cc: Nate Eldredge <nate@thatsmathematics.com>,
Thomas Gleixner <tglx@linutronix.de>,
Ingo Molnar <mingo@kernel.org>, "H. Peter Anvin" <hpa@zytor.com>,
"the arch/x86 maintainers" <x86@kernel.org>,
stable <stable@vger.kernel.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Maarten Baert <maarten-baert@hotmail.com>,
Jan Kara <jack@suse.cz>, George Spelvin <linux@horizon.com>,
Pekka Riikonen <priikone@iki.fi>
Subject: Re: [PATCH] Make math_state_restore() save and restore the interrupt flag
Date: Mon, 3 Feb 2014 10:20:19 -0800 [thread overview]
Message-ID: <CA+55aFyrUTshLPk=21H+6ySOeEVS_zVUAK=2QPrq0aqdQ=3_zg@mail.gmail.com> (raw)
In-Reply-To: <1391410583.3801.6.camel@europa>
On Sun, Feb 2, 2014 at 10:56 PM, Suresh Siddha <sbsiddha@gmail.com> wrote:
>
> Other patch which cleans up the irq_enable/disable logic in
> math_state_restore() has been sent yesterday. You can run your
> experiments with both these patches if you want. But your issue should
> get fixed with just the appended patch here.
>
> Peter, Please push both these patches through normal channels depending
> on the results.
Thinking about it some more, this patch is *almost* not needed at all.
I'm wondering if you should just change the first patch to just always
initialize the fpu when it is allocated, and at execve() time (ie in
flush_thread()).
If we do that, then this:
+ if (!tsk_used_math(tsk))
+ init_fpu(tsk);
can be dropped entirely from math_state_restore(). And quite frankly,
at that point, I think all the changes to __kernel_fpu_end() can go
away, because at that point math_state_restore() really does the right
thing - all the allocations are gone, and all the async task state
games are gone, only the "restore state" remains.
Hmm? So the only thing needed would be to add that "init_fpu()" to the
initial bootmem allocation path and to change flush_thread() (it
currently does "drop_init_fpu()", let's just make it initialize the
FPU state using fpu_finit()), and then we could remove the whole
"used_math" bit entirely, and just say that the FPU is always
initialized.
What do you guys think?
Linus
next prev parent reply other threads:[~2014-02-03 18:20 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-30 22:01 [PATCH] Make math_state_restore() save and restore the interrupt flag Nate Eldredge
2014-01-30 22:24 ` Linus Torvalds
2014-01-31 7:33 ` Suresh Siddha
2014-02-01 19:27 ` Linus Torvalds
2014-02-01 19:35 ` H. Peter Anvin
2014-02-01 19:46 ` Linus Torvalds
2014-02-01 20:00 ` H. Peter Anvin
2014-02-01 20:16 ` Linus Torvalds
2014-02-01 20:16 ` H. Peter Anvin
2014-02-01 21:17 ` George Spelvin
2014-02-01 21:36 ` H. Peter Anvin
2014-02-01 23:40 ` H. Peter Anvin
2014-02-02 0:17 ` Linus Torvalds
2014-02-02 1:19 ` George Spelvin
2014-02-02 1:25 ` H. Peter Anvin
2014-02-02 8:45 ` Pekka Riikonen
2014-02-02 1:06 ` Suresh Siddha
2014-02-02 1:26 ` H. Peter Anvin
2014-02-02 1:35 ` Suresh Siddha
2014-02-02 1:38 ` Linus Torvalds
2014-02-02 1:47 ` Suresh Siddha
2014-02-02 1:51 ` Linus Torvalds
2014-02-02 1:57 ` H. Peter Anvin
2014-02-02 2:05 ` Linus Torvalds
2014-02-02 2:12 ` H. Peter Anvin
2014-02-02 1:59 ` Suresh Siddha
2014-02-02 1:43 ` H. Peter Anvin
2014-02-02 1:47 ` Linus Torvalds
2014-02-02 7:19 ` Suresh Siddha
2014-02-02 19:15 ` Linus Torvalds
2014-02-03 6:56 ` Suresh Siddha
2014-02-03 18:20 ` Linus Torvalds [this message]
2014-02-04 6:03 ` Suresh Siddha
2014-02-06 5:26 ` Nate Eldredge
2014-02-06 5:34 ` George Spelvin
2014-02-13 15:45 ` Maarten Baert
2014-02-13 20:00 ` George Spelvin
2014-03-11 19:36 ` [tip:x86/urgent] x86, fpu: Check tsk_used_math() in kernel_fpu_end() for eager FPU tip-bot for Suresh Siddha
2014-02-27 23:44 ` [PATCH] Make math_state_restore() save and restore the interrupt flag H. Peter Anvin
2014-03-07 23:18 ` H. Peter Anvin
2014-03-08 6:18 ` Suresh Siddha
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CA+55aFyrUTshLPk=21H+6ySOeEVS_zVUAK=2QPrq0aqdQ=3_zg@mail.gmail.com' \
--to=torvalds@linux-foundation.org \
--cc=hpa@zytor.com \
--cc=jack@suse.cz \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@horizon.com \
--cc=maarten-baert@hotmail.com \
--cc=mingo@kernel.org \
--cc=nate@thatsmathematics.com \
--cc=priikone@iki.fi \
--cc=sbsiddha@gmail.com \
--cc=stable@vger.kernel.org \
--cc=tglx@linutronix.de \
--cc=x86@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).