Re: [Qemu-devel] [PATCH] net: detect errors from probing vnet hdr flag for TAP devices

2017-10-30 Thread Jason Wang
On 2017年10月27日 16:55, Daniel P. Berrange wrote: When QEMU sets up a tap based network device backend, it mostly ignores errors reported from various ioctl() calls it makes, assuming the TAP file descriptor is valid. This assumption can easily be violated when the user is passing in a pre-opened

Re: [Qemu-devel] Running Qemu in discrete time/step by step

2017-10-30 Thread Matt
Within the context of a simulator, qemu runs with a limited set of devices so that removes some indeterminism but not all I guess. VMsimint boasts exact reproducibility because they control all outputs/inputs. I tried to look into their patch but the patch is empty and instead there seems to be th

Re: [Qemu-devel] [PATCH v2 1/4] build: allow setting a custom GIT binary for transparent proxying

2017-10-30 Thread Daniel P. Berrange
On Mon, Oct 30, 2017 at 10:49:01AM +1100, Alexey Kardashevskiy wrote: > On 30/10/17 03:29, Daniel P. Berrange wrote: > > On Mon, Oct 30, 2017 at 01:08:56AM +1100, Alexey Kardashevskiy wrote: > >> > >> I run ./scripts/git-submodule.sh on a server (where source directory is > >> writeable), them "rm

Re: [Qemu-devel] [PATCH] net: detect errors from probing vnet hdr flag for TAP devices

2017-10-30 Thread Daniel P. Berrange
On Mon, Oct 30, 2017 at 03:37:33PM +0800, Jason Wang wrote: > > > On 2017年10月27日 16:55, Daniel P. Berrange wrote: > > When QEMU sets up a tap based network device backend, it mostly ignores > > errors > > reported from various ioctl() calls it makes, assuming the TAP file > > descriptor > > is

Re: [Qemu-devel] [PATCH 3/5] target/s390x: tcg stubs

2017-10-30 Thread Thomas Huth
On 30.10.2017 04:05, Philippe Mathieu-Daudé wrote: > missing since 55c3ceef61: > > LINKs390x-softmmu/qemu-system-s390x > target/s390x/cpu.o: In function `s390_cpu_class_init': > target/s390x/cpu.c:500: undefined reference to `s390x_translate_init' > > Suggested-by: Eduardo Habkost >

Re: [Qemu-devel] [PATCH 0/5] build fixes for --disable-tcg

2017-10-30 Thread Thomas Huth
On 30.10.2017 04:05, Philippe Mathieu-Daudé wrote: > Hi, > > This series add missing stubs (as suggested by Eduardo in [1]) to build > the i386/s390x targets with --disable-tcg (the only targets building without > TCG so far). Compiling with --disable-tcg works fine for me with the latest master

Re: [Qemu-devel] [PATCH v4] s390-ccw: print carriage return with new lines

2017-10-30 Thread Christian Borntraeger
On 10/27/2017 06:14 PM, Collin L. Walling wrote: > The sclp console in the s390 bios writes raw data, > leading console emulators (such as virsh console) to > treat a new line ('\n') as just a new line instead > of as a Unix line feed. Because of this, output > appears in a "stair case" pattern. >

Re: [Qemu-devel] [RFC PATCH] vl: only display available accelerators

2017-10-30 Thread Eduardo Habkost
On Mon, Oct 30, 2017 at 01:00:56AM -0300, Philippe Mathieu-Daudé wrote: > examples configuring with '--enable-kvm --disable-tcg' > > - before > > $ qemu-system-x86_64 -accel help > Possible accelerators: kvm, xen, hax, tcg > > $ qemu-system-x86_64 -accel tcg > qemu-system-x86_64: -machin

Re: [Qemu-devel] [PATCH v1 0/5][RFC] Refactoring of AIS support

2017-10-30 Thread Christian Borntraeger
Now I thought about that for a while and I start to think that we cannot implement ais in QEMU and cover all cases. One aspect was certainly passthrough (like you handled in patch 4). Another aspect is that some interrupts might be injected from the kernel - even for emulated devices. e.g. virtio

Re: [Qemu-devel] [PATCH] qemu-iotests: Use -nographic in 182

2017-10-30 Thread Kashyap Chamarthy
On Sun, Oct 29, 2017 at 01:53:38PM +0100, Kevin Wolf wrote: > This avoids that random UI frontend error messages end up in the output. > In particular, we were seeing this line in CI error logs: > > +Unable to init server: Could not connect: Connection refused > > Signed-off-by: Kevin Wolf > ---

Re: [Qemu-devel] [RFC 00/19] KVM: s390/crypto/vfio: guest dedicated crypto adapters

2017-10-30 Thread Christian Borntraeger
adding qemu devel and add Daniel and Erik from libvirt to keep them in the loop. On 10/29/2017 12:11 PM, Cornelia Huck wrote: > On Fri, 13 Oct 2017 13:38:45 -0400 > Tony Krowiak wrote: > >> Tony Krowiak (19): >> KVM: s390: SIE considerations for AP Queue virtualization >> KVM: s390: refact

[Qemu-devel] [PULL 1/4] target/s390x: change CPU type name to "s390x-cpu"

2017-10-30 Thread Christian Borntraeger
From: David Hildenbrand For now, e.g. host-s390-cpu wasn't exposed to the user. cpu-add, -cpu and the CPU model qmp interfaces didn't care about the actual type, as that information was hidden. This changed with CPU hotplug via device_add. Now the type is visible to the user. Before we get that

[Qemu-devel] [PULL 4/4] s390-*.img: update s390 bios with latest fixes

2017-10-30 Thread Christian Borntraeger
includes 7618c0aefed1 ("s390-ccw: print carriage return with new lines") a8fbbf1db7a3 ("s390: set DHCP client architecure id for netboot") Signed-off-by: Christian Borntraeger --- pc-bios/s390-ccw.img | Bin 26416 -> 30520 bytes pc-bios/s390-netboot.img | Bin 83864 -> 83864 bytes 2 files ch

[Qemu-devel] [PULL 0/4] s390: fixups for 2.11

2017-10-30 Thread Christian Borntraeger
tags/s390x-20171030 for you to fetch changes up to 7a4ec18d117d74eedccdadca882331b928135465: s390-*.img: update s390 bios with latest fixes (2017-10-30 09:04:25 +0100) s390x: fixups for 2.11 - missing \r in the BIOS console output

[Qemu-devel] [PULL 2/4] s390x/kvm: use cpu model for gscb on compat machines

2017-10-30 Thread Christian Borntraeger
Starting a guest with hvm on an IBM z14 results in "qemu-system-s390x: Some features requested in the CPU model are not available in the configuration: gs" This is because guarded storage is fenced for compat machines that did not have guarded storage support. While this prevents f

[Qemu-devel] [PULL 3/4] s390-ccw: print carriage return with new lines

2017-10-30 Thread Christian Borntraeger
From: "Collin L. Walling" The sclp console in the s390 bios writes raw data, leading console emulators (such as virsh console) to treat a new line ('\n') as just a new line instead of as a Unix line feed. Because of this, output appears in a "stair case" pattern. Let's print \r\n on every occurr

Re: [Qemu-devel] [PATCH 1/6] CAN bus simple SJA1000 PCI card emulation for QEMU

2017-10-30 Thread KONRAD Frederic
On 10/29/2017 11:43 PM, Pavel Pisa wrote: Hello Fred, thanks much for review and remarks. On Friday 27 of October 2017 16:18:31 KONRAD Frederic wrote: Hi Pavel, On 10/25/2017 01:12 AM, p...@cmp.felk.cvut.cz wrote: From: Pavel Pisa The work is based on Jin Yang GSoC 2013 work funded by Go

Re: [Qemu-devel] [PATCH v2 3/4] fw_cfg: write vmcoreinfo details

2017-10-30 Thread Hatayama, Daisuke
Marc-Andre, Sorry, I missed your original mails from my local data, so I'm replying this mail as a new thread... > From: Marc-Andre Lureau > > If the "etc/vmcoreinfo" file is present, write the addr/size of the > vmcoreinfo ELF note. > > Signed-off-by: Marc-Andre Lureau > --- > drivers/firmw

Re: [Qemu-devel] [PATCH v3 00/32] Migration: postcopy failure recovery

2017-10-30 Thread Juan Quintela
Peter Xu wrote: > On Mon, Oct 16, 2017 at 12:32:10AM -0700, no-re...@patchew.org wrote: >> Hi, >> >> This series failed build test on s390x host. Please find the details below. >> >> Type: series >> Subject: [Qemu-devel] [PATCH v3 00/32] Migration: postcopy failure recovery >> Message-id: 201710

Re: [Qemu-devel] [PATCH v2 3/4] fw_cfg: write vmcoreinfo details

2017-10-30 Thread Hatayama, Daisuke
Resend because the mail address for LKML was wrong in the previous mail... Marc-Andre, Sorry, I missed your original mails from my local data, so I'm replying this mail as a new thread... > From: Marc-Andre Lureau > > If the "etc/vmcoreinfo" file is present, write the addr/size of the > vmcore

Re: [Qemu-devel] [PULL 0/4] qemu-ga patch queue

2017-10-30 Thread Peter Maydell
On 27 October 2017 at 02:18, Michael Roth wrote: > The following changes since commit ae49fbbcd8e4e9d8bf7131add34773f579e1aff7: > > Merge remote-tracking branch 'remotes/rth/tags/pull-tcg-20171025' into > staging (2017-10-25 16:38:57 +0100) > > are available in the git repository at: > > git:

Re: [Qemu-devel] [PATCH v3] xlnx-zcu102: Specify the number of max CPUs

2017-10-30 Thread Peter Maydell
On 30 October 2017 at 03:25, Philippe Mathieu-Daudé wrote: > From: Alistair Francis > > Specify the number of CPUs that can run on ZynqMP. > > Signed-off-by: Alistair Francis > Reviewed-by: Philippe Mathieu-Daudé > Tested-by: Philippe Mathieu-Daudé > --- > V3: > - Fix my ASCII noncompliant la

Re: [Qemu-devel] [Qemu devel V4 PATCH] msf2: Remove dead code reported by Coverity

2017-10-30 Thread Peter Maydell
On 25 October 2017 at 03:29, Subbaraya Sundeep wrote: > Fixed incorrect frame size mask, validated maximum frame > size in spi_write and removed dead code. > > Signed-off-by: Subbaraya Sundeep > --- > v4: > changed %d to %u while logging frame size error. > v3: > Added comment tha

Re: [Qemu-devel] nvme: Add tracing v3

2017-10-30 Thread Doug Gale
On Fri, Oct 20, 2017 at 4:11 PM, Doug Gale wrote: > On Thu, Oct 12, 2017 at 3:07 PM, Doug Gale wrote: > >> From c7f12a5949458fdd195b5e0b52f158e8f114f203 Mon Sep 17 00:00:00 2001 >> From: Doug Gale >> Date: Thu, 12 Oct 2017 14:29:07 -0400 >> Subject: [PATCH] nvme: Add tracing >> >> Add trace out

Re: [Qemu-devel] [Qemu devel PATCH] msf2: Wire up SYSRESETREQ in SoC for system reset

2017-10-30 Thread Peter Maydell
On 29 October 2017 at 04:59, Subbaraya Sundeep wrote: > Implemented system reset by creating SYSRESETREQ gpio > out from nvic. > > Signed-off-by: Subbaraya Sundeep > --- > hw/arm/msf2-soc.c | 11 +++ > 1 file changed, 11 insertions(+) > Applied to target-arm.next, thanks. -- PMM

Re: [Qemu-devel] [PATCH] hw/pci-host/gpex: Improve INTX to gsi routing error checking

2017-10-30 Thread Peter Maydell
On 23 October 2017 at 17:30, Eric Auger wrote: > We exposed gpex_set_irq_num() for machines to set the INTx to > GSI routing. However if the machine forgets to call that > function we currently do not check the association was properly > done. Let's initialize gsi values to -1 and if this value is

Re: [Qemu-devel] [PATCH] qemu-iotests: Use -nographic in 182

2017-10-30 Thread Kevin Wolf
Am 30.10.2017 um 09:39 hat Kashyap Chamarthy geschrieben: > On Sun, Oct 29, 2017 at 01:53:38PM +0100, Kevin Wolf wrote: > > This avoids that random UI frontend error messages end up in the output. > > In particular, we were seeing this line in CI error logs: > > > > +Unable to init server: Could n

[Qemu-devel] [PATCH] vga: fix region checks in wraparound case

2017-10-30 Thread Gerd Hoffmann
Cc: "Dr. David Alan Gilbert" Signed-off-by: Gerd Hoffmann --- hw/display/vga.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/display/vga.c b/hw/display/vga.c index 1d19f6b..a64a094 100644 --- a/hw/display/vga.c +++ b/hw/display/vga.c @@ -1666,9 +1666,9 @@ static void

[Qemu-devel] QMP event missed during startup

2017-10-30 Thread Ross Lagerwall
Hi, I have found an issue where QEMU emits the RESUME event during startup when it starts VM execution, but it is not possible to receive this event. To repro this, run: qemu-system-i386 -m 256 -trace enable=monitor_protocol_event_emit,file=/tmp/out -qmp unix:/tmp/qmp,server,wait QEMU will

Re: [Qemu-devel] [PATCH v3 5/5] RFC: fw_cfg: add DMA write operation in sysfs

2017-10-30 Thread Hatayama, Daisuke
> From: Marc-André Lureau > > Since qemu 2.9, DMA write operations are allowed. However, usage of this > interface from kernel or user-space is strongly discouraged by the > maintainers. This patch is meant for experimentations for now. > Could you (or maintainers?) tell me how experimental the D

Re: [Qemu-devel] [PULL v2 00/39] x86/cpu/numa queue, 2017-10-27

2017-10-30 Thread Peter Maydell
On 27 October 2017 at 15:20, Eduardo Habkost wrote: > Change in v2: > * Include sh4-linux-user fixup sent by Igor > (at: "sh4: simplify superh_cpu_class_by_name()") > > The following changes since commit 74d7fc7f6385158b8a5c524c61baaef1b66f3dac: > > tcg: Avoid setting tcg_initialize if !CONFIG

[Qemu-devel] [PATCH 1/6] qio: Make port 0 work for qio

2017-10-30 Thread Juan Quintela
For tcp sockets we read back what is the socket/address. So we know what is the port that we are listening into. Looked all callers of the function, and they just create the addr, use it, and drop it, so no problem that we always update the port in the address. Signed-off-by: Juan Quintela CC:

[Qemu-devel] [PATCH 0/6] Improve info migrate output on destination

2017-10-30 Thread Juan Quintela
Hi This series (on top of my last pull requset) do: - Make update port for address work for port 0. Right now we only update the _port_ that we ask for, not the one that we receive. - Print global options as everything else (i.e. one by line, and on/off) - Move addr cleanup to the place where

[Qemu-devel] [PATCH 2/6] migration: print features as on off

2017-10-30 Thread Juan Quintela
Once there, do one thing for line Signed-off-by: Juan Quintela --- migration/migration.c | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/migration/migration.c b/migration/migration.c index 4de3b551fe..706d2407ec 100644 --- a/migration/migration.c +++ b/migration

[Qemu-devel] [PATCH 3/6] migration: free addr in the same function that we created it

2017-10-30 Thread Juan Quintela
Otherwise, we can't use it after calling socket_start_incoming_migration Signed-off-by: Juan Quintela --- migration/socket.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/migration/socket.c b/migration/socket.c index dee869044a..3a8232dd2d 100644 --- a/migration/socket.

[Qemu-devel] [PATCH 6/6] migration: make migrate uri parameter optional

2017-10-30 Thread Juan Quintela
Now that we have set an uri migration parameter, we can make it optional in the command line. Signed-off-by: Juan Quintela --- hmp-commands.hx | 4 ++-- hmp.c | 5 +++-- migration/migration.c | 12 +--- qapi/migration.json | 2 +- 4 files changed, 15 insertions

[Qemu-devel] [PATCH 4/6] migration: Create uri parameter

2017-10-30 Thread Juan Quintela
It will be used to store the uri migration parameter. Right now it is just the parameter code. Signed-off-by: Juan Quintela --- hmp.c | 7 +++ migration/migration.c | 13 + qapi/migration.json | 18 +++--- 3 files changed, 35 insertions(+), 3 delet

[Qemu-devel] [PATCH 5/6] migration: Now set the migration uri

2017-10-30 Thread Juan Quintela
Signed-off-by: Juan Quintela --- migration/migration.c | 25 ++--- migration/migration.h | 2 ++ migration/socket.c| 7 +++ 3 files changed, 27 insertions(+), 7 deletions(-) diff --git a/migration/migration.c b/migration/migration.c index 3e48d37880..507226907b 1006

Re: [Qemu-devel] [PATCH 1/6] CAN bus simple SJA1000 PCI card emulation for QEMU

2017-10-30 Thread Marek Vasut
On 10/30/2017 10:19 AM, KONRAD Frederic wrote: [...] >> CAN bus MIOe-3680 PCI (dual SJA1000 channel) emulation added. >>    hw/can/can_mioe3680_pci.c >> >> If you think that this is more logical even that the first >> commits introduce something which cannot be tested/compiles >> only into librar

Re: [Qemu-devel] [PATCH 1/6] CAN bus simple SJA1000 PCI card emulation for QEMU

2017-10-30 Thread KONRAD Frederic
On 10/30/2017 11:51 AM, Marek Vasut wrote: On 10/30/2017 10:19 AM, KONRAD Frederic wrote: [...] CAN bus MIOe-3680 PCI (dual SJA1000 channel) emulation added. hw/can/can_mioe3680_pci.c If you think that this is more logical even that the first commits introduce something which cannot be

Re: [Qemu-devel] [PATCH 1/6] CAN bus simple SJA1000 PCI card emulation for QEMU

2017-10-30 Thread Pavel Pisa
Hello Marek and Konrad, On Monday 30 of October 2017 12:38:12 KONRAD Frederic wrote: > On 10/30/2017 11:51 AM, Marek Vasut wrote: > > On 10/30/2017 10:19 AM, KONRAD Frederic wrote: > > > > [...] > > > >>> CAN bus MIOe-3680 PCI (dual SJA1000 channel) emulation added. > >>> hw/can/can_mioe3680_p

Re: [Qemu-devel] nvme: Add tracing v3

2017-10-30 Thread Philippe Mathieu-Daudé
Hi Doug, On 10/12/2017 04:07 PM, Doug Gale wrote: > From c7f12a5949458fdd195b5e0b52f158e8f114f203 Mon Sep 17 00:00:00 2001 > From: Doug Gale > Date: Thu, 12 Oct 2017 14:29:07 -0400 > Subject: [PATCH] nvme: Add tracing > > Add trace output for commands, errors, and undefined behavior. > Add guest

Re: [Qemu-devel] [RFC PATCH v6 13/12] tweak test 83 verbosity

2017-10-30 Thread Vladimir Sementsov-Ogievskiy
27.10.2017 13:45, Eric Blake wrote: Commenting these two lines is enough to avoid the change to 083.out in 12/12. That is evidence that we may want these two lines to be trace points rather than error messages; or maybe we really do like the extra verbosity in the case of an unexpected communica

Re: [Qemu-devel] [PATCH v1 0/5][RFC] Refactoring of AIS support

2017-10-30 Thread Cornelia Huck
On Mon, 30 Oct 2017 09:28:09 +0100 Christian Borntraeger wrote: > Now I thought about that for a while and I start to think that we cannot > implement ais > in QEMU and cover all cases. > One aspect was certainly passthrough (like you handled in patch 4). > Another aspect is that some interrupts

Re: [Qemu-devel] [PATCH v1 0/5][RFC] Refactoring of AIS support

2017-10-30 Thread Christian Borntraeger
On 10/30/2017 01:42 PM, Cornelia Huck wrote: > On Mon, 30 Oct 2017 09:28:09 +0100 > Christian Borntraeger wrote: > >> Now I thought about that for a while and I start to think that we cannot >> implement ais >> in QEMU and cover all cases. >> One aspect was certainly passthrough (like you hand

Re: [Qemu-devel] [PATCH 0/5] build fixes for --disable-tcg

2017-10-30 Thread Philippe Mathieu-Daudé
Hi Thomas, On 10/30/2017 05:05 AM, Thomas Huth wrote: > Compiling with --disable-tcg works fine for me with the latest master > branch as of today, on both x86 and s390x. Are you sure that these > patches are still required? I guess I didn't pulled /master early enough so I missed Peter's merge.

Re: [Qemu-devel] [RFC PATCH v6 13/12] tweak test 83 verbosity

2017-10-30 Thread Eric Blake
On 10/30/2017 01:34 PM, Vladimir Sementsov-Ogievskiy wrote: > 27.10.2017 13:45, Eric Blake wrote: >> Commenting these two lines is enough to avoid the change to 083.out >> in 12/12.  That is evidence that we may want these two lines to be >> trace points rather than error messages; or maybe we real

Re: [Qemu-devel] [PULL 0/9] Migration pull request

2017-10-30 Thread Peter Maydell
On 29 October 2017 at 13:09, Juan Quintela wrote: > Hi > > This includes the patches reviewed form: > - migrate_set_cache_size series (quintela) > - migrate tests (quintela) > > Please apply. > > Later, Juan. > > The following changes since commit a93ece47fd9edbd4558db24300056c9a57d3bcd4: > > Me

Re: [Qemu-devel] [PATCH 0/6] Improve info migrate output on destination

2017-10-30 Thread Fam Zheng
On Mon, 10/30 12:21, Juan Quintela wrote: > Hi > > This series (on top of my last pull requset) do: For patchew, Based-on: 20171029130934.4555-1-quint...@redhat.com

[Qemu-devel] [PATCH v2 2/3] block/parallals: Do not update header or truncate image when INMIGRATE

2017-10-30 Thread Jeff Cody
If we write or modify the image file while the QEMU run state is INMIGRATE, then the BDRV_O_INACTIVE BDS flag is set. This will cause an assert, since the image is marked inactive. Make sure we obey this flag. Signed-off-by: Jeff Cody --- block/parallels.c | 7 ++- 1 file changed, 2 insert

[Qemu-devel] [PATCH v2 0/3] Don't write headers if BDS is INACTIVE

2017-10-30 Thread Jeff Cody
Changes from v1->v2: * Drop previous parallels patches, just check BDRV_O_INACTIVE now (Kevin) git-backport-diff -r qemu/master.. -u github/master Key: [] : patches are identical [] : number of functional differences between upstream/downstream patch [down] : patch is downstream-only T

Re: [Qemu-devel] [PATCHv4 09/13] lance: move TYPE_LANCE and SysBusPCNetState from lance.c to lance.h

2017-10-30 Thread Mark Cave-Ayland
On 26/10/17 11:12, Mark Cave-Ayland wrote: > On 25/10/17 18:47, Peter Maydell wrote: > >> On 25 October 2017 at 16:59, Mark Cave-Ayland >> wrote: >>> This enables them to be used outside of lance.c. >>> >>> Signed-off-by: Mark Cave-Ayland >>> CC: Jason Wang >> >> >>> diff --git a/include/hw/ne

[Qemu-devel] [PATCH v2 1/3] block/vhdx.c: Don't blindly update the header

2017-10-30 Thread Jeff Cody
The VHDX specification requires that before user data modification of the vhdx image, the VHDX header file and data GUIDs need to be updated. In vhdx_open(), if the image is set to RDWR, we go ahead and update the header. However, just because the image is set to RDWR does not mean we can go ahead

[Qemu-devel] [PATCH v2 3/3] qemu-iotests: update unsupported image formats in 194

2017-10-30 Thread Jeff Cody
Test 194 checks for 'luks' to exclude as an unsupported format, However, most formats are unsupported, due to migration blockers. Rather than specifying a blacklist of unsupported formats, whitelist supported formats (specifically, qcow2, qed, raw, dmg). Signed-off-by: Jeff Cody --- tests/qemu-

[Qemu-devel] [PATCH v12 5/6] migration: add blocktime calculation into migration-test

2017-10-30 Thread Alexey Perevalov
This patch just requests blocktime calculation, and check it in case when UFFD_FEATURE_THREAD_ID feature is set on the host. Reviewed-by: Dr. David Alan Gilbert Signed-off-by: Alexey Perevalov --- tests/migration-test.c | 16 1 file changed, 16 insertions(+) diff --git a/tests

[Qemu-devel] [PATCH v12 4/6] migration: postcopy_blocktime documentation

2017-10-30 Thread Alexey Perevalov
Reviewed-by: Dr. David Alan Gilbert Signed-off-by: Alexey Perevalov --- docs/devel/migration.txt | 13 + 1 file changed, 13 insertions(+) diff --git a/docs/devel/migration.txt b/docs/devel/migration.txt index 4030703..cebfe7a 100644 --- a/docs/devel/migration.txt +++ b/docs/devel/mi

[Qemu-devel] [PATCH v12 0/6] calculate blocktime for postcopy live migration

2017-10-30 Thread Alexey Perevalov
This is 12th version. The rationale for that idea is following: vCPU could suspend during postcopy live migration until faulted page is not copied into kernel. Downtime on source side it's a value - time interval since source turn vCPU off, till destination start runnig vCPU. But that value was pr

[Qemu-devel] [PATCH v12 3/6] migration: calculate vCPU blocktime on dst side

2017-10-30 Thread Alexey Perevalov
This patch provides blocktime calculation per vCPU, as a summary and as a overlapped value for all vCPUs. This approach was suggested by Peter Xu, as an improvements of previous approch where QEMU kept tree with faulted page address and cpus bitmask in it. Now QEMU is keeping array with faulted pa

[Qemu-devel] [PATCH v12 1/6] migration: introduce postcopy-blocktime capability

2017-10-30 Thread Alexey Perevalov
Right now it could be used on destination side to enable vCPU blocktime calculation for postcopy live migration. vCPU blocktime - it's time since vCPU thread was put into interruptible sleep, till memory page was copied and thread awake. Reviewed-by: Dr. David Alan Gilbert Signed-off-by: Alexey P

[Qemu-devel] [PATCH v12 2/6] migration: add postcopy blocktime ctx into MigrationIncomingState

2017-10-30 Thread Alexey Perevalov
This patch adds request to kernel space for UFFD_FEATURE_THREAD_ID, in case when this feature is provided by kernel. PostcopyBlocktimeContext is incapsulated inside postcopy-ram.c, due to it's postcopy only feature. Also it defines PostcopyBlocktimeContext's instance live time. Information from Po

[Qemu-devel] [PATCH v12 6/6] migration: add postcopy total blocktime into query-migrate

2017-10-30 Thread Alexey Perevalov
Postcopy total blocktime is available on destination side only. But query-migrate was possible only for source. This patch adds ability to call query-migrate on destination. To be able to see postcopy blocktime, need to request postcopy-blocktime capability. The query-migrate command will show fol

Re: [Qemu-devel] [PATCHv4 09/13] lance: move TYPE_LANCE and SysBusPCNetState from lance.c to lance.h

2017-10-30 Thread Peter Maydell
On 30 October 2017 at 13:10, Mark Cave-Ayland wrote: > On 26/10/17 11:12, Mark Cave-Ayland wrote: >> On 25/10/17 18:47, Peter Maydell wrote: >>> On 25 October 2017 at 16:59, Mark Cave-Ayland wrote: --- /dev/null +++ b/include/hw/net/lance.h @@ -0,0 +1,41 @@ +/* + * QEMU AM

Re: [Qemu-devel] [PATCH v2 05/10] tests: check that migration parameters are really assigned

2017-10-30 Thread Juan Quintela
Peter Xu wrote: > On Thu, Oct 26, 2017 at 09:52:17AM +0200, Juan Quintela wrote: >> Signed-off-by: Juan Quintela >> --- >> tests/migration-test.c | 27 --- >> 1 file changed, 24 insertions(+), 3 deletions(-) >> >> diff --git a/tests/migration-test.c b/tests/migration-tes

Re: [Qemu-devel] [PATCH 2/5] target/i386: tcg stubs

2017-10-30 Thread Eduardo Habkost
On Mon, Oct 30, 2017 at 12:05:50AM -0300, Philippe Mathieu-Daudé wrote: > missing since 55c3ceef61: > > LINKx86_64-softmmu/qemu-system-x86_64 > target/i386/cpu.o: In function `x86_cpu_common_class_init': > target/i386/cpu.c:4215: undefined reference to `tcg_x86_init' > > Suggested-by:

Re: [Qemu-devel] [PATCH v1 0/5][RFC] Refactoring of AIS support

2017-10-30 Thread Pierre Morel
On 30/10/2017 13:44, Christian Borntraeger wrote: On 10/30/2017 01:42 PM, Cornelia Huck wrote: On Mon, 30 Oct 2017 09:28:09 +0100 Christian Borntraeger wrote: Now I thought about that for a while and I start to think that we cannot implement ais in QEMU and cover all cases. One aspect was

Re: [Qemu-devel] [PATCH v1 0/5][RFC] Refactoring of AIS support

2017-10-30 Thread Christian Borntraeger
On 10/30/2017 02:44 PM, Pierre Morel wrote: > On 30/10/2017 13:44, Christian Borntraeger wrote: >> >> >> On 10/30/2017 01:42 PM, Cornelia Huck wrote: >>> On Mon, 30 Oct 2017 09:28:09 +0100 >>> Christian Borntraeger wrote: >>> Now I thought about that for a while and I start to think that we

Re: [Qemu-devel] [PATCH] qemu-iotests: Use -nographic in 182

2017-10-30 Thread Kashyap Chamarthy
On Mon, Oct 30, 2017 at 11:13:21AM +0100, Kevin Wolf wrote: > Am 30.10.2017 um 09:39 hat Kashyap Chamarthy geschrieben: > > On Sun, Oct 29, 2017 at 01:53:38PM +0100, Kevin Wolf wrote: [...] > > > echo > > > echo "Starting a second QEMU using the same image should fail" > > > -echo 'quit' | $QEM

Re: [Qemu-devel] [PATCH v1 0/5][RFC] Refactoring of AIS support

2017-10-30 Thread Halil Pasic
On 10/30/2017 02:48 PM, Christian Borntraeger wrote: > > > On 10/30/2017 02:44 PM, Pierre Morel wrote: >> On 30/10/2017 13:44, Christian Borntraeger wrote: >>> >>> >>> On 10/30/2017 01:42 PM, Cornelia Huck wrote: On Mon, 30 Oct 2017 09:28:09 +0100 Christian Borntraeger wrote: >>

Re: [Qemu-devel] [PULL 0/3] MIPS queue

2017-10-30 Thread Yongbok Kim
On 23/10/2017 15:46, Yongbok Kim wrote: The following changes since commit e822e81e350825dd94f41ee2538ff1432b812eb9: Merge remote-tracking branch 'remotes/stefanha/tags/block-pull-request' into staging (2017-10-20 15:04:00 +0100) are available in the git repository at: git://github.com/

Re: [Qemu-devel] [PULL 0/4] s390: fixups for 2.11

2017-10-30 Thread Peter Maydell
00) > > are available in the git repository at: > > git://github.com/borntraeger/qemu.git tags/s390x-20171030 > > for you to fetch changes up to 7a4ec18d117d74eedccdadca882331b928135465: > > s390-*.img: update s3

Re: [Qemu-devel] [PULL 0/3] MIPS queue

2017-10-30 Thread Peter Maydell
On 30 October 2017 at 14:09, Yongbok Kim wrote: > On 23/10/2017 15:46, Yongbok Kim wrote: >> >> The following changes since commit >> e822e81e350825dd94f41ee2538ff1432b812eb9: >> >>Merge remote-tracking branch 'remotes/stefanha/tags/block-pull-request' >> into staging (2017-10-20 15:04:00 +010

Re: [Qemu-devel] [PATCH v6 08/25] block: Add bdrv_make_absolute_filename()

2017-10-30 Thread Alberto Garcia
On Fri 29 Sep 2017 06:53:30 PM CEST, Max Reitz wrote: > This is a general function for making a filename that is relative to a > certain BDS absolute. > > It calls bdrv_get_full_backing_filename_from_filename() for now, but > that will be changed in a follow-up patch. > > Signed-off-by: Max Reitz

Re: [Qemu-devel] drive_add: file names with spaces

2017-10-30 Thread Dr. David Alan Gilbert
* Programmingkid (programmingk...@gmail.com) wrote: > > > On Oct 29, 2017, at 6:21 AM, Dr. David Alan Gilbert > > wrote: > > > > * Eric Blake (ebl...@redhat.com) wrote: > >> On 10/28/2017 06:44 AM, Programmingkid wrote: > >>> I'm trying to use an image file that has spaces in its file name (and

Re: [Qemu-devel] [PULL 0/3] MIPS queue

2017-10-30 Thread Yongbok Kim
On 30/10/2017 14:20, Peter Maydell wrote: On 30 October 2017 at 14:09, Yongbok Kim wrote: On 23/10/2017 15:46, Yongbok Kim wrote: The following changes since commit e822e81e350825dd94f41ee2538ff1432b812eb9: Merge remote-tracking branch 'remotes/stefanha/tags/block-pull-request' into sta

Re: [Qemu-devel] [PATCH v6 09/25] block: Fix bdrv_find_backing_image()

2017-10-30 Thread Alberto Garcia
On Fri 29 Sep 2017 06:53:31 PM CEST, Max Reitz wrote: > @@ -4096,22 +4086,31 @@ BlockDriverState > *bdrv_find_backing_image(BlockDriverState *bs, > } else { > /* If not an absolute filename path, make it relative to the > current > * image's filename path */ >

[Qemu-devel] [Bug 1728615] [NEW] qemu-io crashes with SIGABRT and Assertion `c->entries[i].offset != 0' failed

2017-10-30 Thread R.Nageswara Sastry
Public bug reported: git is at HEAD a93ece47fd9edbd4558db24300056c9a57d3bcd4 This is on ppc64le architecture. Re-production steps: 1. Copy the attached files named backing_img.file and test.img to a directory 2. And customize the following command to point to the above directory and run the sam

[Qemu-devel] ping Re: loading bitmaps in invalidate_cache fails

2017-10-30 Thread Vladimir Sementsov-Ogievskiy
Hi! Any ideas? 05.10.2017 12:41, Vladimir Sementsov-Ogievskiy wrote: 12.09.2017 12:46, Kevin Wolf wrote: Am 11.09.2017 um 18:51 hat Vladimir Sementsov-Ogievskiy geschrieben: Hi Kevin! I'm confused with relations of permissions and invalidation, can you please help? Now dirty bitmaps are l

Re: [Qemu-devel] [RFC PATCH] vl: only display available accelerators

2017-10-30 Thread Philippe Mathieu-Daudé
On 10/30/2017 05:20 AM, Eduardo Habkost wrote: > On Mon, Oct 30, 2017 at 01:00:56AM -0300, Philippe Mathieu-Daudé wrote: >> - I don't think this is the nicest way, too much #ifdef'fery in main() > > I suggest using object_class_get_list(TYPE_ACCEL, false). OK, this gives: $ qemu-system-x86_6

Re: [Qemu-devel] [PATCH] vga: fix region checks in wraparound case

2017-10-30 Thread Dr. David Alan Gilbert
* Gerd Hoffmann (kra...@redhat.com) wrote: > Cc: "Dr. David Alan Gilbert" > Signed-off-by: Gerd Hoffmann Makes more sense to me; Reviewed-by: Dr. David Alan Gilbert > --- > hw/display/vga.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/hw/display/vga.c b/hw/d

[Qemu-devel] [Bug 1728615] Re: qemu-io crashes with SIGABRT and Assertion `c->entries[i].offset != 0' failed

2017-10-30 Thread R.Nageswara Sastry
** Attachment added: "images tar file" https://bugs.launchpad.net/qemu/+bug/1728615/+attachment/5000141/+files/images.tar.gz -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1728615 Title: qemu-io

Re: [Qemu-devel] [Qemu-block] [PATCH] qemu-iotests: Use -nographic in 182

2017-10-30 Thread Jeff Cody
On Sun, Oct 29, 2017 at 01:53:38PM +0100, Kevin Wolf wrote: > This avoids that random UI frontend error messages end up in the output. > In particular, we were seeing this line in CI error logs: > > +Unable to init server: Could not connect: Connection refused > > Signed-off-by: Kevin Wolf > ---

Re: [Qemu-devel] drive_add: file names with spaces

2017-10-30 Thread Programmingkid
> On Oct 30, 2017, at 10:32 AM, Dr. David Alan Gilbert > wrote: > > * Programmingkid (programmingk...@gmail.com) wrote: >> >>> On Oct 29, 2017, at 6:21 AM, Dr. David Alan Gilbert >>> wrote: >>> >>> * Eric Blake (ebl...@redhat.com) wrote: On 10/28/2017 06:44 AM, Programmingkid wrote: >>

Re: [Qemu-devel] [RFC PATCH] vl: only display available accelerators

2017-10-30 Thread Peter Maydell
On 30 October 2017 at 15:01, Philippe Mathieu-Daudé wrote: > On 10/30/2017 05:20 AM, Eduardo Habkost wrote: >> On Mon, Oct 30, 2017 at 01:00:56AM -0300, Philippe Mathieu-Daudé wrote: >>> - I don't think this is the nicest way, too much #ifdef'fery in main() >> >> I suggest using object_class_g

Re: [Qemu-devel] [RFC 00/19] KVM: s390/crypto/vfio: guest dedicated crypto adapters

2017-10-30 Thread Tony Krowiak
On 10/30/2017 04:57 AM, Christian Borntraeger wrote: adding qemu devel and add Daniel and Erik from libvirt to keep them in the loop. On 10/29/2017 12:11 PM, Cornelia Huck wrote: On Fri, 13 Oct 2017 13:38:45 -0400 Tony Krowiak wrote: Tony Krowiak (19): KVM: s390: SIE considerations for AP

[Qemu-devel] [Bug 1728635] [NEW] qemu-io crashes with SIGSEGV when did -c aio_write 9233408 28160 on a image_fuzzer image

2017-10-30 Thread R.Nageswara Sastry
Public bug reported: git is at HEAD a93ece47fd9edbd4558db24300056c9a57d3bcd4 This is on ppc64le architecture. Re-production steps: 1. Copy the attached file named test.img to a directory 2. And customize the following command to point to the above directory and run the same. # cp test.img copy.

[Qemu-devel] [Bug 1728635] Re: qemu-io crashes with SIGSEGV when did -c aio_write 9233408 28160 on a image_fuzzer image

2017-10-30 Thread R.Nageswara Sastry
** Attachment added: "test.img tarred" https://bugs.launchpad.net/qemu/+bug/1728635/+attachment/5000176/+files/test.img.tar.gz -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1728635 Title: qemu-

Re: [Qemu-devel] [PATCH v3] monitor: fix dangling CPU pointer

2017-10-30 Thread Igor Mammedov
On Tue, 17 Oct 2017 10:16:22 +0200 Greg Kurz wrote: > If a CPU selected with the "cpu" command is hot-unplugged then "info cpus" > causes QEMU to exit: > > (qemu) device_del cpu1 > (qemu) info cpus > qemu:qemu_cpu_kick_thread: No such process > > This happens because "cpu" stores the pointer to

[Qemu-devel] [Bug 1728639] Re: qemu-io crashes with SIGSEGV when did -c truncate 320000 on a image_fuzzer image

2017-10-30 Thread R.Nageswara Sastry
** Attachment added: "test.img tarred" https://bugs.launchpad.net/qemu/+bug/1728639/+attachment/5000182/+files/test.img.tar.gz -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1728639 Title: qemu-

[Qemu-devel] [Bug 1728639] [NEW] qemu-io crashes with SIGSEGV when did -c truncate 320000 on a image_fuzzer image

2017-10-30 Thread R.Nageswara Sastry
Public bug reported: git is at HEAD a93ece47fd9edbd4558db24300056c9a57d3bcd4 This is on ppc64le architecture. Re-production steps: 1. Copy the attached files named test.img to a directory 2. And customize the following command to point to the above directory and run the same. # mv test.img copy

[Qemu-devel] [PATCH v4] nvme: Add tracing

2017-10-30 Thread Doug Gale
>From 0e27b5dca8f4f32a1b194e1b3544be77dd4f45d9 Mon Sep 17 00:00:00 2001 From: Doug Gale Date: Mon, 30 Oct 2017 09:28:43 -0400 Subject: [PATCH] nvme: Add tracing Add trace output for commands, errors, and undefined behavior. Add guest error log output for undefined behavior. Report invalid undefin

Re: [Qemu-devel] [PATCH v3 5/5] RFC: fw_cfg: add DMA write operation in sysfs

2017-10-30 Thread Michael S. Tsirkin
On Mon, Oct 30, 2017 at 11:07:20AM +, Hatayama, Daisuke wrote: > > From: Marc-André Lureau > > > > Since qemu 2.9, DMA write operations are allowed. However, usage of this > > interface from kernel or user-space is strongly discouraged by the > > maintainers. This patch is meant for experiment

[Qemu-devel] [Bug 1728643] [NEW] qemu-io fails with Assertion `*host_offset != 0' failed

2017-10-30 Thread R.Nageswara Sastry
Public bug reported: git is at HEAD a93ece47fd9edbd4558db24300056c9a57d3bcd4 This is on ppc64le architecture. Re-production steps: 1. Copy the attached files named test.img to a directory 2. And customize the following command to point to the above directory and run the same. # cp test.img copy

[Qemu-devel] [Bug 1728643] Re: qemu-io fails with Assertion `*host_offset != 0' failed

2017-10-30 Thread R.Nageswara Sastry
** Attachment added: "test.img tarred" https://bugs.launchpad.net/qemu/+bug/1728643/+attachment/5000185/+files/test.img_1728643.tar.gz -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1728643 Title:

[Qemu-devel] [PATCH v8 08/14] migration/qemu-file: add qemu_put_counted_string()

2017-10-30 Thread Vladimir Sementsov-Ogievskiy
Add function opposite to qemu_get_counted_string. qemu_put_counted_string puts one-byte length of the string (string should not be longer than 255 characters), and then it puts the string, without last zero byte. Signed-off-by: Vladimir Sementsov-Ogievskiy Reviewed-by: John Snow Reviewed-by: Jua

[Qemu-devel] [PATCH v8 07/14] migration: include migrate_dirty_bitmaps in migrate_postcopy

2017-10-30 Thread Vladimir Sementsov-Ogievskiy
Enable postcopy if dirty bitmap migration is endabled. Signed-off-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Juan Quintela --- migration/migration.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/migration/migration.c b/migration/migration.c index 1526cd4bff..e973837bfd

[Qemu-devel] [PATCH v8 13/14] iotests: add dirty bitmap postcopy test

2017-10-30 Thread Vladimir Sementsov-Ogievskiy
Test - start two vms (vm_a, vm_b) - in a - do writes from set A - do writes from set B - fix bitmap sha256 - clear bitmap - do writes from set A - start migration - than, in b - wait vm start (postcopy should start) - do writes from set B - check bitmap sha256

[Qemu-devel] [PATCH v8 14/14] iotests: add persistent bitmap migration test

2017-10-30 Thread Vladimir Sementsov-Ogievskiy
Test that persistent bitmap migrates and its persistance property migrates too. Signed-off-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/169 | 21 ++--- tests/qemu-iotests/169.out | 4 ++-- 2 files changed, 20 insertions(+), 5 deletions(-) diff --git a/tests/qemu-

[Qemu-devel] [PATCH v8 01/14] block/dirty-bitmap: add bdrv_dirty_bitmap_enable_successor()

2017-10-30 Thread Vladimir Sementsov-Ogievskiy
Enabling bitmap successor is necessary to enable successors of bitmaps being migrated before target vm start. Signed-off-by: Vladimir Sementsov-Ogievskiy --- include/block/dirty-bitmap.h | 1 + block/dirty-bitmap.c | 8 2 files changed, 9 insertions(+) diff --git a/include/bloc

[Qemu-devel] [PATCH v8 06/14] qapi: add dirty-bitmaps migration capability

2017-10-30 Thread Vladimir Sementsov-Ogievskiy
Signed-off-by: Vladimir Sementsov-Ogievskiy Reviewed-by: John Snow Reviewed-by: Eric Blake Reviewed-by: Juan Quintela --- qapi/migration.json | 6 +- migration/migration.h | 1 + migration/migration.c | 9 + 3 files changed, 15 insertions(+), 1 deletion(-) diff --git a/qapi/migr

[Qemu-devel] [PATCH v8 04/14] block/dirty-bitmap: add bdrv_dirty_bitmap_set_frozen

2017-10-30 Thread Vladimir Sementsov-Ogievskiy
Make it possible to set bitmap 'frozen' without a successor. This is needed to protect the bitmap during outgoing bitmap postcopy migration. Signed-off-by: Vladimir Sementsov-Ogievskiy --- include/block/dirty-bitmap.h | 1 + block/dirty-bitmap.c | 22 -- 2 files chan

  1   2   >