On 1/26/21 5:43 AM, Bin Meng wrote:
> From: Bin Meng
>
> The SEND_IF_COND command (CMD8) response is of format R7, but
> current code returns R1 for CMD8. Fix it.
>
> Fixes: 775616c3ae8c ("Partial SD card SPI mode support")
> Signed-off-by: Bin Meng
>
> ---
> When testing with VxWorks driver,
On Jan 26 14:02, Dmitry Fomichev wrote:
> These patches solve a few problems that exist in zoned Write
> ans Zone Append validation code.
>
> Dmitry Fomichev (3):
> hw/block/nvme: Check for zone boundary during append
> hw/block/nvme: Check zone state before checking boundaries
> hw/block/nv
26.01.2021 01:05, Kevin Wolf wrote:
Am 23.01.2021 um 22:04 hat Vladimir Sementsov-Ogievskiy geschrieben:
Add TestEnv class, which will handle test environment in a new python
iotests running framework.
Don't add compat=1.1 for qcow2 IMGOPTS, as v3 is default anyway.
Signed-off-by: Vladimir Sem
On Mon, Jan 25, 2021 at 3:52 PM Philippe Mathieu-Daudé
wrote:
>
> You forgot to Cc the subsystem maintainers...
>
> ./scripts/get_maintainer.pl -f hw/scsi/megasas.c
> Hannes Reinecke (supporter:megasas)
> Paolo Bonzini (supporter:SCSI)
> Fam Zheng (reviewer:SCSI)
I used to only check the MAINT
On Jan 26 09:21, Klaus Jensen wrote:
> On Jan 26 14:02, Dmitry Fomichev wrote:
> > These patches solve a few problems that exist in zoned Write
> > ans Zone Append validation code.
> >
> > Dmitry Fomichev (3):
> > hw/block/nvme: Check for zone boundary during append
> > hw/block/nvme: Check zo
On 26/01/21 08:03, Thomas Huth wrote:
+ artifacts:
+ when: always
+ paths:
+ - build/meson-logs/
+ reports:
+ junit: build/meson-logs/testlog.junit.xml
Ok, I've tried that but it also worked not quite as expected:
https://gitlab.com/huth/qemu/-/pipelines/246463068/test_rep
On 01/25/21 23:45, Paolo Bonzini wrote:
> On 25/01/21 23:15, Laszlo Ersek wrote:
>>
>> - How does this case differ from the registration of types? type_init()
>> is a constructor function, and it generally ends up adding some
>> structures with function pointers (I reckon) via type_table_add(). The
On 26/01/2021 08.36, Philippe Mathieu-Daudé wrote:
On 1/26/21 7:57 AM, Thomas Huth wrote:
Currently, our check-system-* jobs are recompiling the whole sources
again. This happens due to the fact that the jobs are checking out
the whole source tree and required submodules again, and only try
to u
On Jan 26 09:19, Minwoo Im wrote:
> The actual parameter name is 'cross_read' rather than 'cross_zone_read'.
>
> Signed-off-by: Minwoo Im
> ---
> hw/block/nvme.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/block/nvme.c b/hw/block/nvme.c
> index 21aec90637fa..bf91
On Tue, 26 Jan 2021 at 00:12, Joelle van Dyne wrote:
> cc.compiles should work though. Is there a reason why it's not used
> instead of all the compile_prog checks in ./configure ?
As Paolo says, that's because our transition to Meson is
pretty recent. We put in the minimum necessary to move all
Eric Blake writes:
> These cases require a bit more thought to review; in each case, the
> code was appending to a list, but not with a FOOList **tail variable.
>
> Signed-off-by: Eric Blake
> Reviewed-by: Vladimir Sementsov-Ogievskiy
>
> ---
> fix qmp_guest_network_get_interfaces [Vladimir]
F
On 1/26/21 10:08 AM, Thomas Huth wrote:
> On 26/01/2021 08.36, Philippe Mathieu-Daudé wrote:
>> On 1/26/21 7:57 AM, Thomas Huth wrote:
>>> Currently, our check-system-* jobs are recompiling the whole sources
>>> again. This happens due to the fact that the jobs are checking out
>>> the whole source
Am 26.01.2021 um 09:28 hat Vladimir Sementsov-Ogievskiy geschrieben:
> 26.01.2021 01:05, Kevin Wolf wrote:
> > Am 23.01.2021 um 22:04 hat Vladimir Sementsov-Ogievskiy geschrieben:
> > > Add TestEnv class, which will handle test environment in a new python
> > > iotests running framework.
> > >
> >
26.01.2021 12:45, Kevin Wolf wrote:
Am 26.01.2021 um 09:28 hat Vladimir Sementsov-Ogievskiy geschrieben:
26.01.2021 01:05, Kevin Wolf wrote:
Am 23.01.2021 um 22:04 hat Vladimir Sementsov-Ogievskiy geschrieben:
Add TestEnv class, which will handle test environment in a new python
iotests runnin
On Fri, Jan 22, 2021 at 10:40:54AM -0500, Vivek Goyal wrote:
> On Thu, Jan 21, 2021 at 02:44:29PM +, Stefan Hajnoczi wrote:
> > A well-behaved FUSE client does not attempt to open special files with
> > FUSE_OPEN because they are handled on the client side (e.g. device nodes
> > are handled by
** Changed in: qemu
Status: Expired => Incomplete
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1796520
Title:
autogen crashes on qemu-sh4-user after 61dedf2af7
Status in QEMU:
Incomplete
25.01.2021 21:50, Vladimir Sementsov-Ogievskiy wrote:
Add TestEnv class, which will handle test environment in a new python
iotests running framework.
Don't add compat=1.1 for qcow2 IMGOPTS, as v3 is default anyway.
Signed-off-by: Vladimir Sementsov-Ogievskiy
squash-in to fix mypy complains,
25.01.2021 21:50, Vladimir Sementsov-Ogievskiy wrote:
Add TestRunner class, which will run tests in a new python iotests
running framework.
There are some differences with current ./check behavior, most
significant are:
- Consider all tests self-executable, just run them, don't run python
by
Am 26.01.2021 um 11:08 hat Vladimir Sementsov-Ogievskiy geschrieben:
> 26.01.2021 12:45, Kevin Wolf wrote:
> > Am 26.01.2021 um 09:28 hat Vladimir Sementsov-Ogievskiy geschrieben:
> > > 26.01.2021 01:05, Kevin Wolf wrote:
> > > > Am 23.01.2021 um 22:04 hat Vladimir Sementsov-Ogievskiy geschrieben:
Philippe Mathieu-Daudé writes:
> Similarly to commits ae82adc8e29..7f93879e444, use the
> translator_ld*() API introduced in commit 409c1a0bf0f
> to fetch the code on the MIPS target.
>
> Reviewed-by: Jiaxun Yang
> Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alex Bennée
--
Alex Be
On Mon, Jan 25, 2021 at 05:12:23PM +0100, Miklos Szeredi wrote:
> On Thu, Jan 21, 2021 at 3:44 PM Stefan Hajnoczi wrote:
>
> > This patch adds the missing checks to virtiofsd. This is a short-term
> > solution because it does not prevent a compromised virtiofsd process
> > from opening device nod
On Tue, Jan 26, 2021 at 11:18 AM Stefan Hajnoczi wrote:
>
> On Mon, Jan 25, 2021 at 05:12:23PM +0100, Miklos Szeredi wrote:
> > On Thu, Jan 21, 2021 at 3:44 PM Stefan Hajnoczi wrote:
> >
> > > This patch adds the missing checks to virtiofsd. This is a short-term
> > > solution because it does not
If the link test failed, compilation proceeded with RBD disabled,
even if --enable-rbd was used on the configure command line.
Fix that.
Signed-off-by: Paolo Bonzini
---
meson.build | 24 +++-
1 file changed, 15 insertions(+), 9 deletions(-)
diff --git a/meson.build b/meson.
Meson's "static" argument to cc.find_library is a tri-state. By default
Meson *prefers* a shared library, which basically means using -l to
look for it; instead, "static: false" *requires* a shared library. Of
course, "static: true" requires a static library, which is all good
for --enable-static
On Mon, Jan 25, 2021 at 09:02:38AM -1000, Richard Henderson wrote:
> On 1/25/21 8:58 AM, Stefan Weil wrote:
> > I have no evidence that TCI is less reliable than TCG, so I would not write
> > that.
>
> It can't pass make check-tcg.
It also doesn't have anyhere near the same level of automated or
A well-behaved FUSE client does not attempt to open special files with
FUSE_OPEN because they are handled on the client side (e.g. device nodes
are handled by client-side device drivers).
The check to prevent virtiofsd from opening special files is missing in
a few cases, most notably FUSE_OPEN. A
On Tue, Jan 26, 2021 at 10:35:02AM +, Stefan Hajnoczi wrote:
> A well-behaved FUSE client does not attempt to open special files with
> FUSE_OPEN because they are handled on the client side (e.g. device nodes
> are handled by client-side device drivers).
>
> The check to prevent virtiofsd from
Am 26.01.21 um 11:27 schrieb Paolo Bonzini:
> If the link test failed, compilation proceeded with RBD disabled,
> even if --enable-rbd was used on the configure command line.
> Fix that.
>
> Signed-off-by: Paolo Bonzini
> ---
> meson.build | 24 +++-
> 1 file changed, 15 inser
On Mon, 25 Jan 2021 14:53:32 +0100
Christian Borntraeger wrote:
> Secure execution (aka protected virtualization) guests cannot be
> migrated at the moment. If the unpack facility is provided in the cpu
> model, a guest may choose to transition to secure mode, making the
> guest unmigratable at t
On 26/01/2021 10:35, Stefan Hajnoczi wrote:
A well-behaved FUSE client does not attempt to open special files with
FUSE_OPEN because they are handled on the client side (e.g. device nodes
are handled by client-side device drivers).
The check to prevent virtiofsd from opening special files is mis
Commit 0afec75734331 removed the 'change' QMP command, so we can no
longer test it in 118.
Fixes: 0afec75734331a0b52fa3aa4235220eda8c7846f
('qmp: remove deprecated "change" command')
Signed-off-by: Max Reitz
---
tests/qemu-iotests/118 | 17 -
tests/qemu-iotests/118.out
On 1/26/21 11:48 AM, Max Reitz wrote:
> Commit 0afec75734331 removed the 'change' QMP command, so we can no
> longer test it in 118.
>
> Fixes: 0afec75734331a0b52fa3aa4235220eda8c7846f
>('qmp: remove deprecated "change" command')
> Signed-off-by: Max Reitz
> ---
> tests/qemu-iotests/118
On Thu, Jan 21, 2021 at 04:09:09PM +, John Levon wrote:
>
> Hi, please take a look. For context, this addition is against the current
> https://github.com/nutanix/libvfio-user/blob/master/docs/vfio-user.rst
> specification.
>
> kvm@ readers: Stefan suggested this may be of interest from a VFI
On 1/15/21 11:21 PM, Thomas Huth wrote:
> On 15/01/2021 16.09, Philippe Mathieu-Daudé wrote:
>> Tests in fuzz-test's main() already check for the supported
>> architecture before adding tests, therefore this test is not
>> specific to the X86 target. Move it to the generic set.
>
> As long as it d
On 1/15/21 11:39 PM, Alexander Bulekov wrote:
> On 210115 1609, Philippe Mathieu-Daudé wrote:
>> This test fails when QEMU is built without the megasas device,
>> restrict it to its availability.
>
> Should we just make a separate directory for fuzzer tests and have a
> separate source file for ea
On Thu, Jan 21, 2021 at 04:18:55PM -0800, Joe Komlodi wrote:
> Using the cfg.use_non_secure bitfield and the MMU access type, we can
> determine
> if the access should be secure or not.
Reviewed-by: Edgar E. Iglesias
Tested-by: Edgar E. Iglesias
>
> Signed-off-by: Joe Komlodi
> ---
> targ
Some tests/qtest/fuzz-test fail when the device tested is
not available in the build. Fix this by only running the
test when devices are available.
FWIW Alexander Bulekov suggested an improvement, putting each
test in a directory named by the device tested. This series
does not cover that.
Supers
This test fails when QEMU is built without the megasas device,
restrict it to its availability.
Signed-off-by: Philippe Mathieu-Daudé
---
tests/qtest/fuzz-megasas-test.c | 49 +
tests/qtest/fuzz-test.c | 25 -
MAINTAINERS
On Tue, Dec 22, 2020 at 06:39:33PM -0500, Igor Mammedov wrote:
> Implement _DSM according to:
> PCI Firmware Specification 3.1
> 4.6.7. DSM for Naming a PCI or PCI Express Device Under
> Operating Systems
> and wire it up to cold and hot-plugged PCI devices.
> Feature depends o
The length field is already contained in the ip6_ext_hdr structure.
Check it direcly in eth_parse_ipv6_hdr() before calling
_eth_get_rss_ex_dst_addr(), which gets a bit simplified.
Reviewed-by: Miroslav Rezanina
Signed-off-by: Philippe Mathieu-Daudé
---
net/eth.c | 14 +++---
1 file cha
Am 26.01.2021 um 11:48 hat Max Reitz geschrieben:
> Commit 0afec75734331 removed the 'change' QMP command, so we can no
> longer test it in 118.
>
> Fixes: 0afec75734331a0b52fa3aa4235220eda8c7846f
>('qmp: remove deprecated "change" command')
> Signed-off-by: Max Reitz
Thanks, applied to
On Thu, Jan 21, 2021 at 04:18:54PM -0800, Joe Komlodi wrote:
> Using MMUAccessType makes it more clear what the variable's use is.
> No functional change.
Reviewed-by: Edgar E. Iglesias
Tested-by: Edgar E. Iglesias
>
> Signed-off-by: Joe Komlodi
> ---
> target/microblaze/mmu.c | 2 +-
> tar
This test fails when QEMU is built without the virtio-scsi device,
restrict it to its availability.
Reviewed-by: Michael S. Tsirkin
Signed-off-by: Philippe Mathieu-Daudé
---
tests/qtest/fuzz-test.c | 51
tests/qtest/fuzz-virtio-scsi-test.c | 75 +
QEMU fuzzer reported a buffer overflow in _eth_get_rss_ex_dst_addr()
reproducible as:
$ cat << EOF | ./qemu-system-i386 -M pc-q35-5.0 \
-accel qtest -monitor none \
-serial none -nographic -qtest stdio
outl 0xcf8 0x80001010
outl 0xcfc 0xe102
outl 0xcf8 0x80001004
outw 0xcfc 0
I had a look at the patch from Miroslav trying to silence a
compiler warning which in fact is a nasty bug. Here is a fix.
https://www.mail-archive.com/qemu-devel@nongnu.org/msg772735.html
v3: Added Thomas qtest Acked-by
Philippe Mathieu-Daudé (2):
net/eth: Simplify _eth_get_rss_ex_dst_addr()
Signed-off-by: Peter Lieven
---
block/rbd.c | 15 +--
1 file changed, 13 insertions(+), 2 deletions(-)
diff --git a/block/rbd.c b/block/rbd.c
index f68ebcf240..7abd0252c9 100644
--- a/block/rbd.c
+++ b/block/rbd.c
@@ -91,6 +91,7 @@ typedef struct BDRVRBDState {
char *namespace;
On 1/26/21 12:16 PM, Philippe Mathieu-Daudé wrote:
> Some tests/qtest/fuzz-test fail when the device tested is
> not available in the build. Fix this by only running the
> test when devices are available.
Forgot, since v1:
- Do not make the testing generic, keep it restricted to x86 (thuth)
> FW
this series migrates the qemu rbd driver from the old aio emulation
to native coroutines and adds write zeroes support which is important
for block operations.
To achive this we first bump the librbd requirement to the already
outdated luminous release of ceph to get rid of some wrappers and
ifdef
Signed-off-by: Peter Lieven
---
block/rbd.c | 36 +++-
1 file changed, 35 insertions(+), 1 deletion(-)
diff --git a/block/rbd.c b/block/rbd.c
index d11a3c6dd1..35dc1dc90e 100644
--- a/block/rbd.c
+++ b/block/rbd.c
@@ -63,7 +63,8 @@ typedef enum {
RBD_AIO_READ
On Thu, Jan 21, 2021 at 04:18:53PM -0800, Joe Komlodi wrote:
> This property is used to control the security of the following interfaces
> on MicroBlaze:
> M_AXI_DP - data interface
> M_AXI_IP - instruction interface
> M_AXI_DC - dcache interface
> M_AXI_IC - icache interface
>
> It works by enabl
librbd supports 1 byte alignment for all aio operations.
Currently, there is no API call to query limits from the ceph backend.
So drop the bdrv_refresh_limits completely until there is such an API call.
Signed-off-by: Peter Lieven
---
block/rbd.c | 9 -
1 file changed, 9 deletions(-)
Signed-off-by: Peter Lieven
---
block/rbd.c | 253 ++--
1 file changed, 86 insertions(+), 167 deletions(-)
diff --git a/block/rbd.c b/block/rbd.c
index 7abd0252c9..d11a3c6dd1 100644
--- a/block/rbd.c
+++ b/block/rbd.c
@@ -66,22 +66,6 @@ typedef enu
in case the image size changed we should adjust our internally stored size as
well.
Signed-off-by: Peter Lieven
---
block/rbd.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/block/rbd.c b/block/rbd.c
index 1028596c68..f68ebcf240 100644
--- a/block/rbd.c
+++ b/block/rbd.c
@@ -964,6 +964,7
When we started to commit the fuzzer QTest reproducers to
fuzz-test.c in commit d8dd1095019 ("qtest: add fuzz test case"),
we forgot to add the corresponding MAINTAINERS entry. Do it now.
Signed-off-by: Philippe Mathieu-Daudé
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MA
even luminous (version 12.2) is unmaintained for over 3 years now.
Bump the requirement to get rid of the ifdef'ry in the code.
Signed-off-by: Peter Lieven
---
block/rbd.c | 120
meson.build | 13 --
2 files changed, 17 insertions(+), 116
Am 25.01.21 um 23:35 schrieb Richard Henderson:
On 1/25/21 11:02 AM, Stefan Weil wrote:
Am 25.01.21 um 20:02 schrieb Richard Henderson:
On 1/25/21 8:58 AM, Stefan Weil wrote:
I have no evidence that TCI is less reliable than TCG, so I would not write
that.
It can't pass make check-tcg.
Whe
Signed-off-by: Peter Lieven
---
block/rbd.c | 18 +++---
1 file changed, 7 insertions(+), 11 deletions(-)
diff --git a/block/rbd.c b/block/rbd.c
index a191c74619..1028596c68 100644
--- a/block/rbd.c
+++ b/block/rbd.c
@@ -90,6 +90,7 @@ typedef struct BDRVRBDState {
char *snap;
On 26/01/21 11:48, Max Reitz wrote:
Commit 0afec75734331 removed the 'change' QMP command, so we can no
longer test it in 118.
Fixes: 0afec75734331a0b52fa3aa4235220eda8c7846f
('qmp: remove deprecated "change" command')
Signed-off-by: Max Reitz
---
tests/qemu-iotests/118 | 17 -
Add a job for a supported feature: capstone.
Philippe Mathieu-Daudé (2):
docker: Install libcastone on Fedora
gitlab-ci: Test Fedora capstone package
.gitlab-ci.yml | 7 +++
tests/docker/dockerfiles/fedora.docker | 1 +
2 files changed, 8 insertions(+)
--
2.26.2
Proposed fix:
https://www.mail-archive.com/qemu-devel@nongnu.org/msg773408.html
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1879531
Title:
Stack-overflow in _eth_get_rss_ex_dst_addr
Status in QE
On Tue, Jan 26, 2021 at 12:36:49PM +0100, Philippe Mathieu-Daudé wrote:
> Test building the 4 targets using the capstone disassembler
> with the capstone package provided on Fedora.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> .gitlab-ci.yml | 7 +++
> 1 file changed, 7 insertions(+)
>
* Tom Lendacky (thomas.lenda...@amd.com) wrote:
> From: Tom Lendacky
>
> When SEV-ES is enabled, it is not possible modify the guests register
> state after it has been initially created, encrypted and measured.
>
> Normally, an INIT-SIPI-SIPI request is used to boot the AP. However, the
> hyper
As other packages might install the system capstone library,
we want to test it too in our CI.
By default QEMU clones libcapstone from git, but a developer
can force to use the system packaged library using:
$ ./configure --enable-capstone=system
Signed-off-by: Philippe Mathieu-Daudé
---
tes
Test building the 4 targets using the capstone disassembler
with the capstone package provided on Fedora.
Signed-off-by: Philippe Mathieu-Daudé
---
.gitlab-ci.yml | 7 +++
1 file changed, 7 insertions(+)
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index de3a3d25b58..913940656de 100644
---
From: Klaus Jensen
The Maximum Open Resources field shall be less than or equal to the
Maximum Active Resources field.
Verify that constraint and remove the redundant check that the Maximum
Open Resources field does not exceed the number of zones.
Cc: Dmitry Fomichev
Reported-by: Gollu Appalan
On 1/26/21 12:39 PM, Daniel P. Berrangé wrote:
> On Tue, Jan 26, 2021 at 12:36:49PM +0100, Philippe Mathieu-Daudé wrote:
>> Test building the 4 targets using the capstone disassembler
>> with the capstone package provided on Fedora.
>>
>> Signed-off-by: Philippe Mathieu-Daudé
>> ---
>> .gitlab-ci
On 26/01/2021 12.36, Philippe Mathieu-Daudé wrote:
Add a job for a supported feature: capstone.
Philippe Mathieu-Daudé (2):
docker: Install libcastone on Fedora
gitlab-ci: Test Fedora capstone package
.gitlab-ci.yml | 7 +++
tests/docker/dockerfiles/fedora.d
On 1/26/21 1:05 PM, Thomas Huth wrote:
> On 26/01/2021 12.36, Philippe Mathieu-Daudé wrote:
>> Add a job for a supported feature: capstone.
>>
>> Philippe Mathieu-Daudé (2):
>> docker: Install libcastone on Fedora
>> gitlab-ci: Test Fedora capstone package
>>
>> .gitlab-ci.yml
Hi Thomas,
On 1/21/21 6:44 PM, Thomas Huth wrote:
> Since the meson build system rework, the configure script prefers the
> git submodules over the system libraries. So we are testing compilation
> with capstone, fdt and libslirp as a submodule all over the place,
> burning CPU cycles by recompili
From: Klaus Jensen
Firstly, if zoned.max_active is non-zero, zoned.max_open must be less
than or equal to zoned.max_active.
Secondly, if only zones.max_active is set, we have to explicitly set
zones.max_open or we end up with an invalid MAR/MOR configuration. This
is an artifact of the parameter
On 26.01.21 12:18, Kevin Wolf wrote:
Am 26.01.2021 um 11:48 hat Max Reitz geschrieben:
Commit 0afec75734331 removed the 'change' QMP command, so we can no
longer test it in 118.
Fixes: 0afec75734331a0b52fa3aa4235220eda8c7846f
('qmp: remove deprecated "change" command')
Signed-off-by: Ma
* James Bottomley (j...@linux.ibm.com) wrote:
> OVMF is developing a mechanism for depositing a GUIDed table just
> below the known location of the reset vector. The table goes
> backwards in memory so all entries are of the form
>
> |len|
>
> Where is arbtrary size and type, is a uint16_t and
Am 26.01.2021 um 12:30 hat Paolo Bonzini geschrieben:
> On 26/01/21 11:48, Max Reitz wrote:
> > Commit 0afec75734331 removed the 'change' QMP command, so we can no
> > longer test it in 118.
> >
> > Fixes: 0afec75734331a0b52fa3aa4235220eda8c7846f
> > ('qmp: remove deprecated "change" comma
* James Bottomley (j...@linux.ibm.com) wrote:
> If the gpa isn't specified, it's value is extracted from the OVMF
> properties table located below the reset vector (and if this doesn't
> exist, an error is returned). OVMF has defined the GUID for the SEV
> secret area as 4c2eb361-7d9b-4cc3-8081-12
ccd3b3b8112 has deprecated short-hand boolean options (i.e., options
with values). All options without values are interpreted as boolean
options, so this includes the invalid option "snapshot.foo" used in
iotest 178.
So after ccd3b3b8112, 178 fails with:
+qemu-img: warning: short-form boolean
Markus Armbruster (2):
backends/dbus-vmstate: Fix short read error handling
vhost_user_gpu: Drop dead check for g_malloc() failure
backends/dbus-vmstate.c | 5 -
hw/display/vhost-user-gpu.c | 1 -
2 files changed, 4 insertions(+), 2 deletions(-)
--
2.26.2
On 26/01/2021 13.08, Philippe Mathieu-Daudé wrote:
Hi Thomas,
On 1/21/21 6:44 PM, Thomas Huth wrote:
Since the meson build system rework, the configure script prefers the
git submodules over the system libraries. So we are testing compilation
with capstone, fdt and libslirp as a submodule all o
When dbus_vmstate_post_load() fails, it complains to stderr. Except
on short read, where it checks with g_return_val_if_fail(). This
fails silently if G_DISABLE_CHECKS is undefined (it should be), or
else pads the short read with uninitialized bytes.
Replace g_return_val_if_fail() by a proper er
Signed-off-by: Markus Armbruster
---
hw/display/vhost-user-gpu.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/hw/display/vhost-user-gpu.c b/hw/display/vhost-user-gpu.c
index 51f1747c4a..db042c7c5e 100644
--- a/hw/display/vhost-user-gpu.c
+++ b/hw/display/vhost-user-gpu.c
@@ -332,7 +332,6 @@
Am 25.01.2021 um 19:50 hat Vladimir Sementsov-Ogievskiy geschrieben:
> v9:
> 01: new, one more whitespace-fixing patch
> testenv: allow case when we don't have system-arch emulator, but have several
> for another architectures
> change direct os.access(..., os.X_OK) calls to new helper fu
25.01.2021 15:03, Max Reitz wrote:
Disposition (action) for any given signal is global for the process.
When two threads run coroutine-sigaltstack's qemu_coroutine_new()
concurrently, they may interfere with each other: One of them may revert
the SIGUSR2 handler to SIG_DFL, between the other thre
On Tue, Jan 26, 2021 at 4:43 PM Markus Armbruster wrote:
>
> When dbus_vmstate_post_load() fails, it complains to stderr. Except
> on short read, where it checks with g_return_val_if_fail(). This
> fails silently if G_DISABLE_CHECKS is undefined (it should be), or
> else pads the short read with
On Tue, Jan 26, 2021 at 4:47 PM Markus Armbruster wrote:
>
> Signed-off-by: Markus Armbruster
Reviewed-by: Marc-André Lureau
thanks
> ---
> hw/display/vhost-user-gpu.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/hw/display/vhost-user-gpu.c b/hw/display/vhost-user-gpu.c
> index 5
** Tags added: fuzzer
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1879531
Title:
Stack-overflow in _eth_get_rss_ex_dst_addr
Status in QEMU:
New
Bug description:
Hello,
While fuzzing, I fo
On 1/26/21 11:27 AM, Paolo Bonzini wrote:
> Meson's "static" argument to cc.find_library is a tri-state. By default
> Meson *prefers* a shared library, which basically means using -l to
> look for it; instead, "static: false" *requires* a shared library. Of
> course, "static: true" requires a sta
On 26.01.21 13:44, Vladimir Sementsov-Ogievskiy wrote:
25.01.2021 15:03, Max Reitz wrote:
Disposition (action) for any given signal is global for the process.
When two threads run coroutine-sigaltstack's qemu_coroutine_new()
concurrently, they may interfere with each other: One of them may rever
26.01.2021 15:53, Kevin Wolf wrote:
Am 25.01.2021 um 19:50 hat Vladimir Sementsov-Ogievskiy geschrieben:
v9:
01: new, one more whitespace-fixing patch
testenv: allow case when we don't have system-arch emulator, but have several
for another architectures
change direct os.access(..., o
Do not check for Cocoa framework if the OS is not macOS.
Signed-off-by: Philippe Mathieu-Daudé
---
meson.build | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/meson.build b/meson.build
index 35a9eddf5cf..9a9ee5408b9 100644
--- a/meson.build
+++ b/meson.build
@@ -369,7 +36
On Tue, 26 Jan 2021 at 02:08, Stefan Berger wrote:
>
> Peter,
>
> this PR removes an unused tracepoint from tpm_spapr.
>
>Stefan
>
> The following changes since commit 31ee895047bdcf7387e3570cbd2a473c6f744b08:
>
> Merge remote-tracking branch 'remotes/jasowang/tags/net-pull-request' into
On Tue, 26 Jan 2021 at 13:26, Philippe Mathieu-Daudé wrote:
>
> Do not check for Cocoa framework if the OS is not macOS.
>
> Signed-off-by: Philippe Mathieu-Daudé
> ---
> meson.build | 5 -
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/meson.build b/meson.build
> index 35
Am 26.01.2021 um 14:19 hat Vladimir Sementsov-Ogievskiy geschrieben:
> 26.01.2021 15:53, Kevin Wolf wrote:
> > Am 25.01.2021 um 19:50 hat Vladimir Sementsov-Ogievskiy geschrieben:
> > > v9:
> > > 01: new, one more whitespace-fixing patch
> > > testenv: allow case when we don't have system-arch emul
On 1/26/21 3:57 AM, Thomas Huth wrote:
Currently, our check-system-* jobs are recompiling the whole sources
again. This happens due to the fact that the jobs are checking out
the whole source tree and required submodules again, and only try
to use the "build" directory with the binaries and obj
On Tue, Jan 26, 2021 at 12:18:46PM +0100, Philippe Mathieu-Daudé wrote:
The length field is already contained in the ip6_ext_hdr structure.
Check it direcly in eth_parse_ipv6_hdr() before calling
_eth_get_rss_ex_dst_addr(), which gets a bit simplified.
Reviewed-by: Miroslav Rezanina
Signed-off-
On 1/26/21 2:28 PM, Peter Maydell wrote:
> On Tue, 26 Jan 2021 at 13:26, Philippe Mathieu-Daudé
> wrote:
>>
>> Do not check for Cocoa framework if the OS is not macOS.
>>
>> Signed-off-by: Philippe Mathieu-Daudé
>> ---
>> meson.build | 5 -
>> 1 file changed, 4 insertions(+), 1 deletion(-)
Hi Thomas,
On 1/22/21 5:19 AM, Thomas Huth wrote:
On 21/01/2021 21.46, Wainer dos Santos Moschetta wrote:
On 1/21/21 3:28 PM, Thomas Huth wrote:
On 21/01/2021 19.13, Daniel P. Berrangé wrote:
On Thu, Jan 21, 2021 at 03:05:43PM -0300, Wainer dos Santos
Moschetta wrote:
Hi,
On 1/21/21 7:08 A
On Mon, Jan 25, 2021 at 08:50:23PM +0100, BALATON Zoltan wrote:
> On Mon, 25 Jan 2021, Stefan Hajnoczi wrote:
> > Zoltan reminded me that the 'log' backend prints tids/timestamps and this
> > can
> > be unwanted in some cases. It's easier to look at trace output without them
> > and
> > in some c
Am 26.01.2021 um 13:38 hat Max Reitz geschrieben:
> ccd3b3b8112 has deprecated short-hand boolean options (i.e., options
> with values). All options without values are interpreted as boolean
> options, so this includes the invalid option "snapshot.foo" used in
> iotest 178.
>
> So after ccd3b3b81
On 26/01/21 14:47, Philippe Mathieu-Daudé wrote:
-cocoa = dependency('appleframeworks', modules: 'Cocoa', required:
get_option('cocoa'))
+cocoa = not_found
+if targetos == 'darwin'
+ cocoa = dependency('appleframeworks', modules: 'Cocoa', required:
get_option('cocoa'))
+endif
if cocoa.found
On Tue, Jan 26, 2021 at 01:42:40PM +0100, Markus Armbruster wrote:
> Signed-off-by: Markus Armbruster
Reviewed-by: Michael S. Tsirkin
> ---
> hw/display/vhost-user-gpu.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/hw/display/vhost-user-gpu.c b/hw/display/vhost-user-gpu.c
> index 5
On Tue, Jan 19, 2021 at 09:45:19AM +0100, Stefano Garzarella wrote:
> On Mon, Jan 18, 2021 at 04:03:12PM +, Dr. David Alan Gilbert wrote:
> > * Stefano Garzarella (sgarz...@redhat.com) wrote:
> > > Commit 9d7bd0826f introduced a new 'use-disabled-flag' property
> > > set to true by default.
> >
1 - 100 of 304 matches
Mail list logo
