> On 25 Apr 2019, at 20:53, Michael S. Tsirkin wrote:
>
> On Thu, Apr 25, 2019 at 09:38:19AM +0100, Stefan Hajnoczi wrote:
>> On Wed, Apr 24, 2019 at 07:38:57PM +0300, Liran Alon wrote:
>>>
>>>
>>>> On 18 Apr 2019, at 12:41, Stefan Hajnoczi wr
> On 6 Jun 2019, at 11:42, Dr. David Alan Gilbert wrote:
>
> * Liran Alon (liran.a...@oracle.com) wrote:
>> Hi,
>>
>> Looking at QEMU source code, I am puzzled regarding how migration backwards
>> compatibility is preserved regarding X86CPU.
>>
>&
> On 6 Jun 2019, at 12:23, Dr. David Alan Gilbert wrote:
>
> * Liran Alon (liran.a...@oracle.com) wrote:
>>
>>
>>> On 6 Jun 2019, at 11:42, Dr. David Alan Gilbert wrote:
>>>
>>> * Liran Alon (liran.a...@oracle.com) wrote:
>>>&
> On 6 Jun 2019, at 13:39, Dr. David Alan Gilbert wrote:
>
> * Liran Alon (liran.a...@oracle.com) wrote:
>>
>>
>>> On 6 Jun 2019, at 12:23, Dr. David Alan Gilbert wrote:
>>>
>>> * Liran Alon (liran.a...@oracle.com) wrote:
>>>>
> On 6 Jun 2019, at 14:07, Dr. David Alan Gilbert wrote:
>
> * Liran Alon (liran.a...@oracle.com) wrote:
>>
>>
>>> On 6 Jun 2019, at 13:39, Dr. David Alan Gilbert wrote:
>>>
>>> * Liran Alon (liran.a...@oracle.com) wrote:
>>>>
> On 6 Jun 2019, at 16:31, Dr. David Alan Gilbert wrote:
>
>>>
>>> So we still need to tie subsections to machine types; that way
>>> you don't send them to old qemu's and there for you don't have the
>>> problem of the qemu receiving something it doesn't know.
>>
>> I agree that if there is
> On 15 Jun 2019, at 3:42, Paolo Bonzini wrote:
>
> From: Liran Alon
>
> Reviewed-by: Nikita Leshenko
> Signed-off-by: Liran Alon
> Signed-off-by: Paolo Bonzini
> ---
> target/i386/kvm.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
&
> On 15 Jun 2019, at 3:42, Paolo Bonzini wrote:
>
> From: Liran Alon
>
> Kernel commit c4f55198c7c2 ("kvm: x86: Introduce KVM_CAP_EXCEPTION_PAYLOAD")
> introduced a new KVM capability which allows userspace to correctly
> distinguish between pending and
> On 15 Jun 2019, at 3:42, Paolo Bonzini wrote:
>
> From: Liran Alon
>
> Kernel commit 8fcc4b5923af ("kvm: nVMX: Introduce KVM_CAP_NESTED_STATE")
> introduced new IOCTLs to extract and restore KVM internal state used to
> run a VM that is in VMX operation.
&
> On 15 Jun 2019, at 3:42, Paolo Bonzini wrote:
>
> This patch improves the KVM_GET/SET_NESTED_STATE structs by detailing
> the format of VMX nested state in a struct. The VMX nested state is
> accessible through struct kvm_vmx_nested_state though, to avoid
> changing the size of the structs,
> On 15 Jun 2019, at 3:57, Liran Alon wrote:
>
>> On 15 Jun 2019, at 3:42, Paolo Bonzini wrote:
>>
>> From: Liran Alon
>>
>> +static bool is_vmx_enabled(CPUX86State *env)
>> +{
>> +return (IS_INTEL_CPU(env) && (env->cr[4] &
Ping on my last reply.
I would like to reach to an agreement on how v3 should look like before just
implementing what I think is right.
Thanks,
-Liran
> On 3 Nov 2018, at 4:02, Liran Alon wrote:
>
>
>
>> On 2 Nov 2018, at 18:39, Jim Mattson wrote:
>>
>>
> On 8 Nov 2018, at 11:50, Paolo Bonzini wrote:
>
> On 08/11/2018 01:45, Jim Mattson wrote:
>> I have no attachments to the current design. I had used a data[] blob,
>> because I didn't think userspace would have any need to know what was
>> in there. However, I am now seeing the error of my w
> On 8 Nov 2018, at 19:02, Paolo Bonzini wrote:
>
> On 08/11/2018 10:57, Liran Alon wrote:
>>
>>
>>> On 8 Nov 2018, at 11:50, Paolo Bonzini wrote:
>>>
>>> On 08/11/2018 01:45, Jim Mattson wrote:
>>>> I have no attachments to
> On 12 Nov 2018, at 18:54, Daniel P. Berrangé wrote:
>
> On Mon, Nov 12, 2018 at 04:50:54PM +, Dr. David Alan Gilbert wrote:
>> * Daniel P. Berrangé (berra...@redhat.com) wrote:
>>> On Sun, Nov 04, 2018 at 11:19:57PM +0100, Paolo Bonzini wrote:
On 02/11/2018 17:54, Daniel P. Berrangé
> On 12 Nov 2018, at 18:50, Dr. David Alan Gilbert wrote:
>
> * Daniel P. Berrangé (berra...@redhat.com) wrote:
>> On Sun, Nov 04, 2018 at 11:19:57PM +0100, Paolo Bonzini wrote:
>>> On 02/11/2018 17:54, Daniel P. Berrangé wrote:
We have usually followed a rule that new machine types must
> On 13 Nov 2018, at 2:07, Jim Mattson wrote:
>
> On Mon, Nov 12, 2018 at 4:00 PM, Liran Alon wrote:
>>
>>
>>> On 12 Nov 2018, at 18:54, Daniel P. Berrangé wrote:
>>>
>>> On Mon, Nov 12, 2018 at 04:50:54PM +, Dr. David Alan
Gentle ping on v2 of this series.
(I noticed 1st patch of series was already applied)
Thanks,
-Liran
> On 16 Sep 2018, at 15:46, Liran Alon wrote:
>
> Hi,
>
> This series aims to add support for QEMU to be able to migrate VMs that
> are running nested hypervisors. In
Another gentle ping on v2 of this series?
Patch was submitted a month ago.
Thanks,
-Liran
> On 8 Oct 2018, at 20:21, Liran Alon wrote:
>
> Gentle ping on v2 of this series.
> (I noticed 1st patch of series was already applied)
>
> Thanks,
> -Liran
>
>> On 16
Ping.
Patch was submitted almost two months ago and I haven’t seen any respond for
the v2 of this series.
Thanks,
-Liran
> On 15 Oct 2018, at 21:10, Liran Alon wrote:
>
> Another gentle ping on v2 of this series?
> Patch was submitted a month ago.
>
> Thanks,
> -Liran
&g
> On 31 Oct 2018, at 20:19, Paolo Bonzini wrote:
>
> On 31/10/2018 19:17, Eduardo Habkost wrote:
>> On Wed, Oct 31, 2018 at 03:03:34AM +0200, Liran Alon wrote:
>>> Ping.
>>> Patch was submitted almost two months ago and I haven’t seen any respond
>>&g
> On 31 Oct 2018, at 20:59, Dr. David Alan Gilbert wrote:
>
> * Liran Alon (liran.a...@oracle.com) wrote:
>>
>>
>>> On 31 Oct 2018, at 20:19, Paolo Bonzini wrote:
>>>
>>> On 31/10/2018 19:17, Eduardo Habkost wrote:
>>>>
> On 1 Nov 2018, at 15:10, Dr. David Alan Gilbert wrote:
>
> * Liran Alon (liran.a...@oracle.com) wrote:
>>
>>
>>> On 31 Oct 2018, at 20:59, Dr. David Alan Gilbert
>>> wrote:
>>>
>>> * Liran Alon (liran.a...@oracle.com) wrote:
> On 1 Nov 2018, at 17:56, Dr. David Alan Gilbert wrote:
>
> * Liran Alon (liran.a...@oracle.com) wrote:
>>
>>
>>> On 1 Nov 2018, at 15:10, Dr. David Alan Gilbert wrote:
>>>
>>> * Liran Alon (liran.a...@oracle.com) wrote:
>>>>
> On Thu, Nov1, 2018 at 09:45 AM, Jim Mattson wrote:
>> On Thu, Nov 1, 2018 at 8:56 AM, Dr. David Alan Gilbert
>> wrote:
>> So if I have matching host kernels it should always work?
>> What happens if I upgrade the source kernel to increase it's maximum
>> nested size, can I force it to keep t
> On 2 Nov 2018, at 11:40, Paolo Bonzini wrote:
>
> On 02/11/2018 04:46, Liran Alon wrote:
>>> On Thu, Nov1, 2018 at 09:45 AM, Jim Mattson wrote:
>>
>>>> On Thu, Nov 1, 2018 at 8:56 AM, Dr. David Alan Gilbert
>>>> wrote:
>>
>>
> On 2 Nov 2018, at 18:39, Jim Mattson wrote:
>
> On Thu, Nov 1, 2018 at 8:46 PM, Liran Alon wrote:
>
>> Hmm this makes sense.
>>
>> This means though that the patch I have submitted here isn't good enough.
>> My patch currently assumes that
> On 25 Aug 2018, at 15:19, Peng Hao wrote:
>
> diff --git a/hw/pci-host/piix.c b/hw/pci-host/piix.c
> index 0e60834..da73743 100644
> --- a/hw/pci-host/piix.c
> +++ b/hw/pci-host/piix.c
> @@ -327,6 +327,10 @@ static void i440fx_pcihost_realize(DeviceState *dev,
> Error **errp)
>
> sysbu
While at it, also rename var to indicate it is not used only in KVM.
Reviewed-by: Nikita Leshchenko
Reviewed-by: Patrick Colp
Signed-off-by: Liran Alon
---
target/i386/cpu.h | 4 +++-
target/i386/hvf/README.md | 2 +-
target/i386/hvf/hvf.c | 2 +-
target/i386/hvf/x86hvf.c | 4
Hi,
This series aims to add support for QEMU to be able to migrate VMs that
are running nested hypervisors. In order to do so, it utilizes the new
IOCTLs introduced in KVM commit 8fcc4b5923af ("kvm: nVMX: Introduce
KVM_CAP_NESTED_STATE") which were created for this purpose.
1st patch is not reall
by: Nikita Leshchenko
Reviewed-by: Patrick Colp
Signed-off-by: Liran Alon
---
accel/kvm/kvm-all.c | 14 +
include/sysemu/kvm.h | 1 +
target/i386/cpu.h | 2 ++
target/i386/kvm.c | 58 +++
target/i386/mach
>On 14/09/2018 09:16, Paolo Bonzini wrote:
>Heh, I was going to send a similar patch. However, things are a bit
>more complex for two reason.
>
>First, I'd prefer to reuse the hflags and hflags2 fields that we already
>have, and only store the VMCS blob in the subsection. For example,
>HF_SVMI_MA
> On 14 Sep 2018, at 13:59, Paolo Bonzini wrote:
>
> On 14/09/2018 11:54, Liran Alon wrote:
>>> On 14/09/2018 09:16, Paolo Bonzini wrote:
>>> Heh, I was going to send a similar patch. However, things are a bit
>>> more complex for two reason.
>>>
> On 14 Sep 2018, at 18:08, Paolo Bonzini wrote:
>
> On 14/09/2018 16:31, Liran Alon wrote:
>>> There is still a problem, however, in that the same input stream would
>>> be parsed differently depending on the kernel version. In particular,
>>> if in the
> On 15 Sep 2018, at 23:48, Liran Alon wrote:
>
>
>
>> On 14 Sep 2018, at 18:08, Paolo Bonzini wrote:
>>
>> On 14/09/2018 16:31, Liran Alon wrote:
>>>> There is still a problem, however, in that the same input stream would
>>>> be p
While at it, also rename var to indicate it is not used only in KVM.
Reviewed-by: Nikita Leshchenko
Reviewed-by: Patrick Colp
Reviewed-by: Mihai Carabas
Signed-off-by: Liran Alon
---
target/i386/cpu.h | 4 +++-
target/i386/hvf/README.md | 2 +-
target/i386/hvf/hvf.c | 2
by: Nikita Leshchenko
Reviewed-by: Patrick Colp
Reviewed-by: Mihai Carabas
Signed-off-by: Liran Alon
---
accel/kvm/kvm-all.c | 15 +++
include/sysemu/kvm.h | 1 +
target/i386/cpu.h | 2 ++
target/i386/kvm.c | 58
target/i386/mach
Hi,
This series aims to add support for QEMU to be able to migrate VMs that
are running nested hypervisors. In order to do so, it utilizes the new
IOCTLs introduced in KVM commit 8fcc4b5923af ("kvm: nVMX: Introduce
KVM_CAP_NESTED_STATE") which were created for this purpose.
1st patch is not reall
From: Leonid Shatz
Signed-off-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/hw/display/vmware_vga.c b/hw/display/vmware_vga.c
index 0bbb78b9a6fd..60a672530840 100644
FIFO
desync.
Patches 26-27 add vmware-svga capabilities required by Linux
kernel vmware-svga driver.
Patch 28 is a small code refactoring change.
Patch 29 disallows setting screen size to zero width/height to avoid
VNC window surprisingly disappearing.
Regards,
-Liran Alon
From: Leonid Shatz
While we continue to ignore SVGA_CMD_RECT_ROP_FILL, SVGA_CMD_RECT_ROP_COPY
and SVGA_CMD_FENCE commands, we should account for command length, not only
arguments following command code.
Signed-off-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw
From: Leonid Shatz
For better readability of debug output, show these values in decimal notation,
as they are defined in source by decimal integers.
Signed-off-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 6 +++---
1 file changed, 3
Future patches will add handling of commands that are parsed but
deliberately ignored. This change adds required framework for
avoiding printing parsing error messages for these commands,
when we encounter them in the FIFO.
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display
From: Leonid Shatz
Prefer variable names of cursor structure from VMware SVGA DevKit.
In addition, make sure to use the AND mask bpp parameter in
SVGA_CMD_DEFINE_CURSOR.
Signed-off-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 26
Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/hw/display/vmware_vga.c b/hw/display/vmware_vga.c
index b32a625ae9c2..c30ae9b4b204 100644
--- a/hw/display/vmware_vga.c
+++ b/hw/display/vmware_vga.c
@@ -
From: Leonid Shatz
Should not change semantics.
Signed-off-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 60 -
1 file changed, 34 insertions(+), 26 deletions(-)
diff --git a/hw/display
alue, an opaque cookie which is used by internal
debugging tools. Third party drivers should not use this
command."
Signed-off-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
dif
From: Leonid Shatz
Some guests are using 64x64 32bpp cursor pixel array, the old size of 1024
integers
was not sufficient to store such large masks.
Signed-off-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 2 +-
1 file changed, 1
: Liran Alon
---
hw/display/vmware_vga.c | 56 -
1 file changed, 28 insertions(+), 28 deletions(-)
diff --git a/hw/display/vmware_vga.c b/hw/display/vmware_vga.c
index f8c5b64cfd7c..a244f43a866f 100644
--- a/hw/display/vmware_vga.c
+++ b/hw/display
From: Leonid Shatz
Import FIFO register definitions from VMware SVGA Device Developer Kit.
Report number of available registers so that guest device driver can
reserve enough space for registers before command FIFO.
Signed-off-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/hw/display/vmware_vga.c b/hw/display/vmware_vga.c
index fab6443a87e2..675c8755ab48 100644
--- a/hw/display/vmware_vga.c
+++ b/hw/display
-off-by: Liran Alon
---
hw/display/vmware_vga.c | 43 ---
1 file changed, 36 insertions(+), 7 deletions(-)
diff --git a/hw/display/vmware_vga.c b/hw/display/vmware_vga.c
index edd336e65005..bd7202833081 100644
--- a/hw/display/vmware_vga.c
+++ b/hw/display
requires to support the following registers: SVGA_FIFO_CAPABILITIES,
SVGA_FIFO_FLAGS and SVGA_FIFO_3D_HWVERSION.
For more information on how SVGA_FIFO_3D_HWVERSION is negoitated, see
SVGA3D_Init() in VMware SVGA development kit.
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display
/* Must be SVGA_ROP_COPY */
}
SVGAFifoCmdFrontRopFill;
Signed-off-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 9 -
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/hw/display/vmware_vga.c b/hw/display/vmwar
From: Leonid Shatz
This is necessary in order for device to raise interrupts.
Future patches will add functionality to device which will
need this ability.
Signed-off-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 1 +
1 file changed, 1
From: Leonid Shatz
Signed-off-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/hw/display/vmware_vga.c b/hw/display/vmware_vga.c
index 389248b4badf..1803a565fa07 100644
--- a/hw/display
From: Leonid Shatz
Should not change semantics.
This is done as a preparation for future patches.
Signed-off-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 8
1 file changed, 8 insertions(+)
diff --git a/hw/display/vmware_vga.c b
Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 7 ---
1 file changed, 7 deletions(-)
diff --git a/hw/display/vmware_vga.c b/hw/display/vmware_vga.c
index c30ae9b4b204..f0e6b4bc74ba 100644
--- a/hw/display/vmware_vga.c
+++ b/hw/display/vmware_vga.c
or no hint of visible mouse cursor.
Signed-off-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 85 +
1 file changed, 72 insertions(+), 13 deletions(-)
diff --git a/hw/display/vmware_vga.c b/hw
From: Leonid Shatz
Based on Linux kernel
drivers/gpu/drm/vmwgfx/device_include/svga_reg.h.
Signed-off-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 12
1 file changed, 12 insertions(+)
diff --git a/hw/display/vmware_vga.c b
From: Leonid Shatz
Add missing functionality of interrupt mask and status registers.
Writing to interrupt status register clears interrupt request.
Signed-off-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 52
From: Leonid Shatz
This adds tracking of guest cursor position with the help of FIFO
registers reporting pointing device coordindates.
Signed-off-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 38 +-
1
by guest in VMware SVGA development kit
SVGAFIFOFull().
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/hw/display/vmware_vga.c b/hw/display/vmware_vga.c
index 597051ec5c92..dc5f4681f0d3 100644
--- a
FENCE_GOAL.
*/
Signed-off-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 33 +
1 file changed, 25 insertions(+), 8 deletions(-)
diff --git a/hw/display/vmware_vga.c b/hw/display/vmware_vga.c
index dc5f4681f0d3..73
We don't support GMR regions while reporting caps, but some guests may try
to send us some GMR queries and we do our best to ignore them while avoiding
FIFO command crash.
Reported-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c
From: Leonid Shatz
AND/XOR mask is a standard method for defining hardware cursor images.
These are also the names suggested by VMware SVGA DevKit.
Signed-off-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 22 +++---
1 file
From: Leonid Shatz
This in only code refactoring without change in semantics.
Signed-off-by: Leonid Shatz
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 45 +
1 file changed, 25 insertions(+), 20 deletions
Reviewed-by: Darren Kenny
Signed-off-by: Liran Alon
---
hw/display/vmware_vga.c | 66 +
1 file changed, 66 insertions(+)
diff --git a/hw/display/vmware_vga.c b/hw/display/vmware_vga.c
index b2f3456357bd..edd336e65005 100644
--- a/hw/display
> On 10 Aug 2018, at 12:56, Gerd Hoffmann wrote:
>
> Hi,
>
>> +case SVGA_REG_DISPLAY_WIDTH:
>> +if ((s->display_id == 0) || (s->display_id == SVGA_ID_INVALID))
>> +ret = s->new_width ? s->new_width : surface_width(surface);
>> +else
>> +ret = 0;
>>
> On 10 Aug 2018, at 13:14, Gerd Hoffmann wrote:
>
> On Thu, Aug 09, 2018 at 02:46:13PM +0300, Liran Alon wrote:
>> Hi,
>>
>> This patch series aim to fix many issues in vmware-svga emulation
>> which have prevented it from being fully functional in a wide-va
> On 10 Aug 2018, at 13:00, Gerd Hoffmann wrote:
>
> Hi,
>
>> +if (cmd == SVGA_CMD_UPDATE_VERBOSE)
>> +vmsvga_fifo_read(s);
>
> Code style. qemu wants braces unconditionally. There might be more of
> this in following patches. You can use scripts/checkpatch.pl
- kra...@redhat.com wrote:
> Hi,
>
> > > But gdm doesn't start, neither with X11 nor with wayland. Which
> is a
> > > regression, because the xorg vmware driver manages to handle the
> current
> > > qemu emulation just fine so gnome @ X11 works.
> >
> > Interesting. Thanks for finding thi
urns the number of #SMIs that occurred on
> > CPU since boot.
> >
> > KVM commit 52797bf9a875 ("KVM: x86: Add emulation of
> MSR_SMI_COUNT")
> > introduced support for emulating this MSR.
> >
> > This commit adds support for QEMU to save/load th
This MSR returns the number of #SMIs that occurred on
CPU since boot.
KVM commit 52797bf9a875 ("KVM: x86: Add emulation of MSR_SMI_COUNT")
introduced support for emulating this MSR.
This commit adds support for QEMU to save/load this
MSR for migration purposes.
Signed-off-by:
ating this MSR.
>
> This commit adds support for QEMU to save/load this
> MSR for migration purposes.
>
> Signed-off-by: Liran Alon
> Reviewed-by: Konrad Rzeszutek Wilk
> Signed-off-by: Konrad Rzeszutek Wilk
> ---
> target/i386/cpu.c | 1 +
> targe
2020 16:50:08 +0200
Liran Alon wrote:
Microsoft introduced this ACPI table to avoid Windows guests performing
various workarounds for device erratas. As the virtual device emulated
by VMM may not have the errata.
Currently, WAET allows hypervisor to inform guest about two
specific behaviors: On
On 22/03/2020 13:22, Liran Alon wrote:
On 15/03/2020 13:56, Liran Alon wrote:
On 14/03/2020 22:56, Michael S. Tsirkin wrote:
On Sat, Mar 14, 2020 at 10:05:20PM +0200, Liran Alon wrote:
Michael, you can also refer to this VMware time-keeping whitepaper:
https://urldefense.com/v3/__https
pkg_config parameter should be "--exists", not "--exist".
This is probably a typo.
Fixes: 3a67848134d0 ("configure: Enable test and libs for zstd")
Reviewed-by: Bhavesh Davda
Signed-off-by: Liran Alon
---
configure | 2 +-
1 file changed, 1 insertion(+), 1 deletio
Instead of hard-coding the VMX version, make it a VMPORT object property.
This would allow user to control it's value via "-global vmport.vmx-version=X".
Reviewed-by: Nikita Leshenko
Signed-off-by: Liran Alon
---
hw/i386/vmport.c | 6 +-
1 file changed, 5 insertions(+), 1 de
->env.regs[R_EAX].
To fix this issue, change vmport_ioport_read() to explicitly override
cpu->env.regs[R_EAX] with the value it wish to propagate to vCPU EAX
register.
Reviewed-by: Nikita Leshenko
Signed-off-by: Liran Alon
---
hw/i386/vmport.c | 23 +--
1 file chang
Command currently returns that it is unimplemented by setting
the reserved-bit in it's return value.
Following patches will return various useful vCPU information
to guest.
Reviewed-by: Nikita Leshenko
Signed-off-by: Liran Alon
---
hw/i386/vmport.c | 13 +
include/hw/i386
t;) and Linux try_to_enable_x2apic()
function.
Reviewed-by: Nikita Leshenko
Signed-off-by: Liran Alon
---
hw/i386/vmport.c | 9 -
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/hw/i386/vmport.c b/hw/i386/vmport.c
index b33ef9c01d65..2b0a623f19c1 100644
--- a/hw/i386/vmport.c
This command is used by guest to gettimeofday() from host.
See usage example in open-vm-tools TimeSyncReadHost() function.
Reviewed-by: Nikita Leshenko
Signed-off-by: Liran Alon
---
hw/i386/vmport.c | 21 +
include/hw/i386/pc.h | 1 +
2 files changed, 22 insertions
is unimplemented.
Reviewed-by: Nikita Leshenko
Signed-off-by: Liran Alon
---
hw/i386/vmport.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/hw/i386/vmport.c b/hw/i386/vmport.c
index 9319720e8204..a78e20040a79 100644
--- a/hw/i386/vmport.c
+++ b/hw/i386/vmport.c
@@ -72,6 +72,7 @@ static
No functional change.
This is done as a preparation for the following patches that will
introduce several device properties.
Reviewed-by: Nikita Leshenko
Signed-off-by: Liran Alon
---
hw/i386/vmport.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/hw/i386/vmport.c b/hw/i386
No functional change.
This information will be used by following patches.
Reviewed-by: Nikita Leshenko
Signed-off-by: Liran Alon
---
linux-headers/asm-x86/kvm.h | 4
target/i386/cpu.h | 1 +
target/i386/kvm.c | 6 +++---
3 files changed, 8 insertions(+), 3 deletions
As can be seen from VmCheck_GetVersion() in open-vm-tools code,
CMD_GETVERSION should return VMX type in ECX register.
Default is to fake host as VMware ESX server. But user can control
this value by "-global vmport.vmx-type=X".
Reviewed-by: Nikita Leshenko
Signed-off-by: Liran Alo
realized.
Reviewed-by: Nikita Leshenko
Signed-off-by: Liran Alon
---
hw/i386/vmport.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/hw/i386/vmport.c b/hw/i386/vmport.c
index 95d4a23ce9ba..659a323e8448 100644
--- a/hw/i386/vmport.c
+++ b/hw/i386/vmport.c
@@ -68,6 +68,8 @@ static VMPortState
Hi,
This series aims to fix several bugs in VMPort and improve it by supporting
more VMPort commands and make command results more configurable to
user via QEMU command-line.
This functionality was proven to be useful to run various VMware VMs
when attempting to run them as-is on top of QEMU/KVM.
Similar to CMD_GETTIME but lacks the 136-year overflow issue,
by returning full 64-bit of host uSeconds.
Reviewed-by: Nikita Leshenko
Signed-off-by: Liran Alon
---
hw/i386/vmport.c | 17 +
include/hw/i386/pc.h | 1 +
2 files changed, 18 insertions(+)
diff --git a/hw/i386
ko
Signed-off-by: Liran Alon
---
hw/i386/vmport.c | 19 +++
include/hw/i386/pc.h | 1 +
2 files changed, 20 insertions(+)
diff --git a/hw/i386/vmport.c b/hw/i386/vmport.c
index 2b0a623f19c1..95d4a23ce9ba 100644
--- a/hw/i386/vmport.c
+++ b/hw/i386/vmport.c
@@ -150,6 +150,24
VMPort commands supported by QEMU.
Reviewed-by: Nikita Leshenko
Signed-off-by: Liran Alon
---
hw/i386/vmmouse.c| 18 ++
hw/i386/vmport.c | 11 ++-
include/hw/i386/pc.h | 11 ++-
3 files changed, 18 insertions(+), 22 deletions(-)
diff --git a/hw/i386/vmmouse.c
This is VMware documented functionallity that some guests rely on.
Returns the BIOS UUID of the current virtual machine.
Reviewed-by: Nikita Leshenko
Signed-off-by: Liran Alon
---
hw/i386/vmport.c | 14 ++
include/hw/i386/pc.h | 1 +
2 files changed, 15 insertions(+)
diff
On 10/03/2020 2:13, no-re...@patchew.org wrote:
Patchew URL:
https://urldefense.com/v3/__https://patchew.org/QEMU/20200309235411.76587-1-liran.a...@oracle.com/__;!!GqivPVa7Brio!IXfneuptZ4caQyQEEgsIN74dLYkkLHJ8XSqL2iGyrzODUx_Jt8Vk-eKN_QQftVU$
Hi,
This series seems to have some coding style
guest attempt to calibrate TSC.
Signed-off-by: Liran Alon
---
target/i386/cpu.c | 1 +
target/i386/cpu.h | 1 +
target/i386/kvm.c | 35 +--
3 files changed, 31 insertions(+), 6 deletions(-)
diff --git a/target/i386/cpu.c b/target/i386/cpu.c
index 92fafa265914
On 10/03/2020 2:54, no-re...@patchew.org wrote:
Patchew URL:
https://urldefense.com/v3/__https://patchew.org/QEMU/20200309235411.76587-1-liran.a...@oracle.com/__;!!GqivPVa7Brio!N5car8IBDE_QFIGGfZAaPfF3gc3sv_CIDltJvgI8K5bvOLgPcGxKnVicygP6jf4$
Hi,
This series failed the asan build test. Plea
On 10/03/2020 11:29, Michael S. Tsirkin wrote:
On Tue, Mar 10, 2020 at 01:54:09AM +0200, Liran Alon wrote:
No functional change.
This information will be used by following patches.
Reviewed-by: Nikita Leshenko
Signed-off-by: Liran Alon
---
linux-headers/asm-x86/kvm.h | 4
target
On 10/03/2020 11:30, Michael S. Tsirkin wrote:
On Tue, Mar 10, 2020 at 01:54:11AM +0200, Liran Alon wrote:
vmport_register() is also called from other modules such as vmmouse.
Therefore, these modules rely that vmport is realized before those call
sites. If this is violated, vmport_register
On 10/03/2020 11:32, Michael S. Tsirkin wrote:
On Tue, Mar 10, 2020 at 01:54:01AM +0200, Liran Alon wrote:
Instead of hard-coding the VMX version, make it a VMPORT object property.
This would allow user to control it's value via "-global vmport.vmx-version=X".
Reviewed-by:
On 10/03/2020 11:34, Michael S. Tsirkin wrote:
On Tue, Mar 10, 2020 at 01:54:04AM +0200, Liran Alon wrote:
This is VMware documented functionallity that some guests rely on.
Returns the BIOS UUID of the current virtual machine.
Reviewed-by: Nikita Leshenko
Signed-off-by: Liran Alon
So
1 - 100 of 270 matches
Mail list logo