- 20 Sep, 2012 1 commit
-
-
Avi Kivity authored
gpte_access() computes the access permissions of a guest pte and also write-protects clean gptes. This is wrong when we are servicing a write fault (since we'll be setting the dirty bit momentarily) but correct when instantiating a speculative spte, or when servicing a read fault (since we'll want to trap a following write in order to set the dirty bit). It doesn't seem to hurt in practice, but in order to make the code readable, push the write protection out of gpte_access() and into a new protect_clean_gpte() which is called explicitly when needed. Reviewed-by:
Xiao Guangrong <xiaoguangrong@linux.vnet.ibm.com> Signed-off-by:
Avi Kivity <avi@redhat.com>
-
- 17 Sep, 2012 5 commits
-
-
Stefan Fritsch authored
- mention that system time needs to be added to wallclock time - positive tsc_shift means left shift, not right - mention additional 32bit right shift Signed-off-by:
Stefan Fritsch <sf@sfritsch.de> Signed-off-by:
Marcelo Tosatti <mtosatti@redhat.com>
-
Michael S. Tsirkin authored
vcpu mutex can be held for unlimited time so taking it with mutex_lock on an ioctl is wrong: one process could be passed a vcpu fd and call this ioctl on the vcpu used by another process, it will then be unkillable until the owner exits. Call mutex_lock_killable instead and return status. Note: mutex_lock_interruptible would be even nicer, but I am not sure all users are prepared to handle EINTR from these ioctls. They might misinterpret it as an error. Cleanup paths expect a vcpu that can't be used by any userspace so this will always succeed - catch bugs by calling BUG_ON. Catch callers that don't check return state by adding __must_check. Signed-off-by:
Michael S. Tsirkin <mst@redhat.com> Signed-off-by:
-
Avi Kivity authored
Use macros for bitness-insensitive register names, instead of rolling our own. Signed-off-by:
-
Avi Kivity authored
Use macros for bitness-insensitive register names, instead of rolling our own. Signed-off-by:
-
Avi Kivity authored
LTO (link-time optimization) doesn't like local labels to be referred to from a different function, since the two functions may be built in separate compilation units. Use an external variable instead. Signed-off-by:
-
- 12 Sep, 2012 1 commit
-
-
Takuya Yoshikawa authored
find_highest_vector() and count_vectors(): - Instead of using magic values, define and use proper macros. find_highest_vector(): - Remove likely() which is there only for historical reasons and not doing correct branch predictions anymore. Using such heuristics to optimize this function is not worth it now. Let CPUs predict things instead. - Stop checking word[0] separately. This was only needed for doing likely() optimization. - Use for loop, not while, to iterate over the register array to make the code clearer. Note that we actually confirmed that the likely() did wrong predictions by inserting debug code. Acked-by:
Takuya Yoshikawa <yoshikawa.takuya@oss.ntt.co.jp> Signed-off-by:
-
- 10 Sep, 2012 2 commits
-
-
Xiao Guangrong authored
Checking the return of kvm_mmu_get_page is unnecessary since it is guaranteed by memory cache Signed-off-by:
-
Liu, Jinsong authored
KVM lapic timer and tsc deadline timer based on hrtimer, setting a leftmost node to rb tree and then do hrtimer reprogram. If hrtimer not configured as high resolution, hrtimer_enqueue_reprogram do nothing and then make kvm lapic timer and tsc deadline timer fail. Signed-off-by:
Liu, Jinsong <jinsong.liu@intel.com> Signed-off-by:
-
- 09 Sep, 2012 1 commit
-
-
Jan Kiszka authored
Signed-off-by:
Jan Kiszka <jan.kiszka@siemens.com> Signed-off-by:
-
- 06 Sep, 2012 8 commits
-
-
Michael S. Tsirkin authored
interrupt_bitmap is KVM_NR_INTERRUPTS bits in size, so just use that instead of hard-coded constants and math. Signed-off-by:
-
Gleb Natapov authored
Optimize "rep ins" by allowing emulator to write back more than one datum at a time. Introduce new operand type OP_MEM_STR which tells writeback() that dst contains pointer to an array that should be written back as opposite to just one data element. Signed-off-by:
Gleb Natapov <gleb@redhat.com> Signed-off-by:
-
Gleb Natapov authored
Remove unneeded segment argument. Address structure already has correct segment which was put there during decode. Signed-off-by:
-
Gleb Natapov authored
Signed-off-by:
-
Gleb Natapov authored
Current code assumes that IO exit was due to instruction emulation and handles execution back to emulator directly. This patch adds new userspace IO exit completion callback that can be set by any other code that caused IO exit to userspace. Signed-off-by:
-
Marcelo Tosatti authored
Other arches do not need this. Signed-off-by:
-
Marcelo Tosatti authored
PPC must flush all translations before the new memory slot is visible. Signed-off-by:
-
Marcelo Tosatti authored
Introducing kvm_arch_flush_shadow_memslot, to invalidate the translations of a single memory slot. Signed-off-by:
-
- 05 Sep, 2012 8 commits
-
-
Mathias Krause authored
We never modify direct_access_msrs[], msrpm_ranges[], svm_exit_handlers[] or x86_intercept_map[] at runtime. Mark them r/o. Signed-off-by:
Mathias Krause <minipli@googlemail.com> Cc: Joerg Roedel <joerg.roedel@amd.com> Signed-off-by:
-
Mathias Krause authored
We use vmcs_field_to_offset_table[], kvm_vmx_segment_fields[] and kvm_vmx_exit_handlers[] as lookup tables only -- make them r/o. Signed-off-by:
-
Mathias Krause authored
Signed-off-by:
-
Mathias Krause authored
We never change those, make them r/o. Signed-off-by:
-
Mathias Krause authored
We never change emulate_ops[] at runtime so it should be r/o. Signed-off-by:
-
Mathias Krause authored
The opcode tables never change at runtime, therefor mark them const. Signed-off-by:
-
Mathias Krause authored
As the the compiler ensures that the memory operand is always aligned to a 16 byte memory location, use the aligned variant of MOVDQ for read_sse_reg() and write_sse_reg(). Signed-off-by:
-
Mathias Krause authored
Some fields can be constified and/or made static to reduce code and data size. Numbers for a 32 bit build: text data bss dec hex filename before: 3351 80 0 3431 d67 cpuid.o after: 3391 0 0 3391 d3f cpuid.o Signed-off-by:
-
- 04 Sep, 2012 1 commit
-
-
Gleb Natapov authored
kvm_pic_reset() is not used anywhere. Move reset logic from pic_ioport_write() there. Signed-off-by:
-
- 30 Aug, 2012 1 commit
-
-
Marcelo Tosatti authored
Signed-off-by:
-
- 27 Aug, 2012 12 commits
-
-
Avi Kivity authored
We will enter the guest with G and D cleared; as real hardware ignores D in real mode, and G is taken care of by the limit test, we allow more code to run in vm86 mode. Signed-off-by:
-
Avi Kivity authored
Signed-off-by:
-
Avi Kivity authored
While this is undocumented, real processors do not reload the segment limit and access rights when loading a segment register in real mode. Real programs rely on it so we need to comply with this behaviour. Signed-off-by:
-
Avi Kivity authored
emulate_invalid_guest_state=1 doesn't mean we don't munge the segments in the vmcs; we do. So we need to return the real ones (maintained by vmx_set_segment). Signed-off-by:
-
Avi Kivity authored
We want the segment selector, nor segment number. Signed-off-by:
-
Avi Kivity authored
Segment limits are verified in real mode, not just protected mode. Signed-off-by:
-
Avi Kivity authored
When loading a segment in real mode, only the base and selector must be modified. The limit needs to be left alone, otherwise big real mode users will hit a #GP due to limit checking (currently this is suppressed because we don't check limits in real mode). Signed-off-by:
-
Avi Kivity authored
Usually, big real mode uses large (4GB) segments. Currently we don't virtualize this; if any segment has a limit other than 0xffff, we emulate. But if we set the vmx-visible limit to 0xffff, we can use vm86 to virtualize real mode; if an access overruns the segment limit, the guest will #GP, which we will trap and forward to the emulator. This results in significantly faster execution, and less risk of hitting an unemulated instruction. If the limit is less than 0xffff, we retain the existing behaviour. Signed-off-by:
-
Avi Kivity authored
Real mode is always entered from protected mode with dpl=0. Since the dpl doesn't affect execution, and we already override it to 3 in the vmcs (as vmx requires), we can allow execution in that state. Signed-off-by:
-
Avi Kivity authored
Real processors don't change segment limits and attributes while in real mode. Mimic that behaviour. Signed-off-by:
-
Avi Kivity authored
Instead of using struct kvm_save_segment, use struct kvm_segment, which is what the other APIs use. This leads to some simplification. We replace save_rmode_seg() with a call to vmx_save_segment(). Since this depends on rmode.vm86_active, we move the call to before setting the flag. Signed-off-by:
-
Avi Kivity authored
fix_pmode_dataseg() looks up S in ->base instead of ->ar_bytes. Signed-off-by:
-
