Re: [PATCH v8 0/4] Implements the NetBSD Virtual Machine Monitor accelerator

2021-05-08 Thread Reinoud Zandijk
ping? On Wed, Apr 07, 2021 at 04:16:27PM +, Reinoud Zandijk wrote: > The NetBSD team has implemented its new hypervisor called NVMM. It has been > included since NetBSD 9.0 and has been in use now for quite some time. NVMM > adds user-mode capabilities to create and manage virtual

RFC Qemu mipssim patches

2021-02-11 Thread Reinoud Zandijk
devices for it. Thoughts? Reinoud Zandijk, NetBSD developer $NetBSD$ --- hw/mips/Kconfig.orig2020-08-11 19:17:15.0 + +++ hw/mips/Kconfig @@ -20,6 +20,7 @@ config MIPSSIM select ISA_BUS select SERIAL_ISA select MIPSNET +select VIRTIO_MMIO config JAZZ bool

Re: [PATCH v6 1/4] Add NVMM accelerator: configure and build logic

2021-04-01 Thread Reinoud Zandijk
Hi Paolo, thanks for the review! On Thu, Apr 01, 2021 at 10:28:19AM +0200, Paolo Bonzini wrote: > On 31/03/21 22:07, Reinoud Zandijk wrote: > > Signed-off-by: Reinoud Zandijk > > Signed-off-by: Kamil Rytarowski > > Incorrect order for the S-o-b headers, you

Re: [PATCH v6 2/4] Add NVMM accelerator: x86 CPU support

2021-04-01 Thread Reinoud Zandijk
On Thu, Apr 01, 2021 at 10:35:40AM +0200, Paolo Bonzini wrote: > On 31/03/21 22:07, Reinoud Zandijk wrote: > > +void nvmm_vcpu_kick(CPUState *cpu); > > Not defined anywhere. Hmmm, indeed. I think its a leftover of the former patch. Good catch. > > +{ > >

[PATCH v6 4/4] Add NVMM Accelerator: add maintainers for NetBSD/NVMM

2021-03-31 Thread Reinoud Zandijk
Signed-off-by: Reinoud Zandijk Signed-off-by: Kamil Rytarowski --- MAINTAINERS | 11 +++ 1 file changed, 11 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 554be84b32..cf9d81ea7c 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -510,6 +510,15 @@ F: accel/stubs/hax-stub.c F

[PATCH v6 2/4] Add NVMM accelerator: x86 CPU support

2021-03-31 Thread Reinoud Zandijk
Signed-off-by: Reinoud Zandijk Signed-off-by: Kamil Rytarowski --- target/i386/meson.build |1 + target/i386/nvmm/meson.build |4 + target/i386/nvmm/nvmm-accel-ops.c | 111 +++ target/i386/nvmm/nvmm-accel-ops.h | 25 + target/i386/nvmm/nvmm-all.c | 1231

[PATCH v6 3/4] Add NVMM accelerator: acceleration enlightenments

2021-03-31 Thread Reinoud Zandijk
Signed-off-by: Reinoud Zandijk Signed-off-by: Kamil Rytarowski --- include/sysemu/hw_accel.h | 1 + include/sysemu/nvmm.h | 26 ++ target/i386/helper.c | 2 +- 3 files changed, 28 insertions(+), 1 deletion(-) create mode 100644 include/sysemu/nvmm.h diff

[PATCH v6 1/4] Add NVMM accelerator: configure and build logic

2021-03-31 Thread Reinoud Zandijk
Signed-off-by: Reinoud Zandijk Signed-off-by: Kamil Rytarowski --- accel/Kconfig | 3 +++ configure | 12 +--- meson.build | 23 --- meson_options.txt | 2 ++ qemu-options.hx | 8 5 files changed, 38 insertions(+), 10 deletions

[PATCH v6 0/4] Implements the NetBSD Virtual Machine Monitor accelerator

2021-03-31 Thread Reinoud Zandijk
v2: - Included the testing plan as requested by Philippe Mathieu-Daude - Formatting nit fix in qemu-options.hx - Document NVMM in the accel section of qemu-options.hx Signed-off-by: Reinoud Zandijk Signed-off-by: Kamil Rytarowski Reinoud Zandijk (4): Add NVMM accelerator: configure and build l

[PATCH v7 0/4] Implements the NetBSD Virtual Machine Monitor accelerator

2021-04-02 Thread Reinoud Zandijk
phrase old code comments (remove XXX) v1 -> v2: - Included the testing plan as requested by Philippe Mathieu-Daude - Formatting nit fix in qemu-options.hx - Document NVMM in the accel section of qemu-options.hx Signed-off-by: Kamil Rytarowski Signed-off-by: Reinoud Zandijk Reinoud Zandi

[PATCH v7 4/4] Add NVMM Accelerator: add maintainers for NetBSD/NVMM

2021-04-02 Thread Reinoud Zandijk
Signed-off-by: Kamil Rytarowski Signed-off-by: Reinoud Zandijk --- MAINTAINERS | 11 +++ 1 file changed, 11 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 554be84b32..cf9d81ea7c 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -510,6 +510,15 @@ F: accel/stubs/hax-stub.c F

[PATCH v7 2/4] Add NVMM accelerator: x86 CPU support

2021-04-02 Thread Reinoud Zandijk
Signed-off-by: Kamil Rytarowski Signed-off-by: Reinoud Zandijk --- target/i386/meson.build |1 + target/i386/nvmm/meson.build | 10 + target/i386/nvmm/nvmm-accel-ops.c | 111 +++ target/i386/nvmm/nvmm-accel-ops.h | 24 + target/i386/nvmm/nvmm-all.c | 1226

[PATCH v7 3/4] Add NVMM accelerator: acceleration enlightenments

2021-04-02 Thread Reinoud Zandijk
Signed-off-by: Kamil Rytarowski Signed-off-by: Reinoud Zandijk --- include/sysemu/hw_accel.h | 1 + include/sysemu/nvmm.h | 26 ++ target/i386/helper.c | 2 +- 3 files changed, 28 insertions(+), 1 deletion(-) create mode 100644 include/sysemu/nvmm.h diff

[PATCH v7 1/4] Add NVMM accelerator: configure and build logic

2021-04-02 Thread Reinoud Zandijk
Signed-off-by: Kamil Rytarowski Signed-off-by: Reinoud Zandijk --- accel/Kconfig | 3 +++ configure | 8 +++- meson.build | 14 ++ meson_options.txt | 2 ++ qemu-options.hx | 8 5 files changed, 30 insertions(+), 5 deletions(-) diff --git

Re: Windows 10 won't run on default x86_64 machine anymore

2021-03-16 Thread Reinoud Zandijk
On Tue, Mar 16, 2021 at 01:49:57PM +0100, Paolo Bonzini wrote: > On 16/03/21 13:13, Igor Mammedov wrote: > > > Although I don't know about nvmm case, this function also needs to be > > > updated > > > if smi isn't supported. > > can you submit a patch for this please? > nvmm is not part of

Re: Windows 10 won't run on default x86_64 machine anymore

2021-03-16 Thread Reinoud Zandijk
On Mon, Mar 15, 2021 at 06:55:03PM -0700, Isaku Yamahata wrote: > > If I add in '-M pc-i440fx-5.2' it works again with the accelerator. If I add > > in '-M q35' it does work fine with or without the accelerator. > > Anyhow, can you please try "-global PIIX4_PM.smm-compat=on" > (or "-global

Re: Windows 10 won't run on default x86_64 machine anymore

2021-03-16 Thread Reinoud Zandijk
On Mon, Mar 15, 2021 at 06:55:03PM -0700, Isaku Yamahata wrote: > I think tcg case can be explained by x86_machine_is_smm_enabled() > > bool x86_machine_is_smm_enabled(const X86MachineState *x86ms) > ... > if (tcg_enabled() || qtest_enabled()) { > smm_available = true; >

Re: Windows 10 won't run on default x86_64 machine anymore

2021-03-16 Thread Reinoud Zandijk
On Tue, Mar 16, 2021 at 01:53:01PM +0100, Igor Mammedov wrote: > > The image file was downloaded pre-installed from Microsoft for Edge browser > > evaluation. I used it first on Qemu 5.1 IIRC and it kept working in Qemu > > 5.2. > > issue looks similar to: > 0a343a5add7 i386/acpi: restore device

Re: Windows 10 won't run on default x86_64 machine anymore

2021-03-17 Thread Reinoud Zandijk
On Tue, Mar 16, 2021 at 09:34:08PM +0100, Igor Mammedov wrote: > Thanks for reporting it before it was released > please test/review fix that I've just posted: > > "[PATCH for-6.0] x86:acpi:piix4: reinitialize PM1.CNT on reset" Reported-by: Reinoud Zandijk Tested-by: Reinoud Zandijk

Re: Windows 10 won't run on default x86_64 machine anymore

2021-03-17 Thread Reinoud Zandijk
On Wed, Mar 17, 2021 at 09:58:25AM +0100, Reinoud Zandijk wrote: > On Tue, Mar 16, 2021 at 09:34:08PM +0100, Igor Mammedov wrote: > > Thanks for reporting it before it was released > > please test/review fix that I've just posted: > > > > "[PATCH for-6.0] x86:

Re: Windows 10 won't run on default x86_64 machine anymore

2021-03-17 Thread Reinoud Zandijk
On Tue, Mar 16, 2021 at 09:34:08PM +0100, Igor Mammedov wrote: > Thanks for reporting it before it was released > please test/review fix that I've just posted: > > "[PATCH for-6.0] x86:acpi:piix4: reinitialize PM1.CNT on reset" I've tested qemu with the patch, booted the install without

Win10 always takes 100% cpu power even when idle

2021-03-17 Thread Reinoud Zandijk
Hi, when executing Qemu with qemu-system-x86_64 -m 4G -smp cores=2 -M q35 -snapshot \ -drive file=/home/reinoud/Downloads/Win10-demo.raw,format=raw \ -rtc base=localtime,clock=host -spice port=5924,disable-ticketing=on \ -vga qxl -usb -device usb-tablet -net nic -net

Re: Win10 always takes 100% cpu power even when idle

2021-03-17 Thread Reinoud Zandijk
On Wed, Mar 17, 2021 at 04:18:53PM +, Daniel P. Berrangé wrote: > On Wed, Mar 17, 2021 at 05:11:22PM +0100, Reinoud Zandijk wrote: > > when executing Qemu with > > > > qemu-system-x86_64 -m 4G -smp cores=2 -M q35 -snapshot \ > > -drive file=/home/reinoud/Down

Re: [PATCH for-6.0] x86:acpi:piix4: reinitialize PM1.CNT on reset

2021-03-17 Thread Reinoud Zandijk
Reported-by: Reinoud Zandijk Tested-by: Reinoud Zandijk It works fine now with the patch

Is accel/stubs/ still needed?

2021-03-20 Thread Reinoud Zandijk
Hi, a quick question. I forgot to add my accel/stubs/nvmm-stub.h to accel/stubs/meson.build but even when I disable NVMM explicitly or build it on a NetBSD platform that does not support NVMM, it just compiled. So, is it actually still used? With regards, Reinoud signature.asc Description:

Is accel/stubs/ still needed? (resent)

2021-03-20 Thread Reinoud Zandijk
Hi, a quick question. I forgot to add my accel/stubs/nvmm-stub.h to accel/stubs/meson.build but even when I disable NVMM explicitly or build it on a NetBSD platform that does not support NVMM, it just compiled. So, is it actually still used? With regards, Reinoud signature.asc Description:

Windows 10 won't run on default x86_64 machine anymore

2021-03-15 Thread Reinoud Zandijk
Hi, with the introduction of the following patch from 17th of February, Win10 won't boot anymore without explicitly setting the machine to be 5.2 compatible like pc-1440fx-5.2. The default 6.0 will cause it to panic and gives as reason only "ACPI error". It might work to counter a bug in Linux

Re: Win10 always takes 100% cpu power even when idle

2021-03-17 Thread Reinoud Zandijk
Hi, On Wed, Mar 17, 2021 at 05:05:39PM +, Daniel P. Berrangé wrote: > Possibly you need some of the hyperv enlightenments enabled too. Though > I can't recall the precise command line offhand, as I just rely on > virt-install virt-manager to do the right thing for Windows guests. HuperV

Re: Windows 10 won't run on default x86_64 machine anymore

2021-03-15 Thread Reinoud Zandijk
On Mon, Mar 15, 2021 at 06:53:02PM +0100, Igor Mammedov wrote: > Windows 10 1607x64 boots fine when I test it with default machine. > > So > 1) can you provide full QEMU command line used > 2) What Windows build do you use > 3) is it existing guest image (i.e. installed in older QEMU

Re: [PATCH v7 2/4] Add NVMM accelerator: x86 CPU support

2021-04-07 Thread Reinoud Zandijk
On Fri, Apr 02, 2021 at 02:51:41PM -0700, Richard Henderson wrote: > On 4/2/21 1:25 PM, Reinoud Zandijk wrote: > > +i386_softmmu_ss.add(when: 'CONFIG_NVMM', if_true: > > + declare_dependency(link_args: ['-lnvmm']) > > +) > > You shouldn't be playing with link flags dir

[PATCH v8 0/4] Implements the NetBSD Virtual Machine Monitor accelerator

2021-04-07 Thread Reinoud Zandijk
ytarowski Signed-off-by: Reinoud Zandijk Reinoud Zandijk (4): Add NVMM accelerator: configure and build logic Add NVMM accelerator: x86 CPU support Add NVMM accelerator: acceleration enlightenments Add NVMM Accelerator: add maintainers for NetBSD/NVMM MAINTAINERS

[PATCH v8 2/4] Add NVMM accelerator: x86 CPU support

2021-04-07 Thread Reinoud Zandijk
Signed-off-by: Kamil Rytarowski Signed-off-by: Reinoud Zandijk --- target/i386/meson.build |1 + target/i386/nvmm/meson.build |8 + target/i386/nvmm/nvmm-accel-ops.c | 111 +++ target/i386/nvmm/nvmm-accel-ops.h | 24 + target/i386/nvmm/nvmm-all.c | 1226

[PATCH v8 4/4] Add NVMM Accelerator: add maintainers for NetBSD/NVMM

2021-04-07 Thread Reinoud Zandijk
Signed-off-by: Kamil Rytarowski Signed-off-by: Reinoud Zandijk --- MAINTAINERS | 11 +++ 1 file changed, 11 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 554be84b32..cf9d81ea7c 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -510,6 +510,15 @@ F: accel/stubs/hax-stub.c F

[PATCH v8 3/4] Add NVMM accelerator: acceleration enlightenments

2021-04-07 Thread Reinoud Zandijk
Signed-off-by: Kamil Rytarowski Signed-off-by: Reinoud Zandijk --- include/sysemu/hw_accel.h | 1 + include/sysemu/nvmm.h | 26 ++ target/i386/helper.c | 2 +- 3 files changed, 28 insertions(+), 1 deletion(-) create mode 100644 include/sysemu/nvmm.h diff

[PATCH v8 1/4] Add NVMM accelerator: configure and build logic

2021-04-07 Thread Reinoud Zandijk
Signed-off-by: Kamil Rytarowski Signed-off-by: Reinoud Zandijk --- accel/Kconfig | 3 +++ configure | 8 +++- meson.build | 14 ++ meson_options.txt | 2 ++ qemu-options.hx | 8 5 files changed, 30 insertions(+), 5 deletions(-) diff --git

Qemu mipssim patches, mips malta and virtio bugs

2021-02-13 Thread Reinoud Zandijk
Or should I try to contact ppl directly? With regards, Reinoud Zandijk signature.asc Description: PGP signature

applied? Re: [PATCH v1 0/2] Update NVMM support to recent changes, [PATCH v1 1/2] Only check CONFIG_NVMM when NEED_CPU_H is defined, [PATCH v1 2/2] Fix nvmm_ram_block_added() function arguments

2021-08-29 Thread Reinoud Zandijk
Hi :) Have these patches been applied? How can I easily check it without manually checking if they are there in a git pullup? Am I notified normally when patches are applied? With regards, Reinoud On Sun, Jul 18, 2021 at 03:46:48PM +0200, Reinoud Zandijk wrote: > This patchset fixes small N

Re: applied? Re: [PATCH v1 0/2] Update NVMM support to recent changes, [PATCH v1 1/2] Only check CONFIG_NVMM when NEED_CPU_H is defined, [PATCH v1 2/2] Fix nvmm_ram_block_added() function arguments

2021-09-07 Thread Reinoud Zandijk
ping? On Sun, Aug 29, 2021 at 05:39:07PM +0100, Peter Maydell wrote: > On Sun, 29 Aug 2021 at 17:06, Reinoud Zandijk wrote: > > > > Hi :) > > > > Have these patches been applied? How can I easily check it without manually > > checking if they are there in a g

Re: [PATCH v1 1/2] Only check CONFIG_NVMM when NEED_CPU_H is defined

2021-09-08 Thread Reinoud Zandijk
On Tue, Sep 07, 2021 at 06:20:33PM +0200, Philippe Mathieu-Daudé wrote: > On 7/18/21 3:46 PM, Reinoud Zandijk wrote: > > Userland targers will otherwise use a poisoned CONFIG_NVMM > > Typo "targets", but do you mean bsd-user or linux-user? Since its on NetBSD, I guess it

Re: [PATCH 14/16] tests/acceptance/ppc_prep_40p.py: NetBSD 7.1.2 location update

2021-09-30 Thread Reinoud Zandijk
On Mon, Sep 27, 2021 at 05:26:43PM +0200, Philippe Mathieu-Daudé wrote: > On 9/24/21 20:55, Cleber Rosa wrote: > > The NetBSD-7.1.2-prep.iso is no longer available on the CDN, but it's > > still available in the archive. > > > > Let's update its location so that users without the file on cache

[PATCH v1 0/2] Update NVMM support to recent changes

2021-07-18 Thread Reinoud Zandijk
This patchset fixes small NVMM accelerator compilation issues due to changes made in the rest of Qemu since its import. Signed-off-by: Reinoud Zandijk --- Reinoud Zandijk (2): Only check CONFIG_NVMM when NEED_CPU_H is defined Fix nvmm_ram_block_added() function arguments include/sysemu

[PATCH v1 2/2] Fix nvmm_ram_block_added() function arguments

2021-07-18 Thread Reinoud Zandijk
A parameter max_size was added to the RAMBlockNotifier ram_block_added function. Use the max_size for pre allocation of hva space. Signed-off-by: Reinoud Zandijk --- target/i386/nvmm/nvmm-all.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/target/i386/nvmm/nvmm-all.c

[PATCH v1 1/2] Only check CONFIG_NVMM when NEED_CPU_H is defined

2021-07-18 Thread Reinoud Zandijk
Userland targers will otherwise use a poisoned CONFIG_NVMM Signed-off-by: Reinoud Zandijk --- include/sysemu/nvmm.h | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/include/sysemu/nvmm.h b/include/sysemu/nvmm.h index 6d216599b0..833670fccb 100644 --- a/include/sysemu

Re: [PATCH v1 2/2] Fix nvmm_ram_block_added() function arguments

2021-07-19 Thread Reinoud Zandijk
Hi, On Sun, Jul 18, 2021 at 05:38:58PM +0100, Peter Maydell wrote: > On Sun, 18 Jul 2021 at 14:54, Reinoud Zandijk wrote: > > A parameter max_size was added to the RAMBlockNotifier > > ram_block_added function. Use the max_size for pre allocation > > of hva space. > >

Re: [PATCH 08/16] whpx nvmm: Drop useless migrate_del_blocker()

2021-07-22 Thread Reinoud Zandijk
On Tue, Jul 20, 2021 at 02:54:00PM +0200, Markus Armbruster wrote: > There is nothing to delete after migrate_add_blocker() failed. Trying > anyway is safe, but useless. Don't. > > Cc: Sunil Muthuswamy > Cc: Kamil Rytarowski > Cc: Reinoud Zandijk > Signed-off-

Re: [RFC PATCH-for-8.2] .gitlab-ci.d/cirrus.yml: Promote NetBSD job as gating

2023-11-10 Thread Reinoud Zandijk
On Thu, Nov 09, 2023 at 06:15:51PM +0100, Thomas Huth wrote: > On 09/11/2023 17.58, Daniel P. Berrangé wrote: > > On Thu, Nov 09, 2023 at 04:35:56PM +0100, Philippe Mathieu-Daudé wrote: > > > On 9/11/23 16:35, Philippe Mathieu-Daudé wrote: > > > > This Cirrus-CI based job takes ~12min, similarly

Re: [RFC PATCH] docs/about/deprecated: Deprecate 32-bit host systems

2023-02-22 Thread Reinoud Zandijk
On Wed, Feb 22, 2023 at 09:51:57AM +, Daniel P. Berrangé wrote: > On Wed, Feb 22, 2023 at 09:11:13AM +, Bernhard Beschow wrote: > > Are there any plans or ideas to support 128 bit architectures > > such as CHERI in the future? There is already a QEMU fork > > implementing CHERI for RISC V

Re: [RFC PATCH] docs/about/deprecated: Deprecate 32-bit host systems

2023-02-19 Thread Reinoud Zandijk
On Fri, Feb 17, 2023 at 08:22:43AM -1000, Richard Henderson wrote: > On 2/17/23 06:06, Reinoud Zandijk wrote: > > On Fri, Feb 17, 2023 at 11:36:41AM +0100, Markus Armbruster wrote: > > > I feel the discussion petered out without a conclusion. > > > > > > I do

Re: [RFC PATCH] docs/about/deprecated: Deprecate 32-bit host systems

2023-02-19 Thread Reinoud Zandijk
On Fri, Feb 17, 2023 at 12:05:46PM +0100, Stefan Weil wrote: > On Fri, Feb 17, 2023 at 11:36:41AM +0100, Markus Armbruster wrote: > > > Which 32-bit hosts are still useful, and why? > > Citing my previous mail: > >I now checked all downloads of the latests installers since 2022-12-30. > >

Re: [RFC PATCH] docs/about/deprecated: Deprecate 32-bit host systems

2023-02-17 Thread Reinoud Zandijk
nch of 32 bit-only hosts (sparc32, ppc32, armv7, vax, mips32 etc.) that all run Qemu fine. They are all actively maintained and released as part of the main releases. Maintaining 32 bit host support is thus vital for those machines; not everyone runs a 64 bit system. with regards, Reinoud Zandijk

Re: [PATCH 0/7] bsd-user: remove bitrotted NetBSD and OpenBSD bsd-user support

2023-04-01 Thread Reinoud Zandijk
Dear Qemu folks, I'll throw it in the NetBSD group for feedback. With regards, Reinoud On Fri, Mar 31, 2023 at 08:18:26AM -0600, Warner Losh wrote: > The NetBSD and OpenBSD support in bsd-user hasn't built since before the meson > conversion. It's also out of sync with many of the recent

Re: [PATCH v2 05/11] qemu-options: finesse the recommendations around -blockdev

2023-04-06 Thread Reinoud Zandijk
On Tue, Apr 04, 2023 at 06:17:45PM +0200, Kevin Wolf wrote: > Am 04.04.2023 um 17:07 hat Michael Tokarev geschrieben: > > 04.04.2023 16:57, Kevin Wolf пишет: > Maybe -snapshot should error out if -blockdev is in use. You'd generally > expect that either -blockdev is used primarily and snapshots

Re: [RFC PATCH-for-8.2] .gitlab-ci.d/cirrus.yml: Promote NetBSD job as gating

2023-11-11 Thread Reinoud Zandijk
On Fri, Nov 10, 2023 at 10:12:38PM +0100, Reinoud Zandijk wrote: > On Thu, Nov 09, 2023 at 06:15:51PM +0100, Thomas Huth wrote: > > On 09/11/2023 17.58, Daniel P. Berrangé wrote: > > > On Thu, Nov 09, 2023 at 04:35:56PM +0100, Philippe Mathieu-Daudé wrote: ... > > You're r