From: Jan Kiszka jan.kis...@siemens.com
Without CONFIG_INTEL_TXT, the user must not enable this feature in the
BIOS. Otherwise, KVM will not work. Explain this dependency via a kernel
log message.
Signed-off-by: Jan Kiszka jan.kis...@siemens.com
---
arch/x86/kvm/vmx.c |7 ++-
1 files
Am 02.11.2010 08:31, Sheng Yang wrote:
On Tuesday 02 November 2010 15:05:51 Jan Kiszka wrote:
From: Jan Kiszka jan.kis...@siemens.com
Obtail the new pgd pointer before releasing the page containing this
value.
Signed-off-by: Jan Kiszka jan.kis...@siemens.com
---
Who is taking care of
On 11/14/2010 11:18 AM, Jan Kiszka wrote:
From: Jan Kiszkajan.kis...@siemens.com
Without CONFIG_INTEL_TXT, the user must not enable this feature in the
BIOS. Otherwise, KVM will not work. Explain this dependency via a kernel
log message.
Signed-off-by: Jan Kiszkajan.kis...@siemens.com
---
On 11/12/2010 11:20 AM, Stefan Hajnoczi wrote:
Who guarantees that less common virtio-blk and virtio-net guest drivers
for non-Linux OSes are fine with it? Maybe you should add a feature flag
that the guest has to ACK to enable it.
Virtio-blk and virtio-net are fine. Both of those
On Sun, Nov 14, 2010 at 10:34 AM, Avi Kivity a...@redhat.com wrote:
On 11/12/2010 11:20 AM, Stefan Hajnoczi wrote:
Who guarantees that less common virtio-blk and virtio-net guest drivers
for non-Linux OSes are fine with it? Maybe you should add a feature
flag
that the guest has to
Am 14.11.2010 11:30, Avi Kivity wrote:
On 11/14/2010 11:18 AM, Jan Kiszka wrote:
From: Jan Kiszkajan.kis...@siemens.com
Without CONFIG_INTEL_TXT, the user must not enable this feature in the
BIOS. Otherwise, KVM will not work. Explain this dependency via a kernel
log message.
On 11/12/2010 12:30 PM, Xiao Guangrong wrote:
We just need flush tlb if overwrite a writable spte with a read-only one
What are the advantages? Avoid playing with rmap, and avoid a window
where the spte is missing?
(they are worth the patch, just seeing if I'm not missing something)
--
FYI. Long ago we discussed key value approach on top of virtio-serial.
Original Message
Subject: [PATCH]: An implementation of HyperV KVP functionality
Date: Thu, 11 Nov 2010 13:03:10 -0700
From: Ky Srinivasan ksriniva...@novell.com
To: de...@driverdev.osuosl.org,
On 11/12/2010 12:34 PM, Xiao Guangrong wrote:
We can past the page fault to guest directly if gpte's reserved
is set
How can that work? shadow_notrap_nonpresent_pte causes a fault with
PFEC.P=PFEC.RSVD=0, while we need PFEC.P=PFEC.RSVD=1.
Signed-off-by: Xiao
On 11/14/2010 12:41 PM, Jan Kiszka wrote:
Am 14.11.2010 11:30, Avi Kivity wrote:
On 11/14/2010 11:18 AM, Jan Kiszka wrote:
From: Jan Kiszkajan.kis...@siemens.com
Without CONFIG_INTEL_TXT, the user must not enable this feature in the
BIOS. Otherwise, KVM will not work. Explain this
On 11/14/2010 12:37 PM, Stefan Hajnoczi wrote:
On Sun, Nov 14, 2010 at 10:34 AM, Avi Kivitya...@redhat.com wrote:
On 11/12/2010 11:20 AM, Stefan Hajnoczi wrote:
Who guarantees that less common virtio-blk and virtio-net guest drivers
for non-Linux OSes are fine with it? Maybe you
On 11/12/2010 03:16 AM, Huang Ying wrote:
On Thu, 2010-11-11 at 17:39 +0800, Avi Kivity wrote:
On 11/11/2010 02:56 AM, Huang Ying wrote:
On Thu, 2010-11-11 at 00:49 +0800, Anthony Liguori wrote:
On 11/10/2010 02:34 AM, Avi Kivity wrote:
Why the gpa -hva mapping is not
On 11/14/2010 01:08 PM, Avi Kivity wrote:
4) Use -mempath on /dev/shm and poison a page in the backing file
If we can poison a page in the backing file, how do we know the
corresponding gpa and hpa?
I think you currently can't know it's gpa (why do you want to?); the
upcoming NUMA
On 11/14/2010 01:05 PM, Avi Kivity wrote:
I agree, but let's enable virtio-ioeventfd carefully because bad code
is out there.
Sure. Note as long as the thread waiting on ioeventfd doesn't consume
too much cpu, it will awaken quickly and we won't have the
transaction per timeslice effect.
Jan Kiszka wrote:
Strange, should work. I would suggest to post your full kernel log,
maybe there is some enlightening message hidden.
I don't think it is a problem of your kernel version, but I'm able to
pass through devices on OpenSUSE 11.3 with
kernel-desktop-2.6.36-90.1.x86_64 from their
I'd like to pick up this patch and rework it based on your
suggestions. Need it for [1]. Most opinions seem to be pointing to a
generic software mac filtering in the VLAN layer with hw support if
necessary. Here's what I had in mind:
- For the generic mac filtering we can use something similar to
As usual, make it the default guest on the default
kvm autotest run.
Signed-off-by: Lucas Meneghel Rodrigues l...@redhat.com
---
client/tests/kvm/get_started.py|6 +++---
client/tests/kvm/tests.cfg.sample | 14 +++---
client/tests/kvm/tests_base.cfg.sample | 26
Use device ioports to create unique device path.
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/isa-bus.c | 16
1 files changed, 16 insertions(+), 0 deletions(-)
diff --git a/hw/isa-bus.c b/hw/isa-bus.c
index c0ac7e9..c423c1b 100644
--- a/hw/isa-bus.c
+++ b/hw/isa-bus.c
New get_fw_dev_path callback will be used for build device path usable
by firmware in contrast to qdev qemu internal device path.
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/qdev.h |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/hw/qdev.h b/hw/qdev.h
index
Add fw_name to DeviceInfo to use in device path building. In
contrast to name fw_name should refer to functionality device
provides instead of particular device model like name does.
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/fdc.c|1 +
hw/ide/isa.c|1 +
hw/ide/qdev.c
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/ide/cmd646.c |4 ++--
hw/ide/internal.h |3 ++-
hw/ide/isa.c |2 +-
hw/ide/piix.c |4 ++--
hw/ide/qdev.c |3 ++-
hw/ide/via.c |4 ++--
6 files changed, 11 insertions(+), 9 deletions(-)
diff --git
If bootindex is specified on command line a string that describes device
in firmware readable way is added into sorted list. Later this list will
be passed into firmware to control boot order.
Signed-off-by: Gleb Natapov g...@redhat.com
---
block_int.h |4 +++-
hw/e1000.c |4
Prints out mmio or pio used to access child device.
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/pci_host.c |2 ++
hw/sysbus.c | 30 ++
hw/sysbus.h |4
3 files changed, 36 insertions(+), 0 deletions(-)
diff --git a/hw/pci_host.c
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/pci.c | 108 -
1 files changed, 85 insertions(+), 23 deletions(-)
diff --git a/hw/pci.c b/hw/pci.c
index 962886e..114b435 100644
--- a/hw/pci.c
+++ b/hw/pci.c
@@ -43,12 +43,14 @@
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/usb-bus.c | 42 ++
1 files changed, 42 insertions(+), 0 deletions(-)
diff --git a/hw/usb-bus.c b/hw/usb-bus.c
index 256b881..8b4583c 100644
--- a/hw/usb-bus.c
+++ b/hw/usb-bus.c
@@ -5,11 +5,13 @@
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/ide/qdev.c | 13 +
1 files changed, 13 insertions(+), 0 deletions(-)
diff --git a/hw/ide/qdev.c b/hw/ide/qdev.c
index 88ff657..01a181b 100644
--- a/hw/ide/qdev.c
+++ b/hw/ide/qdev.c
@@ -24,9 +24,12 @@
/*
Ports on root hub will have NULL here. This is needed to reconstruct
path from device to its root hub to build device path.
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/usb-bus.c |3 ++-
hw/usb-hub.c |2 +-
hw/usb-musb.c |2 +-
hw/usb-ohci.c |2 +-
hw/usb-uhci.c |2 +-
Extend -option-rom command to have additional parameter ,bootindex=.
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/loader.c| 16 +++-
hw/loader.h|8
hw/multiboot.c |3 ++-
hw/ne2000.c|2 +-
hw/nseries.c |2 +-
hw/pc.c|7
Store all io ports used by device in ISADevice structure.
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/cs4231a.c |1 +
hw/fdc.c |3 +++
hw/gus.c |4
hw/ide/isa.c |2 ++
hw/isa-bus.c | 25 +
hw/isa.h |4
This is current state of the patch series for people to comment on.
I am using open firmware naming scheme to specify device path names.
In this submission I addressed all comment from previous one and added
option rom support and rebased to qemu upstream.
Kevin can you double check that the
Change fw_cfg_add_file() to get full file path as a parameter instead
of building one internally. Two reasons for that. First caller may need
to know how file is named. Second this moves policy of file naming out
from fw_cfg. Platform may want to use more then two levels of
directories for
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/fw_cfg.c | 14 ++
hw/fw_cfg.h |4 +++-
sysemu.h|1 +
vl.c| 51 +++
4 files changed, 69 insertions(+), 1 deletions(-)
diff --git a/hw/fw_cfg.c b/hw/fw_cfg.c
Action that depends on fully initialized device model should register
with this notifier chain.
Signed-off-by: Gleb Natapov g...@redhat.com
---
sysemu.h |2 ++
vl.c | 15 +++
2 files changed, 17 insertions(+), 0 deletions(-)
diff --git a/sysemu.h b/sysemu.h
index
Am 14.11.2010 14:21, Erik Brakkee wrote:
Jan Kiszka wrote:
Strange, should work. I would suggest to post your full kernel log,
maybe there is some enlightening message hidden.
I don't think it is a problem of your kernel version, but I'm able to
pass through devices on OpenSUSE 11.3 with
On Sun, Nov 14, 2010 at 05:39:34PM +0200, Gleb Natapov wrote:
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/pci.c | 108 -
1 files changed, 85 insertions(+), 23 deletions(-)
diff --git a/hw/pci.c b/hw/pci.c
index
On Sun, Nov 14, 2010 at 05:39:41PM +0200, Gleb Natapov wrote:
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/fw_cfg.c | 14 ++
hw/fw_cfg.h |4 +++-
sysemu.h|1 +
vl.c| 51 +++
4 files changed, 69
On Sun, Nov 14, 2010 at 08:27:20PM +0200, Michael S. Tsirkin wrote:
On Sun, Nov 14, 2010 at 05:39:34PM +0200, Gleb Natapov wrote:
+static char *pcibus_get_fw_dev_path(DeviceState *dev)
+{
+PCIDevice *d = (PCIDevice *)dev;
+char path[50], name[33];
+int off;
+
+off =
On Sun, Nov 14, 2010 at 08:41:37PM +0200, Michael S. Tsirkin wrote:
On Sun, Nov 14, 2010 at 05:39:41PM +0200, Gleb Natapov wrote:
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/fw_cfg.c | 14 ++
hw/fw_cfg.h |4 +++-
sysemu.h|1 +
vl.c| 51
On Sun, Nov 14, 2010 at 3:39 PM, Gleb Natapov g...@redhat.com wrote:
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/fw_cfg.c | 14 ++
hw/fw_cfg.h | 4 +++-
sysemu.h | 1 +
vl.c | 51 +++
4 files changed,
On Sun, Nov 14, 2010 at 08:49:59PM +, Blue Swirl wrote:
On Sun, Nov 14, 2010 at 3:39 PM, Gleb Natapov g...@redhat.com wrote:
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/fw_cfg.c | 14 ++
hw/fw_cfg.h | 4 +++-
sysemu.h | 1 +
vl.c | 51
On Sun, Nov 14, 2010 at 08:52:37PM +0200, Gleb Natapov wrote:
On Sun, Nov 14, 2010 at 08:41:37PM +0200, Michael S. Tsirkin wrote:
On Sun, Nov 14, 2010 at 05:39:41PM +0200, Gleb Natapov wrote:
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/fw_cfg.c | 14 ++
On Sun, Nov 14, 2010 at 08:52:37PM +0200, Gleb Natapov wrote:
+
+len = strlen(bootpath);
+list = qemu_realloc(list, total + len + 1);
+list[total++] = len;
+memcpy(list[total], bootpath, len);
+total += len;
+c++;
+
On Sun, Nov 14, 2010 at 8:54 PM, Michael S. Tsirkin m...@redhat.com wrote:
On Sun, Nov 14, 2010 at 08:49:59PM +, Blue Swirl wrote:
On Sun, Nov 14, 2010 at 3:39 PM, Gleb Natapov g...@redhat.com wrote:
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/fw_cfg.c | 14 ++
On Sun, Nov 14, 2010 at 3:39 PM, Gleb Natapov g...@redhat.com wrote:
Extend -option-rom command to have additional parameter ,bootindex=.
This patch is broken:
CCarm-softmmu/palm.o
/src/qemu/hw/palm.c: In function 'palmte_init':
/src/qemu/hw/palm.c:237: error: incompatible type for
On Sun, Nov 14, 2010 at 09:13:03PM +, Blue Swirl wrote:
On Sun, Nov 14, 2010 at 8:54 PM, Michael S. Tsirkin m...@redhat.com wrote:
On Sun, Nov 14, 2010 at 08:49:59PM +, Blue Swirl wrote:
On Sun, Nov 14, 2010 at 3:39 PM, Gleb Natapov g...@redhat.com wrote:
Signed-off-by: Gleb
On Sun, Nov 14, 2010 at 3:39 PM, Gleb Natapov g...@redhat.com wrote:
Signed-off-by: Gleb Natapov g...@redhat.com
---
hw/fw_cfg.c | 14 ++
hw/fw_cfg.h | 4 +++-
sysemu.h | 1 +
vl.c | 51 +++
4 files changed,
On Sun, 2010-11-14 at 19:08 +0800, Avi Kivity wrote:
On 11/12/2010 03:16 AM, Huang Ying wrote:
On Thu, 2010-11-11 at 17:39 +0800, Avi Kivity wrote:
On 11/11/2010 02:56 AM, Huang Ying wrote:
On Thu, 2010-11-11 at 00:49 +0800, Anthony Liguori wrote:
On 11/10/2010 02:34 AM, Avi
(2010/11/14 14:58), Chris Wright wrote:
* Hidetoshi Seto (seto.hideto...@jp.fujitsu.com) wrote:
+/*
+ * Fallback: use utimes() instead of utimensat().
+ * See commit 74bc02b2d2272dc88fb98d43e631eb154717f517 for known
problem.
+ */
+struct timeval tv[2];
+int i;
+
This patch introduce a fallback mechanism for old systems that do not
support utimensat(). This fix build failure with following warnings:
hw/virtio-9p-local.c: In function 'local_utimensat':
hw/virtio-9p-local.c:479: warning: implicit declaration of function 'utimensat'
* Hidetoshi Seto (seto.hideto...@jp.fujitsu.com) wrote:
This patch introduce a fallback mechanism for old systems that do not
support utimensat(). This fix build failure with following warnings:
hw/virtio-9p-local.c: In function 'local_utimensat':
hw/virtio-9p-local.c:479: warning: implicit
On Sun, Nov 14, 2010 at 05:39:41PM +0200, Gleb Natapov wrote:
+/*
+ * This function returns device list as an array in a below format:
+ * +-+-+---+-+---+--
+ * | n | l1 | devpath1| l2 | devpath2 | ...
+ *
On Sat, 2010-11-13 at 23:05 +0200, Michael S. Tsirkin wrote:
On Fri, Nov 12, 2010 at 10:47:21AM -0700, Alex Williamson wrote:
This not only makes pci_find_capability a directly lookup, but also
allows us to better track added capabilities and avoids the proliferation
of random additional
On 11/14/2010 06:56 PM, Avi Kivity wrote:
On 11/12/2010 12:34 PM, Xiao Guangrong wrote:
We can past the page fault to guest directly if gpte's reserved
is set
How can that work? shadow_notrap_nonpresent_pte causes a fault with
PFEC.P=PFEC.RSVD=0, while we need PFEC.P=PFEC.RSVD=1.
Ah, i
the gpa caused ept violation is below:
most of them is F202.(4060217344)
error gpa is 4060217344
error gpa is 4060217344
error gpa is 4060217344
error gpa is 4060217344
error gpa is 4060217344
error gpa is 4060217344
error gpa is 4060217344
error gpa is 4060217344
error gpa is 4060217344
the address is the Region 1 of virtio_net.
why virtio_net use this address caused ept violation?
00:04.0 Ethernet controller: Unknown device 1af4:1000
Subsystem: Unknown device 1af4:0001
Control: I/O+ Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr-
Stepping- SERR- FastB2B-
On Friday 12 November 2010 19:25:16 Michael S. Tsirkin wrote:
On Fri, Nov 12, 2010 at 06:54:01PM +0800, Sheng Yang wrote:
On Friday 12 November 2010 18:47:29 Michael S. Tsirkin wrote:
On Fri, Nov 12, 2010 at 06:13:48PM +0800, Sheng Yang wrote:
On Friday 12 November 2010 17:53:13 Michael
On Mon, Nov 15, 2010 at 03:37:21PM +0800, Sheng Yang wrote:
We can back to them if there is someone really did it in that way. But
for all hypervisors using QEmu, I think we haven't seen such kind of
behavior yet.
I would rather stick to the spec than go figure out what do
On Monday 15 November 2010 15:42:50 Michael S. Tsirkin wrote:
On Mon, Nov 15, 2010 at 03:37:21PM +0800, Sheng Yang wrote:
We can back to them if there is someone really did it in that way.
But for all hypervisors using QEmu, I think we haven't seen such
kind of behavior yet.
I
On Mon, Nov 15, 2010 at 09:40:08AM +0200, Gleb Natapov wrote:
On Sun, Nov 14, 2010 at 10:40:33PM -0500, Kevin O'Connor wrote:
On Sun, Nov 14, 2010 at 05:39:41PM +0200, Gleb Natapov wrote:
+/*
+ * This function returns device list as an array in a below format:
+ *
59 matches
Mail list logo
