[Qemu-devel] [PATCH RFC 1/2] s390x/virtio-ccw: respond to READ_STATUS command

2015-09-11 Thread Cornelia Huck
From: Pierre Morel This patch adds the response to the READ_STATUS CCW command. Signed-off-by: Pierre Morel Signed-off-by: Cornelia Huck --- hw/s390x/virtio-ccw.c | 20 hw/s390x/virtio-ccw.h

[Qemu-devel] [PATCH RFC 2/2] s390x/virtio-ccw: set revision 2 as maximum revision number

2015-09-11 Thread Cornelia Huck
From: Pierre Morel We have everything needed for virtio-ccw revision 2 wired up now. Bump the maximum supported revision reported to the guest so they can make use of it. Signed-off-by: Pierre Morel Signed-off-by: Cornelia Huck

Re: [Qemu-devel] [PATCH v4 00/11] qemu-log, perfmap and other logging tweaks

2015-09-11 Thread Alex Bennée
Michael Tokarev writes: > 03.08.2015 12:14, Alex Bennée wrote: >> Hi, >> >> This is mostly just a re-base to keep current with master. I've added >> a couple of outstanding s-o-b and r-b tags. There are also two new >> patches in this series which seemed to be worth keeping

Re: [Qemu-devel] [PATCH 7/9] opengl: add egl-context.[ch] helpers

2015-09-11 Thread Marc-André Lureau
On Wed, Sep 9, 2015 at 1:20 PM, Gerd Hoffmann wrote: > Add helper functions to manage opengl contexts using egl. > > Signed-off-by: Gerd Hoffmann > --- > include/ui/egl-context.h | 14 ++ > ui/Makefile.objs | 1 + > ui/egl-context.c

Re: [Qemu-devel] [PATCH v5 08/14] blockjob: Simplify block_job_finish_sync

2015-09-11 Thread Max Reitz
On 07.09.2015 09:34, Fam Zheng wrote: > With job->completed and job->ret to replace BlockFinishData. > > Signed-off-by: Fam Zheng > --- > blockjob.c | 27 ++- > 1 file changed, 6 insertions(+), 21 deletions(-) Reviewed-by: Max Reitz

Re: [Qemu-devel] [PATCH v5 06/14] blockjob: Add .commit and .abort block job actions

2015-09-11 Thread Max Reitz
On 07.09.2015 09:34, Fam Zheng wrote: > Signed-off-by: Fam Zheng > --- > include/block/blockjob.h | 18 ++ > 1 file changed, 18 insertions(+) Reviewed-by: Max Reitz signature.asc Description: OpenPGP digital signature

Re: [Qemu-devel] I want to recover my QEMU password.

2015-09-11 Thread Stefan Weil
Am 11.09.2015 um 17:38 schrieb SkyKiDS: > Hi, there! > > Long long time ago, in 2010, I am a member of QEMU wiki, just had an > account, and contributed some news or corrected mis-spells. > But now, I forgot my password of the site, so hope to recover my password. > Anyone can help me? > > My

Re: [Qemu-devel] [Qemu-block] [PATCH v2 11/11] block: Only poll block layer fds in bdrv_aio_poll

2015-09-11 Thread Fam Zheng
On Fri, 09/11 11:54, Paolo Bonzini wrote: > > > On 11/09/2015 11:44, Fam Zheng wrote: > > > > > That would be a step back. Using GSource is useful because it lets > > > > > you integrate libraries such as GTK+. > > > > > > > > Can we move GTK to a separate GSource thread? > > > > > > I think

[Qemu-devel] [PULL 06/26] maint: remove unused include for assert.h

2015-09-11 Thread Michael Tokarev
From: "Daniel P. Berrange" A number of files were including assert.h but not using any of the functions it provides Signed-off-by: Daniel P. Berrange Reviewed-by: Markus Armbruster Signed-off-by: Michael Tokarev

[Qemu-devel] [PULL 09/26] maint: remove unused include for strings.h

2015-09-11 Thread Michael Tokarev
From: "Daniel P. Berrange" A number of files were including strings.h but not using any of the functions it provides Signed-off-by: Daniel P. Berrange Reviewed-by: Markus Armbruster Signed-off-by: Michael Tokarev

[Qemu-devel] [PULL 13/26] docs: fix a qga/qapi-schema.json comment

2015-09-11 Thread Michael Tokarev
From: Marc-André Lureau For consistency with the rest of the comment blocks. Signed-off-by: Marc-André Lureau Reviewed-by: Eric Blake Signed-off-by: Michael Tokarev --- qga/qapi-schema.json | 3 ++-

[Qemu-devel] [PATCH] checkpatch: Escape left braces in regex

2015-09-11 Thread Fam Zheng
Latest perl now deprecates "{" literal in regex and print warnings like "unescaped left brace in regex is deprecated". Add escape to keep it happy. Signed-off-by: Fam Zheng --- scripts/checkpatch.pl | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-)

[Qemu-devel] [PULL 26/26] virtio-vga: enable for i386

2015-09-11 Thread Michael Tokarev
From: Gerd Hoffmann This one just syncs x86_64 and i386. Cc: Paolo Bonzini Cc: qemu-triv...@nongnu.org Signed-off-by: Gerd Hoffmann Signed-off-by: Michael Tokarev --- default-configs/i386-softmmu.mak | 1 + 1 file

[Qemu-devel] [PULL 24/26] hw/gpio/zaurus: Remove meaningless blank Property

2015-09-11 Thread Michael Tokarev
From: Shannon Zhao Signed-off-by: Shannon Zhao Signed-off-by: Shannon Zhao Signed-off-by: Michael Tokarev --- hw/gpio/zaurus.c | 5 - 1 file changed, 5 deletions(-) diff --git a/hw/gpio/zaurus.c

[Qemu-devel] [PULL 03/26] i6300esb: fix timer overflow

2015-09-11 Thread Michael Tokarev
From: Laurent Vivier We use muldiv64() to compute the time to wait: timeout = muldiv64(get_ticks_per_sec(), timeout, 3300); but get_ticks_per_sec() is 10^9 (30 bit value) and timeout is a 35 bit value. Whereas muldiv64 is: uint64_t muldiv64(uint64_t a,

[Qemu-devel] [PATCH PULL 03/11] crypto: move crypto objects out of libqemuutil.la

2015-09-11 Thread Daniel P. Berrange
Future patches will be adding more crypto related APIs which rely on QOM infrastructure. This creates a problem, because QOM relies on library constructors to register objects. When you have a file in a static .a library though which is only referenced by a constructor the linker is dumb and will

[Qemu-devel] [PATCH PULL 02/11] tests: remove repetition in unit test object deps

2015-09-11 Thread Daniel P. Berrange
Most of the unit tests have identical sets of object deps. For example all block unit tests need to depend on $(block-obj-y) libqemuutil.a libqemustub.a Currently each unit test repeats this list of test deps. This list of deps will grow as future patches add more modules to the build, so

[Qemu-devel] [PATCH PULL 01/11] qapi: allow override of default enum prefix naming

2015-09-11 Thread Daniel P. Berrange
The camel_to_upper() method applies some heuristics to turn a mixed case type name into an all-uppercase name. This is used for example, to generate enum constant name prefixes. The heuristics don't also generate a satisfactory name though. eg { 'enum': 'QCryptoTLSCredsEndpoint', 'data':

[Qemu-devel] [PATCH PULL 05/11] crypto: introduce new base module for TLS credentials

2015-09-11 Thread Daniel P. Berrange
Introduce a QCryptoTLSCreds class to act as the base class for storing TLS credentials. This will be later subclassed to provide handling of anonymous and x509 credential types. The subclasses will be user creatable objects, so instances can be created & deleted via 'object-add' and 'object-del'

[Qemu-devel] [PATCH PULL 07/11] crypto: introduce new module for TLS x509 credentials

2015-09-11 Thread Daniel P. Berrange
Introduce a QCryptoTLSCredsX509 class which is used to manage x509 certificate TLS credentials. This will be the preferred credential type offering strong security characteristics Example CLI configuration: $QEMU -object tls-creds-x509,id=tls0,endpoint=server,\

[Qemu-devel] [PATCH PULL 06/11] crypto: introduce new module for TLS anonymous credentials

2015-09-11 Thread Daniel P. Berrange
Introduce a QCryptoTLSCredsAnon class which is used to manage anonymous TLS credentials. Use of this class is generally discouraged since it does not offer strong security, but it is required for backwards compatibility with the current VNC server implementation. Simple example CLI configuration:

[Qemu-devel] [PATCH PULL 04/11] qom: allow QOM to be linked into tools binaries

2015-09-11 Thread Daniel P. Berrange
The qom objects are currently added to common-obj-y which is only linked into the system emulators. The later crypto patches will depend on QOM infrastructure and will also be used from tools binaries. Thus the QOM objects are moved into a new qom-obj-y variable which can be referenced when

[Qemu-devel] [PATCH PULL 00/11] Extract TLS handling code from VNC server

2015-09-11 Thread Daniel P. Berrange
The following changes since commit 7b9c09f7d486647784c605739d69b708a7249c9b: Merge remote-tracking branch 'remotes/sstabellini/tags/xen-2015-09-10-tag' into staging (2015-09-10 18:25:52 +0100) are available in the git repository at: git://github.com/berrange/qemu.git

[Qemu-devel] [PATCH PULL 11/11] ui: convert VNC server to use QCryptoTLSSession

2015-09-11 Thread Daniel P. Berrange
Switch VNC server over to using the QCryptoTLSSession object for the TLS session. This removes the direct use of gnutls from the VNC server code. It also removes most knowledge about TLS certificate handling from the VNC server code. This has the nice effect that all the CONFIG_VNC_TLS

Re: [Qemu-devel] [PATCH 18/20] tcg: Save insn data and use it in cpu_restore_state_from_tb

2015-09-11 Thread Sergey Fedorov
On 10.09.2015 16:49, Peter Maydell wrote: >> @@ -2406,6 +2411,8 @@ static inline int tcg_gen_code_common(TCGContext *s, >> > check_regs(s); >> > #endif >> > } >> > +tcg_debug_assert(num_insns >= 0); > This is claiming that every TB will have at least one insn_start, > right? I

Re: [Qemu-devel] [Qemu-block] [PATCH v2 11/11] block: Only poll block layer fds in bdrv_aio_poll

2015-09-11 Thread Paolo Bonzini
On 11/09/2015 13:01, Fam Zheng wrote: > On Fri, 09/11 12:46, Paolo Bonzini wrote: >> >> >> On 11/09/2015 12:40, Fam Zheng wrote: >>> On Fri, 09/11 11:54, Paolo Bonzini wrote: On 11/09/2015 11:44, Fam Zheng wrote: That would be a step back. Using GSource is useful because

Re: [Qemu-devel] [Qemu-block] [PATCH v2 11/11] block: Only poll block layer fds in bdrv_aio_poll

2015-09-11 Thread Fam Zheng
On Fri, 09/11 12:46, Paolo Bonzini wrote: > > > On 11/09/2015 12:40, Fam Zheng wrote: > > On Fri, 09/11 11:54, Paolo Bonzini wrote: > >> > >> > >> On 11/09/2015 11:44, Fam Zheng wrote: > >> That would be a step back. Using GSource is useful because it lets > >> you integrate libraries

[Qemu-devel] [PULL 15/26] linux-user: Fix warnings caused by missing 'static' attribute

2015-09-11 Thread Michael Tokarev
From: Stefan Weil Warnings from the Sparse static analysis tool: linux-user/main.c:40:12: warning: symbol 'filename' was not declared. Should it be static? linux-user/main.c:41:12: warning: symbol 'argv0' was not declared. Should it be static? linux-user/main.c:42:5:

[Qemu-devel] [PULL 10/26] maint: avoid useless "if (foo) free(foo)" pattern

2015-09-11 Thread Michael Tokarev
From: "Daniel P. Berrange" The free() and g_free() functions both happily accept NULL on any platform QEMU builds on. As such putting a conditional 'if (foo)' check before calls to 'free(foo)' merely serves to bloat the lines of code. Signed-off-by: Daniel P. Berrange

[Qemu-devel] [PULL 17/26] Target-ppc: Remove unnecessary variable

2015-09-11 Thread Michael Tokarev
From: Shraddha Barke Compress lines and remove the variable ret. Change made using Coccinelle script @@ expression ret; @@ - if (ret) return ret; - return 0; + return ret; @@ local idexpression ret; expression e; @@ - ret = e; - return ret; + return e; @@ type T;

[Qemu-devel] [PULL 25/26] hw/arm/spitz: Remove meaningless blank Property

2015-09-11 Thread Michael Tokarev
From: Shannon Zhao Signed-off-by: Shannon Zhao Signed-off-by: Shannon Zhao Signed-off-by: Michael Tokarev --- hw/arm/spitz.c | 5 - 1 file changed, 5 deletions(-) diff --git a/hw/arm/spitz.c

Re: [Qemu-devel] [PATCH v3] ppc/spapr: Implement H_RANDOM hypercall in QEMU

2015-09-11 Thread Eric Blake
On 09/11/2015 03:17 AM, Thomas Huth wrote: > The PAPR interface defines a hypercall to pass high-quality > hardware generated random numbers to guests. Recent kernels can > already provide this hypercall to the guest if the right hardware > random number generator is available. But in case the

[Qemu-devel] [PATCH PULL 10/11] ui: fix return type for VNC I/O functions to be ssize_t

2015-09-11 Thread Daniel P. Berrange
Various VNC server I/O functions return 'long' and then also pass this to a method accepting 'int'. All these should be ssize_t to match the signature of read/write APIs and thus avoid potential for integer truncation / wraparound. Signed-off-by: Daniel P. Berrange

Re: [Qemu-devel] [Qemu-block] [PATCH v2 11/11] block: Only poll block layer fds in bdrv_aio_poll

2015-09-11 Thread Paolo Bonzini
On 11/09/2015 12:40, Fam Zheng wrote: > On Fri, 09/11 11:54, Paolo Bonzini wrote: >> >> >> On 11/09/2015 11:44, Fam Zheng wrote: >> That would be a step back. Using GSource is useful because it lets >> you integrate libraries such as GTK+. > > Can we move GTK to a separate

Re: [Qemu-devel] [PATCH 18/20] tcg: Save insn data and use it in cpu_restore_state_from_tb

2015-09-11 Thread Sergey Fedorov
On 11.09.2015 13:32, Peter Maydell wrote: > On 11 September 2015 at 11:29, Sergey Fedorov wrote: >> On 10.09.2015 16:49, Peter Maydell wrote: @@ -2406,6 +2411,8 @@ static inline int tcg_gen_code_common(TCGContext *s, > check_regs(s); > #endif >

[Qemu-devel] [PULL 21/26] typofixes - v4

2015-09-11 Thread Michael Tokarev
From: Veres Lajos Signed-off-by: Veres Lajos Signed-off-by: Michael Tokarev --- disas/i386.c| 2 +- disas/s390.c| 4 ++-- docs/specs/ppc-spapr-hcalls.txt | 4 ++-- docs/writing-qmp-commands.txt | 2

Re: [Qemu-devel] [PATCH v2 00/11] aio: Introduce handler type to fix nested aio_poll for dataplane

2015-09-11 Thread Fam Zheng
On Fri, 09/11 12:39, Kevin Wolf wrote: > Am 29.07.2015 um 06:42 hat Fam Zheng geschrieben: > > v2: Switch to disable/enable model. [Paolo] > > > > Most existing nested aio_poll()'s in block layer are inconsiderate of > > dispatching potential new r/w requests from ioeventfds and nbd exports, > >

Re: [Qemu-devel] [PATCH RFC 0/3] qapi: more introspection tweaks

2015-09-11 Thread Marc-André Lureau
Hi On Fri, Sep 11, 2015 at 12:59 AM, Eric Blake wrote: > http://repo.or.cz/qemu/ericb.git/shortlog/refs/heads/qapi > git fetch git://repo.or.cz/qemu/ericb.git qapi My pending qapi patches are no rebased on your branch (https://github.com/elmarco/qemu qapi branch). I dropped

[Qemu-devel] [PATCH PULL 08/11] crypto: add sanity checking of TLS x509 credentials

2015-09-11 Thread Daniel P. Berrange
If the administrator incorrectly sets up their x509 certificates, the errors seen at runtime during connection attempts are very obscure and difficult to diagnose. This has been a particular problem for people using openssl to generate their certificates instead of the gnutls certtool, because the

[Qemu-devel] [PATCH PULL 09/11] crypto: introduce new module for handling TLS sessions

2015-09-11 Thread Daniel P. Berrange
Introduce a QCryptoTLSSession object that will encapsulate all the code for setting up and using a client/sever TLS session. This isolates the code which depends on the gnutls library, avoiding #ifdefs in the rest of the codebase, as well as facilitating any possible future port to other TLS

Re: [Qemu-devel] [PATCH v2 00/11] aio: Introduce handler type to fix nested aio_poll for dataplane

2015-09-11 Thread Kevin Wolf
Am 29.07.2015 um 06:42 hat Fam Zheng geschrieben: > v2: Switch to disable/enable model. [Paolo] > > Most existing nested aio_poll()'s in block layer are inconsiderate of > dispatching potential new r/w requests from ioeventfds and nbd exports, which > might result in responsiveness issues (e.g.

Re: [Qemu-devel] [PATCH 18/20] tcg: Save insn data and use it in cpu_restore_state_from_tb

2015-09-11 Thread Peter Maydell
On 11 September 2015 at 11:29, Sergey Fedorov wrote: > On 10.09.2015 16:49, Peter Maydell wrote: >>> @@ -2406,6 +2411,8 @@ static inline int tcg_gen_code_common(TCGContext *s, >>> > check_regs(s); >>> > #endif >>> > } >>> > +tcg_debug_assert(num_insns >=

Re: [Qemu-devel] [PATCH 01/34] linux-user: Fix array bounds in errno conversion

2015-09-11 Thread Peter Maydell
On 6 September 2015 at 00:56, Timothy E Baldwin wrote: > Check array bounds in host_to_target_errno() and target_to_host_errno(). > > Signed-off-by: Timothy Edward Baldwin > --- > linux-user/syscall.c | 4 ++-- > 1 file

Re: [Qemu-devel] [PATCH] qom/object.h: remove some child/parent doc

2015-09-11 Thread Andreas Färber
Am 11.09.2015 um 12:13 schrieb Michael Tokarev: > 07.09.2015 18:46, Andreas Färber wrote: >> Am 04.09.2015 um 23:01 schrieb marcandre.lur...@redhat.com: >>> From: Marc-André Lureau >>> >>> It looks like this documentation is obsolete: a child object may lookup >>> its

[Qemu-devel] [PULL 04/26] maint: remove double semicolons in many files

2015-09-11 Thread Michael Tokarev
From: "Daniel P. Berrange" A number of source files have statements accidentally terminated by a double semicolon - eg 'foo = bar;;'. This is harmless but a mistake none the less. The tcg/ia64/tcg-target.c file is whitelisted because it has valid use of ';;' in a comment

[Qemu-devel] [PULL 20/26] qapi-schema: remove legacy<> from doc

2015-09-11 Thread Michael Tokarev
From: Marc-André Lureau The legacy<> type is no longer used since 7ce7ffe02. Signed-off-by: Marc-André Lureau Reviewed-by: Eric Blake Signed-off-by: Michael Tokarev --- qapi-schema.json | 15

[Qemu-devel] [PULL 12/26] trivial: remove trailing newline from error_report

2015-09-11 Thread Michael Tokarev
From: John Snow Minor cleanup. Signed-off-by: John Snow Reviewed-by: Gonglei Reviewed-by: Stefan Hajnoczi Signed-off-by: Michael Tokarev --- hw/ide/ahci.c | 2 +- hw/vfio/platform.c | 2

[Qemu-devel] [PULL 11/26] maint: avoid useless "if (foo) free(foo)" pattern

2015-09-11 Thread Michael Tokarev
From: Markus Armbruster My Coccinelle semantic patch finds a few more, because it also fixes up the equally pointless conditional if (foo) { free(foo); foo = NULL; } Result (feel free to squash it into your patch): Signed-off-by: Markus Armbruster

[Qemu-devel] [PATCH 04/29] Introduce QDict

2015-09-11 Thread Programmingkid
Could you make a tutorial on how to use the QDict type?

Re: [Qemu-devel] [PATCH V3 1/2] hw/sd : move sd.h to include/hw/sd/

2015-09-11 Thread Alistair Francis
On Thu, Sep 10, 2015 at 4:25 AM, Sai Pavan Boddu wrote: > Create new folder in includes for sd headers. I'd change the commit message to something like: sd.h: Move sd.h to include/hw/sd/ Create a sd directory under include/hw/ and move sd.h to that folder. > >

Re: [Qemu-devel] [PATCH V3 2/2] sdhci: Split sdhci.h for public and internal device usage

2015-09-11 Thread Alistair Francis
On Thu, Sep 10, 2015 at 4:25 AM, Sai Pavan Boddu wrote: > Split sdhci.h into sdhci-common.h and sdhci.h separating common > register declarations and object declarations. And move sdhci.h to > include/hw/sd/. > > Signed-off-by: Sai Pavan Boddu >

Re: [Qemu-devel] [PATCH 04/29] Introduce QDict

2015-09-11 Thread Luiz Capitulino
On Fri, 11 Sep 2015 20:22:38 -0400 Programmingkid wrote: > Could you make a tutorial on how to use the QDict type? There are several examples in tests/check-qdict.c.

[Qemu-devel] [PULL 22/26] hw/s390x/s390-virtio-bus: Remove meaningless blank Property

2015-09-11 Thread Michael Tokarev
From: Shannon Zhao Signed-off-by: Shannon Zhao Signed-off-by: Shannon Zhao Acked-by: Cornelia Huck Signed-off-by: Michael Tokarev --- hw/s390x/s390-virtio-bus.c | 5 -

[Qemu-devel] [PULL 08/26] maint: remove unused include for signal.h

2015-09-11 Thread Michael Tokarev
From: "Daniel P. Berrange" A number of files were including signal.h but not using any of the functions it provides Signed-off-by: Daniel P. Berrange Reviewed-by: Markus Armbruster Signed-off-by: Michael Tokarev

Re: [Qemu-devel] [RFC PATCH] spapr: Reduce creation of LMB DR connectors from O(n^3) to O(n^2)

2015-09-11 Thread Paolo Bonzini
On 10/09/2015 08:28, David Gibson wrote: > The dynamic reconfiguration (hotplug) code for the pseries machine type > uses a "DR connector" QOM object for each resource it will be possible > to hotplug. Each of these is added to its owner using > object_property_add_child(owner,

[Qemu-devel] [PATCH 3/7] chardev: add a scenario engine backend

2015-09-11 Thread Victor CLEMENT
This scenario backend implements the scenario interaction API for all character device emulators. It provides a callback registration function which registers a chr_write callback. It will call the user defined callback on incoming data. To write to this character device, qemu_chr_be_write can be

[Qemu-devel] [PATCH 4/7] scenario-engine: add utilities

2015-09-11 Thread Victor CLEMENT
This scenario utilities file currently implements one function. This function is the event callback registration function which walks through all guest devices to find those of a given type then registers their event callback. Signed-off-by: Victor CLEMENT ---

[Qemu-devel] [PATCH 1/7] configure: add --enable-scenario-engine option

2015-09-11 Thread Victor CLEMENT
Add an option to ./configure script to enable the scenario engine. It defines the CONFIG_SCENARIO constant. Signed-off-by: Victor CLEMENT --- configure | 8 1 file changed, 8 insertions(+) diff --git a/configure b/configure index cd219d8..466a71f 100755 ---

[Qemu-devel] [PATCH 2/7] gpio-pl061: add a scenario engine interaction API

2015-09-11 Thread Victor CLEMENT
Add a scenario API to interact with guest GPIOs. It implements read and write functions in order to check or modify the guest GPIO data register from the scenario test file. It also provides an event notification callback to the scenario engine by calling a user-defined handler after each

Re: [Qemu-devel] [PATCH PULL 00/11] Extract TLS handling code from VNC server

2015-09-11 Thread Daniel P. Berrange
On Fri, Sep 11, 2015 at 06:38:06AM -0600, Eric Blake wrote: > On 09/11/2015 06:10 AM, Daniel P. Berrange wrote: > > The following changes since commit 7b9c09f7d486647784c605739d69b708a7249c9b: > > > > Merge remote-tracking branch > > 'remotes/sstabellini/tags/xen-2015-09-10-tag' into staging

[Qemu-devel] [PATCH v2 3/4] virtio-ccw: feature bits > 31 handling

2015-09-11 Thread Cornelia Huck
We currently switch off the VERSION_1 feature bit if the guest has not negotiated at least revision 1. As no feature bits beyond 31 are valid however unless VERSION_1 has been negotiated, make sure that legacy guests never see a feature bit beyond 31. Reviewed-by: David Hildenbrand

[Qemu-devel] [PATCH v2 4/4] virtio-ccw: enable virtio-1

2015-09-11 Thread Cornelia Huck
Let's enable revision 1 for virtio-ccw devices. We can always offer VERSION_1 as drivers in legacy mode won't be able to see it anyway. We have to introduce a way to set a lower maximum revision for a device to accommodate the following cases: - compat machines (to enforce legacy only) -

[Qemu-devel] [PATCH v2 2/4] virtio-ccw: support ring size changes

2015-09-11 Thread Cornelia Huck
Wire up changing the ring size for virtio-1 devices. Signed-off-by: Cornelia Huck --- hw/s390x/virtio-ccw.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/hw/s390x/virtio-ccw.c b/hw/s390x/virtio-ccw.c index d36373e..85e2a5d 100644

[Qemu-devel] [PULL 14/26] opts: produce valid command line in qemu_opts_print

2015-09-11 Thread Michael Tokarev
From: Kővágó, Zoltán This will let us print options in a format that the user would actually write it on the command line (foo=bar,baz=asd,etc=def), without prepending a spurious comma at the beginning of the list, or quoting values unnecessarily. This patch provides the

Re: [Qemu-devel] [PATCH v3 0/2] update-linux-headers changes

2015-09-11 Thread Denis V. Lunev
On 09/10/2015 12:39 PM, Paolo Bonzini wrote: Patches 1 and 2 updates the Linux headers, which required an unexpected change to hw/pci/pcie_aer.c. Patches 3 and 4 modify the update-linux-headers script so that asm-x86/hyperv.h can be moved to standard-headers. This is required by the Hyper-V

Re: [Qemu-devel] [PATCH PULL 00/11] Extract TLS handling code from VNC server

2015-09-11 Thread Eric Blake
On 09/11/2015 06:10 AM, Daniel P. Berrange wrote: > The following changes since commit 7b9c09f7d486647784c605739d69b708a7249c9b: > > Merge remote-tracking branch 'remotes/sstabellini/tags/xen-2015-09-10-tag' > into staging (2015-09-10 18:25:52 +0100) > > are available in the git repository

Re: [Qemu-devel] [PATCH PULL 01/11] qapi: allow override of default enum prefix naming

2015-09-11 Thread Eric Blake
On 09/11/2015 06:10 AM, Daniel P. Berrange wrote: > The camel_to_upper() method applies some heuristics to turn > a mixed case type name into an all-uppercase name. This is > used for example, to generate enum constant name prefixes. > > The heuristics don't also generate a satisfactory name >

[Qemu-devel] [PATCH 5/7] scenario-engine: add a time based event scheduler

2015-09-11 Thread Victor CLEMENT
The scheduler reads an event description file containing timestamps and parameters then schedules those events. A user defined function is called when those events expire. It is used for precise time simulations. Signed-off-by: Victor CLEMENT --- Makefile.objs

Re: [Qemu-devel] [PATCH PULL 01/11] qapi: allow override of default enum prefix naming

2015-09-11 Thread Daniel P. Berrange
On Fri, Sep 11, 2015 at 06:37:08AM -0600, Eric Blake wrote: > On 09/11/2015 06:10 AM, Daniel P. Berrange wrote: > > The camel_to_upper() method applies some heuristics to turn > > a mixed case type name into an all-uppercase name. This is > > used for example, to generate enum constant name

Re: [Qemu-devel] [PATCH 4/4] virtio-ccw: enable virtio-1

2015-09-11 Thread Cornelia Huck
On Fri, 4 Sep 2015 10:54:29 +0200 Cornelia Huck wrote: > Let's enable revision 1 for virtio-ccw devices. We can always offer > VERSION_1 as drivers in legacy mode won't be able to see it anyway. > > We have to introduce a way to set a lower maximum revision for a

[Qemu-devel] [PATCH v2 0/4] virtio-1/virtio-ccw related patches

2015-09-11 Thread Cornelia Huck
v2 of my recent virtio patches. Patch 1 is the stateless version I sent out before. Patch 4 replaces the broken version I sent out previously. R-bs have been added. I can take patches 2-4 through the s390x tree. Cornelia Huck (4): virtio: ring sizes vs. reset virtio-ccw: support ring size

Re: [Qemu-devel] [ANNOUNCE] QEMU 2.3.1 Stable released

2015-09-11 Thread Sona Sarmadi
Hi Michael, > I am pleased to announce that the QEMU v2.3.1 stable release is now available > at: > > http://wiki.qemu.org/download/qemu-2.3.1.tar.bz2 > > v2.3.1 is now tagged in the official qemu.git repository, and the stable-2.3 > branch has been updated accordingly: > >

[Qemu-devel] [PATCH v2 1/4] virtio: ring sizes vs. reset

2015-09-11 Thread Cornelia Huck
We allow guests to change the size of the virtqueue rings by supplying a number of buffers that is different from the number of buffers the device was initialized with. Current code has some problems, however, since reset does not reset the ringsizes to the default values (as this is not saved

[Qemu-devel] [PULL 02/26] Trivial: fix commandline help message

2015-09-11 Thread Michael Tokarev
From: Andrey Korolyov Fix obvious typo in printed help for qemu-nbd. Signed-off-by: Andrey Korolyov Signed-off-by: Michael Tokarev --- qemu-nbd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qemu-nbd.c b/qemu-nbd.c index

Re: [Qemu-devel] [PATCH v2 00/11] aio: Introduce handler type to fix nested aio_poll for dataplane

2015-09-11 Thread Kevin Wolf
Am 11.09.2015 um 13:46 hat Fam Zheng geschrieben: > On Fri, 09/11 12:39, Kevin Wolf wrote: > > Am 29.07.2015 um 06:42 hat Fam Zheng geschrieben: > > > v2: Switch to disable/enable model. [Paolo] > > > > > > Most existing nested aio_poll()'s in block layer are inconsiderate of > > > dispatching

[Qemu-devel] [PULL 19/26] disas/microblaze: Remove unused code

2015-09-11 Thread Michael Tokarev
From: Stefan Weil Signed-off-by: Stefan Weil Reviewed-by: Edgar E. Iglesias Signed-off-by: Michael Tokarev --- disas/microblaze.c | 150 - 1 file changed, 150

[Qemu-devel] [PATCH v4] monitor: allow device_del to accept QOM paths

2015-09-11 Thread Daniel P. Berrange
Currently device_del requires that the client provide the device short ID. device_add allows devices may be created without giving an ID, at which point there is no way to delete them with device_del. The QOM object path, however, provides an alternative way to identify the devices. Allowing

Re: [Qemu-devel] [PATCH v4] monitor: allow device_del to accept QOM paths

2015-09-11 Thread Eric Blake
On 09/11/2015 06:33 AM, Daniel P. Berrange wrote: > Currently device_del requires that the client provide the > device short ID. device_add allows devices may be created > without giving an ID, at which point there is no way to > delete them with device_del. The QOM object path, however, >

[Qemu-devel] [PATCH 7/7] scenario engine: add a Qemu option to start it

2015-09-11 Thread Victor CLEMENT
Add an option to Qemu in order to start the scenario engine at the end of Qemu initialization. Signed-off-by: Victor CLEMENT --- qemu-options.hx | 12 vl.c| 45 - 2 files changed, 56 insertions(+),

[Qemu-devel] [PATCH 0/7] Qemu scenario engine

2015-09-11 Thread Victor CLEMENT
This patch introduce a testing component for Qemu guest OS: the scenario engine. The Qemu scenario engine is a framework designed to test the software running on the guest machine. It allows interacting with the guest virtual devices by stimulating inputs and reading outputs from outside of the

[Qemu-devel] [PATCH 6/7] scenario engine: provide a scenario file template

2015-09-11 Thread Victor CLEMENT
This template implements the mandatory functions of the main scenario source file and provides an example use case. Signed-off-by: Victor CLEMENT --- Makefile.objs | 1 + include/scenario/scenario.h | 5 ++ scenario/scenario.c | 168

Re: [Qemu-devel] [PATCH 0/7] Qemu scenario engine

2015-09-11 Thread Andreas Färber
Am 11.09.2015 um 14:50 schrieb Victor CLEMENT: > This patch introduce a testing component for Qemu guest OS: the scenario > engine. > > The Qemu scenario engine is a framework designed to test the software running > on the guest machine. It allows interacting with the guest virtual devices by >

Re: [Qemu-devel] [PATCH 3/7] chardev: add a scenario engine backend

2015-09-11 Thread Eric Blake
On 09/11/2015 06:50 AM, Victor CLEMENT wrote: > This scenario backend implements the scenario interaction API for all > character device emulators. > It provides a callback registration function which registers a chr_write > callback. It will call the user defined callback on incoming data. > To

Re: [Qemu-devel] [PATCH] Makefile: suppress command echoing in silent mode (make -s)

2015-09-11 Thread Jérôme Forissier
On 09/11/2015 09:15 AM, Michael Tokarev wrote: > 02.09.2015 12:14, Jerome Forissier wrote: >> Signed-off-by: Jerome Forissier > > Please Cc qemu-devel on patches. OK, sorry. I wrongly assumed that trivial patches were to be sent to qemu-trivial only to avoid

Re: [Qemu-devel] [PATCH 33/34] linux-user: Support for restarting system calls for CRIS targets

2015-09-11 Thread Peter Maydell
On 11 September 2015 at 15:18, Edgar E. Iglesias wrote: > On Thu, Sep 10, 2015 at 08:12:21PM +0100, Peter Maydell wrote: >> On 6 September 2015 at 00:57, Timothy E Baldwin >> wrote: >> > Signed-off-by: Timothy Edward Baldwin

Re: [Qemu-devel] [PATCH v2] ui/cocoa.m: Add Mount image file menu item

2015-09-11 Thread Programmingkid
On Sep 11, 2015, at 3:40 AM, Markus Armbruster wrote: > Only reviewing around qmp_device_add() and such, ignoring the GUI part. > > Programmingkid writes: > >> Add "Mount Image File..." and a "Eject Image File" menu items to >> cocoa interface. This patch makes

Re: [Qemu-devel] [PATCH 33/34] linux-user: Support for restarting system calls for CRIS targets

2015-09-11 Thread Edgar E. Iglesias
On Fri, Sep 11, 2015 at 03:20:31PM +0100, Peter Maydell wrote: > On 11 September 2015 at 15:18, Edgar E. Iglesias > wrote: > > On Thu, Sep 10, 2015 at 08:12:21PM +0100, Peter Maydell wrote: > >> On 6 September 2015 at 00:57, Timothy E Baldwin > >>

Re: [Qemu-devel] [PATCH 18/34] linux-user: Fix race between multiple signals

2015-09-11 Thread Peter Maydell
On 6 September 2015 at 00:57, Timothy E Baldwin wrote: > If multiple host signals are recieved in quick succession they would > be queued in TaskState then delivered to the guest in spite of > signals being blocked. Fixed by keeping host signals blocked until >

Re: [Qemu-devel] [PATCH 8/9] gtk/opengl: add opengl context and scanout support (egl)

2015-09-11 Thread Marc-André Lureau
On Wed, Sep 9, 2015 at 1:20 PM, Gerd Hoffmann wrote: > This allows virtio-gpu to render in 3d mode. > Uses egl, for gtk versions 3.14 and older. > > Signed-off-by: Gerd Hoffmann > --- > include/ui/gtk.h | 16 +++ > ui/gtk-egl.c | 129 >

Re: [Qemu-devel] [PATCH 27/34] linux-user: pause() should not pause if signal pending

2015-09-11 Thread Peter Maydell
On 6 September 2015 at 00:57, Timothy E Baldwin wrote: > Signed-off-by: Timothy Edward Baldwin > --- > linux-user/syscall.c | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/linux-user/syscall.c

Re: [Qemu-devel] [PATCH 2/2] PCI-e device multi-function hot-add support

2015-09-11 Thread Igor Mammedov
On Thu, 10 Sep 2015 20:12:23 +0800 Cao jin wrote: > In case user regret when hot-add multi-function, we should roll back, > device_del the function added but still not worked. > > Signed-off-by: Cao jin > --- > hw/pci/pcie.c | 28

Re: [Qemu-devel] [PATCH 24/34] linux-user: Restart execve() if signal pending

2015-09-11 Thread Peter Maydell
On 6 September 2015 at 00:57, Timothy E Baldwin wrote: > Without this SIGTERM could fail to terminate the process, as the > signal lost in QEMU's queue. > > Signed-off-by: Timothy Edward Baldwin > --- > linux-user/syscall.c |

Re: [Qemu-devel] [PATCH v1 04/10] target-arm: Add VTCR_EL2

2015-09-11 Thread Edgar E. Iglesias
On Tue, Sep 08, 2015 at 03:19:37PM +0100, Peter Maydell wrote: > On 3 September 2015 at 21:14, Edgar E. Iglesias > wrote: > > From: "Edgar E. Iglesias" > > > > Signed-off-by: Edgar E. Iglesias > > --- > >

Re: [Qemu-devel] [PULL v1 00/21] Multi-arch queue

2015-09-11 Thread Peter Crosthwaite
On Fri, Sep 11, 2015 at 2:31 AM, Peter Maydell wrote: > On 11 September 2015 at 06:44, Peter Crosthwaite > wrote: >> Flush of multi-arch pre-requisite work. >> >> >> multi-arch queue: >> * add EM_MOXIE

[Qemu-devel] [PATCH 2/4] Fix bad error handling after memory_region_init_ram()

2015-09-11 Thread Markus Armbruster
Symptom: $ qemu-system-x86_64 -m 1000 Unexpected error in ram_block_add() at /work/armbru/qemu/exec.c:1456: upstream-qemu: cannot set up guest memory 'pc.ram': Cannot allocate memory Aborted (core dumped) Root cause: commit ef701d7 screwed up handling of out-of-memory

[Qemu-devel] [PATCH 4/4] memory: Fix bad error handling in memory_region_init_ram_ptr()

2015-09-11 Thread Markus Armbruster
Commit ef701d7 screwed up handling of out-of-memory conditions. Before the commit, we report the error and exit(1), in one place. The commit lifts the error handling up the call chain some, to three places. Fine. Except it uses _abort in these places, changing the behavior from exit(1) to

Re: [Qemu-devel] [PATCH 33/34] linux-user: Support for restarting system calls for CRIS targets

2015-09-11 Thread Edgar E. Iglesias
On Thu, Sep 10, 2015 at 08:12:21PM +0100, Peter Maydell wrote: > On 6 September 2015 at 00:57, Timothy E Baldwin > wrote: > > Signed-off-by: Timothy Edward Baldwin > > --- > > > > WARNING - NOT TESTED > > > >

Re: [Qemu-devel] [PATCH 19/34] linux-user: Restart fork() if signals pending

2015-09-11 Thread Peter Maydell
On 6 September 2015 at 00:57, Timothy E Baldwin wrote: > If there is a signal pending during fork() the signal handler will > erroneously be called in both the parent and child, so handle any > pending signals first. > > Signed-off-by: Timothy Edward Baldwin

Re: [Qemu-devel] [PATCH 28/34] linux-user: Restart sigaction() if signal pending

2015-09-11 Thread Peter Maydell
On 6 September 2015 at 00:57, Timothy E Baldwin wrote: > Signed-off-by: Timothy Edward Baldwin > --- > linux-user/signal.c | 12 +--- > 1 file changed, 9 insertions(+), 3 deletions(-) > > diff --git

Re: [Qemu-devel] [PATCH 25/34] linux-user: Restart exit() if signal pending

2015-09-11 Thread Peter Maydell
On 6 September 2015 at 00:57, Timothy E Baldwin wrote: > Without this a signal could vanish on thread exit. > > Signed-off-by: Timothy Edward Baldwin > --- > linux-user/syscall.c | 8 ++-- > 1 file changed, 6

<    1   2   3   4   >