[Qemu-devel] [PATCH v3 RFC 1/2] linux-headers: sync linux headers

2018-06-07 Thread Dongjiu Geng
Import KVM_CAP_ARM_INJECT_SERROR_ESR and struct kvm_vcpu_events for arm64, they are added manually in the kernel tree, so it will be added after re-running scripts/update-linux-headers.sh. Signed-off-by: Dongjiu Geng --- linux-headers/asm-arm/kvm.h | 12

[Qemu-devel] [PATCH v3 RFC 0/2] add support for VCPU event states

2018-06-07 Thread Dongjiu Geng
support for KVM_GET/SET_VCPU_EVENTS to get/set the SError exception state, and support the state migration. change since v2: 1. add header definition for arm platform change since v1: 1. update the code to fix the build errors Dongjiu Geng (2): linux-headers: sync linux headers target: arm:

Re: [Qemu-devel] [PATCH v3 0/2] backup: Use copy offloading

2018-06-07 Thread Stefan Hajnoczi
On Tue, Jun 05, 2018 at 10:06:29PM +0800, Fam Zheng wrote: > Based-on: <20180529055959.32002-1-f...@redhat.com> > ([PATCH v7 00/10] qemu-img convert with copy offloading) > > This enhances the backup job to make use of the copy offloading API. It > eliminates the necessity to use the bounce

[Qemu-devel] [PATCH] qemu-img: align is_allocated_sectors to 4k

2018-06-07 Thread Peter Lieven
We currently don't enforce that the sparse segments we detect during convert are aligned. This leads to unnecessary and costly read-modify-write cycles either internally in Qemu or in the background on the storage device as nearly all modern filesystems or hardware has a 4k alignment internally.

Re: [Qemu-devel] [PATCH] file-posix: Consolidate the locking error message

2018-06-07 Thread Markus Armbruster
Daniel P. Berrangé writes: > On Fri, Jun 01, 2018 at 05:18:35PM +0800, Fam Zheng wrote: >> When hot-plugging a block device fails due to image locking errors, >> users won't see the helpful 'Is another process using the image?' >> message in QMP because currently the error hint is not carried

[Qemu-devel] OpenRISC: SMP support for more than 2 cores

2018-06-07 Thread Davidson Francis
Dear all, Currently Qemu supports only 2 cores when SMP enabled for or1k architecure, so I would like to know if there is a quick way to increase the number of cores by changing a few lines of code or to accomplish this, will requires significant changes in the source code? I'm asking because

Re: [Qemu-devel] [PATCH 1/4] hw/block/fdc: Replace error_setg(_abort) by error_report() + abort()

2018-06-07 Thread Markus Armbruster
Philippe Mathieu-Daudé writes: > On 05/30/2018 05:23 PM, John Snow wrote: >> On 05/29/2018 01:48 PM, Philippe Mathieu-Daudé wrote: >>> Use error_report() + abort() instead of error_setg(_abort), >>> as suggested by the "qapi/error.h" documentation: >>> >>> Please don't error_setg(_fatal,

Re: [Qemu-devel] [PATCH v4 06/14] spapr: prepare for multi stage hotplug handlers

2018-06-07 Thread Igor Mammedov
On Tue, 5 Jun 2018 09:51:26 +0200 David Hildenbrand wrote: > On 05.06.2018 03:08, David Gibson wrote: > > On Thu, May 17, 2018 at 10:15:19AM +0200, David Hildenbrand wrote: > >> For multi stage hotplug handlers, we'll have to do some error handling > >> in some hotplug functions, so let's use

Re: [Qemu-devel] [PATCH v9 3/7] monitor: more comments on lock-free elements

2018-06-07 Thread Markus Armbruster
Peter Xu writes: > Add some explicit comments for both Readline and cpu_set/cpu_get helpers > that they do not need the mon_lock protection. > > Signed-off-by: Peter Xu Reviewed-by: Markus Armbruster

Re: [Qemu-devel] [PATCH v9 5/7] monitor: remove event_clock_type

2018-06-07 Thread Markus Armbruster
Peter Xu writes: > Instead, use a dynamic function to detect which clock we'll use. The > problem is that the old code will let monitor initialization depends on > qtest_enabled(). After this change, we don't have such a dependency any > more. > > Suggested-by: Markus Armbruster >

Re: [Qemu-devel] [PATCH v2 8/8] qapi: query-blockstat: add driver specific file-posix stats

2018-06-07 Thread Anton Nefedov
On 3/2/2018 6:59 PM, Markus Armbruster wrote: Eric Blake writes: On 01/19/2018 06:50 AM, Anton Nefedov wrote: A block driver can provide a callback to report driver-specific statistics. file-posix driver now reports discard statistics Signed-off-by: Anton Nefedov Reviewed-by: Vladimir

Re: [Qemu-devel] [PATCH v9 7/7] monitor: add lock to protect mon_fdsets

2018-06-07 Thread Markus Armbruster
Peter Xu writes: > Introduce a new global big lock for mon_fdsets. Take it where needed. > > The monitor_fdset_get_fd() handling is a bit tricky: now we need to call > qemu_mutex_unlock() which might pollute errno, so we need to make sure > the correct errno be passed up to the callers. To

Re: [Qemu-devel] [PATCH v12 2/4] i386: Verify if topoext feature can be supported

2018-06-07 Thread Eduardo Habkost
On Thu, Jun 07, 2018 at 02:24:18PM +, Moger, Babu wrote: [...] > > > +/* On AMD systems, check if we can support topoext feature */ > > > +if (IS_AMD_CPU(env) && > > > +(env->features[FEAT_8000_0001_ECX] & CPUID_EXT3_TOPOEXT)) { > > > +if

Re: [Qemu-devel] [PATCH 03/17] iotests: ask qemu for supported formats

2018-06-07 Thread Markus Armbruster
Thomas Huth writes: > On 07.06.2018 08:57, Markus Armbruster wrote: >> Thomas Huth writes: >> >>> On 05.06.2018 00:40, Eric Blake wrote: On 06/04/2018 05:34 AM, Thomas Huth wrote: > On 04.06.2018 09:18, Markus Armbruster wrote: >> Roman Kagan writes: >> >>> Add helper

Re: [Qemu-devel] [PATCH v7 4/5] virtio-balloon: VIRTIO_BALLOON_F_FREE_PAGE_HINT

2018-06-07 Thread Wei Wang
On 06/07/2018 02:32 PM, Peter Xu wrote: On Thu, Jun 07, 2018 at 01:24:29PM +0800, Wei Wang wrote: On 06/06/2018 07:02 PM, Peter Xu wrote: On Wed, Jun 06, 2018 at 06:04:23PM +0800, Wei Wang wrote: On 06/06/2018 01:42 PM, Peter Xu wrote: IMHO migration states do not suite here. IMHO bitmap

Re: [Qemu-devel] [PATCH v2 4/6] hmp: Add info commands for preconfig

2018-06-07 Thread Igor Mammedov
On Thu, 7 Jun 2018 11:21:21 +0100 "Dr. David Alan Gilbert (git)" wrote: > From: "Dr. David Alan Gilbert" > > Allow a bunch of the info commands to be used in preconfig. > Could probably add most of them. > > Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Igor Mammedov > --- >

[Qemu-devel] [Bug 1775555] Re: guest migration 100% cpu freeze bug

2018-06-07 Thread Dion Bosschieter
guest xml definition: vps25 0cf4666d-6855-b3a8-12da-2967563f 8388608 8388608 4 /machine hvm Westmere destroy restart restart /usr/bin/kvm 734003200

Re: [Qemu-devel] [RFC] qapi: command category to stimulate high-level machine devices

2018-06-07 Thread Stefan Hajnoczi
On Mon, Jun 04, 2018 at 05:14:25PM -0500, Eric Blake wrote: > On 06/01/2018 10:16 PM, Steffen Görtz wrote: > > From: Steffen Goertz > > > > Add a new category "stimulate" to host commands that > > act upon high-level devices associated with machines/boards. > > > > This is patch is part of an

Re: [Qemu-devel] [PATCH v2 2/4] ftgmac100: add IEEE 802.1Q VLAN support

2018-06-07 Thread Peter Maydell
On 30 May 2018 at 07:17, Cédric Le Goater wrote: > The ftgmac100 NIC supports VLAN tag insertion and the MAC engine also > has a control to remove VLAN tags from received packets. > > The VLAN control bits and VLAN tag information are contained in the > second word of the transmit and receive

Re: [Qemu-devel] [PATCH 2/2] jobs: fix verb references in docs

2018-06-07 Thread Jeff Cody
On Wed, Jun 06, 2018 at 07:02:57PM -0400, John Snow wrote: > These point to the job versions now, not the blockjob versions which > don't really exist anymore. > > Except set-speed, which does. It sticks out like a sore thumb. This > patch doesn't fix that, but it doesn't make it any worse,

Re: [Qemu-devel] [PATCH v4 01/14] memory-device: drop assert related to align and start of address space

2018-06-07 Thread Igor Mammedov
On Mon, 4 Jun 2018 12:53:42 +0200 David Hildenbrand wrote: > >> Let me explain a little bit why I don't like such restrictions (for > >> which I don't see a need yet): > > (*) being conservative is good here because we can always relax restrictions > > in future if it's needed without breaking

[Qemu-devel] [PATCH] hw/arm/smmuv3: fix smmu emulation when guest smmu is in passthrough mode

2018-06-07 Thread Jia He
There is an exception when I passes iommu.passthrough=1 to guest's kernel boot parameter(host QDF2400 kernel 4.17, guest kernel 4.14). The guest will be hang when booting up. When guest smmu is in passthrough mode, entry.perm will not be assigned to flag in smmuv3_translate. It seems not be

Re: [Qemu-devel] [PATCH v9 1/7] monitor: rename out_lock to mon_lock

2018-06-07 Thread Markus Armbruster
Peter Xu writes: > The out_lock is protecting a few Monitor fields. In the future the > monitor code will start to run in multiple threads. We are going to > turn it into a bigger lock to protect not only the out buffer but also > most of the rest. > > Since at it, rearrange the Monitor struct

[Qemu-devel] [PATCH v2 03/10] block: Add a mechanism for passing a block driver a block configuration

2018-06-07 Thread Ari Sundholm
A block driver may need to know about the block configuration, most critically the sector sizes, of a block backend for alignment purposes or for some other reason. It doesn't seem like qemu has an existing mechanism for a block backend to convey the required information to the relevant block

Re: [Qemu-devel] [RFC v2 3/4] monitor: remove "x-oob", turn oob on by default

2018-06-07 Thread Markus Armbruster
Peter Xu writes: > There was a regression reported by Eric Auger before with OOB: > > http://lists.gnu.org/archive/html/qemu-devel/2018-03/msg06231.html > > The fix is 951702f39c ("monitor: bind dispatch bh to iohandler context", > 2018-04-10), which is in master already. > > For the bug, we

Re: [Qemu-devel] -enable-kvm and friens (was: Re: [PATCH 03/17] iotests: ask qemu for supported formats)

2018-06-07 Thread Paolo Bonzini
On 07/06/2018 13:27, Thomas Huth wrote: >> As to "-enable-kvm", I don't see anything wrong with users using it, or >> even with occasionally adding more options like it. However, we should >> warn developers that such simple options should be syntactic sugar for a >> structured (i.e.

[Qemu-devel] [Bug 1775555] Re: guest migration 100% cpu freeze bug

2018-06-07 Thread Dr. David Alan Gilbert
hangs like this after migration are a pain to debug; especially with that really rare recurrence rate. The fact the RIP is changing and is moving in and out of the kernel suggests something is happening; so it might be that we've corrupted some memory, or got a device in a mess where the device

Re: [Qemu-devel] -enable-kvm and friens

2018-06-07 Thread Thomas Huth
On 07.06.2018 13:42, Paolo Bonzini wrote: > On 07/06/2018 13:27, Thomas Huth wrote: >>> As to "-enable-kvm", I don't see anything wrong with users using it, or >>> even with occasionally adding more options like it. However, we should >>> warn developers that such simple options should be

Re: [Qemu-devel] [PATCH v2 0/4] pc-bios/s390-ccw: Allow network booting via pxelinux.cfg

2018-06-07 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 1528374154-14680-1-git-send-email-th...@redhat.com Subject: [Qemu-devel] [PATCH v2 0/4] pc-bios/s390-ccw: Allow network booting via pxelinux.cfg === TEST SCRIPT BEGIN ===

Re: [Qemu-devel] [PATCH 03/17] iotests: ask qemu for supported formats

2018-06-07 Thread Daniel P . Berrangé
On Thu, Jun 07, 2018 at 09:50:41AM +0200, Thomas Huth wrote: > On 07.06.2018 08:57, Markus Armbruster wrote: > > Thomas Huth writes: > > > >> On 05.06.2018 00:40, Eric Blake wrote: > >>> On 06/04/2018 05:34 AM, Thomas Huth wrote: > On 04.06.2018 09:18, Markus Armbruster wrote: > > Roman

Re: [Qemu-devel] [PATCH V8 10/17] qmp event: Add COLO_EXIT event to notify users while exited COLO

2018-06-07 Thread Markus Armbruster
Eric Blake writes: > On 06/03/2018 12:05 AM, Zhang Chen wrote: >> From: zhanghailiang >> >> If some errors happen during VM's COLO FT stage, it's important to >> notify the users of this event. Together with 'x-colo-lost-heartbeat', >> Users can intervene in COLO's failover work immediately. >>

Re: [Qemu-devel] [PATCH 1/2] jobs: fix stale wording

2018-06-07 Thread Jeff Cody
On Wed, Jun 06, 2018 at 07:02:56PM -0400, John Snow wrote: > During the design for manual completion, we decided not to use the > "manual" property as a shorthand for both auto-dismiss and auto-finalize. > > Fix the wording. > > Signed-off-by: John Snow Reviewed-by: Jeff Cody > --- >

Re: [Qemu-devel] [PATCH V8 11/17] qapi: Add new command to query colo status

2018-06-07 Thread Markus Armbruster
Zhang Chen writes: > Libvirt or other high level software can use this command query colo status. > You can test this command like that: > {'execute':'query-colo-status'} > > Signed-off-by: Zhang Chen > --- > migration/colo.c| 39 +++ >

Re: [Qemu-devel] [Qemu-block] [PATCH 1/5] block: Add blklogwrites

2018-06-07 Thread Ari Sundholm
On 06/07/2018 03:30 PM, Stefan Hajnoczi wrote: On Mon, Jun 04, 2018 at 03:10:47PM +0300, Ari Sundholm wrote: On 06/04/2018 12:51 PM, Stefan Hajnoczi wrote: On Fri, Jun 01, 2018 at 05:24:53PM +0300, Ari Sundholm wrote: On 06/01/2018 04:32 PM, Stefan Hajnoczi wrote: On Fri, Jun 01, 2018 at

Re: [Qemu-devel] [PATCH v4 04/14] pc: prepare for multi stage hotplug handlers

2018-06-07 Thread Igor Mammedov
On Mon, 4 Jun 2018 13:27:01 +0200 David Hildenbrand wrote: > On 31.05.2018 16:13, Igor Mammedov wrote: > > On Wed, 30 May 2018 16:13:32 +0200 > > David Hildenbrand wrote: > > > >> On 30.05.2018 15:08, Igor Mammedov wrote: > >>> On Thu, 17 May 2018 10:15:17 +0200 > >>> David Hildenbrand

Re: [Qemu-devel] [PATCH 4/6] hmp: Add info commands for preconfig

2018-06-07 Thread Dr. David Alan Gilbert
* Markus Armbruster (arm...@redhat.com) wrote: > Peter Xu writes: > > > On Tue, Jun 05, 2018 at 01:26:34PM +0100, Dr. David Alan Gilbert (git) > > wrote: > >> From: "Dr. David Alan Gilbert" > >> > >> Allow a bunch of the info commands to be used in preconfig. > >> Could probably add most of

Re: [Qemu-devel] [PATCH 0/4] qapi/error: converts error_setg(_fatal) to error_report() + exit()

2018-06-07 Thread Markus Armbruster
Philippe Mathieu-Daudé writes: > Hi, > > This series converts error_setg(_fatal) to error_report() + exit() as > suggested by the "qapi/error.h" documentation. Appreciated! The series actually converts two anti-patterns. 1. From if (COND) { error_setg(_fatal, ...); } to

Re: [Qemu-devel] [PATCH v4 04/14] pc: prepare for multi stage hotplug handlers

2018-06-07 Thread David Hildenbrand
On 07.06.2018 15:44, Igor Mammedov wrote: > On Mon, 4 Jun 2018 13:27:01 +0200 > David Hildenbrand wrote: > >> On 31.05.2018 16:13, Igor Mammedov wrote: >>> On Wed, 30 May 2018 16:13:32 +0200 >>> David Hildenbrand wrote: >>> On 30.05.2018 15:08, Igor Mammedov wrote: > On Thu, 17

[Qemu-devel] [PATCH v2 04/10] hw/scsi/scsi-disk: Always apply block configuration to block driver

2018-06-07 Thread Ari Sundholm
This allows the block driver to use the block configuration of the new SCSI device. One use for this information is to set request limits using this information. Signed-off-by: Ari Sundholm --- hw/scsi/scsi-disk.c | 1 + 1 file changed, 1 insertion(+) diff --git a/hw/scsi/scsi-disk.c

[Qemu-devel] [PATCH v2 00/10] New block driver: blklogwrites

2018-06-07 Thread Ari Sundholm
This patch series adds a new block driver, blklogwrites, to QEMU. The driver is given two block devices: a raw device backed by an image or a host block device, and a log device, typically backed by a file, on which writes to the raw device are logged. The logging format used is the same as in

[Qemu-devel] [PATCH v2 08/10] hw/block/fdc: Always apply block configuration to block driver

2018-06-07 Thread Ari Sundholm
This allows the block driver to use the block configuration of the new floppy device. One use for this information is to set request limits using this information. Signed-off-by: Ari Sundholm --- hw/block/fdc.c | 4 1 file changed, 4 insertions(+) diff --git a/hw/block/fdc.c

Re: [Qemu-devel] An emulation failure occurs, if I hotplug vcpus immediately after the VM start

2018-06-07 Thread David Hildenbrand
On 07.06.2018 13:02, Paolo Bonzini wrote: > On 07/06/2018 12:37, David Hildenbrand wrote: >> >> I have a related requirement, which would be to atomically grow a >> memory regions. So instead of region_del(old)+region_add(new), I would >> have to do it in one shot (atomically). >> >> AFAICS an

Re: [Qemu-devel] [PATCH 1/6] hmp: Add flag for preconfig commands

2018-06-07 Thread Dr. David Alan Gilbert
* Markus Armbruster (arm...@redhat.com) wrote: > "Dr. David Alan Gilbert (git)" writes: > > > From: "Dr. David Alan Gilbert" > > > > Add a flag to command definitions to allow them to be used in preconfig > > and check it. > > > > Signed-off-by: Dr. David Alan Gilbert > > --- > > monitor.c |

[Qemu-devel] [PATCH v2 3/4] pc-bios/s390-ccw/net: Add support for pxelinux-style config files

2018-06-07 Thread Thomas Huth
Since it is quite cumbersome to manually create a combined kernel with initrd image for network booting, we now support loading via pxelinux configuration files, too. In these files, the kernel, initrd and command line parameters can be specified seperately, and the firmware then takes care of

[Qemu-devel] [PATCH v2 2/4] pc-bios/s390-ccw/net: Update code for the latest changes in SLOF

2018-06-07 Thread Thomas Huth
The ip_version information now has to be stored in the filename_ip_t structure, and there is now a common function called tftp_get_error_info() which can be used to get the error string for a TFTP error code. We can also get rid of some superfluous "(char *)" casts now. Signed-off-by: Thomas Huth

Re: [Qemu-devel] [PATCH] vl.c: make default main_loop_wait() timeout independed of slirp

2018-06-07 Thread Stefan Hajnoczi
On Tue, Jun 05, 2018 at 09:47:44AM +0100, Daniel P. Berrangé wrote: > On Tue, Jun 05, 2018 at 10:27:03AM +0200, Igor Mammedov wrote: > > On Mon, 4 Jun 2018 22:04:13 -0300 > > Eduardo Habkost wrote: > > > > > On Mon, Jun 04, 2018 at 10:14:38PM +0200, Igor Mammedov wrote: > > > > Since commit

Re: [Qemu-devel] [PATCH] hw/i2c: Add trace events

2018-06-07 Thread Peter Maydell
On 6 June 2018 at 20:18, Philippe Mathieu-Daudé wrote: > Signed-off-by: Philippe Mathieu-Daudé > --- > $ qemu-system ... -d trace:i2c_recv,trace:i2c_send > > 4486@1528311614.709959:i2c_recv recv(addr:0x50) data:0x00 > 4486@1528311614.709994:i2c_send send(addr:0x50) data:0x05 >

Re: [Qemu-devel] [PATCH] file-posix: Consolidate the locking error message

2018-06-07 Thread Daniel P . Berrangé
On Thu, Jun 07, 2018 at 03:20:24PM +0200, Markus Armbruster wrote: > Daniel P. Berrangé writes: > > > On Fri, Jun 01, 2018 at 05:18:35PM +0800, Fam Zheng wrote: > >> When hot-plugging a block device fails due to image locking errors, > >> users won't see the helpful 'Is another process using the

Re: [Qemu-devel] [PATCH] hw/arm/smmuv3: fix smmu emulation when guest smmu is in passthrough mode

2018-06-07 Thread Peter Maydell
On 7 June 2018 at 14:38, Jia He wrote: > There is an exception when I passes iommu.passthrough=1 to guest's > kernel boot parameter(host QDF2400 kernel 4.17, guest kernel 4.14). > The guest will be hang when booting up. > > When guest smmu is in passthrough mode, entry.perm will not be assigned >

Re: [Qemu-devel] [PATCH 3/4] hw/arm/sysbus-fdt: Replace error_setg(_fatal) by error_report() + exit()

2018-06-07 Thread Markus Armbruster
Philippe Mathieu-Daudé writes: > Use error_report() + exit() instead of error_setg(_fatal), > as suggested by the "qapi/error.h" documentation: > >Please don't error_setg(_fatal, ...), use error_report() and >exit(), because that's more obvious. > > This fixes CID 1352173: > "Passing

Re: [Qemu-devel] An emulation failure occurs, if I hotplug vcpus immediately after the VM start

2018-06-07 Thread David Hildenbrand
On 07.06.2018 13:13, Gonglei (Arei) wrote: > >> -Original Message- >> From: David Hildenbrand [mailto:da...@redhat.com] >> Sent: Thursday, June 07, 2018 6:40 PM >> Subject: Re: An emulation failure occurs,if I hotplug vcpus immediately >> after the >> VM start >> >> On 06.06.2018 15:57,

[Qemu-devel] [PATCH v6 2/2] vl: fix use of --daemonize with --preconfig

2018-06-07 Thread Igor Mammedov
When using --daemonize, the initial lead process will fork a child and then wait to be notified that setup is complete via a pipe, before it exits. When using --preconfig there is an extra call to main_loop() before the notification is done from os_setup_post(). Thus the parent process won't exit

[Qemu-devel] [PATCH v3 RFC 2/2] target: arm: Add support for VCPU event states

2018-06-07 Thread Dongjiu Geng
This patch extends the qemu-kvm state sync logic with support for KVM_GET/SET_VCPU_EVENTS, giving access to yet missing SError exception. And also it can support the exception state migration. Signed-off-by: Dongjiu Geng --- target/arm/cpu.h | 5 target/arm/kvm64.c | 64

[Qemu-devel] [PATCH v2 4/4] pc-bios/s390-ccw/net: Try to load pxelinux.cfg file accoring to the UUID

2018-06-07 Thread Thomas Huth
With the STSI instruction, we can get the UUID of the current VM instance, so we can support loading pxelinux config files via UUID in the file name, too. Signed-off-by: Thomas Huth --- pc-bios/s390-ccw/netmain.c | 56 +- 1 file changed, 55

Re: [Qemu-devel] An emulation failure occurs, if I hotplug vcpus immediately after the VM start

2018-06-07 Thread Paolo Bonzini
On 07/06/2018 13:36, David Hildenbrand wrote: >> The dirty bitmap would be synced in kvm_region_del (so it's not true >> that kvm_region_del would disappear, but almost :)). > I was rather concerned when doing a KVM_SET_USER_MEMORY_REGIONS while > some (already present) memory region is performing

Re: [Qemu-devel] [PATCH v2 2/4] ftgmac100: add IEEE 802.1Q VLAN support

2018-06-07 Thread Cédric Le Goater
On 06/07/2018 02:41 PM, Peter Maydell wrote: > On 30 May 2018 at 07:17, Cédric Le Goater wrote: >> The ftgmac100 NIC supports VLAN tag insertion and the MAC engine also >> has a control to remove VLAN tags from received packets. >> >> The VLAN control bits and VLAN tag information are contained

Re: [Qemu-devel] [PATCH v4 01/14] memory-device: drop assert related to align and start of address space

2018-06-07 Thread David Hildenbrand
>> And another point against introducing a check for the maimum alignment >> would be: There might be users with a specified alignment > max page >> size. Changing the behavior will break these setups (strange but true). > check is already there implicitly as assert, and you're removing it >

Re: [Qemu-devel] [PATCH v9 4/7] monitor: fix comment for monitor_lock

2018-06-07 Thread Markus Armbruster
Peter Xu writes: > Fix typo in d622cb5879c. Meanwhile move these variables close to each > other. monitor_qapi_event_state can be declared static, add that. > > Reported-by: Markus Armbruster > Signed-off-by: Peter Xu Reviewed-by: Markus Armbruster

Re: [Qemu-devel] [PATCH 1/4] hw/block/fdc: Replace error_setg(_abort) by error_report() + abort()

2018-06-07 Thread Philippe Mathieu-Daudé
On 06/07/2018 10:41 AM, Markus Armbruster wrote: > Philippe Mathieu-Daudé writes: >> On 05/30/2018 05:23 PM, John Snow wrote: >>> On 05/29/2018 01:48 PM, Philippe Mathieu-Daudé wrote: Use error_report() + abort() instead of error_setg(_abort), as suggested by the "qapi/error.h"

Re: [Qemu-devel] [PATCH v9 6/7] monitor: move init global earlier

2018-06-07 Thread Markus Armbruster
Peter Xu writes: > Before this patch, monitor fd helpers might be called even earlier than > monitor_init_globals(). This can be problematic. > > After previous work, now monitor_init_globals() does not depend on > accelerator initialization any more. Call it earlier (before CLI > parsing;

Re: [Qemu-devel] [PATCH] monitor: postpone monitor_qmp_cleanup_queues

2018-06-07 Thread Markus Armbruster
Peter Xu writes: > Previously we cleanup the queues when we got CLOSED event. It was used we clean up > to make sure we won't leftover replies/events of a old client to a new we won't send leftover replies/events of an old client > client. Now this patch postpones that until OPENED. What

Re: [Qemu-devel] An emulation failure occurs, if I hotplug vcpus immediately after the VM start

2018-06-07 Thread Gonglei (Arei)
> -Original Message- > From: David Hildenbrand [mailto:da...@redhat.com] > Sent: Thursday, June 07, 2018 6:40 PM > Subject: Re: An emulation failure occurs,if I hotplug vcpus immediately after > the > VM start > > On 06.06.2018 15:57, Paolo Bonzini wrote: > > On 06/06/2018 15:28,

Re: [Qemu-devel] [Qemu-block] [PATCH 1/5] block: Add blklogwrites

2018-06-07 Thread Stefan Hajnoczi
On Mon, Jun 04, 2018 at 03:10:47PM +0300, Ari Sundholm wrote: > On 06/04/2018 12:51 PM, Stefan Hajnoczi wrote: > > On Fri, Jun 01, 2018 at 05:24:53PM +0300, Ari Sundholm wrote: > > > On 06/01/2018 04:32 PM, Stefan Hajnoczi wrote: > > > > On Fri, Jun 01, 2018 at 12:17:19AM +0300, Ari Sundholm

Re: [Qemu-devel] storing machine data in qcow images?

2018-06-07 Thread Daniel P . Berrangé
On Thu, Jun 07, 2018 at 01:17:24PM +0200, Andrea Bolognani wrote: > On Thu, 2018-06-07 at 11:22 +0100, Daniel P. Berrangé wrote: > > On Thu, Jun 07, 2018 at 12:02:29PM +0200, Andrea Bolognani wrote: > > > While hints might be considered a reasonable fit for qcow2, I think > > > it's pretty hard to

Re: [Qemu-devel] [PATCH] CODING_STYLE: Define our preferred form for multiline comments

2018-06-07 Thread Stefan Hajnoczi
On Mon, Jun 04, 2018 at 07:17:56PM -0600, Alex Williamson wrote: > Note that the kernel coding style, except for certain exceptions, is: > > /* > * This is a > * multi-line > * comment > */ Amen, the one true way. :-) Stefan signature.asc Description: PGP signature

Re: [Qemu-devel] [PATCH] hw/arm: Remove the deprecated xlnx-ep108 machine

2018-06-07 Thread Peter Maydell
On 5 June 2018 at 22:18, Thomas Huth wrote: > It has been marked as deprecated since QEMU v2.11, so it is time to > remove this now. The xlnx-zcu102 machine is very much the same and > can be used as a replacement instead. > > Signed-off-by: Thomas Huth Applied to target-arm.next, thanks. --

Re: [Qemu-devel] [PATCH] CODING_STYLE: Define our preferred form for multiline comments

2018-06-07 Thread Peter Maydell
On 7 June 2018 at 13:40, Stefan Hajnoczi wrote: > On Mon, Jun 04, 2018 at 07:17:56PM -0600, Alex Williamson wrote: >> Note that the kernel coding style, except for certain exceptions, is: >> >> /* >> * This is a >> * multi-line >> * comment >> */ > > Amen, the one true way. :-) OK,

Re: [Qemu-devel] [PATCH v12 2/4] i386: Verify if topoext feature can be supported

2018-06-07 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Wednesday, June 6, 2018 5:06 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qemu-devel@nongnu.org; >

[Qemu-devel] [PATCH v2 01/10] block: Move two block permission constants to the relevant enum

2018-06-07 Thread Ari Sundholm
This allows using the two constants outside of block.c, which will happen in a subsequent patch. Signed-off-by: Ari Sundholm --- block.c | 6 -- include/block/block.h | 7 +++ 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/block.c b/block.c index

[Qemu-devel] [PATCH v2 6/6] block/blklogwrites: Use the block device logical sector size when logging writes

2018-06-07 Thread Ari Sundholm
The guest OS may perform writes which are aligned to the logical sector size instead of the physical one, so logging at this granularity records the writes performed on the block device most faithfully. Signed-off-by: Ari Sundholm --- block/blklogwrites.c | 47

Re: [Qemu-devel] [PATCH v7 4/5] virtio-balloon: VIRTIO_BALLOON_F_FREE_PAGE_HINT

2018-06-07 Thread Wei Wang
On 06/07/2018 02:58 PM, Peter Xu wrote: On Thu, Jun 07, 2018 at 01:29:22PM +0800, Wei Wang wrote: [...] +static const VMStateDescription vmstate_virtio_balloon_free_page_report = { +.name = "virtio-balloon-device/free-page-report", +.version_id = 1, +.minimum_version_id = 1, +

Re: [Qemu-devel] [PATCH] CODING_STYLE: Define our preferred form for multiline comments

2018-06-07 Thread Markus Armbruster
Peter Maydell writes: > On 5 June 2018 at 08:46, Cornelia Huck wrote: >> On Tue, 5 Jun 2018 06:33:22 +0200 >> Thomas Huth wrote: >>> On 05.06.2018 03:17, Alex Williamson wrote: >>> > On Mon, 4 Jun 2018 17:21:40 +0100 >>> > Peter Maydell wrote: >>> >> +Multiline comments blocks should have a

Re: [Qemu-devel] [PATCH 1/6] hmp: Add flag for preconfig commands

2018-06-07 Thread Markus Armbruster
"Dr. David Alan Gilbert (git)" writes: > From: "Dr. David Alan Gilbert" > > Add a flag to command definitions to allow them to be used in preconfig > and check it. > > Signed-off-by: Dr. David Alan Gilbert > --- > monitor.c | 19 +++ > 1 file changed, 19 insertions(+) > > diff

[Qemu-devel] [PATCH v2 0/4] pc-bios/s390-ccw: Allow network booting via pxelinux.cfg

2018-06-07 Thread Thomas Huth
This patch series adds pxelinux.cfg-style network booting to the s390-ccw firmware. The core pxelinux.cfg loading and parsing logic has recently been merged to SLOF, so these patches now just have to make sure to call the right functions to get the config file loaded and parsed. Once this is done,

Re: [Qemu-devel] [PATCH 4/6] hmp: Add info commands for preconfig

2018-06-07 Thread Markus Armbruster
Peter Xu writes: > On Tue, Jun 05, 2018 at 01:26:34PM +0100, Dr. David Alan Gilbert (git) wrote: >> From: "Dr. David Alan Gilbert" >> >> Allow a bunch of the info commands to be used in preconfig. >> Could probably add most of them. > > I guess some of them may not work yet during preconfig.

[Qemu-devel] [PATCH v2 1/4] roms: Update SLOF submodule to current status

2018-06-07 Thread Thomas Huth
We need the latest version of SLOF's libnet for adding pxelinux.cfg support in the s390-ccw bios, too. Signed-off-by: Thomas Huth --- I assume that Alexey will send a SLOF update soon, too, which supplies the SLOF.bin file for these changes. roms/SLOF | 2 +- 1 file changed, 1 insertion(+),

Re: [Qemu-devel] [PATCH v2] vhost-user-blk: start vhost when guest kicks

2018-06-07 Thread Stefan Hajnoczi
On Wed, Jun 06, 2018 at 09:24:48PM +0800, Yongji Xie wrote: > Some old guests (before commit 7a11370e5: "virtio_blk: enable VQs early") > kick virtqueue before setting VIRTIO_CONFIG_S_DRIVER_OK. This violates > the virtio spec. But virtio 1.0 transitional devices support this behaviour. > So we

Re: [Qemu-devel] An emulation failure occurs, if I hotplug vcpus immediately after the VM start

2018-06-07 Thread David Hildenbrand
On 07.06.2018 14:36, Paolo Bonzini wrote: > On 07/06/2018 13:36, David Hildenbrand wrote: >>> The dirty bitmap would be synced in kvm_region_del (so it's not true >>> that kvm_region_del would disappear, but almost :)). >> I was rather concerned when doing a KVM_SET_USER_MEMORY_REGIONS while >>

Re: [Qemu-devel] [RFC v3] qapi: command category to stimulate high-level machine devices

2018-06-07 Thread Gerd Hoffmann
Hi, > I prefer the WebSocket route over creating a fake display that will not > be able to implement complex widgets well. Makes sense. > Gerd: What is your preference? Do you want board-specific fake displays > inside the QEMU process as the long-term direction for UIs? Well, it isn't the

Re: [Qemu-devel] storing machine data in qcow images?

2018-06-07 Thread Dr. David Alan Gilbert
* Daniel P. Berrangé (berra...@redhat.com) wrote: > On Thu, Jun 07, 2018 at 01:17:24PM +0200, Andrea Bolognani wrote: > > On Thu, 2018-06-07 at 11:22 +0100, Daniel P. Berrangé wrote: > > > On Thu, Jun 07, 2018 at 12:02:29PM +0200, Andrea Bolognani wrote: > > > > While hints might be considered a

Re: [Qemu-devel] [PATCH V6 0/7] nvdimm: guarantee persistence of QEMU writes to persistent memory

2018-06-07 Thread Stefan Hajnoczi
On Fri, Jun 01, 2018 at 01:27:14AM -0700, no-re...@patchew.org wrote: > /tmp/qemu-test/src/stubs/pmem.c: In function 'pmem_persist': > /tmp/qemu-test/src/stubs/pmem.c:23:1: error: control reaches end of non-void > function [-Werror=return-type] > } > ^ > cc1: all warnings being treated as

Re: [Qemu-devel] [PATCH 4/4] device_tree: Replace error_setg(_fatal) by error_report() + exit()

2018-06-07 Thread Markus Armbruster
Philippe Mathieu-Daudé writes: > Use error_report() + exit() instead of error_setg(_fatal), > as suggested by the "qapi/error.h" documentation: > >Please don't error_setg(_fatal, ...), use error_report() and >exit(), because that's more obvious. > > Suggested-by: Eric Blake >

Re: [Qemu-devel] storing machine data in qcow images?

2018-06-07 Thread Andrea Bolognani
On Thu, 2018-06-07 at 14:49 +0100, Dr. David Alan Gilbert wrote: > Including the nvram and efi makes me nervous; but I can see why together > they might work. However, there's no guarantee that EFI has been tested > with the QEMU it's used on and ... that could be trouble. If the QEMU binary

[Qemu-devel] [PATCH v2 10/10] block/blklogwrites: Use the block device logical sector size when logging writes

2018-06-07 Thread Ari Sundholm
The guest OS may perform writes which are aligned to the logical sector size instead of the physical one, so logging at this granularity records the writes performed on the block device most faithfully. Signed-off-by: Ari Sundholm --- block/blklogwrites.c | 47

Re: [Qemu-devel] [PATCH] slirp: Add Samuel Thibault's staging tree for slirp

2018-06-07 Thread Thomas Huth
On 06.06.2018 15:14, Samuel Thibault wrote: > Ping? I'm not sure who I am supposed to get a review from, or if I have > to have one at all? > > Samuel > > Samuel Thibault, le jeu. 31 mai 2018 21:48:43 +0200, a ecrit: >> Signed-off-by: Samuel Thibault >> --- >> MAINTAINERS | 1 + >> 1 file

Re: [Qemu-devel] [PATCH 4/6] hmp: Add info commands for preconfig

2018-06-07 Thread Markus Armbruster
"Dr. David Alan Gilbert" writes: > * Markus Armbruster (arm...@redhat.com) wrote: >> "Dr. David Alan Gilbert" writes: >> >> > * Markus Armbruster (arm...@redhat.com) wrote: >> >> Peter Xu writes: >> >> >> >> > On Tue, Jun 05, 2018 at 01:26:34PM +0100, Dr. David Alan Gilbert (git) >> >> >

Re: [Qemu-devel] [PATCH v2 6/6] block/blklogwrites: Use the block device logical sector size when logging writes

2018-06-07 Thread Ari Sundholm
Oops, it seems this old patch found its way in this series somehow. My apologies. And sorry for this top-post. Best regards, Ari Sundholm a...@tuxera.com On 06/07/2018 05:40 PM, Ari Sundholm wrote: The guest OS may perform writes which are aligned to the logical sector size instead of the

[Qemu-devel] [PATCH v2 06/10] hw/block/virtio-blk: Always apply block configuration to block driver

2018-06-07 Thread Ari Sundholm
This allows the block driver to use the block configuration of the new VirtIO block device. One use for this information is to set request limits using this information. Signed-off-by: Ari Sundholm --- hw/block/virtio-blk.c | 1 + 1 file changed, 1 insertion(+) diff --git

[Qemu-devel] [PATCH v2 1/4] hw/block/fdc: Replace error_setg(_abort) by assert()

2018-06-07 Thread Philippe Mathieu-Daudé
Use assert() instead of error_setg(_abort), as suggested by the "qapi/error.h" documentation: Please don't error_setg(_fatal, ...), use error_report() and exit(), because that's more obvious. Likewise, don't error_setg(_abort, ...), use assert(). Signed-off-by: Philippe Mathieu-Daudé

[Qemu-devel] [PATCH v2 00/10] New block driver: blklogwrites

2018-06-07 Thread Ari Sundholm
This patch series adds a new block driver, blklogwrites, to QEMU. The driver is given two block devices: a raw device backed by an image or a host block device, and a log device, typically backed by a file, on which writes to the raw device are logged. The logging format used is the same as in

Re: [Qemu-devel] [PATCH] gitignore: Ignore qapi/qapi-*-job.[ch]

2018-06-07 Thread Jeff Cody
On Thu, Jun 07, 2018 at 10:43:59AM -0400, Yaowei Bai wrote: > They were introduced by commit bf42508f24ee(job: Introduce > qapi/job.json) but forgot to ignore them in .gitignore. > > Signed-off-by: Yaowei Bai Good patch, but Eric Blake already sent an identical one that has two r-b's on it

Re: [Qemu-devel] [PATCH 4/6] hmp: Add info commands for preconfig

2018-06-07 Thread Markus Armbruster
"Dr. David Alan Gilbert" writes: > * Markus Armbruster (arm...@redhat.com) wrote: >> Peter Xu writes: >> >> > On Tue, Jun 05, 2018 at 01:26:34PM +0100, Dr. David Alan Gilbert (git) >> > wrote: >> >> From: "Dr. David Alan Gilbert" >> >> >> >> Allow a bunch of the info commands to be used in

[Qemu-devel] [PULL 6/7] 9p: Properly check/translate flags in unlinkat

2018-06-07 Thread Greg Kurz
From: Keno Fischer The 9p-local code previously relied on P9_DOTL_AT_REMOVEDIR and AT_REMOVEDIR having the same numerical value and deferred any errorchecking to the syscall itself. However, while the former assumption is true on Linux, it is not true in general. 9p-handle did this properly

[Qemu-devel] [PULL 0/7] 9p patches 2018-06-07

2018-06-07 Thread Greg Kurz
The following changes since commit 5d328d7d2f1fd4fb160bcfb6e4eb838720274438: Merge remote-tracking branch 'remotes/awilliam/tags/vfio-update-20180605.0' into staging (2018-06-07 08:59:28 +0100) are available in the Git repository at: https://github.com/gkurz/qemu.git tags/for-upstream for

Re: [Qemu-devel] [PATCH 4/6] hmp: Add info commands for preconfig

2018-06-07 Thread Dr. David Alan Gilbert
* Markus Armbruster (arm...@redhat.com) wrote: > "Dr. David Alan Gilbert" writes: > > > * Markus Armbruster (arm...@redhat.com) wrote: > >> Peter Xu writes: > >> > >> > On Tue, Jun 05, 2018 at 01:26:34PM +0100, Dr. David Alan Gilbert (git) > >> > wrote: > >> >> From: "Dr. David Alan Gilbert"

Re: [Qemu-devel] storing machine data in qcow images?

2018-06-07 Thread Dr. David Alan Gilbert
* Andrea Bolognani (abolo...@redhat.com) wrote: > On Thu, 2018-06-07 at 15:45 +0100, Dr. David Alan Gilbert wrote: > > * Andrea Bolognani (abolo...@redhat.com) wrote: > > > On Thu, 2018-06-07 at 14:49 +0100, Dr. David Alan Gilbert wrote: > > > > Including the nvram and efi makes me nervous; but I

[Qemu-devel] [PATCH v3 07/10] hw/block/nvme: Always apply block configuration to block driver

2018-06-07 Thread Ari Sundholm
This allows the block driver to use the block configuration of the new NVMe device. One use for this information is to set request limits using this information. Signed-off-by: Ari Sundholm --- hw/block/nvme.c | 1 + 1 file changed, 1 insertion(+) diff --git a/hw/block/nvme.c b/hw/block/nvme.c

[Qemu-devel] [PATCH v4 08/28] ppc/xive: introduce the XIVE Event Queues

2018-06-07 Thread Cédric Le Goater
The Event Queue Descriptor (EQD) table is an internal table of the XIVE routing sub-engine, the IVRE. It specifies on which Event Queue the Event Queue data should be posted when an exception occurs (and later on pulled by the OS) and which Virtual Processor to notify. The routing algorithm of

[Qemu-devel] [PATCH v1 2/2] exec: check that alignment is a power of two

2018-06-07 Thread David Hildenbrand
Right now we can crash QEMU using e.g. qemu-system-x86_64 -m 256M,maxmem=20G,slots=2 \ -object memory-backend-file,id=mem0,size=12288,mem-path=/dev/zero,align=12288 \ -device pc-dimm,id=dimm1,memdev=mem0 qemu-system-x86_64: util/mmap-alloc.c:115: qemu_ram_mmap: Assertion

[Qemu-devel] [PATCH v4 01/28] sparp_pci: simplify how the PCI LSIs are allocated

2018-06-07 Thread Cédric Le Goater
PCI LSIs are today allocated one by one using the IRQ alloc_block routine. Change the code sequence to first allocate a PCI_NUM_PINS block. It will help us providing a generic IRQ framework to the machine. Signed-off-by: Cédric Le Goater Reviewed-by: Greg Kurz Signed-off-by: Cédric Le Goater

  1   2   3   4   5   >