sched/x86: Add a comment clarifying LDT context switching
authorAndy Lutomirski <luto@amacapital.net>
Mon, 6 Oct 2014 19:36:47 +0000 (12:36 -0700)
committerIngo Molnar <mingo@kernel.org>
Tue, 28 Oct 2014 10:11:57 +0000 (11:11 +0100)
commitc4a7bba29bc6d9fcd5842d712b6b3d493b682b17
treeb95b1763bfd07b0e9fa1427adb1d293dfdd51e1b
parent2c7577a7583747c9b71f26dced7f696b739da745
sched/x86: Add a comment clarifying LDT context switching

The code is correct, but only for a rather subtle reason.  This
confused me for quite a while when I read switch_mm, so clarify
the code to avoid confusing other people, too.

TBH, I wouldn't be surprised if this code was only correct by
accident.

[ I wouldn't normally send a comment-only patch, but it took me a long
  time to first figure out wtf was going on here, and then to figure
  out why this wasn't exploitable by malicious code, and then to
  figure out why this oddity had no user-visible effect at all.  Let's
  spare future readers the same confusion. ]

Signed-off-by: Andy Lutomirski <luto@amacapital.net>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Link: http://lkml.kernel.org/r/36275c99801a87d8dcf0502a41cf4e2ad81aae46.1412623954.git.luto@amacapital.net
Signed-off-by: Ingo Molnar <mingo@kernel.org>
arch/x86/include/asm/mmu_context.h