From: Paolo Bonzini <pbonzini@redhat.com>
To: Jan Kiszka <jan.kiszka@siemens.com>,
Nadav Amit <namit@cs.technion.ac.il>
Cc: kvm@vger.kernel.org, nadav.amit@gmail.com
Subject: Re: [PATCH 12/21] KVM: x86: MOV to CR3 can set bit 63
Date: Tue, 10 Feb 2015 17:18:48 +0100 [thread overview]
Message-ID: <54DA2F68.8080901@redhat.com> (raw)
In-Reply-To: <54DA2EBA.10803@siemens.com>
On 10/02/2015 17:15, Jan Kiszka wrote:
>> > #define CR3_L_MODE_RESERVED_BITS 0xFFFFFF0000000000ULL
>> > +#define CR3_PCID_INVD (1UL << 63)
> 1ULL (for i386)
Already fixed:
commit cfaa790a3fb8a7efa98f4a6457e19dc3a0db35d3
Author: Borislav Petkov <bp@suse.de>
Date: Thu Jan 15 09:44:56 2015 +0100
kvm: Fix CR3_PCID_INVD type on 32-bit
arch/x86/kvm/emulate.c: In function ‘check_cr_write’:
arch/x86/kvm/emulate.c:3552:4: warning: left shift count >= width of type
rsvd = CR3_L_MODE_RESERVED_BITS & ~CR3_PCID_INVD;
happens because sizeof(UL) on 32-bit is 4 bytes but we shift it 63 bits
to the left.
Signed-off-by: Borislav Petkov <bp@suse.de>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> Paolo, there is no 32-bit test build anymore on your side, right?
Reinstating it has been on my todo list for a while. But as of now
I'm not doing 32-bit tests.
> I was
> about to drop them from kvm-kmod as well, but at least 2 remained in
> place and caught this.
next prev parent reply other threads:[~2015-02-10 16:19 UTC|newest]
Thread overview: 59+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-02 9:54 [PATCH 00/21] Fixes for various KVM bugs Nadav Amit
2014-11-02 9:54 ` [PATCH 01/21] KVM: x86: decode_modrm does not regard modrm correctly Nadav Amit
2014-11-05 11:14 ` Paolo Bonzini
2014-11-02 9:54 ` [PATCH 02/21] KVM: x86: No error-code on real-mode exceptions Nadav Amit
2014-11-02 9:54 ` [PATCH 03/21] KVM: x86: Emulator should set DR6 upon GD like real CPU Nadav Amit
2014-11-02 9:54 ` [PATCH 04/21] KVM: x86: Clear DR6[0:3] on #DB during handle_dr Nadav Amit
2014-11-02 9:54 ` [PATCH 05/21] KVM: x86: Breakpoints do not consider CS.base Nadav Amit
2014-11-02 9:54 ` [PATCH 06/21] KVM: x86: Emulator MOV-sreg uses incorrect size Nadav Amit
2014-11-05 11:28 ` Paolo Bonzini
2014-11-02 9:54 ` [PATCH 07/21] KVM: x86: Emulator considers imm as memory operand Nadav Amit
2014-11-05 11:36 ` Paolo Bonzini
2014-11-02 9:54 ` [PATCH 08/21] KVM: x86: Reset FPU state during reset Nadav Amit
2014-11-05 12:04 ` Paolo Bonzini
2014-11-05 13:20 ` Nadav Amit
2014-11-05 14:55 ` Paolo Bonzini
2014-11-05 20:31 ` Nadav Amit
2014-11-06 8:58 ` Paolo Bonzini
2014-11-06 9:13 ` Nadav Amit
2014-11-06 9:44 ` Paolo Bonzini
2014-11-06 9:56 ` Nadav Amit
2014-11-06 10:44 ` Paolo Bonzini
2014-11-06 17:38 ` Radim Krčmář
2014-11-02 9:54 ` [PATCH 09/21] KVM: x86: SYSCALL cannot clear eflags[1] Nadav Amit
2014-11-02 9:54 ` [PATCH 10/21] KVM: x86: Wrong flags on CMPS and SCAS emulation Nadav Amit
2014-11-02 9:54 ` [PATCH 11/21] KVM: x86: Emulate push sreg as done in Core Nadav Amit
2014-11-02 9:54 ` [PATCH 12/21] KVM: x86: MOV to CR3 can set bit 63 Nadav Amit
2015-02-10 16:15 ` Jan Kiszka
2015-02-10 16:18 ` Paolo Bonzini [this message]
2015-02-10 16:34 ` Jan Kiszka
2015-02-10 16:42 ` Paolo Bonzini
2014-11-02 9:54 ` [PATCH 13/21] KVM: x86: Do not update EFLAGS on faulting emulation Nadav Amit
2014-11-02 9:54 ` [PATCH 14/21] KVM: x86: Software disabled APIC should still deliver NMIs Nadav Amit
2014-11-05 12:30 ` Paolo Bonzini
2014-11-05 20:45 ` Nadav Amit
2014-11-06 9:34 ` Paolo Bonzini
2014-11-06 16:45 ` Radim Krčmář
2014-11-10 17:35 ` Paolo Bonzini
2014-11-10 18:06 ` Radim Krčmář
2014-11-14 15:00 ` Paolo Bonzini
2014-11-26 17:01 ` Nadav Amit
2014-11-26 18:00 ` Paolo Bonzini
2014-11-27 13:39 ` Radim Krčmář
2014-11-27 21:45 ` Nadav Amit
2014-11-27 22:26 ` Radim Krčmář
2014-12-01 16:30 ` Paolo Bonzini
2014-12-01 17:49 ` Radim Krčmář
2014-11-02 9:54 ` [PATCH 15/21] KVM: x86: Combine the lgdt and lidt emulation logic Nadav Amit
2014-11-02 9:54 ` [PATCH 16/21] KVM: x86: Inject #GP when loading system segments with non-canonical base Nadav Amit
2014-11-02 9:54 ` [PATCH 17/21] KVM: x86: Remove redundant and incorrect cpl check on task-switch Nadav Amit
2014-11-02 9:54 ` [PATCH 18/21] KVM: x86: Emulator mis-decodes VEX instructions on real-mode Nadav Amit
2014-11-08 7:25 ` Paolo Bonzini
2014-11-02 9:54 ` [PATCH 19/21] KVM: x86: Warn on APIC base relocation Nadav Amit
2014-11-02 9:55 ` [PATCH 20/21] KVM: x86: MOVNTI emulation min opsize is not respected Nadav Amit
2014-11-05 12:18 ` Paolo Bonzini
2014-11-05 19:58 ` Nadav Amit
2014-11-05 19:58 ` Nadav Amit
2014-11-06 9:23 ` Paolo Bonzini
2014-11-02 9:55 ` [PATCH 21/21] KVM: x86: Return UNHANDLABLE on unsupported SYSENTER Nadav Amit
2014-11-05 12:31 ` [PATCH 00/21] Fixes for various KVM bugs Paolo Bonzini
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=54DA2F68.8080901@redhat.com \
--to=pbonzini@redhat.com \
--cc=jan.kiszka@siemens.com \
--cc=kvm@vger.kernel.org \
--cc=nadav.amit@gmail.com \
--cc=namit@cs.technion.ac.il \
/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).