linux-kernel - Re: [RFC] per-cpu preempt

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 12 Aug 2013 11:53:25 -0700
From:	Linus Torvalds <torvalds@...ux-foundation.org>
To:	"H. Peter Anvin" <hpa@...or.com>
Cc:	Peter Zijlstra <peterz@...radead.org>,
	Ingo Molnar <mingo@...nel.org>,
	Thomas Gleixner <tglx@...utronix.de>,
	Mike Galbraith <bitbucket@...ine.de>,
	Andi Kleen <ak@...ux.intel.com>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [RFC] per-cpu preempt_count

On Mon, Aug 12, 2013 at 10:51 AM, H. Peter Anvin <hpa@...or.com> wrote:
>
> So we would have code looking something like:
>
>         decl %fs:preempt_count
>         jnz 1f
>         cmpb $0,%fs:need_resched
>         je 1f
>         call __preempt_schedule
> 1:
>
> It's a nontrivial amount of code, but would seem a fair bit better than
> what we have now, at least.

Well, we currently don't even bother checking the preempt count at
all, and we just optimistically assume that we don't nest in the
common case. The preempt count is then re-checked in
__preempt_schedule, I think.

Which sounds like a fair approach.

So the code would be simplified to just

         decl %fs:preempt_count
         cmpb $0,%fs:need_resched
         jne .. unlikely branch that calls __preempt_schedule

which is not horrible. Not *quite* as nice as just doing a single
"decl+js", but hey, certainly better than what we have now.

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