Peter Xu wrote:
> On Tue, May 08, 2018 at 05:16:15PM +0200, Juan Quintela wrote:
>> Peter Xu wrote:
>> > Introducing a new state "postcopy-paused", which can be used when the
>> > postcopy migration is paused. It is targeted for postcopy network
>> > failure recovery.
>> >
>> > Reviewed-by: Dr. D
On Thu, 05/03 11:27, Stefan Hajnoczi wrote:
> On Wed, Apr 18, 2018 at 11:04:22AM +0800, Fam Zheng wrote:
> > +static void iscsi_save_designator(IscsiLun *lun,
> > + struct
> > scsi_inquiry_device_identification *inq_di)
> > +{
> > +struct scsi_inquiry_device_de
Thomas Huth writes:
> On 07.05.2018 09:14, Markus Armbruster wrote:
> [...]
>> Two (possibly confused) questions:
>>
>> 1. The user can add nics without convenience options:
>>
>> $ upstream-qemu -display none -nodefaults -device e1000
>> upstream-qemu: warning: nic e1000.0 has no peer
Hi,
This series failed docker-mingw@fedora build test. Please find the testing
commands and
their output below. If you have Docker installed, you can probably reproduce it
locally.
Type: series
Message-id: 20180509055802.28423-1-f...@redhat.com
Subject: [Qemu-devel] [PATCH 0/5] vmdk: Implement x
Since v1:
- corrected the CRC16 offset
- addressed Peter Maydell comments
- add Edgar Iglesias S-o-b for Alistair Francis original patch at Xilinx
- use SD_FG_MAX to have a clearer loop
- rename uhs_enabled/uhs_mode and add documentation
- check all functions in group are correct before set
Add more descriptive comments to keep a clear separation
between static property vs runtime changeable.
Suggested-by: Peter Maydell
Signed-off-by: Philippe Mathieu-Daudé
---
hw/sd/sd.c | 14 +-
1 file changed, 9 insertions(+), 5 deletions(-)
diff --git a/hw/sd/sd.c b/hw/sd/sd.c
ind
Per the Physical Layer Simplified Spec. "4.3.10.4 Switch Function Status":
The block length is predefined to 512 bits
and "4.10.2 SD Status":
The SD Status contains status bits that are related to the SD Memory Card
proprietary features and may be used for future application-specific usage
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alistair Francis
---
hw/sd/sd.c | 8
1 file changed, 8 insertions(+)
diff --git a/hw/sd/sd.c b/hw/sd/sd.c
index d8dd88f872..a0e8a1b46a 100644
--- a/hw/sd/sd.c
+++ b/hw/sd/sd.c
@@ -94,6 +94,10 @@ struct SDState {
BlockBackend *bl
Signed-off-by: Fam Zheng
---
tests/qemu-iotests/214 | 268 +++
tests/qemu-iotests/214.out | 304 +
tests/qemu-iotests/group | 1 +
3 files changed, 573 insertions(+)
create mode 100755 tests/qemu-iotests/214
Signed-off-by: Fam Zheng
---
tests/qemu-iotests/common.filter | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/qemu-iotests/common.filter b/tests/qemu-iotests/common.filter
index c5f4bcf578..0debefed18 100644
--- a/tests/qemu-iotests/common.filter
+++ b/tests/qemu-iotests/common.filter
@
[based on a patch from Alistair Francis
from qemu/xilinx tag xilinx-v2015.2]
Signed-off-by: Edgar E. Iglesias
[PMD: rebased, changed magic by definitions, use stw_be_p, add tracing,
check all functions in group are correct before setting the values]
Signed-off-by: Philippe Mathieu-Daudé
---
h
The extracted vmdk_init_extent takes a BlockBackend object and
initializes the format metadata. It is the common part between "qemu-img
create" and "blockdev-create".
Add a "BlockBackend *pbb" parameter to vmdk_create_extent, to return the
opened BB to the caller in the next patch.
Signed-off-by:
This variant of qapi_enum_parse can do case insensitive compare.
Signed-off-by: Fam Zheng
---
include/qapi/util.h | 2 ++
qapi/qapi-util.c| 20
2 files changed, 18 insertions(+), 4 deletions(-)
diff --git a/include/qapi/util.h b/include/qapi/util.h
index a7c3c64148..2c
This adds vmdk support to x-blockdev-create command.
Unlike other formats, several VMDK subformats consists of multiple files
(extents). In the QAPI interface we use an array of BlockdevRef to pass in. The
management tool need to figure out how many extent files are needed but that is
simple enoug
This makes VMDK support x-blockdev-create. The implementation reuses the
image creation code in vmdk_co_create_opts which now acceptes a callback
pointer to "retrieve" BlockBackend pointers from the caller. This way we
separate the logic between file/extent acquisition and initialization.
The QAPI
On Mon, 05/07 13:09, Alistair Francis wrote:
> Avocado is not trivial to setup on non-Fedora systems. To simplfying
> future testing add a docker test image that runs Avocado tests.
>
> Signed-off-by: Alistair Francis
> ---
> tests/docker/dockerfiles/fedora.docker | 12 +--
> tests/docke
On 09.05.2018 00:14, Paolo Bonzini wrote:
> From: Daniel P. Berrangé
Here the accent is right ^
> The existing QemuOpts parsing code uses a fixed size 128 byte buffer
> for storing the parameter keys. If a key exceeded this size it was
> silently truncate and no error reported to the user. This
Am Wed, 9 May 2018 00:15:42 +0200
schrieb Paolo Bonzini :
> Actually, --program-prefix= and --exec-prefix= have a meaning, they
> cannot just be ignored. For now I've removed this line; what is the
> exact incantation used by SUSE Linux?
Like in every other rpm.rpm I guess:
/usr/lib/rpm/macros:
Hi Peter,
On 03/09/2018 02:03 PM, Peter Maydell wrote:
> On 9 March 2018 at 15:36, Philippe Mathieu-Daudé wrote:
[...]
>> static void sd_function_switch(SDState *sd, uint32_t arg)
>> {
>> -int i, mode, new_func;
>> -mode = !!(arg & 0x8000);
>> -
>> -sd->data[0] = 0x00;
On Sun, May 06, 2018 at 04:04:02PM +0100, Peter Maydell wrote:
> On 6 May 2018 at 14:39, David Gibson wrote:
> > Well, I'm biased of course, but I think we'd be better off just
> > ditching the wrapper. In its present form it is so limited as to not
> > really add any value. If it was rewritten
The file "qemu-options.h", "qemu-options.hx" and "qemu-options-wrapper.h"
in the main directory are currently without maintainer according to our
get_maintainers.pl script. Considering that the command line options are
a public interface and thus quite important, this is quite a bad state.
So I'd l
On 09.05.2018 07:02, Philippe Mathieu-Daudé wrote:
> Hi Thomas,
>
> On 05/09/2018 01:38 AM, Thomas Huth wrote:
>> The "trace-events" and "qemu-option-trace.texi" files in the top directory
>> are currently "unmaintained" according to scripts/get_maintainer.pl. They
>> obviously belong to the Traci
On 08.05.2018 22:39, Laurent Vivier wrote:
> Update the variable checked by the loop condition (expDiff).
> Backport the update from Previous.
>
> Fixes: 591596b77a ("target/m68k: add fmod/frem")
> Signed-off-by: Laurent Vivier
> ---
> target/m68k/softfloat.c | 1 +
> 1 file changed, 1 insertion
Hi Thomas,
On 05/09/2018 01:38 AM, Thomas Huth wrote:
> The "trace-events" and "qemu-option-trace.texi" files in the top directory
> are currently "unmaintained" according to scripts/get_maintainer.pl. They
> obviously belong to the Tracing section, so add an entry for them there.
I agree qemu-op
在 2018/5/8 下午6:37, Daniel P. Berrangé 写道:
On Mon, May 07, 2018 at 01:04:17PM -0500, Eric Blake wrote:
On 05/06/2018 10:32 PM, Yi Min Zhao wrote:
In the subject line: s/avoid to compile/avoid compiling/
If CONFIG_SECCOMP is undefined, the option 'elevatorprivileges' remains
s/elevator/eleva
The "trace-events" and "qemu-option-trace.texi" files in the top directory
are currently "unmaintained" according to scripts/get_maintainer.pl. They
obviously belong to the Tracing section, so add an entry for them there.
Signed-off-by: Thomas Huth
---
MAINTAINERS | 2 ++
1 file changed, 2 inser
Similar to previous patch, but introduce a new global big lock for
mon_fdsets. Take it where needed.
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Peter Xu
---
monitor.c | 64 +--
1 file changed, 53 insertions(+), 11 deletions(-)
diff --git a/
mon->fds were protected by BQL. Now protect it by mon_lock so that it
can even be used in monitor iothread.
Signed-off-by: Peter Xu
---
monitor.c | 22 ++
1 file changed, 18 insertions(+), 4 deletions(-)
diff --git a/monitor.c b/monitor.c
index 14c681dc8a..d6c3c08932 100644
Add some explicit comment for both Readline and cpu_set/cpu_get helpers
that they do not need the mon_lock protection.
Signed-off-by: Peter Xu
---
monitor.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/monitor.c b/monitor.c
index d6c3c08932..ae5bca9d7c 100644
--- a/mo
v5:
- collect r-bs and rebase
- move two close()s outside critical section [Dave]
- move comment to end of line [Stefan]
v4:
- fix a s/cur_mon/mon/ typo
v3:
- add comment for fields that are protected by monitor lock [Stefan]
- drop most of patch 2, only keep the protections for mon->fds [Stefan]
The out_lock was only protecting a few Monitor fields. In the future
the monitor code will start to run in multiple threads. We turn it into
a bigger lock to protect not only the out buffer but also all the rest.
For now we share the lock. We can split the lock when needed.
Since at it, arrange
On Wed, May 9, 2018 at 1:10 AM, Dr. David Alan Gilbert
wrote:
> * Juan Quintela (quint...@redhat.com) wrote:
>> Lidong Chen wrote:
>> > if qio_channel_rdma_readv return QIO_CHANNEL_ERR_BLOCK, the destination
>> > qemu
>> > crash.
>> >
>> > The backtrace is:
>> > (gdb) bt
>> > #0 0x0
On Tue, May 08, 2018 at 05:16:15PM +0200, Juan Quintela wrote:
> Peter Xu wrote:
> > Introducing a new state "postcopy-paused", which can be used when the
> > postcopy migration is paused. It is targeted for postcopy network
> > failure recovery.
> >
> > Reviewed-by: Dr. David Alan Gilbert
> > Si
Since not all modelled controllers use the CRC verification (which is
somehow expensive), let the controller have a configurable property
to enable verification.
So far only the Milkymist controller uses it.
This silent the Coverity warning:
"Code block is unreachable because of the syntactic
Hi Peter, Paolo,
On 04/24/2018 07:21 PM, Philippe Mathieu-Daudé wrote:
> The libfdt does not guarantee than fdt_getprop() returns a pointer
> aligned to the property size.
>
> Assuming the base of the fdt is aligned, a 32-bit property returns
> a 32-bit aligned pointer. This is however not guaran
Hi Peter, Kevin,
On 04/16/2018 05:52 PM, Alistair Francis wrote:
> On Mon, Apr 16, 2018 at 9:05 AM, mar.krzeminski
> wrote:
>> W dniu 15.04.2018 o 22:31, Philippe Mathieu-Daudé pisze:
>>>
>>> >From the datasheet (3368J–SEEPR) description:
>>> The AT25128A/256A provides 131,072/262,144 bits o
Signed-off-by: Philippe Mathieu-Daudé
---
include/hw/sd/sd.h | 10 ++
hw/sd/sd.c | 2 +-
hw/sd/sdmmc-internal.c | 6 ++
3 files changed, 17 insertions(+), 1 deletion(-)
diff --git a/include/hw/sd/sd.h b/include/hw/sd/sd.h
index c854ed6a14..752d8edd6c 100644
--- a/in
Hi Paolo,
On 04/22/2018 04:22 PM, Richard Henderson wrote:
> On 04/21/2018 01:21 PM, Philippe Mathieu-Daudé wrote:
>> This fixes the following ASan warning:
>>
>> $ qemu-system-xtensa -M kc705 -m 128M -semihosting -nographic -monitor
>> null -kernel Image.elf
>> include/hw/elf_ops.h:179:5: ru
Signed-off-by: Philippe Mathieu-Daudé
---
include/hw/sd/sd.h | 10 ++
hw/sd/sdmmc-internal.c | 6 ++
tests/sdcard-test.c| 12
3 files changed, 28 insertions(+)
diff --git a/include/hw/sd/sd.h b/include/hw/sd/sd.h
index 752d8edd6c..83399cd42d 100644
--- a/include
Replace the SDRequest argument using directly {uint8_t cmd, uint32_t arg},
it will be easier to remove the SDRequest struct in the next commit.
Signed-off-by: Philippe Mathieu-Daudé
---
hw/sd/sd.c | 104 +++--
1 file changed, 53 insertions(+), 51 d
Let's return TRUE when the CRC is valid.
Signed-off-by: Philippe Mathieu-Daudé
---
hw/sd/sd.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/hw/sd/sd.c b/hw/sd/sd.c
index d8dad94fc4..6fc8daa5b8 100644
--- a/hw/sd/sd.c
+++ b/hw/sd/sd.c
@@ -467,13 +467,13 @@ static voi
Signed-off-by: Philippe Mathieu-Daudé
---
hw/sd/ssi-sd.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/hw/sd/ssi-sd.c b/hw/sd/ssi-sd.c
index 77e446bb94..0375f0b959 100644
--- a/hw/sd/ssi-sd.c
+++ b/hw/sd/ssi-sd.c
@@ -95,11 +95,11 @@ static uint32_t ssi_sd_transfer(SSISlave
Per the Physical Layer Simplified Spec. "3.6 Bus Protocol":
There are two types of Data packet format for the SD card.
(1) Usual data (8-bit width): The usual data (8-bit width) are sent in
LSB (Least Significant Byte) first, MSB (Most Significant Byte) last
sequence. But in the i
Move the calc_checksum() functions out so we will able to write
qtests for them.
Signed-off-by: Philippe Mathieu-Daudé
---
include/hw/sd/sd.h | 20
hw/sd/sd.c | 33 -
hw/sd/sdmmc-internal.c | 35
Signed-off-by: Philippe Mathieu-Daudé
---
tests/sdcard-test.c | 74 +
1 file changed, 74 insertions(+)
diff --git a/tests/sdcard-test.c b/tests/sdcard-test.c
index 81789d1f88..2288a05cdf 100644
--- a/tests/sdcard-test.c
+++ b/tests/sdcard-test.c
@@ -14
It will help when moving this around for qtesting in the next commit.
Signed-off-by: Philippe Mathieu-Daudé
---
hw/sd/sd.c | 21 ++---
1 file changed, 18 insertions(+), 3 deletions(-)
diff --git a/hw/sd/sd.c b/hw/sd/sd.c
index 27a70896cd..06607115a7 100644
--- a/hw/sd/sd.c
+++ b
Fix style to keep patchew/checkpatch happy when moving this code
in the next patch.
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alistair Francis
---
hw/sd/sd.c | 16 ++--
1 file changed, 10 insertions(+), 6 deletions(-)
diff --git a/hw/sd/sd.c b/hw/sd/sd.c
index eb6dd5482e..
Signed-off-by: Philippe Mathieu-Daudé
---
hw/sd/sd.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/hw/sd/sd.c b/hw/sd/sd.c
index 235e0518d6..eb6dd5482e 100644
--- a/hw/sd/sd.c
+++ b/hw/sd/sd.c
@@ -236,11 +236,11 @@ static const int sd_cmd_class[SDMMC_CMD_MAX] = {
Using sd_frame48_init() we silent the Coverity warning:
"Use of an uninitialized variable (CWE-457)"
and fixes the following issues (all "Uninitialized scalar variable"):
- CID1386072 (hw/sd/sdhci.c::sdhci_end_transfer)
- CID1386074 (hw/sd/bcm2835_sdhost.c::bcm2835_sdhost_send_command)
- CID13
On 8/5/18 10:46 pm, Cornelia Huck wrote:
> On Tue, 8 May 2018 08:25:03 -0400
> Tony Krowiak wrote:
>
>> Added entries for the s390 adjunct processor (AP) support.
>>
>> Signed-off-by: Tony Krowiak
>> ---
>> MAINTAINERS | 14 ++
>> 1 files changed, 14 insertions(+), 0 deletions(-)
Hi,
This series emerged after last Coverity scan and Peter suggestion in:
http://lists.nongnu.org/archive/html/qemu-devel/2018-04/msg05046.html
(3) "proper" implementation of CRC, so that an sd controller
can either (a) mark the SDRequest as "no CRC" and have
sd_req_crc_validate() alw
The load/store API will ease further code movement.
Per the Physical Layer Simplified Spec. "3.6 Bus Protocol":
"In the CMD line the Most Significant Bit (MSB) is transmitted
first, the Least Significant Bit (LSB) is the last."
Signed-off-by: Philippe Mathieu-Daudé
---
hw/sd/bcm2835_sdhos
On Wed, May 9, 2018 at 2:40 AM, Dr. David Alan Gilbert
wrote:
> * Lidong Chen (jemmy858...@gmail.com) wrote:
>> When cancel migration during RDMA precopy, the source qemu main thread hangs
>> sometime.
>>
>> The backtrace is:
>> (gdb) bt
>> #0 0x7f249eabd43d in write () from /lib64/l
On Wed, May 9, 2018 at 1:19 AM, Dr. David Alan Gilbert
wrote:
> * Lidong Chen (jemmy858...@gmail.com) wrote:
>> rdma_delete_block function deletes RDMALocalBlock base on index field,
>> but not update the index field. So when next time invoke rdma_delete_block,
>> it will not work correctly.
>>
>>
On Tue, May 8, 2018 at 10:19 PM, Dr. David Alan Gilbert
wrote:
> * Lidong Chen (jemmy858...@gmail.com) wrote:
>> Because qio_channel_rdma_writev and qio_channel_rdma_readv maybe invoked
>> by different threads concurrently, this patch removes unnecessary variables
>> len in QIOChannelRDMA and use
On Tue, 05/08 11:43, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> tests/docker/Makefile.include | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
> index de87341528..ef1a3e62eb 1
On 05/08/2018 06:36 PM, John Snow wrote:
When we've reached the concluded state, we need to expose the error
state if applicable. Add the new field.
This should be sufficient for determining if a job completed
successfully or not after concluding; if we want to discriminate
based on how it faile
On Thu, Apr 12, 2018 at 11:56 AM, Richard Henderson <
richard.hender...@linaro.org> wrote:
> On 04/12/2018 01:29 AM, Emilio G. Cota wrote:
> > To ease an eventual merge I'll be updating the patches' R-b tags as
> > they come in this branch:
> > https://github.com/cota/qemu/tree/next_page_overflo
When we've reached the concluded state, we need to expose the error
state if applicable. Add the new field.
This should be sufficient for determining if a job completed
successfully or not after concluding; if we want to discriminate
based on how it failed more mechanically, we can always add an
e
On Wed, May 9, 2018 at 1:22 AM, Peter Maydell
wrote:
> On 6 May 2018 at 00:35, Michael Clark wrote:
> > The following changes since commit c8b7e627b4269a3bc3ae41d9f42054
> 7a47e6d9b9:
> >
> > Merge remote-tracking branch 'remotes/ericb/tags/pull-nbd-2018-05-04'
> into staging (2018-05-04 14:42
On Tue, May 8, 2018 at 8:10 AM, Philippe Mathieu-Daudé wrote:
> There are only two files requiring this typedef (strongarm.h
> and hw/arm/pxa.h), let them include "hw/pcmcia.h" directly to
> simplify qemu/typedefs.h.
>
> Signed-off-by: Philippe Mathieu-Daudé
I think this is heading in the right
On 05/08/2018 04:36 PM, Richard Henderson wrote:
> In 6001f7729e12 we partially attempt to address the branch
> displacement overflow caused by 15fa08f845.
>
> However, gcc/testsuite/gcc.target/aarch64/advsimd-intrinsics/vqtbX.c
> is a testcase that contains a TB so large as to overflow anyway.
>
From: Philippe Mathieu-Daudé
This kludge was added in a825ca06137, but a cleaner and more generic
fix is now available (see ##COMMIT_CONFIGURE_LIBFDT_LDFLAGS_SHA##).
Signed-off-by: Philippe Mathieu-Daudé
Message-Id: <20180415230522.24404-4-f4...@amsat.org>
Signed-off-by: Paolo Bonzini
---
.sh
From: Philippe Mathieu-Daudé
QEMU requires libfdt version >= 1.4.2.
If the host has an older libfdt installed, the configure script will use
a (git cloned) local version.
Example with Debian 8:
$ dpkg-query --showformat='${Version}\n' --show libfdt-dev
1.4.0+dfsg-1
$ ./configure
From: Thomas Huth
We've never documented this option in our qemu-doc, so apart from the users
that already used the old qemu-kvm fork before, most users should not be
aware of this option at all. It's been marked as deprecated in the source
code for a long time already, and officially marked as d
From: Thomas Huth
Deprecated since the beginning when it was added for compatibility with
the ancient qemu-kvm fork of QEMU, and it even printed out the deprecation
warning since right from the start (i.e. QEMU v1.3.0), so it's really time
to remove this now.
Reviewed-by: Markus Armbruster
Sign
On 05/08/2018 10:49 AM, Philippe Mathieu-Daudé wrote:
> The ALLWINNER_AHCI() macro is only used in ahci-allwinner.c.
>
extra internal!
> Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: John Snow
Feel free to stage in trivial, thanks.
On Tue, May 8, 2018 at 7:42 AM, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alistair Francis
Alistair
> ---
> hw/riscv/riscv_htif.c | 1 -
> hw/riscv/sifive_e.c | 1 -
> hw/riscv/sifive_u.c | 1 -
> hw/riscv/spike.c | 1 -
> hw/riscv/virt.c
From: Daniel P. Berrangé
The existing QemuOpts parsing code uses a fixed size 128 byte buffer
for storing the parameter keys. If a key exceeded this size it was
silently truncate and no error reported to the user. This behaviour was
reasonable & harmless because traditionally the key names are al
On 05/08/2018 11:10 AM, Philippe Mathieu-Daudé wrote:
> Files requiring AllwinnerAHCIState already include "hw/ide/ahci.h".
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> include/hw/ide/ahci.h | 4 ++--
> include/qemu/typedefs.h | 1 -
> 2 files changed, 2 insertions(+), 3 deletions(-)
>
From: Daniel P. Berrangé
Instead of re-using the get_opt_name() method from QemuOpts to split a
string on ':', just use g_strsplit().
Signed-off-by: Daniel P. Berrangé
Message-Id: <20180416111743.8473-2-berra...@redhat.com>
Signed-off-by: Paolo Bonzini
Signed-off-by: Daniel P. Berrangé
---
From: David Hildenbrand
If no slots were defined we try to allocate an empty bitmap, which
fails.
Signed-off-by: David Hildenbrand
Reviewed-by: Marcel Apfelbaum
Reviewed-by: Thomas Huth
Message-Id: <20180427120515.24067-1-da...@redhat.com>
Signed-off-by: Paolo Bonzini
---
hw/mem/pc-dimm.c |
For now, this reduces the text size very slightly due to the newly-added
inlining:
text size before: 9301965
text size after: 9300645
Later, however, the declarations in include/exec/memory_ldst.inc.h will be
reused for the MemoryRegionCache slow path functions.
Signed-off-by: Paolo Bonzin
On 18/04/2018 11:23, Paolo Bonzini wrote:
> On 18/04/2018 09:50, Olaf Hering wrote:
>> Extend the list of recognized, but ignored options from rpms %configure
>> macro. This fixes build on hosts running SUSE Linux.
>>
>> Signed-off-by: Olaf Hering
>> ---
>> configure | 3 +++
>> 1 file changed, 3
From: Jan Kiszka
Align with other mkdir calls.
Signed-off-by: Jan Kiszka
Reviewed-by: Philippe Mathieu-Daudé
Message-Id: <0dd4c8f5-d60e-e564-652f-cd0101f6e...@web.de>
Message-Id: <20180415230522.24404-5-f4...@amsat.org>
Signed-off-by: Paolo Bonzini
---
Makefile | 2 +-
1 file changed, 1 ins
From: Peter Xu
We will conditionally have a wrapper layer depending on whether the host
has the PTHREAD_SETNAME capability. It complicates stuff. Let's keep
the wrapper there; we opt out the pthread_setname_np() call only.
Signed-off-by: Peter Xu
Message-Id: <20180412053444.17801-1-pet...@red
osdep.h is only needed for files that are compiled directly.
Remove it from included C source files, and rename them to
*.inc.c so that scripts/clean-includes knows to skip them.
Cc: Eric Blake
Cc: Michael S. Tsirkin
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Paolo Bonzini
---
target/cris/tr
From: Philippe Mathieu-Daudé
The configure script outputs "yes" regardless which libfdt is used:
./configure
[...]
fdt support yes
Sometimes you can have both system and local git version available,
change the configure script to display which library got selected:
debian8$ dpkg-
This will be used to process IOMMUs in a MemoryRegionCache. This
includes a small bugfix, in that the returned page_mask is now
correctly -1 if the IOMMU memory region maps the entire address
space directly. Previously, address_space_get_iotlb_entry would
return ~TARGET_PAGE_MASK.
Reviewed-by: P
From: Daniel P. Berrangé
The existing QemuOpts parsing code uses a fixed size 1024 byte buffer
for storing the option values. If a value exceeded this size it was
silently truncated and no error reported to the user. Long option values
is not a common scenario, but it is conceivable that they wil
From: Daniel P. Berrangé
Per supported platforms doc, the various min glib on relevant distros is:
RHEL-7: 2.50.3
Debian (Stretch): 2.50.3
Debian (Jessie): 2.42.1
OpenBSD (Ports): 2.54.3
FreeBSD (Ports): 2.50.3
OpenSUSE Leap 15: 2.54.3
Ubuntu (Xenial): 2.48.0
macOS (Homebrew): 2.
Reported by Coverity.
Signed-off-by: Paolo Bonzini
---
target/i386/sev.c | 32 +---
1 file changed, 17 insertions(+), 15 deletions(-)
diff --git a/target/i386/sev.c b/target/i386/sev.c
index c011671..2395171 100644
--- a/target/i386/sev.c
+++ b/target/i386/sev.c
@@ -
The "id" property is unnecessary and can be replaced simply with
object_get_canonical_path_component. This patch mostly undoes commit
e1ff3c67e8 ("monitor: fix qmp/hmp query-memdev not reporting IDs of
memory backends", 2017-01-12).
Signed-off-by: Paolo Bonzini
---
backends/hostmem.c | 26
From: Vitaly Kuznetsov
KVM recently gained support for Hyper-V Reenlightenment MSRs which are
required to make KVM-on-Hyper-V enable TSC page clocksource to its guests
when INVTSC is not passed to it (and it is not passed by default in Qemu
as it effectively blocks migration).
Signed-off-by: Vit
From: Thomas Huth
The dangling remainder of the -tdf option revealed a deficiency in our
option parsing: Options that have been declared, but are not supported
in the switch-case statement in vl.c and not handled in the OS-specifc
os_parse_cmd_args() functions are currently silently ignored. We s
From: Markus Armbruster
When resume of a stopped guest immediately runs into block device
errors, the BLOCK_IO_ERROR event is sent before the RESUME event.
Reproducer:
1. Create a scratch image
$ dd if=/dev/zero of=scratch.img bs=1M count=100
Size doesn't actually matter.
2. Prepare blk
From: Thomas Huth
The qemu-doc already states that this option is only maintained for
backward compatibility and "-device virtconsole" should be used
instead. So let's take the next step and mark this option officially
as deprecated.
Reviewed-by: Markus Armbruster
Signed-off-by: Thomas Huth
Me
From: Daniel P. Berrangé
Describe the policy the project uses to decide which OS are supported as
build platforms. This will:
- Allow maintainers to determine when the minimum version of a 3rd
party piece of software can be increased without negatively
impacting supported platforms.
From: Roman Kagan
As of mainline linux commit 5a485803221777013944cbd1a7cd5c62efba3ffa
"x86/hyper-v: move hyperv.h out of uapi" by Vitaly Kuznetsov, no linux
uapi header includes it, so we no longer need to create a stub for it.
Cc: Vitaly Kuznetsov
Signed-off-by: Roman Kagan
Message-Id: <2018
The following changes since commit cc8f8ba754bba17eea9791d67b572eb26e30b4ce:
Merge remote-tracking branch
'remotes/ehabkost/tags/machine-next-pull-request' into staging (2018-05-08
15:25:17 +0100)
are available in the git repository at:
git://github.com/bonzini/qemu.git tags/for-upstream
Just return NULL; any callers that cause a change in behavior
would have caused an assertion failure before, so this is safe.
Signed-off-by: Paolo Bonzini
---
include/qom/object.h | 1 +
qom/object.c | 5 +++--
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/include/qom/ob
From: Thomas Huth
The -tdf options has been removed with d07aa197c5a1556449361a0cbb5108e2,
but apparently I forgot to remove the corresponding two lines from
qemu-options.hx, so this option is still "available" and just silently
ignored. Kill it now for good.
Reviewed-by: Markus Armbruster
Sign
Prepare for extracting the IOMMU part to a separate function. Mostly
cosmetic; the only semantic change is that, if there is more than one
cascaded IOMMU and the second one fails to translate, *plen_out is now
adjusted according to the page mask of the first IOMMU.
Reviewed-by: Peter Xu
Signed-o
From: Olaf Hering
Extend the list of recognized, but ignored options from rpms %configure
macro. This fixes build on hosts running SUSE Linux.
Cc: qemu-sta...@nongnu.org
Signed-off-by: Olaf Hering
Message-Id: <20180418075045.27393-1-o...@aepfle.de>
Signed-off-by: Paolo Bonzini
---
configure |
MemoryRegionCache was reverted to "normal" address_space_* operations
for 2.9, due to lack of support for IOMMUs. Reinstate the
optimizations, caching only the IOMMU translation at address_cache_init
but not the IOMMU lookup and target AddressSpace translation are not
cached; now that MemoryRegion
From: Cédric Le Goater
Commit 9b0605f9837b ("cpus: tcg: unregister thread with RCU, fix
exiting of loop on unplug") changed the exit condition of the loop in
the vCPU thread function but forgot to remove the beginning 'while (1)'
statement. The resulting code :
while (1) {
...
From: Peter Xu
Otherwise it can warn this:
ERROR: space prohibited between function name and open parenthesis '('
When with things like this:
typedef gboolean (*it_tree_iterator)(ITValue start, ITValue end);
CC: Paolo Bonzini
CC: "Daniel P. Berrangé"
CC: Vladimir Sementsov-Ogievskiy
C
On Wed, May 9, 2018 at 2:42 AM, Philippe Mathieu-Daudé
wrote:
> Signed-off-by: Philippe Mathieu-Daudé
>
Reviewed-by: Michael Clark
---
> hw/riscv/riscv_htif.c | 1 -
> hw/riscv/sifive_e.c | 1 -
> hw/riscv/sifive_u.c | 1 -
> hw/riscv/spike.c | 1 -
> hw/riscv/virt.c | 1 -
>
On Wed, May 9, 2018 at 8:49 AM, Peter Maydell
wrote:
> On 8 May 2018 at 21:07, Michael Clark wrote:
> > The following changes since commit c8b7e627b4269a3bc3ae41d9f42054
> 7a47e6d9b9:
> >
> > Merge remote-tracking branch 'remotes/ericb/tags/pull-nbd-2018-05-04'
> into staging (2018-05-04 14:42
This patch adds gen_io_start()/gen_io_end() to various instructions as required
in order to boot my OpenBIOS test images on qemu-system-sparc64 with icount
enabled.
Signed-off-by: Mark Cave-Ayland
---
target/sparc/translate.c | 97
1 file changed,
1 - 100 of 387 matches
Mail list logo
