From: Stacey Son s...@freebsd.org
This change adds HOST_VARIANT_DIR so the various BSD OS dependent
code can be seperated into its own directories rather than
using #ifdef's. This may also allow an BSD variant OS to host
another BSD variant's executible as a target.
Signed-off-by: Stacey Son
On Thu, Jun 05, 2014 at 06:58:17PM +0200, Alexander Graf wrote:
On 05.06.14 18:52, Paolo Bonzini wrote:
Il 05/06/2014 18:45, Alexander Graf ha scritto:
Only if you were using -cpu somethingThatHasAVX, though, no?
Yes. The same argument goes the other way around. I want to use AVX
On Thu, Jun 05, 2014 at 07:38:49PM +0200, Paolo Bonzini wrote:
Il 05/06/2014 19:17, Eduardo Habkost ha scritto:
If you don't want MONITOR/MWAIT you shouldn't be using a CPU model
containing MONITOR/MWAIT in the first place. If you use -cpu
somethingWithMONITOR, that means you are already
On Thu, Jun 05, 2014 at 06:45:16PM +0200, Alexander Graf wrote:
On 05.06.14 18:44, Paolo Bonzini wrote:
Il 05/06/2014 18:40, Alexander Graf ha scritto:
kvm_set_cpuid(cpuid);
but enabling all experimental features inside KVM just because we want
one or two of them is very
From: Gabriel L. Somlo gso...@gmail.com
Add tests to find and verify the smbios entry point structure,
and to walk and perform checks on the actual smbios tables.
Suggested-by: Michael S. Tsirkin m...@redhat.com
Signed-off-by: Gabriel Somlo so...@cmu.edu
Reviewed-by: Michael S. Tsirkin
From: Stacey Son s...@freebsd.org
This change moves host OS and arch dependent code for the sysarch
system call related to the -strace functionality into the
appropriate host OS and target arch directories.
Signed-off-by: Stacey Son s...@freebsd.org
Signed-off-by: Sean Bruno sbr...@freebsd.org
On Thu, Jun 05, 2014 at 07:39:42PM +0200, Paolo Bonzini wrote:
Il 05/06/2014 19:19, Eduardo Habkost ha scritto:
On Thu, Jun 05, 2014 at 06:57:57PM +0200, Paolo Bonzini wrote:
Il 05/06/2014 18:54, Alexander Graf ha scritto:
What about:
- letting -cpu foo,+emulatedfeature just work
-
Sorry for following the discussion backwards, but I see now that you
started with a proposal that would cover both cases (the one you care
about, and the one I care about), make both of us happy, but it was lost
in favour of other suggestions I disagreed with:
On Thu, Jun 05, 2014 at 06:24:22PM
On 4 June 2014 08:44, Gerd Hoffmann kra...@redhat.com wrote:
Hi,
This pull adds the keyboard delay support for the input layer
and it updates the multiseat.txt doc file.
please pull,
Gerd
The following changes since commit e00fcfeab3d452cba3d0a08991a39ab15df66424:
Merge
On 04.06.2014 14:37, Stefan Hajnoczi wrote:
On Sat, May 31, 2014 at 08:43:10PM +0200, Max Reitz wrote:
Instead of using the main loop function qemu_set_fd_handler2(), use the
AIO function in the context of the exported BDS.
Managing fd handlers shouldn't be necessary at the NBD code level.
On Thu, 2014-06-05 at 16:53 +1000, Gavin Shan wrote:
The patch introduces helper function vfio_pci_container_ioctl() to
pass ioctl commands to the specified VFIO container that is identified
by IOMMU group id. On sPAPR platform, each container only has one
IOMMU group.
Signed-off-by: Gavin
On Thu, 2014-06-05 at 14:12 +0200, Alexander Graf wrote:
On 05.06.14 08:53, Gavin Shan wrote:
The series of patches adds support EEH for VFIO PCI devices on sPAPR
platform.
It requires corresponding host kernel support. Also, it is based on top of
Alexey's VFIO-for-sPAPR git repository.
On 4 June 2014 12:20, Paolo Bonzini pbonz...@redhat.com wrote:
The following changes since commit 1a381811b495651ddfc2b50d7c8cfaaf375816b0:
Merge remote-tracking branch 'remotes/rth/tags/pull-tcg-s390-20140515' into
staging (2014-05-15 18:56:08 +0100)
are available in the git repository
* Dr. David Alan Gilbert (dgilb...@redhat.com) wrote:
* Markus Armbruster (arm...@redhat.com) wrote:
* Dr. David Alan Gilbert (dgilb...@redhat.com) wrote:
snip
Load/save flags to let a user perform pre-save/post-load checking
Odd. I'd expect separate visitors, one for save, one for
On 5 June 2014 18:27, Sean Bruno sbr...@freebsd.org wrote:
This is the first patch-set being pulled from current development
of FreeBSD user mode code. This work is based on Stacey Son's developement
work from Caimbridge.
Refresh the syscalls for FreeBSD. This is a resend as I haven't seen
On 4 June 2014 13:43, Alexander Graf ag...@suse.de wrote:
Hi Peter,
This is my current patch queue for ppc. Please pull.
Alex
The following changes since commit e00fcfeab3d452cba3d0a08991a39ab15df66424:
Merge remote-tracking branch
Sorry for replying to my own message, but I believe we can now summarize
a possible solution that makes everybody happy, and the plans for it:
On Thu, Jun 05, 2014 at 03:02:53PM -0300, Eduardo Habkost wrote:
On Thu, Jun 05, 2014 at 07:39:42PM +0200, Paolo Bonzini wrote:
Il 05/06/2014 19:19,
On Thu, Jun 05, 2014 at 04:12:08PM -0300, Eduardo Habkost wrote:
In the meantime, we could:
* Include the less fine-tuned allow-emulation (or
allow-experimental-features) option, which is implemented by this
series, for people who use enforce and/or don't care too much about
On Thu, 2014-06-05 at 15:49 +1000, Alexey Kardashevskiy wrote:
To perform DMA mapping via TCE table correctly, the guest must
know where DMA window is located on the PCI bus. A hypervisor is
expected to provide such information. Since QEMU has no control
over this setting, we need a way to
On Thu, 2014-06-05 at 15:50 +1000, Alexey Kardashevskiy wrote:
This turns the sPAPR support on and enables VFIO container use
in the kernel.
This extends vfio_connect_container to support VFIO_SPAPR_TCE_IOMMU type
in the host kernel.
This registers a memory listener which sPAPR IOMMU will
The following changes since commit e00fcfeab3d452cba3d0a08991a39ab15df66424:
Merge remote-tracking branch
'remotes/awilliam/tags/vfio-pci-for-qemu-20140602.0' into staging (2014-06-03
14:37:43 +0100)
are available in the git repository at:
git://github.com/mdroth/qemu.git
From: Gonglei arei.gong...@huawei.com
token should be closed in all conditions.
So move CloseHandle(token) to out branch.
Signed-off-by: Wang Rui moon.wang...@huawei.com
Signed-off-by: Gonglei arei.gong...@huawei.com
Signed-off-by: Michael Roth mdr...@linux.vnet.ibm.com
---
qga/commands-win32.c
On 06/05/2014 01:24 PM, Borislav Petkov wrote:
On Thu, Jun 05, 2014 at 04:12:08PM -0300, Eduardo Habkost wrote:
In the meantime, we could:
* Include the less fine-tuned allow-emulation (or
allow-experimental-features) option, which is implemented by this
series, for people who use
On Thu, Jun 05, 2014 at 01:45:06PM -0600, Eric Blake wrote:
On 06/05/2014 01:24 PM, Borislav Petkov wrote:
On Thu, Jun 05, 2014 at 04:12:08PM -0300, Eduardo Habkost wrote:
In the meantime, we could:
* Include the less fine-tuned allow-emulation (or
allow-experimental-features)
The new option will allow slow emulated features (the ones returned by
GET_EMULATED_CPUID) to be enabled. We don't want to allow them to be
enabled by accident, so they will be enabled only if emulation is
explicitly allowed by the user.
Use x- prefix on the property name, to document that it is
On 4 June 2014 22:15, Richard Henderson r...@twiddle.net wrote:
TCG patches accumulated over the last 2 weeks.
Please pull.
r~
The following changes since commit e00fcfeab3d452cba3d0a08991a39ab15df66424:
Merge remote-tracking branch
On 24/05/14 13:44, Mark Cave-Ayland wrote:
This patchset primarily addresses Andreas' comments with regard to cg3
init/realize. The
first patch addresses this, followed by a second patch to address a previous
comment made
by Paolo.
The last two patches then alter tcx in a similar fashion to
On 28/05/14 08:35, Mark Cave-Ayland wrote:
This patchset implements IOMMU translation for the APB PCI host bridge as used
by sun4u.
With this patchset applied, my Debian squeeze/wheezy test images can now boot
and start executing the installer (including CDROM and network access) in
-nographic
Hi Peter,
This branch contains updates for SPARC, particularly QOM fixes for CG3/TCX as
requested by Andreas, an additional CG3 register boundary check requested by
Paolo and my APB IOMMU implementation for SPARC64. Please pull.
ATB,
Mark.
The following changes since commit
On 27/05/14 01:01, Alexander Graf wrote:
The timer registers on our KeyLargo macio emulation are read as byte reversed
from the big endian guest, so we better expose them endian reversed as well.
This fixes initial hickups of booting Mac OS X with -M mac99 for me.
Signed-off-by: Alexander
On 5 June 2014 15:21, Paolo Bonzini pbonz...@redhat.com wrote:
The following changes since commit e00fcfeab3d452cba3d0a08991a39ab15df66424:
Merge remote-tracking branch
'remotes/awilliam/tags/vfio-pci-for-qemu-20140602.0' into staging (2014-06-03
14:37:43 +0100)
are available in the git
This change fails to pass make check for me:
dcs-xen-54:~/qemu/outrm -rf *;../configure --enable-debug --disable-pie
--enable-trace-backend=stderr;make -j8 make check make check-block echo ok
...
GTESTER check-qtest-sh4eb
GTESTER check-qtest-sparc
Registering `memory' which already exists
On 5 June 2014 18:17, Michael S. Tsirkin m...@redhat.com wrote:
The following changes since commit c5fa6c86d0765f837515d1c10654c621724a77e0:
Merge remote-tracking branch 'remotes/qmp-unstable/queue/qmp' into staging
(2014-05-19 14:10:01 +0100)
are available in the git repository at:
On 06/05/2014 07:19 AM, Kevin Wolf wrote:
Signed-off-by: Kevin Wolf kw...@redhat.com
---
block/rbd.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/block/rbd.c b/block/rbd.c
index 09af484..898fcfe 100644
--- a/block/rbd.c
+++ b/block/rbd.c
@@ -684,13 +684,16 @@ static
On 5 June 2014 20:41, Michael Roth mdr...@linux.vnet.ibm.com wrote:
The following changes since commit e00fcfeab3d452cba3d0a08991a39ab15df66424:
Merge remote-tracking branch
'remotes/awilliam/tags/vfio-pci-for-qemu-20140602.0' into staging (2014-06-03
14:37:43 +0100)
are available in
On 05/07/2014 03:58 AM, Chunyan Liu wrote:
Export qemu_opt_find for qcow2 driver using it.
After replacing QEMUOptionParameter with QemuOpts, qcow2 driver will
use qemu_opt_find to judge if an option is explicitly set, to replace
the usage of .assigned in QEMUOptionParameter.
Signed-off-by:
On 30.05.14 23:24, Eduardo Habkost wrote:
Second try, now changing only the pseries machine code at hw/ppc/spapr.c.
Thanks, applied to ppc-next.
Alex
When parsing the -smp option, check if the values (that can be anywhere
in the [0, UINT64_MAX] range) are in the int range before setting the
int globals smp_cpus, max_cpus, smp_cores, smp_threads.
Without this, it was posbible to make smp_cpus and max_cpus negative.
Signed-off-by: Eduardo
On 05.06.14 19:48, Eduardo Habkost wrote:
On Thu, Jun 05, 2014 at 06:58:17PM +0200, Alexander Graf wrote:
On 05.06.14 18:52, Paolo Bonzini wrote:
Il 05/06/2014 18:45, Alexander Graf ha scritto:
Only if you were using -cpu somethingThatHasAVX, though, no?
Yes. The same argument goes the
On 5 June 2014 21:30, Mark Cave-Ayland mark.cave-ayl...@ilande.co.uk wrote:
Hi Peter,
This branch contains updates for SPARC, particularly QOM fixes for CG3/TCX as
requested by Andreas, an additional CG3 register boundary check requested by
Paolo and my APB IOMMU implementation for SPARC64.
On 05.06.14 21:57, Eduardo Habkost wrote:
The new option will allow slow emulated features (the ones returned by
GET_EMULATED_CPUID) to be enabled. We don't want to allow them to be
enabled by accident, so they will be enabled only if emulation is
explicitly allowed by the user.
Use x- prefix
On 05/07/2014 03:58 AM, Chunyan Liu wrote:
Signed-off-by: Dong Xu Wang wdon...@linux.vnet.ibm.com
Signed-off-by: Chunyan Liu cy...@suse.com
---
Changes:
* remove exporting qemu_opt_find from qcow2 patch and commit separately.
* remove unnecessary indention changes in bdrv_qcow2 as Eric
At the advice of Peter Maydell, break out these patches to inplement new
syscalls into substantially smaller, bite sized chunks for review. These
patches are based off a tree that Peter has kindly setup at:
git://git.linaro.org/people/peter.maydell/qemu-arm.git bsd-user.next
I've made clear
Signed-off-by: Sean Bruno sbr...@freebsd.org
---
bsd-user/qemu.h | 26 +++
bsd-user/strace.c | 126 +-
2 files changed, 94 insertions(+), 58 deletions(-)
diff --git a/bsd-user/qemu.h b/bsd-user/qemu.h
index ddc74ed..b8a34c7 100644
Signed-off-by: Sean Bruno sbr...@freebsd.org
---
bsd-user/freebsd/strace.list | 12
1 file changed, 12 insertions(+)
diff --git a/bsd-user/freebsd/strace.list b/bsd-user/freebsd/strace.list
index 32b3c55..b09b130 100644
--- a/bsd-user/freebsd/strace.list
+++
Signed-off-by: Sean Bruno sbr...@freebsd.org
---
bsd-user/freebsd/strace.list | 21 +++--
bsd-user/strace.c| 22 ++
2 files changed, 41 insertions(+), 2 deletions(-)
diff --git a/bsd-user/freebsd/strace.list b/bsd-user/freebsd/strace.list
index
Signed-off-by: Sean Bruno sbr...@freebsd.org
---
bsd-user/arm/syscall.h | 36
bsd-user/arm/syscall.h.orig| 0
bsd-user/arm/target_arch_sysarch.h | 78 ++
bsd-user/freebsd/os-strace.h | 29 +
Signed-off-by: Sean Bruno sbr...@freebsd.org
---
bsd-user/freebsd/strace.list | 2 +-
bsd-user/strace.c| 19 +++
2 files changed, 20 insertions(+), 1 deletion(-)
diff --git a/bsd-user/freebsd/strace.list b/bsd-user/freebsd/strace.list
index 3235f16..718357b 100644
Signed-off-by: Sean Bruno sbr...@freebsd.org
---
bsd-user/freebsd/strace.list | 13 +
1 file changed, 13 insertions(+)
diff --git a/bsd-user/freebsd/strace.list b/bsd-user/freebsd/strace.list
index b09b130..ab52ca8 100644
--- a/bsd-user/freebsd/strace.list
+++
Signed-off-by: Sean Bruno sbr...@freebsd.org
---
bsd-user/freebsd/strace.list | 18 ++
1 file changed, 14 insertions(+), 4 deletions(-)
diff --git a/bsd-user/freebsd/strace.list b/bsd-user/freebsd/strace.list
index b2a159e..61d11ca 100644
--- a/bsd-user/freebsd/strace.list
+++
From: Stacey Son s...@freebsd.org
This change adds HOST_VARIANT_DIR so the various BSD OS dependent
code can be seperated into its own directories rather than
using #ifdef's. This may also allow an BSD variant OS to host
another BSD variant's executible as a target.
Signed-off-by: Stacey Son
Signed-off-by: Sean Bruno sbr...@freebsd.org
---
bsd-user/freebsd/strace.list | 9 +
1 file changed, 9 insertions(+)
diff --git a/bsd-user/freebsd/strace.list b/bsd-user/freebsd/strace.list
index ab52ca8..b2a159e 100644
--- a/bsd-user/freebsd/strace.list
+++
On 06/05/2014 04:28 PM, Sean Bruno wrote:
In the subject, s/out/our/ ?
Please trim your subject lines to 60 characters or so; and also include
a topic. If you have more information (and that is often the case),
then include a blank line after the summary before providing it. Thus,
your commit
On 06/06/2014 02:51 AM, Alexander Graf wrote:
On 05.06.14 16:33, Alexey Kardashevskiy wrote:
On 06/05/2014 11:36 PM, Alexander Graf wrote:
On 05.06.14 15:33, Alexey Kardashevskiy wrote:
On 06/05/2014 11:15 PM, Alexander Graf wrote:
On 05.06.14 15:10, Alexey Kardashevskiy wrote:
On
Add monitor command to change mouse cursor position when input
device is in absolute mode.
Signed-off-by: Marcelo Tosatti mtosa...@redhat.com
---
hmp-commands.hx | 22 +++---
monitor.c | 35 ++-
2 files changed, 53 insertions(+), 4
On Thu, Jun 05, 2014 at 01:23:53PM +0200, Paolo Bonzini wrote:
Both Marcelo's rtc patches and Peter's MemoryRegion patches showed
the interest in having link properties with a custom representation
and/or a different way to store the property.
Such properties would still be links for the
On 06.06.14 01:17, Alexey Kardashevskiy wrote:
On 06/06/2014 02:51 AM, Alexander Graf wrote:
On 05.06.14 16:33, Alexey Kardashevskiy wrote:
On 06/05/2014 11:36 PM, Alexander Graf wrote:
On 05.06.14 15:33, Alexey Kardashevskiy wrote:
On 06/05/2014 11:15 PM, Alexander Graf wrote:
On 05.06.14
On 06/06/2014 05:31 AM, Alex Williamson wrote:
On Thu, 2014-06-05 at 15:50 +1000, Alexey Kardashevskiy wrote:
This turns the sPAPR support on and enables VFIO container use
in the kernel.
This extends vfio_connect_container to support VFIO_SPAPR_TCE_IOMMU type
in the host kernel.
This
On 06/06/2014 05:27 AM, Alex Williamson wrote:
On Thu, 2014-06-05 at 15:49 +1000, Alexey Kardashevskiy wrote:
To perform DMA mapping via TCE table correctly, the guest must
know where DMA window is located on the PCI bus. A hypervisor is
expected to provide such information. Since QEMU has no
On 06/06/2014 09:36 AM, Alexander Graf wrote:
On 06.06.14 01:17, Alexey Kardashevskiy wrote:
On 06/06/2014 02:51 AM, Alexander Graf wrote:
On 05.06.14 16:33, Alexey Kardashevskiy wrote:
On 06/05/2014 11:36 PM, Alexander Graf wrote:
On 05.06.14 15:33, Alexey Kardashevskiy wrote:
On
Public bug reported:
I'm trying to get a VXWorks image running inside a qemu guest. I have
the machine running, however, the vxworks image only has support for the
82544EI device so I had to change the device ID in e1000.c to get the
device even recognized so I'm not sure if this is a bug or an
On Thu, Jun 05, 2014 at 02:09:14PM +0200, Alexander Graf wrote:
On 05.06.14 08:53, Gavin Shan wrote:
The emulation for EEH RTAS requests from guest isn't covered
by QEMU yet and the patch implements them.
The patch defines constants used by EEH RTAS calls and adds
callback
On Thu, Jun 05, 2014 at 02:11:21PM +0200, Alexander Graf wrote:
On 05.06.14 08:53, Gavin Shan wrote:
The patch introduces helper function vfio_pci_container_ioctl() to
pass ioctl commands to the specified VFIO container that is identified
by IOMMU group id. On sPAPR platform, each container only
On Thu, Jun 05, 2014 at 12:27:23PM -0600, Alex Williamson wrote:
On Thu, 2014-06-05 at 16:53 +1000, Gavin Shan wrote:
The patch introduces helper function vfio_pci_container_ioctl() to
pass ioctl commands to the specified VFIO container that is identified
by IOMMU group id. On sPAPR platform,
This patch adds a new subtest, it hotplugs 29 * 8 = 232 virtio-blk
devices to guest, and try to hot-unplug them.
Note: the hot-unplug can't work without cooperation of guest OS.
Signed-off-by: Amos Kong ak...@redhat.com
Reviewed-by: Stefan Hajnoczi stefa...@redhat.com
---
I want to add a new subtest in virtio-blk-test.c, it will start
guest without network. The original pci_init() did nothing, but
it's good to reserve a very simple initialization testing.
Signed-off-by: Amos Kong ak...@redhat.com
---
tests/virtio-blk-test.c | 13 ++---
1 file changed, 6
It's worth to add a hotplug test to qtest, but without
cooperation of guest OS, new devices can't be initialized
by guest, and hot-unplug doesn't work.
However, the new test can cover some part of code of
hotplug/unplug.
I will write another subtest to test hotplug with pci support.
V2: move
Signed-off-by: Amos Kong ak...@redhat.com
Reviewed-by: Stefan Hajnoczi stefa...@redhat.com
---
tests/blockdev-test.c | 23 ++-
1 file changed, 2 insertions(+), 21 deletions(-)
diff --git a/tests/blockdev-test.c b/tests/blockdev-test.c
index c940e00..c9127c0 100644
---
This patch wraps a helper function to execute human command by
one QMP command (human-monitor-command). It also checks the return
string.
Signed-off-by: Amos Kong ak...@redhat.com
---
tests/libqtest.c | 26 ++
tests/libqtest.h | 9 +
2 files changed, 35
On Fri, Jun 06, 2014 at 12:24:26AM +0200, Alexander Graf wrote:
But can we drop the EMULATED name somehow? Can we rename [1] the ioctl
to say GET_UNSUPPORTED_CPUID or something along those lines? The name
is just a really really bad pick.
What do you mean, a bad pick :-P? I added extra care in
Hi, Amos
-Original Message-
From: Amos Kong [mailto:ak...@redhat.com]
Sent: Friday, June 06, 2014 9:12 AM
To: qemu-devel@nongnu.org
Cc: stefa...@gmail.com; afaer...@suse.de; Gonglei (Arei)
Subject: [PATCH v3 3/4] virtio-blk-test.c: add hotplug subtest
This patch adds a new
On Fri, Jun 06, 2014 at 09:40:56AM +1000, Alexey Kardashevskiy wrote:
On 06/06/2014 05:27 AM, Alex Williamson wrote:
On Thu, 2014-06-05 at 15:49 +1000, Alexey Kardashevskiy wrote:
To perform DMA mapping via TCE table correctly, the guest must
know where DMA window is located on the PCI bus. A
The current script isn't terribly friendly -- you basically get one chance
to run it per boot w/out manually recovering the system state.
It also doesn't lend itself to being made into an init script w/out a lot
of manual editing.
Rewrite the sucker with proper functions so that people can
For later reusing by dataplane code.
Signed-off-by: Fam Zheng f...@redhat.com
---
hw/block/virtio-blk.c | 11 ---
include/hw/virtio/virtio-blk.h | 11 +++
2 files changed, 11 insertions(+), 11 deletions(-)
diff --git a/hw/block/virtio-blk.c b/hw/block/virtio-blk.c
index
The block layer will handle the unaligned request.
Signed-off-by: Fam Zheng f...@redhat.com
---
hw/block/dataplane/virtio-blk.c | 25 -
1 file changed, 25 deletions(-)
diff --git a/hw/block/dataplane/virtio-blk.c b/hw/block/dataplane/virtio-blk.c
index c10b7b7..3d1e9e1
This unifies the request structure used by dataplane and non-dataplane code,
while dropping unnessary fields for bounce buffer and read flag.
Applies on top of Stefan's block tree.
v3: Address Paolo's comments:
[06/09] virtio-blk: Use VirtIOBlockReq.in to drop VirtIOBlockReq.inhdr
Add
This will make converging with dataplane code easier.
Add virtio_blk_free_request to handle the freeing of request internal
fields.
Signed-off-by: Fam Zheng f...@redhat.com
---
hw/block/virtio-blk.c | 85 +++---
include/hw/virtio/virtio-blk.h | 2 +-
Since it's set but not used.
Signed-off-by: Fam Zheng f...@redhat.com
---
hw/block/dataplane/virtio-blk.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/hw/block/dataplane/virtio-blk.c b/hw/block/dataplane/virtio-blk.c
index 3d1e9e1..4e5e458 100644
--- a/hw/block/dataplane/virtio-blk.c
+++
In current virtio spec, inhdr is a single byte, and is unlikely to
change for both functionality and compatibility considerations.
Non-dataplane uses .in, and we are on the way to converge them. So
let's unify it to get cleaner code.
Remove .inhdr and use .in.
Signed-off-by: Fam Zheng
Field inhdr is added temporarily for a more mechanical change, and
will be dropped in the next commit.
Signed-off-by: Fam Zheng f...@redhat.com
---
hw/block/dataplane/virtio-blk.c | 34 +++---
include/hw/virtio/virtio-blk.h | 4
2 files changed, 19
The virtio code currently assumes that the outhdr is in its own iovec.
This is not guaranteed by the spec, so we should relax this assumption.
Convert the VirtIOBlockReq.out field to structrue so that we can use
iov_to_buf and then discard the header from the beginning of iovec.
Suggested-by:
VirtIOBlockReq is allocated in process_request, and freed in command
functions.
Signed-off-by: Fam Zheng f...@redhat.com
---
hw/block/dataplane/virtio-blk.c | 102 +---
1 file changed, 44 insertions(+), 58 deletions(-)
diff --git
out_sg is checked by iov_to_buf below, so it can be dropped.
Add assert and iov_discard_back around in_sg, as the in_sg is handled in
dataplane code.
Signed-off-by: Fam Zheng f...@redhat.com
---
hw/block/virtio-blk.c | 15 ---
1 file changed, 8 insertions(+), 7 deletions(-)
diff
On Thu, 06/05 17:11, Peter Maydell wrote:
On 5 June 2014 14:38, Rainer Müller rai...@codingfarm.de wrote:
On 2014-03-14 13:29, Paolo Bonzini wrote:
Il 13/03/2014 19:48, Peter Maydell ha scritto:
Yep, here we are:
LIBSSH2_LIBS=-L/opt/local/lib -lssh2 -Wl,-headerpad_max_install_names
On 06/05/2014 07:47 PM, Mike Frysinger wrote:
The current script isn't terribly friendly -- you basically get one chance
to run it per boot w/out manually recovering the system state.
It also doesn't lend itself to being made into an init script w/out a lot
of manual editing.
Rewrite the
On Fri, Jun 06, 2014 at 03:21:04AM +0200, Borislav Petkov wrote:
On Fri, Jun 06, 2014 at 12:24:26AM +0200, Alexander Graf wrote:
But can we drop the EMULATED name somehow? Can we rename [1] the ioctl
to say GET_UNSUPPORTED_CPUID or something along those lines? The name
is just a really
Matthew Gamble mgam...@mgamble.ca wrote:
I'm trying to get a VXWorks image running inside a qemu guest. I have
the machine running, however, the vxworks image only has support for the
82544EI device so I had to change the device ID in e1000.c to get the
device even recognized so I'm not sure
Gabriel,
I tried your suggestion and while the OS doesn't detect a link, it does
send the following right after toggling the link:
e1000: set_ics 4, ICR 4, IMR 0
e1000: set_ics 4, ICR 4, IMR 0
Without diving into the intel programming guide myself does that help in
any way? Perhaps it's
Paolo Bonzini wrote on 2014-06-03:
Il 30/05/2014 10:59, Tiejun Chen ha scritto:
+static int create_pch_isa_bridge(PCIBus *bus, XenHostPCIDevice *hdev)
+{
+struct PCIDevice *dev;
+
+char rid;
+
+dev = pci_create(bus, PCI_DEVFN(0x1f, 0), intel-pch-isa-bridge);
This is really a
On Thu, Jun 05, 2014 at 10:55:39PM -0400, Matthew Gamble wrote:
Gabriel,
I tried your suggestion and while the OS doesn't detect a link, it does
send the following right after toggling the link:
e1000: set_ics 4, ICR 4, IMR 0
e1000: set_ics 4, ICR 4, IMR 0
Without diving into the
On Thu, 06/05 14:15, Markus Armbruster wrote:
Signed-off-by: Markus Armbruster arm...@redhat.com
---
hw/block/virtio-blk.c | 24 ++--
1 file changed, 14 insertions(+), 10 deletions(-)
diff --git a/hw/block/virtio-blk.c b/hw/block/virtio-blk.c
index 8a568e5..f2b4dca
On Thu, 06/05 14:15, Markus Armbruster wrote:
When a device model's I/O operation fails, we execute the error
action. This lets layers above QEMU implement thin provisioning, or
attempt to correct errors before they reach the guest. But when the
I/O operation fails because its invalid,
On Thu, 06/05 14:15, Markus Armbruster wrote:
The block layer fails such reads and writes just fine. However, they
then get treated like valid operations that fail: the error action
gets executed. Unwanted; reporting the error to the guest is the only
sensible action.
Reject them before
On 06/05/2014 09:18 PM, Fam Zheng wrote:
On Thu, 06/05 14:15, Markus Armbruster wrote:
When a device model's I/O operation fails, we execute the error
action. This lets layers above QEMU implement thin provisioning, or
attempt to correct errors before they reach the guest. But when the
I/O
Hi, all
I ran two linux guest on the same kvm host, then start the netserver on one vm,
start netperf on the other one, netperf command and test result shown as below,
netperf -H 196.5.5.71 -t TCP_STREAM -l 60 -- -m 1400 -M 1400
MIGRATED TCP STREAM TEST from 0.0.0.0 (0.0.0.0) port 0 AF_INET to
Yet another try with VFIO on SPAPR (server PPC64).
This adds VFIO support on SPAPR for the existing VFIO-SPAPR-TCE driver
in the upstream kernel.
Individual patches have more detailed commit logs.
Please comment. Thanks!
Changes:
v8:
* added placeholder for KVM_CAP_SPAPR_TCE_VFIO capability
*
While most operations with VFIO IOMMU driver are generic and used inside
vfio.c, there are still some operations which only specific VFIO IOMMU
drivers implement. The first example of it will be reading a DMA window
start from the host.
This adds a helper which passes an ioctl request to the
POWER KVM supports an KVM_CAP_SPAPR_TCE capability which allows allocating
TCE tables in the host kernel memory and handle H_PUT_TCE requests
targeted to specific LIOBN (logical bus number) right in the host without
switching to QEMU. At the moment this is used for emulated devices only
and the
The patch adds a spapr-pci-vfio-host-bridge device type
which is a PCI Host Bridge with VFIO support. The new device
inherits from the spapr-pci-host-bridge device and adds an iommu
property which is an IOMMU id. This ID represents a minimal entity
for which IOMMU isolation can be guaranteed. In
Hi!
checkpatch.pl often complains on things like this:
===
ERROR: need consistent spacing around '*' (ctx:WxV)
#57: FILE: hw/misc/vfio.c:4323:
+int vfio_container_ioctl(AddressSpace *as, int32_t groupid,
^
total: 1 errors, 0 warnings, 46 lines checked
===
301 - 400 of 414 matches
Mail list logo