diff options
author | Andy Lutomirski <luto@amacapital.net> | 2014-09-23 13:50:54 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2014-10-28 06:22:11 -0400 |
commit | 9c0080ef93bce34db130db6dc3bd946348e6a203 (patch) | |
tree | de4e8e425a37cdc576a83547595febc935410cb9 /arch/x86/vdso | |
parent | 258801563ba1be05793f2417ae30557b43a24e4e (diff) |
x86/vdso: Make the PER_CPU segment start out accessed
The first userspace attempt to read or write the PER_CPU segment
will write the accessed bit to the GDT. This is visible to
userspace using the LAR instruction, and it also pointlessly
dirties a cache line.
Set the segment's accessed bit at boot to prevent userspace
access to segments from having side effects.
Signed-off-by: Andy Lutomirski <luto@amacapital.net>
Link: http://lkml.kernel.org/r/ac63814ca4c637a08ec2fd0360d67ca67560a9ee.1411494540.git.luto@amacapital.net
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'arch/x86/vdso')
-rw-r--r-- | arch/x86/vdso/vma.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/x86/vdso/vma.c b/arch/x86/vdso/vma.c index 261b1349acc9..0c7997467be0 100644 --- a/arch/x86/vdso/vma.c +++ b/arch/x86/vdso/vma.c | |||
@@ -264,7 +264,7 @@ static void vsyscall_set_cpu(int cpu) | |||
264 | d = (struct desc_struct) { | 264 | d = (struct desc_struct) { |
265 | .limit0 = cpu | ((node & 0xf) << 12), | 265 | .limit0 = cpu | ((node & 0xf) << 12), |
266 | .limit = node >> 4, | 266 | .limit = node >> 4, |
267 | .type = 4, /* RO data, expand down */ | 267 | .type = 5, /* RO data, expand down, accessed */ |
268 | .dpl = 3, /* Visible to user code */ | 268 | .dpl = 3, /* Visible to user code */ |
269 | .s = 1, /* Not a system segment */ | 269 | .s = 1, /* Not a system segment */ |
270 | .p = 1, /* Present */ | 270 | .p = 1, /* Present */ |