On 6/6/20 11:03 AM, Erik Smit wrote:
> The hardware supports configurable descriptor sizes, configured in the DBLAC
> register.
>
> Most drivers use the default 4 word descriptor, which is currently hardcoded,
> but Aspeed SDK configures 8 words to store extra data.
>
> ---
> The implementation o
** Changed in: qemu
Status: New => Confirmed
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1874073
Title:
openrisc_sim.c:87:42: error: 'cpu_irqs[0]' may be used uninitialized
in this funct
On 6/8/20 8:03 AM, Markus Armbruster wrote:
> Markus Armbruster writes:
>
>> Peter Maydell writes:
>>
>>> On Fri, 29 May 2020 at 17:23, Christophe de Dinechin
>>> wrote:
On 2020-05-26 at 20:51 CEST, Eric Blake wrote...
> diff --git a/hw/openrisc/openrisc_sim.c b/hw/openrisc/openrisc_si
On 6/8/20 5:45 AM, Emilio G. Cota wrote:
> On Tue, Jun 02, 2020 at 16:46:22 +0100, Alex Bennée wrote:
>> This may well end up being anonymous but it should always be unique.
>>
>> Signed-off-by: Alex Bennée
>> ---
>> include/qemu/qemu-plugin.h | 5 +
>> plugins/api.c | 18
On 6/8/20 6:45 AM, Markus Armbruster wrote:
> Philippe Mathieu-Daudé writes:
>
>> On 2/28/20 6:32 PM, Markus Armbruster wrote:
[...]
>>> warn_reportf_err() is a convenience function to error_prepend(),
>>> warn_report() and free @local_err.
> [...]
>> Why warn_reportf_err() doesn't take a 'Error
On 6/5/20 4:56 PM, Markus Armbruster wrote:
> qdev_prop_set_netdev() fails when the property already has a non-null
> value. Seems to go back to commit 30c367ed44
> "qdev-properties-system.c: Allow vlan or netdev for -device, not
> both", v1.7.0. Board code doesn't expect failure, and crashes:
>
Markus Armbruster writes:
> Peter Maydell writes:
>
>> On Fri, 29 May 2020 at 17:23, Christophe de Dinechin
>> wrote:
>>> On 2020-05-26 at 20:51 CEST, Eric Blake wrote...
>>> > diff --git a/hw/openrisc/openrisc_sim.c b/hw/openrisc/openrisc_sim.c
>>> > index d08ce6181199..95011a8015b4 100644
>>>
On Tue, May 26, 2020 at 09:40:27PM +0100, David CARLIER wrote:
> From b24a6702beb2a4e2a9c1c03b69c6d1dd07d4cf08 Mon Sep 17 00:00:00 2001
> From: David Carlier
> Date: Tue, 26 May 2020 21:35:27 +0100
> Subject: [PATCH] util/oslib: current process full path resolution on MacOS
>
> Using existing lib
On 6/5/20 4:56 PM, Markus Armbruster wrote:
> We always pass &error_abort. Drop the parameter, use &error_abort
> directly.
>
> Cc: Cédric Le Goater
> Cc: Peter Maydell
> Cc: Andrew Jeffery
> Cc: Joel Stanley
> Cc: qemu-...@nongnu.org
> Signed-off-by: Markus Armbruster
Reviewed-by: Philippe
On 6/8/20 7:20 AM, Markus Armbruster wrote:
> Philippe Mathieu-Daudé writes:
>
>> On 6/5/20 4:56 PM, Markus Armbruster wrote:
>>> qdev_prop_set_drive() can fail. None of the other qdev_prop_set_FOO()
>>> can; they abort on error.
>>>
>>> To clean up this inconsistency, rename qdev_prop_set_drive
From: Vladimir Sementsov-Ogievskiy
Sent: Saturday, June 6, 2020 11:18 AM
To: qemu-bl...@nongnu.org
Cc: qemu-devel@nongnu.org ; mre...@redhat.com
; kw...@redhat.com ; ebl...@redhat.com
; Denis Lunev ; Andrey Shinkevich
; Vladimir Sementsov-Ogievskiy
Subject:
Philippe Mathieu-Daudé writes:
> On 6/5/20 4:56 PM, Markus Armbruster wrote:
>> qdev_prop_set_drive() can fail. None of the other qdev_prop_set_FOO()
>> can; they abort on error.
>>
>> To clean up this inconsistency, rename qdev_prop_set_drive() to
>> qdev_prop_set_drive_err(), and create a qde
On 05/06/2020 23.48, Richard Henderson wrote:
> On 6/5/20 1:32 PM, Peter Maydell wrote:
>> On Fri, 5 Jun 2020 at 20:20, Laurent Vivier wrote:
>>> I was thinking this kind of problem would be detected by the travis-ci
>>> builds, but in fact ppc64 and s390 builds don't build other architecture
>>>
Philippe Mathieu-Daudé writes:
[...]
> Why warn_reportf_err() doesn't take a 'Error **err' instead, to set err
> to NULL after freeing *err?
Why doesn't free() take a void ** argument, to set the pointer to null
after freeing what it points to? Why doesn't close() take an int *
argument?
[...]
Hi, Alexandar,
On Mon, Jun 8, 2020 at 4:00 AM Aleksandar Markovic
wrote:
>
> On Sun, Jun 7, 2020 at 3:13 AM chen huacai wrote:
> >
> > On Sat, Jun 6, 2020 at 4:01 AM Aleksandar Markovic
> > wrote:
> > >
> > > суб, 6. јун 2020. у 09:32 Aleksandar Markovic
> > > је написао/ла:
> > > >
> > > > ут
On Fri, Jun 05, 2020 at 13:34:22 -0400, Robert Foley wrote:
> Disable a few tests under CONFIG_TSAN, which
> run into a known TSan issue that results in a hang.
> https://github.com/google/sanitizers/issues/1116
>
> The disabled tests under TSan include all the qtests as well as
> the test-char, t
On Tue, Jun 02, 2020 at 16:46:22 +0100, Alex Bennée wrote:
> This may well end up being anonymous but it should always be unique.
>
> Signed-off-by: Alex Bennée
> ---
> include/qemu/qemu-plugin.h | 5 +
> plugins/api.c | 18 ++
> 2 files changed, 23 insertions(+
On 2020/6/5 11:30, Richard Henderson wrote:
On 6/4/20 7:50 PM, LIU Zhiwei wrote:
So no scalar insns will require changes within a translation block.
Not true -- scalar insns can encode rm into the instruction.
I think there is a error in gen_set_rm
static void gen_set_rm(DisasContext *ctx
On 2020/6/5 10:19, LIU Zhiwei wrote:
On 2020/6/4 21:32, Peter Maydell wrote:
On Thu, 4 Jun 2020 at 13:15, LIU Zhiwei wrote:
I see many UART implementations have a G_IO_OUT | G_IO_HUP callback function.
In hw/serial.c, it is serial_watch_cb, setting by the following code,
s->watch_tag
It's reported an error of implicit conversion from "unsigned long" to
"double" when compiling with Clang 10. Simply make the encoding rate 0
when the encoded_size is 0.
Fixes: e460a4b1a4
Reported-by: Richard Henderson
Signed-off-by: Wei Wang
---
migration/ram.c | 4 +---
1 file changed, 1 inser
ping
On 6/4/2020 4:51 PM, Ying Fang wrote:
Hi Richard,
Recently we are doing some tests on forward migration based on
arm virt machine. And we found the patch below breaks forward
migration compatibility from virt-4.2 to virt-5.0 above machine
type. The patch which breaks this down given by git
@Chris Wulff @Marek Vasut could you please take a look this issue and patch?
Thanks in advance!
-Original Message-
From: Thomas Huth
Sent: Friday, June 5, 2020 3:07 PM
To: Wu, Wentong ; qemu-devel@nongnu.org
Cc: Chris Wulff ; Marek Vasut
Subject: Re: [RFC] hw: nios2: update interrupt_r
Hi Sven, could you review thiw one-line patch?
On 4/18/20 2:25 AM, Marek Vasut wrote:
> The tulip driver claims to emulate dec21143 and it does not emulate dec21142.
> The dec21142 and dec21143 can be discerned by the PCI revision register,
> where dec21142 reports value < 0x20 and dec21143 value
From: "Catherine A. Frederick"
Signed-off-by: Catherine A. Frederick
---
Okay, I removed the bad "fix" on sar_i64, and the asserts in the various
functions.
Crossing my fingers here.
tcg/ppc/tcg-target.inc.c | 34 +-
1 file changed, 29 insertions(+), 5 deletio
Patchew URL:
https://patchew.org/QEMU/1591559185-31287-1-git-send-email-aleksandar.qemu.de...@gmail.com/
Hi,
This series failed the docker-quick@centos7 build test. Please find the testing
commands and
their output below. If you have Docker installed, you can probably reproduce it
locally.
=
On Sun, Jun 7, 2020 at 3:13 AM chen huacai wrote:
>
> On Sat, Jun 6, 2020 at 4:01 AM Aleksandar Markovic
> wrote:
> >
> > суб, 6. јун 2020. у 09:32 Aleksandar Markovic
> > је написао/ла:
> > >
> > > уто, 2. јун 2020. у 04:40 Huacai Chen је
> > > написао/ла:
> > > >
> > > > Add Loongson-3 based
From: Zhang Chen
Previous operation(like vm_start and replication_start_all) will consume
extra time for first forced synchronization, so reduce it in this patch.
Signed-off-by: Zhang Chen
Reviewed-by: zhanghailiang
Reviewed-by: Lukas Straub
Tested-by: Lukas Straub
---
migration/colo.c | 5
From: Huacai Chen
Loongson-3 CPU family include Loongson-3A R1/R2/R3/R4 and Loongson-3B
R1/R2. Loongson-3A R1 is the oldest and its ISA is the smallest, while
Loongson-3A R4 is the newest and its ISA is almost the superset of all
others. To reduce complexity, we just define two CPU types:
1) "Lo
From: Andrea Oliveri
Enable hardware page table walker and CMGCR features for P5600 that
supports both.
Signed-off-by: Andrea Oliveri
Reviewed-by: Aleksandar Markovic
Signed-off-by: Aleksandar Markovic
Message-Id:
---
target/mips/translate_init.inc.c | 9 +
1 file changed, 5 inserti
The original coversion function is used for regular and MSA floating
point instructions handling. Since there are some nuanced differences
between regular and MSA floating point exception handling, provide two
instances of the conversion function, rather than just a single common
one. Inline both i
After demacroing RINT., this macro is not needed anymore.
Reviewed-by: Aleksandar Rikalo
Signed-off-by: Aleksandar Markovic
Message-Id: <20200518200920.17344-15-aleksandar.qemu.de...@gmail.com>
---
target/mips/fpu_helper.c | 13 -
1 file changed, 13 deletions(-)
diff --git a/target
This is just a cosmetic change to enable tools like gcov, gdb,
callgrind, etc. to better display involved source code.
Reviewed-by: Aleksandar Rikalo
Signed-off-by: Aleksandar Markovic
Message-Id: <20200518200920.17344-14-aleksandar.qemu.de...@gmail.com>
---
target/mips/fpu_helper.c | 20 ++
From: Zhang Chen
MIGRATION_STATUS_ACTIVE is invalid here, handle it by default case.
Suggested-by: Lukas Straub
Signed-off-by: Zhang Chen
---
migration/migration.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/migration/migration.c b/migration/migration.c
index 9059238e3d..9958b1520
From: Zhang Chen
No need to reuse MIGRATION_STATUS_ACTIVE boot COLO.
Signed-off-by: Zhang Chen
Reviewed-by: zhanghailiang
---
migration/colo.c | 2 --
migration/migration.c | 17 ++---
2 files changed, 10 insertions(+), 9 deletions(-)
diff --git a/migration/colo.c b/migrati
After demacroing CLASS., this macro is not needed anymore.
Reviewed-by: Aleksandar Rikalo
Signed-off-by: Aleksandar Markovic
Message-Id: <20200518200920.17344-13-aleksandar.qemu.de...@gmail.com>
---
target/mips/fpu_helper.c | 39 ---
1 file changed, 39 deleti
After demacroing ., these macros
are not needed anymore.
Reviewed-by: Aleksandar Rikalo
Signed-off-by: Aleksandar Markovic
Message-Id: <20200518200920.17344-11-aleksandar.qemu.de...@gmail.com>
---
target/mips/fpu_helper.c | 50
1 file changed, 50
From: Zhang Chen
This series optimize some code of COLO, please review.
Zhang Chen (3):
migration/colo: Optimize COLO boot code path
migration/colo: Update checkpoint time lately
migration/migration.c: Remove MIGRATION_STATUS_ACTIVE in
migration_iteration_finish
migration/colo.c
From: Huacai Chen
MIPS has two types of KVM: TE & VZ, and TE is the default type. Now we
can't create a VZ guest in QEMU because it lacks the kvm_type() hook in
MachineClass. Besides, libvirt uses a null-machine to detect the kvm
capability, so by default it will return "KVM not supported" on a V
Use consistently 'l' and 'h' for low and high halves.
Reviewed-by: Aleksandar Rikalo
Signed-off-by: Aleksandar Markovic
Message-Id: <20200518200920.17344-16-aleksandar.qemu.de...@gmail.com>
---
target/mips/fpu_helper.c | 62
1 file changed, 31 in
This is just a cosmetic change to enable tools like gcov, gdb,
callgrind, etc. to better display involved source code.
Reviewed-by: Aleksandar Rikalo
Signed-off-by: Aleksandar Markovic
Message-Id: <20200518200920.17344-10-aleksandar.qemu.de...@gmail.com>
---
target/mips/fpu_helper.c | 44 ++
This is just a cosmetic change to enable tools like gcov, gdb,
callgrind, etc. to better display involved source code.
Reviewed-by: Aleksandar Rikalo
Signed-off-by: Aleksandar Markovic
Message-Id: <20200518200920.17344-8-aleksandar.qemu.de...@gmail.com>
---
target/mips/fpu_helper.c | 40 +++
This is just a cosmetic change to enable tools like gcov, gdb,
callgrind, etc. to better display involved source code.
Reviewed-by: Aleksandar Rikalo
Signed-off-by: Aleksandar Markovic
Message-Id: <20200518200920.17344-9-aleksandar.qemu.de...@gmail.com>
---
target/mips/fpu_helper.c | 44 +++
This is just a cosmetic change to enable tools like gcov, gdb,
callgrind, etc. to better display involved source code.
Reviewed-by: Aleksandar Rikalo
Signed-off-by: Aleksandar Markovic
Message-Id: <20200518200920.17344-7-aleksandar.qemu.de...@gmail.com>
---
target/mips/fpu_helper.c | 41 +++
This is just a cosmetic change to enable tools like gcov, gdb,
callgrind, etc. to better display involved source code.
Reviewed-by: Aleksandar Rikalo
Signed-off-by: Aleksandar Markovic
Message-Id: <20200518200920.17344-12-aleksandar.qemu.de...@gmail.com>
---
target/mips/fpu_helper.c | 70 ++
This is just a cosmetic change to enable tools like gcov, gdb,
callgrind, etc. to better display involved source code.
Reviewed-by: Aleksandar Rikalo
Signed-off-by: Aleksandar Markovic
Message-Id: <20200518200920.17344-3-aleksandar.qemu.de...@gmail.com>
---
target/mips/fpu_helper.c | 37 +++
After demacroing ., this macro is not
needed anymore.
Reviewed-by: Aleksandar Rikalo
Signed-off-by: Aleksandar Markovic
Message-Id: <20200518200920.17344-6-aleksandar.qemu.de...@gmail.com>
---
target/mips/fpu_helper.c | 39 ---
1 file changed, 39 deletions(-)
This is just a cosmetic change to enable tools like gcov, gdb,
callgrind, etc. to better display involved source code.
Reviewed-by: Aleksandar Rikalo
Signed-off-by: Aleksandar Markovic
Message-Id: <20200518200920.17344-5-aleksandar.qemu.de...@gmail.com>
---
target/mips/fpu_helper.c | 37 +++
This is just a cosmetic change to enable tools like gcov, gdb,
callgrind, etc. to better display involved source code.
Reviewed-by: Aleksandar Rikalo
Signed-off-by: Aleksandar Markovic
Message-Id: <20200518200920.17344-4-aleksandar.qemu.de...@gmail.com>
---
target/mips/fpu_helper.c | 37 +++
The following changes since commit 175198ad91d8bac540159705873b4ffe4fb94eab:
Merge remote-tracking branch 'remotes/cohuck/tags/s390x-20200605' into
staging (2020-06-05 17:45:59 +0100)
are available in the git repository at:
https://github.com/AMarkovic/qemu tags/mips-queue-june-07-2020
for
Stefan Brankovic wants to use his new email address for his future
work in QEMU.
CC: Stefan Brankovic
Signed-off-by: Aleksandar Markovic
Reviewed-by: Stefan Brankovic
Message-Id: <20200602085215.12585-3-aleksandar.qemu.de...@gmail.com>
---
.mailmap | 1 +
1 file changed, 1 insertion(+)
diff -
This is just a cosmetic change to enable tools like gcov, gdb,
callgrind, etc. to better display involved source code.
Reviewed-by: Aleksandar Rikalo
Signed-off-by: Aleksandar Markovic
Message-Id: <20200518200920.17344-2-aleksandar.qemu.de...@gmail.com>
---
target/mips/fpu_helper.c | 38 +++
Filip Bozuta wants to use his new email address for his future
work in QEMU.
CC: Filip Bozuta
Signed-off-by: Aleksandar Markovic
Reviewed-by: Filip Bozuta
Message-Id: <20200602085215.12585-2-aleksandar.qemu.de...@gmail.com>
---
.mailmap | 1 +
1 file changed, 1 insertion(+)
diff --git a/.mail
> -Original Message-
> From: Lukas Straub
> Sent: Sunday, June 7, 2020 3:57 AM
> To: Zhang, Chen
> Cc: Dr . David Alan Gilbert ; qemu-dev de...@nongnu.org>; Zhanghailiang ;
> Zhang Chen
> Subject: Re: [PATCH V2 1/2] migration/colo: Optimize COLO boot code path
>
> On Thu, 4 Jun 202
On Sun, 2020-06-07 at 15:51 +0300, Maxim Levitsky wrote:
> On Tue, 2020-06-02 at 17:31 +0200, Kevin Wolf wrote:
> > Am 14.05.2020 um 06:45 hat Klaus Jensen geschrieben:
> > > From: Klaus Jensen
> > >
> > > Changes since v5
> > >
> > > * Prefixed all patches with "hw/block/nvme" t
On Tue, 2020-06-02 at 18:29 +0200, Max Reitz wrote:
> On 18.05.20 14:20, Maxim Levitsky wrote:
> > Hi!
> > Here is the updated series of my patches, incorporating all the feedback I
> > received.
>
> You asked me on IRC what to do to get this series to move forward;
> considering I don’t think th
On Sun, Jun 7, 2020 at 7:21 AM Thomas Huth wrote:
>
> The file tests/qtest/bios-tables-test-allowed-diff.h is currently only
> assigned to the qtest section according MAINTAINERS. However, this file
> normally only gets updated when the ACPI tables changed - something the
> qtest maintainers don't
On Tue, 2020-06-02 at 17:31 +0200, Kevin Wolf wrote:
> Am 14.05.2020 um 06:45 hat Klaus Jensen geschrieben:
> > From: Klaus Jensen
> >
> > Changes since v5
> >
> > * Prefixed all patches with "hw/block/nvme" to avoid confusion with the
> > nvme block driver.
> >
> > * Added pa
I should have included [SeaBIOS]. Sorry.
On 2020-06-07 10:56, Frédéric Pierret (fepitre) wrote:
> Newer distro like CentOS 8 does not include any reference
> to 'python' binary but only 'python2'.
> ---
> Makefile | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Makefile
Newer distro like CentOS 8 does not include any reference
to 'python' binary but only 'python2'.
---
Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Makefile b/Makefile
index c20be15..995dc86 100644
--- a/Makefile
+++ b/Makefile
@@ -22,7 +22,7 @@ LD=$(CROSS_PREFIX)ld
I see, thanks for the clarification.
However sometimes builds usually do tend to work on Ubuntu but fail to
work on Debian since it's not always a 1-1 (as in this case) - so you
might want to consider to keep testing Debian together with Ubuntu.
Regarding the Ubuntu 20 problem - have you tried "e
60 matches
Mail list logo
