[PATCH] qemu-img: add support for rate limit in qemu-img convert

2020-10-17 Thread Zhengui li
From: Zhengui Currently, there is no rate limit for qemu-img convert. This may cause the task of qemu-img convert to consume all the bandwidth of the storage. This will affect the IO performance of other processes and virtual machines under shared storage. So we add support for offline rate

[PATCH] qemu-img: add support for offline rate limit in qemu-img commit

2020-10-17 Thread Zhengui li
From: Zhengui Currently, there is no rate limit for qemu-img commit. This may cause the task of qemu-img commit to consume all the bandwidth of the storage. This will affect the IO performance of other processes and virtual machines under shared storage. So we add support for offline rate limit

Re: [PATCH] qemu-img: add support for offline rate limit in qemu-img commit

2020-10-17 Thread no-reply
Patchew URL: https://patchew.org/QEMU/1602999097-24744-1-git-send-email-lizhen...@huawei.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 1602999097-24744-1-git-send-email-lizhen...@huawei.com Subject: [PATCH]

Re: [PATCH] qemu-img: add support for rate limit in qemu-img convert

2020-10-17 Thread no-reply
Patchew URL: https://patchew.org/QEMU/1602999390-21324-1-git-send-email-lizhen...@huawei.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 1602999390-21324-1-git-send-email-lizhen...@huawei.com Subject: [PATCH]

Re: [PATCH] drivers/virt: vmgenid: add vm generation id driver

2020-10-17 Thread Colm MacCarthaigh
On 16 Oct 2020, at 21:02, Jann Horn wrote: On Sat, Oct 17, 2020 at 5:36 AM Willy Tarreau wrote: But in userspace, we just need a simple counter. There's no need for us to worry about anything else, like timestamps or whatever. If we repeatedly fork a paused VM, the forked VMs will see the

Re: [PATCH] drivers/virt: vmgenid: add vm generation id driver

2020-10-17 Thread Willy Tarreau
On Sat, Oct 17, 2020 at 03:40:08AM +0200, Jann Horn wrote: > [adding some more people who are interested in RNG stuff: Andy, Jason, > Theodore, Willy Tarreau, Eric Biggers. also linux-api@, because this > concerns some pretty fundamental API stuff related to RNG usage] > > On Fri, Oct 16, 2020 at

Re: [PATCH] drivers/virt: vmgenid: add vm generation id driver

2020-10-17 Thread Colm MacCarthaigh
On 16 Oct 2020, at 22:01, Jann Horn wrote: On Sat, Oct 17, 2020 at 6:34 AM Colm MacCarthaigh wrote: For user-space, even a single bit would do. We added MADVISE_WIPEONFORK so that userspace libraries can detect fork()/clone() robustly, for the same reasons. It just wipes a page as the

[PATCH v4 1/5] [RISCV_PM] Add J-extension into RISC-V

2020-10-17 Thread Alexey Baturo
Signed-off-by: Alexey Baturo --- target/riscv/cpu.c | 4 target/riscv/cpu.h | 2 ++ 2 files changed, 6 insertions(+) diff --git a/target/riscv/cpu.c b/target/riscv/cpu.c index 0bbfd7f457..fe6bab4a52 100644 --- a/target/riscv/cpu.c +++ b/target/riscv/cpu.c @@ -438,6 +438,9 @@ static void

[PATCH v4 3/5] [RISCV_PM] Print new PM CSRs in QEMU logs

2020-10-17 Thread Alexey Baturo
Signed-off-by: Alexey Baturo --- target/riscv/cpu.c | 25 + 1 file changed, 25 insertions(+) diff --git a/target/riscv/cpu.c b/target/riscv/cpu.c index d63031eb08..5916ebe5c2 100644 --- a/target/riscv/cpu.c +++ b/target/riscv/cpu.c @@ -255,6 +255,31 @@ static void

[PATCH v4 5/5] [RISCV_PM] Implement address masking functions required for RISC-V Pointer Masking extension

2020-10-17 Thread Alexey Baturo
From: Anatoly Parshintsev Signed-off-by: Anatoly Parshintsev --- target/riscv/cpu.h | 19 +++ target/riscv/translate.c | 34 -- 2 files changed, 51 insertions(+), 2 deletions(-) diff --git a/target/riscv/cpu.h b/target/riscv/cpu.h index

[PATCH v4 4/5] [RISCV_PM] Support pointer masking for RISC-V for i/c/f/d/a types of instructions

2020-10-17 Thread Alexey Baturo
Signed-off-by: Alexey Baturo --- target/riscv/insn_trans/trans_rva.c.inc | 3 +++ target/riscv/insn_trans/trans_rvd.c.inc | 2 ++ target/riscv/insn_trans/trans_rvf.c.inc | 2 ++ target/riscv/insn_trans/trans_rvi.c.inc | 2 ++ target/riscv/translate.c| 14 ++ 5

Re: [PULL 07/37] cpus: extract out hax-specific code to target/i386/

2020-10-17 Thread Volker Rümelin
>> Hi Claudio, >> >> is there a reason why you removed current_cpu = cpu; from >> hax_cpu_thread_fn() when you moved that function to target/i386/hax-cpus.c? >> This change broke HAX on Windows. Adding back that line makes it work again. > > Hello Volker, I see the change in the history and it

Re: [PATCH] drivers/virt: vmgenid: add vm generation id driver

2020-10-17 Thread Willy Tarreau
On Sat, Oct 17, 2020 at 08:55:34AM +0200, Jann Horn wrote: > My suggestion is to use a counter *in the UAPI*, not in the hypervisor > protocol. (And as long as that counter can only miss increments in a > cryptographically negligible fraction of cases, everything's fine.) OK I got it now and I

Re: aio_poll() assertion fail on Windows

2020-10-17 Thread Volker Rümelin
> Whilst testing a Windows build of git master of qemu-system-ppc in > MSYS2/MingW64 I noticed the following assertion message in the console after > booting into OpenBIOS and then closing the GTK GUI window without booting a > client OS: > > $ ./qemu-system-ppc > ** >

[PATCH v4 0/5] RISC-V Pointer Masking implementation

2020-10-17 Thread Alexey Baturo
Hi folks, Addressing code style issues that were found by patchew. Also big thanks to Richard Henderson for reviewing the series and giving great comments! Thanks Alexey Baturo (4): [RISCV_PM] Add J-extension into RISC-V [RISCV_PM] Support CSRs required for RISC-V PM extension except for

[PATCH v4 2/5] [RISCV_PM] Support CSRs required for RISC-V PM extension except for ones in hypervisor mode

2020-10-17 Thread Alexey Baturo
Signed-off-by: Alexey Baturo --- target/riscv/cpu.c | 1 + target/riscv/cpu.h | 12 ++ target/riscv/cpu_bits.h | 66 ++ target/riscv/csr.c | 271 4 files changed, 350 insertions(+) diff --git a/target/riscv/cpu.c

Re: [PULL 09/14] qmp: Move dispatcher to a coroutine

2020-10-17 Thread Volker Rümelin
> From: Kevin Wolf > > This moves the QMP dispatcher to a coroutine and runs all QMP command > handlers that declare 'coroutine': true in coroutine context so they > can avoid blocking the main loop while doing I/O or waiting for other > events. > > For commands that are not declared safe to run

Re: [PATCH] drivers/virt: vmgenid: add vm generation id driver

2020-10-17 Thread Willy Tarreau
On Sat, Oct 17, 2020 at 07:01:31AM +0200, Jann Horn wrote: > Microsoft's documentation > (http://go.microsoft.com/fwlink/?LinkId=260709) says that the VM > Generation ID that we get after a fork "is a 128-bit, > cryptographically random integer value". If multiple people use the > same image, it

Re: [PATCH] drivers/virt: vmgenid: add vm generation id driver

2020-10-17 Thread Willy Tarreau
On Sat, Oct 17, 2020 at 07:52:48AM +0200, Jann Horn wrote: > On Sat, Oct 17, 2020 at 7:37 AM Willy Tarreau wrote: > > On Sat, Oct 17, 2020 at 07:01:31AM +0200, Jann Horn wrote: > > > Microsoft's documentation > > > (http://go.microsoft.com/fwlink/?LinkId=260709) says that the VM > > > Generation

Re: [PATCH v2 2/3] grackle: use qdev gpios for PCI IRQs

2020-10-17 Thread David Gibson
On Fri, Oct 16, 2020 at 07:53:10AM +0100, Mark Cave-Ayland wrote: > On 16/10/2020 07:45, Howard Spoelstra wrote: > > > Hi, > > > > I see compilation of the current ppc-for-5.2 branch fail with: > > > > ../hw/pci-host/grackle.c: In function ‘grackle_realize’: > > ../hw/pci-host/grackle.c:68:11:

Re: [PATCH v2 1/3] macio: don't reference serial_hd() directly within the device

2020-10-17 Thread David Gibson
On Fri, Oct 16, 2020 at 08:00:06AM +0100, Mark Cave-Ayland wrote: > On 16/10/2020 01:16, David Gibson wrote: > > > On Tue, Oct 13, 2020 at 12:49:20PM +0100, Mark Cave-Ayland wrote: > > > Instead use qdev_prop_set_chr() to configure the ESCC serial chardevs at > > > the > > > Mac Old World and

Re: [PATCH] drivers/virt: vmgenid: add vm generation id driver

2020-10-17 Thread Jann Horn
On Sat, Oct 17, 2020 at 8:44 AM Willy Tarreau wrote: > On Sat, Oct 17, 2020 at 07:52:48AM +0200, Jann Horn wrote: > > On Sat, Oct 17, 2020 at 7:37 AM Willy Tarreau wrote: > > > On Sat, Oct 17, 2020 at 07:01:31AM +0200, Jann Horn wrote: > > > > Microsoft's documentation > > > >

Re: [PATCH] tests/acceptance: add MIPS record/replay tests

2020-10-17 Thread Philippe Mathieu-Daudé
On 10/16/20 6:50 PM, Philippe Mathieu-Daudé wrote: On 10/16/20 5:40 PM, Philippe Mathieu-Daudé wrote: On 10/15/20 1:25 PM, Pavel Dovgalyuk wrote: This patch adds MIPS-targeted acceptance tests for record/replay functions. Signed-off-by: Pavel Dovgalyuk ---   0 files changed diff --git

Re: [PATCH v2 0/2] target/arm: Fix tlb flush page vs tbi

2020-10-17 Thread Philippe Mathieu-Daudé
On 10/16/20 11:07 PM, Richard Henderson wrote: Since the FAR_ELx fix at 38d931687fa1, it is reported that page granularity flushing is broken. This makes sense, since TCG will record the entire virtual address in its TLB, not simply the 56 significant bits. With no other TCG support, the ARM

Re: [PATCH v11 06/12] linux-user/elfload: Adjust iteration over phdr

2020-10-17 Thread Philippe Mathieu-Daudé
On 10/16/20 8:42 PM, Richard Henderson wrote: The second loop uses a loop induction variable, and the first does not. Transform the first to match the second, to simplify a following patch moving code between them. Signed-off-by: Richard Henderson Reviewed-by: Philippe Mathieu-Daudé ---

Re: [PATCH 1/5] m48t59-isa: remove legacy m48t59_init_isa() function

2020-10-17 Thread Philippe Mathieu-Daudé
On 10/16/20 8:27 PM, Mark Cave-Ayland wrote: This function is no longer used within the codebase. Signed-off-by: Mark Cave-Ayland --- hw/rtc/m48t59-isa.c | 25 - include/hw/rtc/m48t59.h | 2 -- 2 files changed, 27 deletions(-) Reviewed-by: Philippe

Re: [PATCH v3 08/15] hw/misc/bcm2835_cprman: add a PLL channel skeleton implementation

2020-10-17 Thread Philippe Mathieu-Daudé
On 10/10/20 6:05 PM, Philippe Mathieu-Daudé wrote: On 10/10/20 3:57 PM, Luc Michel wrote: PLLs are composed of multiple channels. Each channel outputs one clock signal. They are modeled as one device taking the PLL generated clock as input, and outputting a new clock. A channel shares the CM

Re: [PATCH] tests/acceptance: add MIPS record/replay tests

2020-10-17 Thread Philippe Mathieu-Daudé
On 10/17/20 11:05 AM, Philippe Mathieu-Daudé wrote: On 10/16/20 6:50 PM, Philippe Mathieu-Daudé wrote: On 10/16/20 5:40 PM, Philippe Mathieu-Daudé wrote: On 10/15/20 1:25 PM, Pavel Dovgalyuk wrote: This patch adds MIPS-targeted acceptance tests for record/replay functions. Signed-off-by:

Re: [PATCH 5/5] m48t59: remove legacy m48t59_init() function

2020-10-17 Thread Mark Cave-Ayland
On 17/10/2020 10:53, Philippe Mathieu-Daudé wrote: On 10/16/20 8:27 PM, Mark Cave-Ayland wrote: Now that all of the callers of this function have been switched to use qdev properties, this legacy init function can now be removed. Signed-off-by: Mark Cave-Ayland ---   hw/rtc/m48t59.c

Re: [PULL v2 00/22] Build system + misc changes for 2020-10-16

2020-10-17 Thread Peter Maydell
On Fri, 16 Oct 2020 at 16:56, Paolo Bonzini wrote: > > The following changes since commit 3e40748834923798aa57e3751db13a069e2c617b: > > Merge remote-tracking branch 'remotes/rth/tags/pull-mb-20201014' into > staging (2020-10-15 20:30:24 +0100) > > are available in the Git repository at: > >

Re: [PATCH] drivers/virt: vmgenid: add vm generation id driver

2020-10-17 Thread Jason A. Donenfeld
After discussing this offline with Jann a bit, I have a few general comments on the design of this. First, the UUID communicated by the hypervisor should be consumed by the kernel -- added as another input to the rng -- and then userspace should be notified that it should reseed any userspace

Re: [PULL v2 00/22] Build system + misc changes for 2020-10-16

2020-10-17 Thread Peter Maydell
On Sat, 17 Oct 2020 at 14:09, Peter Maydell wrote: > > On Fri, 16 Oct 2020 at 16:56, Paolo Bonzini wrote: > > > > The following changes since commit 3e40748834923798aa57e3751db13a069e2c617b: > > > > Merge remote-tracking branch 'remotes/rth/tags/pull-mb-20201014' into > > staging (2020-10-15

Re: [PULL v2 00/22] Build system + misc changes for 2020-10-16

2020-10-17 Thread Paolo Bonzini
On 17/10/20 15:09, Peter Maydell wrote: > Traceback (most recent call last): > File "scripts/mtest2make.py", line 37, in > introspect = json.load(sys.stdin) > File "/usr/lib/python3.6/json/__init__.py", line 299, in load > parse_constant=parse_constant,

[PULL 12/44] target/mips/op_helper: Document Invalidate/Writeback opcodes as no-op

2020-10-17 Thread Philippe Mathieu-Daudé
QEMU does not model caches, so there is not much to do with the Invalidate/Writeback opcodes. Make it explicit adding a comment. Suggested-by: Jiaxun Yang Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Reviewed-by: Jiaxun Yang Message-Id:

[PULL 13/44] target/mips/op_helper: Log unimplemented cache opcode

2020-10-17 Thread Philippe Mathieu-Daudé
In case the guest uses a cache opcode we are not expecting, log it to give us a chance to notice it, in case we should actually do something. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Reviewed-by: Jiaxun Yang Message-Id: <20200813181527.22551-4-f4...@amsat.org> ---

[PULL 25/44] hw/mips/jazz: Correct CPU frequencies

2020-10-17 Thread Philippe Mathieu-Daudé
The Magnum 4000PC CPU runs at 100 MHz, and the Acer PICA-61 CPU at ~134 MHz. Signed-off-by: Philippe Mathieu-Daudé Message-Id: <20201012095804.3335117-17-f4...@amsat.org> --- hw/mips/jazz.c | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/hw/mips/jazz.c

[PULL 19/44] target/mips/cpu: Make cp0_count_rate a property

2020-10-17 Thread Philippe Mathieu-Daudé
Since not all CPU implementations use a cores use a CP0 timer at half the frequency of the CPU, make this variable a property. Signed-off-by: Philippe Mathieu-Daudé Message-Id: <20201012095804.3335117-11-f4...@amsat.org> --- target/mips/cpu.h | 9 + target/mips/cpu.c | 19

[PULL 06/44] target/mips: Demacro helpers for MF.

2020-10-17 Thread Philippe Mathieu-Daudé
From: Aleksandar Markovic Remove function definitions via macros to achieve better code clarity. Signed-off-by: Aleksandar Markovic Reviewed-by: Philippe Mathieu-Daudé Message-Id: <1602103041-32017-3-git-send-email-aleksandar.qemu.de...@gmail.com> Signed-off-by: Philippe Mathieu-Daudé ---

[PULL 40/44] MAINTAINERS: Remove myself

2020-10-17 Thread Philippe Mathieu-Daudé
From: Aleksandar Markovic I have been working on project other than QEMU for some time, and would like to devote myself to that project. It is impossible for me to find enough time to perform maintainer's duties with needed meticulousness and patience. I wish prosperous future to QEMU and all

[PULL 17/44] target/mips: Move cp0_count_ns to CPUMIPSState

2020-10-17 Thread Philippe Mathieu-Daudé
Currently the CP0 timer period is fixed at 10 ns, corresponding to a fixed CPU frequency of 200 MHz (using half the speed of the CPU). In few commits we will be able to use a different CPU frequency. In preparation, move the cp0_count_ns variable to CPUMIPSState so we can modify it.

[PULL 16/44] target/mips/cp0_timer: Document TIMER_PERIOD origin

2020-10-17 Thread Philippe Mathieu-Daudé
TIMER_PERIOD value of '10 ns' can be explained looking at commit 6af0bf9c7c3doc, where the CPU frequency is 200 MHz and CP0 default count rate is half the frequency of the CPU. Document that. Signed-off-by: Philippe Mathieu-Daudé Message-Id: <20201012095804.3335117-8-f4...@amsat.org> ---

[PULL 39/44] docs/system: Update MIPS CPU documentation

2020-10-17 Thread Philippe Mathieu-Daudé
From: Huacai Chen Add Loongson-3A CPU models description. Signed-off-by: Huacai Chen Message-Id: <1602059975-10115-10-git-send-email-che...@lemote.com> [PMD: Split patch in 2: CPU / machine] Signed-off-by: Philippe Mathieu-Daudé --- docs/system/cpu-models-mips.rst.inc | 10 -- 1 file

[PULL v2 3/5] tests/9pfs: wipe local 9pfs test directory

2020-10-17 Thread Christian Schoenebeck
Before running the first 9pfs test case, make sure the test directory for running the 9pfs 'local' tests on is entirely empty. For that reason simply delete the test directory (if any) before (re)creating it on test suite startup. Note: The preferable precise behaviour would be the test directory

[PULL 33/44] hw/mips/malta: Use clearer qdev style

2020-10-17 Thread Philippe Mathieu-Daudé
In order to be consistent with the other code base uses, rewrite slightly how the MIPS_MALTA object is created. No logical change. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Message-Id: <20201012160503.3472140-3-f4...@amsat.org> --- hw/mips/malta.c | 8 1

[PULL 29/44] hw/mips/cps: Do not allow use without input clock

2020-10-17 Thread Philippe Mathieu-Daudé
Now than all QOM users provides the input clock, do not allow using a CPS without input clock connected. Signed-off-by: Philippe Mathieu-Daudé Message-Id: <20201012095804.3335117-21-f4...@amsat.org> --- hw/mips/cps.c | 5 + 1 file changed, 5 insertions(+) diff --git a/hw/mips/cps.c

[PULL 43/44] MAINTAINERS: Remove duplicated Malta test entries

2020-10-17 Thread Philippe Mathieu-Daudé
The Malta tests are already covered in the Malta section. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Thomas Huth Message-Id: <20201013101659.3557154-3-f4...@amsat.org> --- MAINTAINERS | 2 -- 1 file changed, 2 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index

Re: [PATCH v11 05/12] linux-user/elfload: Fix coding style in load_elf_image

2020-10-17 Thread Philippe Mathieu-Daudé
On 10/16/20 8:42 PM, Richard Henderson wrote: Fixing this now will clarify following patches. Signed-off-by: Richard Henderson Reviewed-by: Philippe Mathieu-Daudé --- linux-user/elfload.c | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git

[PATCH v1] migration: using trace_ to replace DPRINTF

2020-10-17 Thread Bihong Yu
Signed-off-by: Bihong Yu --- migration/block.c | 36 ++-- migration/page_cache.c | 13 +++-- migration/trace-events | 13 + 3 files changed, 34 insertions(+), 28 deletions(-) diff --git a/migration/block.c b/migration/block.c index

Re: [PATCH 4/5] ppc405_boards: use qdev properties instead of legacy m48t59_init() function

2020-10-17 Thread Philippe Mathieu-Daudé
On 10/16/20 10:38 PM, BALATON Zoltan via wrote: On Fri, 16 Oct 2020, Mark Cave-Ayland wrote: Signed-off-by: Mark Cave-Ayland --- hw/ppc/ppc405_boards.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/hw/ppc/ppc405_boards.c b/hw/ppc/ppc405_boards.c index

Re: [PATCH 4/5] ppc405_boards: use qdev properties instead of legacy m48t59_init() function

2020-10-17 Thread BALATON Zoltan via
On Sat, 17 Oct 2020, Philippe Mathieu-Daudé wrote: On 10/16/20 10:38 PM, BALATON Zoltan via wrote: On Fri, 16 Oct 2020, Mark Cave-Ayland wrote: Signed-off-by: Mark Cave-Ayland --- hw/ppc/ppc405_boards.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git

Re: [PULL 0/5] 9p queue 2020-10-15

2020-10-17 Thread Christian Schoenebeck
On Samstag, 17. Oktober 2020 12:50:13 CEST Peter Maydell wrote: > On Thu, 15 Oct 2020 at 22:04, Christian Schoenebeck > > wrote: > > The following changes since commit 57c98ea9acdcef5021f5671efa6475a5794a51c4: > > Merge remote-tracking branch > >

[PULL 00/44] mips-next patches for 2020-10-17

2020-10-17 Thread Philippe Mathieu-Daudé
The following changes since commit 7daf8f8d011cdd5d3e86930ed2bde969425c790c: Merge remote-tracking branch 'remotes/kevin/tags/for-upstream' into staging= (2020-10-16 17:39:01 +0100) are available in the Git repository at: https://gitlab.com/philmd/qemu.git tags/mips-next-20201017 for you

[PULL 07/44] target/mips: Demacro helpers for .

2020-10-17 Thread Philippe Mathieu-Daudé
From: Aleksandar Markovic Remove function definitions via macros to achieve better code clarity. Signed-off-by: Aleksandar Markovic Reviewed-by: Philippe Mathieu-Daudé Message-Id: <1602103041-32017-4-git-send-email-aleksandar.qemu.de...@gmail.com> Signed-off-by: Philippe Mathieu-Daudé ---

[PULL 02/44] hw/qdev-clock: Display error hint when clock is missing from device

2020-10-17 Thread Philippe Mathieu-Daudé
Instead of directly aborting, display a hint to help the developer figure out the problem (likely trying to connect a clock to a device pre-dating the Clock API, thus not expecting clocks). Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Luc Michel Reviewed-by: Damien Hedde Reviewed-by:

[PULL 04/44] target/mips: Fix some comment spelling errors

2020-10-17 Thread Philippe Mathieu-Daudé
From: zhaolichang There are many spelling errors in the comments in target/mips/. Use spellcheck to check the spelling errors. Signed-off-by: zhaolichang Reviewed-by: David Edmondson Reviewed-by: Philippe Mathieu-Daudé Message-Id: <20201009064449.2336-7-zhaolich...@huawei.com> Signed-off-by:

[PULL 10/44] target/mips: Add loongson-ext lsdc2 group of instructions

2020-10-17 Thread Philippe Mathieu-Daudé
From: Jiaxun Yang LDC2/SDC2 opcodes have been rewritten as "load & store with offset" group of instructions by loongson-ext ASE. This patch add implementation of these instructions: gslbx: load 1 bytes to GPR gslhx: load 2 bytes to GPR gslwx: load 4 bytes to GPR gsldx: load 8 bytes to

[PULL 26/44] hw/mips/cps: Expose input clock and connect it to CPU cores

2020-10-17 Thread Philippe Mathieu-Daudé
Expose a qdev input clock named 'clk-in', and connect it to each core to forward-propagate the clock. Signed-off-by: Philippe Mathieu-Daudé Message-Id: <20201012095804.3335117-18-f4...@amsat.org> --- include/hw/mips/cps.h | 2 ++ hw/mips/cps.c | 4 2 files changed, 6 insertions(+)

[PULL 28/44] hw/mips/malta: Set CPU frequency to 320 MHz

2020-10-17 Thread Philippe Mathieu-Daudé
The CoreLV card with ID 0x420's CPU clocked at 320 MHz. Create a 'cpuclk' output clock and connect it to the CPU input clock. Signed-off-by: Philippe Mathieu-Daudé Message-Id: <20201012095804.3335117-20-f4...@amsat.org> --- hw/mips/malta.c | 19 --- 1 file changed, 16

[PULL v2 2/5] tests/9pfs: introduce local tests

2020-10-17 Thread Christian Schoenebeck
This patch introduces 9pfs test cases using the 9pfs 'local' filesystem driver which reads/writes/creates/deletes real files and directories. In this initial version, there is only one local test which actually only checks if the 9pfs 'local' device was created successfully. Before the 9pfs

[PULL 32/44] hw/mips/malta: Move gt64120 related code together

2020-10-17 Thread Philippe Mathieu-Daudé
The 'empty_slot' region created is related to the gt64120. Move its creation close to the gt64120 instance creation. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Message-Id: <20201012160503.3472140-2-f4...@amsat.org> --- hw/mips/malta.c | 13 ++--- 1 file

[PULL 08/22] add ninja to dockerfiles, CI configurations and test VMs

2020-10-17 Thread Paolo Bonzini
Reviewed-by: Daniel P. Berrangé Acked-by: Alex Bennée Signed-off-by: Paolo Bonzini --- .cirrus.yml| 6 +++--- .travis.yml| 13 + tests/docker/dockerfiles/centos7.docker| 1 +

[PULL 10/22] build: replace ninjatool with ninja

2020-10-17 Thread Paolo Bonzini
Now that the build is done entirely by Meson, there is no need to keep the Makefile conversion. Instead, we can ask Ninja about the targets it exposes and forward them. The main advantages are, from smallest to largest: - reducing the possible namespace pollution within the Makefile - removal

[PULL v3 00/22] Build system + misc changes for 2020-10-16

2020-10-17 Thread Paolo Bonzini
The following changes since commit 3e40748834923798aa57e3751db13a069e2c617b: Merge remote-tracking branch 'remotes/rth/tags/pull-mb-20201014' into staging (2020-10-15 20:30:24 +0100) are available in the Git repository at: https://gitlab.com/bonzini/qemu.git tags/for-upstream for you to

Re: [RFC PATCH v3] target/mips: Increase number of TLB entries on the 34Kf core (16 -> 64)

2020-10-17 Thread Philippe Mathieu-Daudé
On 10/16/20 7:28 PM, Richard Henderson wrote: On 10/16/20 6:33 AM, Philippe Mathieu-Daudé wrote: Per "MIPS32 34K Processor Core Family Software User's Manual, Revision 01.13" page 8 in "Joint TLB (JTLB)" section: "The JTLB is a fully associative TLB cache containing 16, 32, or

Re: [PATCH 3/5] sun4u: use qdev properties instead of legacy m48t59_init() function

2020-10-17 Thread Philippe Mathieu-Daudé
On 10/16/20 8:27 PM, Mark Cave-Ayland wrote: Signed-off-by: Mark Cave-Ayland Reviewed-by: Philippe Mathieu-Daudé --- hw/sparc64/sun4u.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/hw/sparc64/sun4u.c b/hw/sparc64/sun4u.c index ad5ca2472a..05e659c8a4 100644

Re: [PATCH 2/5] sun4m: use qdev properties instead of legacy m48t59_init() function

2020-10-17 Thread Philippe Mathieu-Daudé
On 10/16/20 8:27 PM, Mark Cave-Ayland wrote: Signed-off-by: Mark Cave-Ayland --- hw/sparc/sun4m.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/hw/sparc/sun4m.c b/hw/sparc/sun4m.c index 54a2b2f9ef..a9bb60f2b2 100644 --- a/hw/sparc/sun4m.c +++

Re: [PATCH v1] migration: using trace_ to replace DPRINTF

2020-10-17 Thread Philippe Mathieu-Daudé
On 10/17/20 11:35 AM, Bihong Yu wrote: Signed-off-by: Bihong Yu --- migration/block.c | 36 ++-- migration/page_cache.c | 13 +++-- migration/trace-events | 13 + 3 files changed, 34 insertions(+), 28 deletions(-) ... diff --git

Re: [PULL 0/5] 9p queue 2020-10-15

2020-10-17 Thread Peter Maydell
On Thu, 15 Oct 2020 at 22:04, Christian Schoenebeck wrote: > > The following changes since commit 57c98ea9acdcef5021f5671efa6475a5794a51c4: > > Merge remote-tracking branch 'remotes/kraxel/tags/ui-20201014-pull-request' > into staging (2020-10-14 13:56:06 +0100) > > are available in the Git

Re: [PATCH 2/5] sun4m: use qdev properties instead of legacy m48t59_init() function

2020-10-17 Thread Mark Cave-Ayland
On 17/10/2020 10:42, Philippe Mathieu-Daudé wrote: On 10/16/20 8:27 PM, Mark Cave-Ayland wrote: Signed-off-by: Mark Cave-Ayland ---   hw/sparc/sun4m.c | 8 +++-   1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/hw/sparc/sun4m.c b/hw/sparc/sun4m.c index 54a2b2f9ef..a9bb60f2b2

Re: [PATCH v2 2/3] grackle: use qdev gpios for PCI IRQs

2020-10-17 Thread Mark Cave-Ayland
On 16/10/2020 01:26, BALATON Zoltan via wrote: As said in previous message the i2c and SPD patches are not quite ready yet so I've omitted those from this series, I may rework them later once this part is merged and can rebase the rest on top of that. We would also need your screamer patches

[PULL 11/44] target/mips/op_helper: Convert multiple if() to switch case

2020-10-17 Thread Philippe Mathieu-Daudé
The cache operation is encoded in bits [20:18] of the instruction. The 'op' argument of helper_cache() contains the bits [20:16]. Extract the 3 bits and parse them using a switch case. This allow us to handle multiple cache types (the cache type is encoded in bits [17:16]). Previously the if()

[PULL 42/44] MAINTAINERS: Downgrade MIPS Boston to 'Odd Fixes', fix Paul Burton mail

2020-10-17 Thread Philippe Mathieu-Daudé
Paul's Wavecomp email has been bouncing for months. He told us he "no longer has access to modern MIPS CPUs or Boston hardware, and wouldn't currently have time to spend on them if he did." [1] but "perhaps that might change in the future." [2]. Be fair and downgrade the status of the Boston board

[PULL 15/44] target/mips/cp0_timer: Explicit unit in variable name

2020-10-17 Thread Philippe Mathieu-Daudé
Name variables holding nanoseconds with the '_ns' suffix. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Aleksandar Markovic Message-Id: <20201012095804.3335117-7-f4...@amsat.org> --- target/mips/cp0_timer.c | 19 ++- 1 file changed, 10 insertions(+), 9 deletions(-) diff

Re: io_uring possibly the culprit for qemu hang (linux-5.4.y)

2020-10-17 Thread Ju Hyung Park
Hi Jens. On Sat, Oct 17, 2020 at 3:07 AM Jens Axboe wrote: > > Would be great if you could try 5.4.71 and see if that helps for your > issue. > Oh wow, yeah it did fix the issue. I'm able to reliably turn off and start the VM multiple times in a row. Double checked by confirming QEMU is

[PULL 36/44] hw/mips: Rename TYPE_MIPS_BOSTON to TYPE_BOSTON

2020-10-17 Thread Philippe Mathieu-Daudé
From: Eduardo Habkost This will make the type name constant consistent with the name of the type checking macro. Signed-off-by: Eduardo Habkost Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé Message-Id: <20200902224311.1321159-19-ehabk...@redhat.com> Signed-off-by:

[PULL v2 0/5] 9p queue (previous 2020-10-15)

2020-10-17 Thread Christian Schoenebeck
-20201017 for you to fetch changes up to fa4551e3f4416cc8c62086ac430b1ceb4f03eb6b: tests/9pfs: add local Tmkdir test (2020-10-17 15:58:39 +0200) 9pfs: add tests using local fs driver The currently existing 9pfs test cases are all

Re: io_uring possibly the culprit for qemu hang (linux-5.4.y)

2020-10-17 Thread Jens Axboe
On 10/17/20 8:29 AM, Ju Hyung Park wrote: > Hi Jens. > > On Sat, Oct 17, 2020 at 3:07 AM Jens Axboe wrote: >> >> Would be great if you could try 5.4.71 and see if that helps for your >> issue. >> > > Oh wow, yeah it did fix the issue. > > I'm able to reliably turn off and start the VM multiple

Re: [PULL 0/9] x86 queue, 2020-10-15

2020-10-17 Thread Peter Maydell
On Thu, 15 Oct 2020 at 17:45, Eduardo Habkost wrote: > > The following changes since commit 57c98ea9acdcef5021f5671efa6475a5794a51c4: > > Merge remote-tracking branch 'remotes/kraxel/tags/ui-20201014-pull-request' > into staging (2020-10-14 13:56:06 +0100) > > are available in the Git

Re: [PULL 09/31] hw/core/clock: introduce clock object

2020-10-17 Thread Philippe Mathieu-Daudé
Hi Damien, Peter, On 4/30/20 1:51 PM, Peter Maydell wrote: This object may be used to represent a clock inside a clock tree. A clock may be connected to another clock so that it receives update, through a callback, whenever the source/parent clock is updated. Although only the root clock of a

Re: [PATCH 4/5] ppc405_boards: use qdev properties instead of legacy m48t59_init() function

2020-10-17 Thread Mark Cave-Ayland
On 17/10/2020 11:44, BALATON Zoltan via wrote: On Sat, 17 Oct 2020, Philippe Mathieu-Daudé wrote: On 10/16/20 10:38 PM, BALATON Zoltan via wrote: On Fri, 16 Oct 2020, Mark Cave-Ayland wrote: Signed-off-by: Mark Cave-Ayland --- hw/ppc/ppc405_boards.c | 10 +- 1 file changed, 9

Re: [PATCH v8 0/5] Mac Old World ROM experiment (ppc/mac_* clean ups and loading binary ROM)

2020-10-17 Thread Mark Cave-Ayland
On 16/10/2020 13:19, BALATON Zoltan via wrote: On Fri, 16 Oct 2020, Mark Cave-Ayland wrote: On 16/10/2020 00:47, BALATON Zoltan via wrote: This is the cut down version of the earlier series omitting unfinished patches that I plan to rework later and rebased to Mark's qemu-macppc branch.

Re: [PULL v2 00/22] Build system + misc changes for 2020-10-16

2020-10-17 Thread Paolo Bonzini
On 17/10/20 15:22, Peter Maydell wrote: > I'm updating various build machines and will give the pullreq a > retry, but in the meantime: > > The NetBSD tests/vm setup fails slightly differently: > > ERROR: Could not detect Ninja v1.7 or newer That's because ninja is apparently a chat program on

Re: [PATCH 5/5] m48t59: remove legacy m48t59_init() function

2020-10-17 Thread Philippe Mathieu-Daudé
On 10/17/20 1:19 PM, Mark Cave-Ayland wrote: On 17/10/2020 10:53, Philippe Mathieu-Daudé wrote: On 10/16/20 8:27 PM, Mark Cave-Ayland wrote: Now that all of the callers of this function have been switched to use qdev properties, this legacy init function can now be removed. Signed-off-by:

[PULL 18/44] target/mips/cpu: Calculate the CP0 timer period using the CPU frequency

2020-10-17 Thread Philippe Mathieu-Daudé
The CP0 timer period is a function of the CPU frequency. Start using the default values, which will be replaced by properties in the next commits. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Jiaxun Yang Message-Id: <20201012095804.3335117-10-f4...@amsat.org> --- target/mips/cpu.c | 4

[PULL 05/44] target/mips: Demacro helpers for .

2020-10-17 Thread Philippe Mathieu-Daudé
From: Aleksandar Markovic Remove function definitions via macros to achieve better code clarity. Signed-off-by: Aleksandar Markovic Reviewed-by: Philippe Mathieu-Daudé Message-Id: <1602103041-32017-2-git-send-email-aleksandar.qemu.de...@gmail.com> Signed-off-by: Philippe Mathieu-Daudé ---

[PULL 03/44] hw/core/clock: Add the clock_new helper function

2020-10-17 Thread Philippe Mathieu-Daudé
From: Luc Michel This function creates a clock and parents it to another object with a given name. It calls clock_setup_canonical_path before returning the new clock. This function is useful to create clocks in devices when one doesn't want to expose it at the qdev level (as an input or an

[PULL v2 1/5] tests/9pfs: change qtest name prefix to synth

2020-10-17 Thread Christian Schoenebeck
All existing 9pfs test cases are using the 'synth' fs driver so far, which means they are not accessing real files, but a purely simulated (in RAM only) file system. Let's make this clear by changing the prefix of the individual qtest case names from 'fs/' to 'synth/'. That way they'll be easily

[PULL 24/44] hw/mips/mipssim: Correct CPU frequency

2020-10-17 Thread Philippe Mathieu-Daudé
The MIPSsim machine CPU frequency is too fast running at 200 MHz, while it should be 12 MHz for the 24K and 6 MHz for the 5K core. Ref: Linux commit c78cbf49c4ed ("Support for MIPSsim, the cycle accurate MIPS simulator.") Signed-off-by: Philippe Mathieu-Daudé Message-Id:

[PULL 27/44] hw/mips/boston: Set CPU frequency to 1 GHz

2020-10-17 Thread Philippe Mathieu-Daudé
The I6400 can run at 1 GHz or more. Create a 'cpuclk' output clock and connect it to the CPU input clock. Signed-off-by: Philippe Mathieu-Daudé Message-Id: <20201012095804.3335117-19-f4...@amsat.org> --- hw/mips/boston.c | 13 + 1 file changed, 13 insertions(+) diff --git

[PULL 21/44] target/mips/cpu: Introduce mips_cpu_create_with_clock() helper

2020-10-17 Thread Philippe Mathieu-Daudé
Introduce an helper to create a MIPS CPU and connect it to a reference clock. This helper is not MIPS specific, but so far only MIPS CPUs need it. Suggested-by: Huacai Chen Signed-off-by: Philippe Mathieu-Daudé Message-Id: <20201012095804.3335117-13-f4...@amsat.org> --- target/mips/cpu.h | 12

[PULL 38/44] tests/acceptance: Add MIPS record/replay tests

2020-10-17 Thread Philippe Mathieu-Daudé
From: Pavel Dovgalyuk This patch adds MIPS-targeted acceptance tests for record/replay functions. Signed-off-by: Pavel Dovgalyuk Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé Message-Id: <160276110297.2705.10918105269658307206.stgit@pasha-ThinkPad-X280> [PMD: Moved

[PULL 31/44] hw/mips/malta: Fix FPGA I/O region size

2020-10-17 Thread Philippe Mathieu-Daudé
The FPGA present on the CoreCard has an I/O region 1MiB wide. Refs: - Atlas User’s Manual (Document Number: MD5) - Malta User’s Manual (Document Number: MD00048) Fixes: ea85df72b60 ("mips_malta: convert to memory API") Signed-off-by: Philippe Mathieu-Daudé Message-Id:

[PULL v2 5/5] tests/9pfs: add local Tmkdir test

2020-10-17 Thread Christian Schoenebeck
This test case uses the 9pfs 'local' driver to create a directory and then checks if the expected directory was actually created (as real directory) on host side. This patch introduces a custom split() implementation, because the test code requires non empty array elements as result. For that

Re: [PATCH 4/5] ppc405_boards: use qdev properties instead of legacy m48t59_init() function

2020-10-17 Thread Artyom Tarasenko
On Fri, Oct 16, 2020 at 10:38 PM BALATON Zoltan wrote: > > On Fri, 16 Oct 2020, Mark Cave-Ayland wrote: > > Signed-off-by: Mark Cave-Ayland > > --- > > hw/ppc/ppc405_boards.c | 10 +- > > 1 file changed, 9 insertions(+), 1 deletion(-) > > > > diff --git a/hw/ppc/ppc405_boards.c

Re: [PATCH v11 08/12] linux-user/elfload: Use Error for load_elf_image

2020-10-17 Thread Philippe Mathieu-Daudé
On 10/16/20 8:42 PM, Richard Henderson wrote: This is a bit clearer than open-coding some of this with a bare c string. c -> C? Signed-off-by: Richard Henderson --- linux-user/elfload.c | 37 - 1 file changed, 20 insertions(+), 17 deletions(-)

Re: [PATCH v11 09/12] linux-user/elfload: Use Error for load_elf_interp

2020-10-17 Thread Philippe Mathieu-Daudé
On 10/16/20 8:42 PM, Richard Henderson wrote: This is slightly clearer than just using strerror, though the different forms produced by error_setg_file_open and error_setg_errno isn't entirely convenient. Signed-off-by: Richard Henderson --- linux-user/elfload.c | 15 --- 1 file

Re: [PATCH 5/5] m48t59: remove legacy m48t59_init() function

2020-10-17 Thread Philippe Mathieu-Daudé
On 10/16/20 8:27 PM, Mark Cave-Ayland wrote: Now that all of the callers of this function have been switched to use qdev properties, this legacy init function can now be removed. Signed-off-by: Mark Cave-Ayland --- hw/rtc/m48t59.c | 35 ---

Re: [PULL 0/3] Usb 20201016 patches

2020-10-17 Thread Peter Maydell
On Fri, 16 Oct 2020 at 06:28, Gerd Hoffmann wrote: > > The following changes since commit 57c98ea9acdcef5021f5671efa6475a5794a51c4: > > Merge remote-tracking branch 'remotes/kraxel/tags/ui-20201014-pull-request' > into staging (2020-10-14 13:56:06 +0100) > > are available in the Git repository

Re: [PATCH 4/5] ppc405_boards: use qdev properties instead of legacy m48t59_init() function

2020-10-17 Thread Mark Cave-Ayland
On 17/10/2020 10:45, Philippe Mathieu-Daudé wrote: On 10/16/20 10:38 PM, BALATON Zoltan via wrote: On Fri, 16 Oct 2020, Mark Cave-Ayland wrote: Signed-off-by: Mark Cave-Ayland --- hw/ppc/ppc405_boards.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git

[Bug 1900241] [NEW] [regression][powerpc] some vcpus are found offline inside guest with different vsmt setting from qemu-cmdline and breaks subsequent vcpu hotplug operation (xive)

2020-10-17 Thread Satheesh Rajendran
Public bug reported: Env: Host: Power9 HW ppc64le # lscpu Architecture:ppc64le Byte Order: Little Endian CPU(s): 128 On-line CPU(s) list: 24-31,40-159 Thread(s) per core: 4 Core(s) per socket:

  1   2   >