diff options
| author | Andy Lutomirski <luto@kernel.org> | 2021-01-19 18:40:55 +0100 |
|---|---|---|
| committer | Borislav Petkov <bp@suse.de> | 2021-01-21 20:08:53 +0100 |
| commit | 8ece53ef7f428ee3f8eab936268b1a3fe2725e6b (patch) | |
| tree | d301122cac7f7b02c221441b4824b91144b0ec80 /arch/x86/include/asm/vm86.h | |
| parent | x86: Remove definition of DEBUG (diff) | |
| download | linux-8ece53ef7f428ee3f8eab936268b1a3fe2725e6b.tar.xz linux-8ece53ef7f428ee3f8eab936268b1a3fe2725e6b.zip | |
x86/vm86/32: Remove VM86_SCREEN_BITMAP support
The implementation was rather buggy. It unconditionally marked PTEs
read-only, even for VM_SHARED mappings. I'm not sure whether this is
actually a problem, but it certainly seems unwise. More importantly, it
released the mmap lock before flushing the TLB, which could allow a racing
CoW operation to falsely believe that the underlying memory was not
writable.
I can't find any users at all of this mechanism, so just remove it.
Signed-off-by: Andy Lutomirski <luto@kernel.org>
Signed-off-by: Borislav Petkov <bp@suse.de>
Acked-by: Stas Sergeev <stsp2@yandex.ru>
Link: https://lkml.kernel.org/r/f3086de0babcab36f69949b5780bde851f719bc8.1611078018.git.luto@kernel.org
Diffstat (limited to 'arch/x86/include/asm/vm86.h')
| -rw-r--r-- | arch/x86/include/asm/vm86.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/arch/x86/include/asm/vm86.h b/arch/x86/include/asm/vm86.h index 26efbec94448..9e8ac5073ecb 100644 --- a/arch/x86/include/asm/vm86.h +++ b/arch/x86/include/asm/vm86.h @@ -36,7 +36,6 @@ struct vm86 { unsigned long saved_sp0; unsigned long flags; - unsigned long screen_bitmap; unsigned long cpu_type; struct revectored_struct int_revectored; struct revectored_struct int21_revectored; |