Re: [PATCH v3 07/18] machine: Add a new function init_apicid_fn in MachineClass

On Tue, 03 Dec 2019 18:37:42 -0600 Babu Moger wrote: > Add a new function init_apicid_fn in MachineClass to initialize the mode > specific handlers to decode the apic ids. > > Signed-off-by: Babu Moger > --- > include/hw/boards.h |1 + > vl.c|3 +++ > 2 files changed,

Re: [PATCH v3 06/18] hw/core: Add core complex id in X86CPU topology

On Tue, 03 Dec 2019 18:37:35 -0600 Babu Moger wrote: > Introduce last level cache id(llc_id) in x86CPU topology. This information is > required to build the topology in EPIC mode. can you add a reference to spec here so one could look for detailed information about this? > Signed-off-by:

Re: [PATCH 2/2] mirror: Don't let an operation wait for itself

On 1/28/20 9:17 AM, Kevin Wolf wrote: mirror_wait_for_free_in_flight_slot() just picks a random operation to wait for. However, when mirror_co_read() waits for free slots, its MirrorOp is already in s->ops_in_flight, so if not enough slots are immediately available, an operation can end up

[Bug 1859656] Re: [2.6] Unable to reboot s390x KVM machine after initial deploy

I doubled check today again with sfeole and he confirmed that it worked for him as well (and not only for me) when he just started using MAAS KVM on s390x and had his initial setup... Anyway, I'm now sure on how much effort we should spent on recreating the old situation (I may try some old

Re: [PATCH 1/2] mirror: Store MirrorOp.co for debuggability

On 1/28/20 9:17 AM, Kevin Wolf wrote: If a coroutine is launched, but the coroutine pointer isn't stored anywhere, debugging any problems inside the coroutine is quite hard. Let's store the coroutine pointer of a mirror operation in MirrorOp to have it available in the debugger. Signed-off-by:

Re: [PATCH v22 5/9] ACPI: Record the Generic Error Status Block address

On Tue, 28 Jan 2020 15:41:10 +0100 Igor Mammedov wrote: > On Wed, 8 Jan 2020 19:32:19 +0800 > Dongjiu Geng wrote: > > in addition to comments of others: > > > Record the GHEB address via fw_cfg file, when recording > > a error to CPER, it will use this address to find out > > Generic Error

Re: python3 script header and rhel8

On Tue, Jan 28, 2020 at 03:44:04PM +, Vladimir Sementsov-Ogievskiy wrote: > Hi all! > > Hmm Qemu dropped support for python2, and anyway python2 is not supported now > at all. > > Still, we have a lot of "#!/usr/bin/env python" headings in our scripts, > which is > unsupported by rhel8 by

Re: [PATCH 00/11] target/hppa patch queue

On Mon, 27 Jan 2020 at 19:01, Richard Henderson wrote: > > Version 4 fixes trivial conflicts with > > commit 4f67d30b5e74e060b8dbe10528829b47345cd6e8 > Author: Marc-André Lureau > Date: Fri Jan 10 19:30:32 2020 +0400 > > qdev: set properties with device_class_set_props() > > > r~ > > > The

Re: python3 script header and rhel8

On 1/28/20 4:44 PM, Vladimir Sementsov-Ogievskiy wrote: Hi all! Hmm Qemu dropped support for python2, and anyway python2 is not supported now at all. Still, we have a lot of "#!/usr/bin/env python" headings in our scripts, which is unsupported by rhel8 by default. So, for example, because of

Re: [PATCH v4 0/4] Improve default object property_add uint helpers

Hi Felipe, On Fri, Jan 24, 2020 at 11:49 AM Felipe Franciosi wrote: > > Hi Marc-Andre and Paolo, > > > On Dec 20, 2019, at 3:15 PM, Marc-André Lureau > > wrote: > > > > Hi > > > > On Thu, Dec 19, 2019 at 10:02 PM Felipe Franciosi > > wrote: > >> > >> This improves the family of

Re: [PATCH v3 04/18] hw/i386: Introduce initialize_topo_info to initialize X86CPUTopoInfo

On Tue, 03 Dec 2019 18:37:21 -0600 Babu Moger wrote: > Initialize all the parameters in one function initialize_topo_info. > > Signed-off-by: Babu Moger > Reviewed-by: Eduardo Habkost > --- > hw/i386/pc.c | 28 +++- > 1 file changed, 15 insertions(+), 13

Re: [PATCH v3 03/18] hw/i386: Consolidate topology functions

On Tue, 03 Dec 2019 18:37:15 -0600 Babu Moger wrote: > Now that we have all the parameters in X86CPUTopoInfo, we can just pass the > structure to calculate the offsets and width. > > Signed-off-by: Babu Moger Reviewed-by: Igor Mammedov > --- > include/hw/i386/topology.h | 64 >

Re: [PATCH v3 02/18] hw/i386: Introduce X86CPUTopoInfo to contain topology info

On Tue, 03 Dec 2019 18:37:08 -0600 Babu Moger wrote: > This is an effort to re-arrange few data structure for better readability. > Add X86CPUTopoInfo which will have all the topology informations required > to build the cpu topology. There is no functional changes. > > Signed-off-by: Babu

python3 script header and rhel8

Hi all! Hmm Qemu dropped support for python2, and anyway python2 is not supported now at all. Still, we have a lot of "#!/usr/bin/env python" headings in our scripts, which is unsupported by rhel8 by default. So, for example, because of such line in tests/qemu-iotests/nbd-fault-injector.py,

Re: Making QEMU easier for management tools and applications

Cc: Eric for netdev_add QAPIfication. Kevin Wolf writes: > Am 28.01.2020 um 11:16 hat Daniel P. Berrangé geschrieben: >> On Mon, Jan 27, 2020 at 11:38:49PM +0100, Paolo Bonzini wrote: >> > Il lun 27 gen 2020, 21:11 John Snow ha scritto: >> > >> > > >> > > > ./qemu-core <> > > { >> > >

Re: [PATCH v22 7/9] ACPI: Record Generic Error Status Block(GESB) table

On Wed, 8 Jan 2020 19:32:21 +0800 Dongjiu Geng wrote: > kvm_arch_on_sigbus_vcpu() error injection uses source_id as > index in etc/hardware_errors to find out Error Status Data > Block entry corresponding to error source. So supported source_id > values should be assigned here and not be changed

Re: [PATCH v2 0/7] ARM virt: Add NVDIMM support

Hi Shameer, On 1/17/20 6:45 PM, Shameer Kolothum wrote: > This series adds NVDIMM support to arm/virt platform. > The series reuses some of the patches posted by Eric > in his earlier attempt here[1]. > > Patch #1 is a fix to the Guest reboot issue on NVDIMM > hot add case described here[2] and

[PATCH 1/2] mirror: Store MirrorOp.co for debuggability

If a coroutine is launched, but the coroutine pointer isn't stored anywhere, debugging any problems inside the coroutine is quite hard. Let's store the coroutine pointer of a mirror operation in MirrorOp to have it available in the debugger. Signed-off-by: Kevin Wolf --- block/mirror.c | 2 ++

[PATCH 2/2] mirror: Don't let an operation wait for itself

mirror_wait_for_free_in_flight_slot() just picks a random operation to wait for. However, when mirror_co_read() waits for free slots, its MirrorOp is already in s->ops_in_flight, so if not enough slots are immediately available, an operation can end up waiting for itself to complete, which results

[PATCH 0/2] mirror: Fix hang (operation waiting for itself)

Kevin Wolf (2): mirror: Store MirrorOp.co for debuggability mirror: Don't let an operation wait for itself block/mirror.c | 23 ++- 1 file changed, 14 insertions(+), 9 deletions(-) -- 2.20.1

Re: [PATCH rc3 01/30] target/avr: Add basic parameters for new AVR platform

Sounds good to me. On Tue, Jan 28, 2020 at 3:49 PM Aleksandar Markovic < aleksandar.m.m...@gmail.com> wrote: > > > On Tuesday, January 28, 2020, Michael Rolnik wrote: > >> Hi all. >> >> I am totally lost in the email. >> Are there any action items for me / someone else ? >> >> Regards, >>

Re: [PATCH v6 2/2] s390: diagnose 318 info reset and migration support

On Tue, 28 Jan 2020 09:37:46 -0500 Collin Walling wrote: > On 1/27/20 12:35 PM, Cornelia Huck wrote: > > On Mon, 27 Jan 2020 11:39:02 -0500 > > Collin Walling wrote: > > > >> On 1/27/20 6:47 AM, Cornelia Huck wrote: > >>> On Fri, 24 Jan 2020 17:14:04 -0500 > >>> Collin Walling wrote: >

Re: [PATCH v2 3/4] linux-user: remove gemu_log from the linux-user tree

Le 17/01/2020 à 20:28, Josh Kunz a écrit : > Now that all uses have been migrated to `qemu_log' it is no longer > needed. > > Signed-off-by: Josh Kunz > --- > linux-user/main.c | 9 - > linux-user/qemu.h | 1 - > 2 files changed, 10 deletions(-) > > diff --git a/linux-user/main.c

Re: [PATCH v2 2/4] linux-user: Use `qemu_log' for strace

Le 17/01/2020 à 20:28, Josh Kunz a écrit : > This change switches linux-user strace logging to use the newer `qemu_log` > logging subsystem rather than the older `gemu_log` (notice the "g") > logger. `qemu_log` has several advantages, namely that it allows logging > to a file, and provides a more

Re: [question] hw/arm/virt: about the default gic-version in accelerated mode

On Tue, Jan 28, 2020 at 02:53:25PM +0100, Auger Eric wrote: > Hi, > > On 1/28/20 1:41 PM, Andrew Jones wrote: > > On Tue, Jan 28, 2020 at 01:34:06PM +0100, Auger Eric wrote: > >> Hi Drew, > >> > >> On 1/28/20 1:29 PM, Andrew Jones wrote: > >>> On Tue, Jan 28, 2020 at 10:52:50AM +, Peter

Re: [PATCH v2 1/4] linux-user: Use `qemu_log' for non-strace logging

Le 17/01/2020 à 20:28, Josh Kunz a écrit : > Since most calls to `gemu_log` are actually logging unimplemented features, > this change replaces most non-strace calls to `gemu_log` with calls to > `qemu_log_mask(LOG_UNIMP, ...)`. This allows the user to easily log to > a file, and to mask out

Re: [PATCH] Handling SIGSETXID used by glibc NPTL setuid/setgid

Le 28/01/2020 à 13:45, Peter Maydell a écrit : > On Tue, 28 Jan 2020 at 12:33, Taylor Simpson wrote: >> >> I proposed a similar patch guarded by #ifdef TARGET_HEXAGON. I understand >> we don't want to break things that are working, and we don't want to open >> the door for a bunch of #ifdef's

Re: [PATCH v22 5/9] ACPI: Record the Generic Error Status Block address

On Wed, 8 Jan 2020 19:32:19 +0800 Dongjiu Geng wrote: in addition to comments of others: > Record the GHEB address via fw_cfg file, when recording > a error to CPER, it will use this address to find out > Generic Error Data Entries and write the error. > > Make the HEST GHES to a GED device.

Re: [GSoC/Outreachy QEMU proposal] Extend support for ioctls in QEMU linux-user mode

On Thu, 23 Jan 2020 at 15:33, Aleksandar Markovic wrote: > PART III: > > a) Develop unit tests for selected ioctls that are already supported in > QEMU. These should probably just be part of the external Linux Test Project test suite. Incidentally a related topic would be to look at recent

Re: [PATCH v6 2/2] s390: diagnose 318 info reset and migration support

On 1/27/20 12:35 PM, Cornelia Huck wrote: > On Mon, 27 Jan 2020 11:39:02 -0500 > Collin Walling wrote: > >> On 1/27/20 6:47 AM, Cornelia Huck wrote: >>> On Fri, 24 Jan 2020 17:14:04 -0500 >>> Collin Walling wrote: >>> DIAGNOSE 0x318 (diag318) is a privileged s390x instruction that must

Re: [PATCH v6 2/2] s390: diagnose 318 info reset and migration support

On 1/28/20 6:24 AM, Cornelia Huck wrote: > On Mon, 27 Jan 2020 18:05:36 -0500 > Collin Walling wrote: > >> On 1/27/20 12:35 PM, Cornelia Huck wrote: >>> On Mon, 27 Jan 2020 11:39:02 -0500 >>> Collin Walling wrote: >>> On 1/27/20 6:47 AM, Cornelia Huck wrote: > On Fri, 24 Jan 2020

Re: [GSoC/Outreachy QEMU proposal] Extend support for ioctls in QEMU linux-user mode

Le 27/01/2020 à 17:30, Aleksandar Markovic a écrit : > > > On Mon, Jan 27, 2020 at 10:30 AM Stefan Hajnoczi > wrote: >> >> On Thu, Jan 23, 2020 at 02:34:01PM +0100, Aleksandar Markovic wrote: >> > *Extend support for ioctls in QEMU linux-user mode* >> > >> > >> > >>

[PATCH v2 3/4] Introduce the NVMM impl

From: Maxime Villard Implements the NetBSD Virtual Machine Monitor (NVMM) target. Which acts as a hypervisor accelerator for QEMU on the NetBSD platform. This enables QEMU much greater speed over the emulated x86_64 path's that are taken on NetBSD today. Signed-off-by: Maxime Villard

[PATCH v2 1/4] Add the NVMM vcpu API

From: Maxime Villard Adds support for the NetBSD Virtual Machine Monitor (NVMM) stubs and introduces the nvmm.h sysemu API for managing the vcpu scheduling and management. Signed-off-by: Maxime Villard Signed-off-by: Kamil Rytarowski Reviewed-by: Sergio Lopez --- accel/stubs/Makefile.objs |

[PATCH v2 4/4] Add the NVMM acceleration enlightenments

From: Maxime Villard Implements the NVMM accelerator cpu enlightenments to actually use the nvmm-all accelerator on NetBSD platforms. Signed-off-by: Maxime Villard Signed-off-by: Kamil Rytarowski Reviewed-by: Sergio Lopez --- cpus.c| 58

[PATCH v2 2/4] Add the NetBSD Virtual Machine Monitor accelerator.

From: Maxime Villard Introduces the configure support for the new NetBSD Virtual Machine Monitor that allows for hypervisor acceleration from usermode components on the NetBSD platform. Signed-off-by: Maxime Villard Signed-off-by: Kamil Rytarowski Reviewed-by: Sergio Lopez --- configure

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

Hello QEMU Community! Over the past year the NetBSD team has been working hard on a new user-mode API for our hypervisor that will be released as part of the upcoming NetBSD 9.0. This new API adds user-mode capabilities to create and manage virtual machines, configure memory mappings for guest

Re: [PATCH] tests/acceptance: Add a test for the canon-a1100 machine

On 1/28/20 10:16 AM, Thomas Huth wrote: On 27/01/2020 21.56, Philippe Mathieu-Daudé wrote: Hey Wainer, On 1/27/20 6:45 PM, Wainer dos Santos Moschetta wrote: On 1/27/20 1:41 PM, Philippe Mathieu-Daudé wrote: On 1/27/20 4:39 PM, Thomas Huth wrote: On 27/01/2020 16.18, Philippe

Re: [question] hw/arm/virt: about the default gic-version in accelerated mode

Hi, On 1/28/20 1:41 PM, Andrew Jones wrote: > On Tue, Jan 28, 2020 at 01:34:06PM +0100, Auger Eric wrote: >> Hi Drew, >> >> On 1/28/20 1:29 PM, Andrew Jones wrote: >>> On Tue, Jan 28, 2020 at 10:52:50AM +, Peter Maydell wrote: On Tue, 28 Jan 2020 at 10:47, Auger Eric wrote: > When

Re: [PATCH rc3 01/30] target/avr: Add basic parameters for new AVR platform

On Tuesday, January 28, 2020, Michael Rolnik wrote: > Hi all. > > I am totally lost in the email. > Are there any action items for me / someone else ? > > Regards, > Michael Rolnik > > I think it would be helpful if we have rc4 with two action items: - remove CONFIG_USER_ONLY references, and

Re: Making QEMU easier for management tools and applications

> > Consider chardev-add. Example: > > > > {"execute": "chardev-add", > > "arguments": {"id": "bar", > >"backend": {"type": "file", > >"data": {"out": "/tmp/bar.log" > Well, first of all: Do we need compatibility? Well, libvirt

Re: [PATCH] tests/acceptance: Add a test for the canon-a1100 machine

On 1/27/20 6:56 PM, Philippe Mathieu-Daudé wrote: Hey Wainer, On 1/27/20 6:45 PM, Wainer dos Santos Moschetta wrote: On 1/27/20 1:41 PM, Philippe Mathieu-Daudé wrote: On 1/27/20 4:39 PM, Thomas Huth wrote: On 27/01/2020 16.18, Philippe Mathieu-Daudé wrote: On 1/27/20 3:41 PM, Thomas Huth

Re: [RFC PATCH] hw/arm/virt: Support NMI injection

Hi Gavin, On 1/28/20 6:48 AM, Gavin Shan wrote: [including more folks into the discussion] On Fri, 17 Jan 2020 at 14:00, Peter Maydell wrote: On Thu, 19 Dec 2019 at 04:06, Gavin Shan wrote: This supports NMI injection for virtual machine and currently it's only supported on GICv3

Re: [PATCH] .travis.yml: Add description to each job

On 1/28/20 11:18 AM, Philippe Mathieu-Daudé wrote: On Tue, Jan 28, 2020 at 1:55 PM Alex Bennée wrote: Thomas Huth writes: On 25/01/2020 19.31, Philippe Mathieu-Daudé wrote: The NAME variable can be used to describe nicely a job (see [*]). As we currently have 32 jobs, use it. This helps

Re: [PATCH rc3 23/30] hw/core/loader: Let load_elf populate the processor-specific flags

On Tuesday, January 28, 2020, BALATON Zoltan wrote: > On Tue, 28 Jan 2020, Aleksandar Markovic wrote: > >> On Sunday, January 26, 2020, Aleksandar Markovic < >> aleksandar.marko...@rt-rk.com> wrote: >> >> From: Philippe Mathieu-Daudé >>> >>> Some platforms (like AVR) need to determine cpu type

Re: [PATCH rc3 01/30] target/avr: Add basic parameters for new AVR platform

Hi all. I am totally lost in the email. Are there any action items for me / someone else ? Regards, Michael Rolnik On Mon, Jan 27, 2020 at 10:54 AM Michael Rolnik wrote: > Thanks for you help guys. > > On Mon, Jan 27, 2020 at 12:55 AM Aleksandar Markovic < > aleksandar.marko...@rt-rk.com>

Re: [PATCH] .travis.yml: Add description to each job

On 1/26/20 5:54 AM, Thomas Huth wrote: On 25/01/2020 19.31, Philippe Mathieu-Daudé wrote: The NAME variable can be used to describe nicely a job (see [*]). As we currently have 32 jobs, use it. This helps for quickly finding a particular job. before:

Re: [PATCH v2 2/4] Add the NetBSD Virtual Machine Monitor accelerator.

On Tue, Jan 28, 2020 at 11:51:33AM +0100, Kamil Rytarowski wrote: > From: Maxime Villard > > Introduces the configure support for the new NetBSD Virtual Machine Monitor > that > allows for hypervisor acceleration from usermode components on the NetBSD > platform. > > Signed-off-by: Maxime

Re: [PATCH] .travis.yml: Add description to each job

On Tue, Jan 28, 2020 at 1:55 PM Alex Bennée wrote: > Thomas Huth writes: > > On 25/01/2020 19.31, Philippe Mathieu-Daudé wrote: > >> The NAME variable can be used to describe nicely a job (see [*]). > >> As we currently have 32 jobs, use it. This helps for quickly > >> finding a particular job.

Re: [PATCH v2 4/7] hw/arm/virt: Add nvdimm hot-plug infrastructure

Hi Shameer, On 1/17/20 6:45 PM, Shameer Kolothum wrote: > From: Kwangwoo Lee > > Prepare pre-plug and plug handlers for NVDIMM support. > Please note nvdimm_support is not yet enabled. > > Signed-off-by: Kwangwoo Lee > Signed-off-by: Eric Auger > Signed-off-by: Shameer Kolothum > --- >

KVM call for agenda for 2020-02-11

Hi Please, send any topic that you are interested in covering. At the end of Monday I will send an email with the agenda or the cancellation of the call, so hurry up. After discussions on the QEMU Summit, we are going to have always open a KVM call where you can add topics. Call details:

Re: [PATCH] .travis.yml: Add description to each job

Thomas Huth writes: > On 25/01/2020 19.31, Philippe Mathieu-Daudé wrote: >> The NAME variable can be used to describe nicely a job (see [*]). >> As we currently have 32 jobs, use it. This helps for quickly >> finding a particular job. >> >> before:

Re: KVM call for agenda for 2020-01-28

Juan Quintela wrote: > Hi > > Please, send any topic that you are interested in covering. Hi As there is no topics on the agenda, call gets cancelled. Happy hacking, Juan. > > At the end of Monday I will send an email with the agenda or the > cancellation of the call, so hurry up. > > After

Re: Making QEMU easier for management tools and applications

Am 28.01.2020 um 13:36 hat Markus Armbruster geschrieben: > Kevin Wolf writes: > > > Am 27.01.2020 um 21:11 hat John Snow geschrieben: > [...] > >> (The argument here is: It's a little harder and a little longer to type, > >> but the benefits from the schema organization may improve productivity

Re: Performance hit in qemu-system-ppc

Hello, cc-ing the ppc list, maintainer and original author of the patch could increase the chances that they actually see your message which they might miss on qemu-devel. David explicitely said he usually cannot read all mails on qemu-devel so likely should be cc-d on all PPC issues (or also

Re: [PATCH] Handling SIGSETXID used by glibc NPTL setuid/setgid

On Tue, 28 Jan 2020 at 12:33, Taylor Simpson wrote: > > I proposed a similar patch guarded by #ifdef TARGET_HEXAGON. I understand we > don't want to break things that are working, and we don't want to open the > door for a bunch of #ifdef's here. However, more than one scenario needs > this

Re: [PATCH rc3 23/30] hw/core/loader: Let load_elf populate the processor-specific flags

On Tue, 28 Jan 2020, Aleksandar Markovic wrote: On Sunday, January 26, 2020, Aleksandar Markovic < aleksandar.marko...@rt-rk.com> wrote: From: Philippe Mathieu-Daudé Some platforms (like AVR) need to determine cpu type by reading the ELF flags (field e_flags oin ELF header). This patch

Re: [PATCH] iscsi: Cap block count from GET LBA STATUS (CVE-2020-1711)

Am 28.01.2020 um 13:30 hat Philippe Mathieu-Daudé geschrieben: > Hi guys, > > (Cc'ing Jon) > > On 1/23/20 5:59 PM, Kevin Wolf wrote: > > Am 23.01.2020 um 13:44 hat Felipe Franciosi geschrieben: > > > When querying an iSCSI server for the provisioning status of blocks (via > > > GET LBA STATUS),

Re: [question] hw/arm/virt: about the default gic-version in accelerated mode

On Tue, Jan 28, 2020 at 01:34:06PM +0100, Auger Eric wrote: > Hi Drew, > > On 1/28/20 1:29 PM, Andrew Jones wrote: > > On Tue, Jan 28, 2020 at 10:52:50AM +, Peter Maydell wrote: > >> On Tue, 28 Jan 2020 at 10:47, Auger Eric wrote: > >>> When arm virt machine is run in accelerated mode with

Re: [PATCH REPOST v3 78/80] hostmem: fix strict bind policy

On Tue, 28 Jan 2020 13:07:40 +0100 Igor Mammedov wrote: > On Mon, 27 Jan 2020 15:41:45 +0100 > Halil Pasic wrote: > > > On Mon, 27 Jan 2020 08:39:25 +0100 > > Igor Mammedov wrote: [..] > > > > > > one should be able to use memory-backend property to make it work > > > instead of -m

Re: Making QEMU easier for management tools and applications

Kevin Wolf writes: > Am 27.01.2020 um 21:11 hat John Snow geschrieben: [...] >> (The argument here is: It's a little harder and a little longer to type, >> but the benefits from the schema organization may improve productivity >> of using QEMU directly instead of harming it.) > > I think this is

Re: [question] hw/arm/virt: about the default gic-version in accelerated mode

Hi Drew, On 1/28/20 1:29 PM, Andrew Jones wrote: > On Tue, Jan 28, 2020 at 10:52:50AM +, Peter Maydell wrote: >> On Tue, 28 Jan 2020 at 10:47, Auger Eric wrote: >>> When arm virt machine is run in accelerated mode with "-cpu host >>> -machine virt", the default gic version is 2. >>> >>> I

RE: [PATCH] Handling SIGSETXID used by glibc NPTL setuid/setgid

I proposed a similar patch guarded by #ifdef TARGET_HEXAGON. I understand we don't want to break things that are working, and we don't want to open the door for a bunch of #ifdef's here. However, more than one scenario needs this capability. Would a suitable compromise be to provide a hook

Re: [PATCH] iscsi: Cap block count from GET LBA STATUS (CVE-2020-1711)

Hi guys, (Cc'ing Jon) On 1/23/20 5:59 PM, Kevin Wolf wrote: Am 23.01.2020 um 13:44 hat Felipe Franciosi geschrieben: When querying an iSCSI server for the provisioning status of blocks (via GET LBA STATUS), Qemu only validates that the response descriptor zero's LBA matches the one requested.

Re: [question] hw/arm/virt: about the default gic-version in accelerated mode

On Tue, Jan 28, 2020 at 10:52:50AM +, Peter Maydell wrote: > On Tue, 28 Jan 2020 at 10:47, Auger Eric wrote: > > When arm virt machine is run in accelerated mode with "-cpu host > > -machine virt", the default gic version is 2. > > > > I understand the rationale with TCG where we don't have

Re: [PATCH] tests/acceptance: Add a test for the canon-a1100 machine

On 27/01/2020 21.56, Philippe Mathieu-Daudé wrote: > Hey Wainer, > > On 1/27/20 6:45 PM, Wainer dos Santos Moschetta wrote: >> On 1/27/20 1:41 PM, Philippe Mathieu-Daudé wrote: >>> On 1/27/20 4:39 PM, Thomas Huth wrote: On 27/01/2020 16.18, Philippe Mathieu-Daudé wrote: > On 1/27/20 3:41

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

On 28.01.2020 13:08, Sergio Lopez wrote: > On Tue, Jan 28, 2020 at 12:11:51PM +0100, Kamil Rytarowski wrote: >> On 28.01.2020 10:10, Sergio Lopez wrote: >>> On Tue, Jan 07, 2020 at 01:53:57PM +0100, Kamil Rytarowski wrote: Hello QEMU Community! Over the past year the NetBSD team has

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

On Tue, Jan 28, 2020 at 12:11:51PM +0100, Kamil Rytarowski wrote: > On 28.01.2020 10:10, Sergio Lopez wrote: > > On Tue, Jan 07, 2020 at 01:53:57PM +0100, Kamil Rytarowski wrote: > >> Hello QEMU Community! > >> > >> Over the past year the NetBSD team has been working hard on a new > >> user-mode

Re: [PATCH REPOST v3 78/80] hostmem: fix strict bind policy

On Mon, 27 Jan 2020 15:41:45 +0100 Halil Pasic wrote: > On Mon, 27 Jan 2020 08:39:25 +0100 > Igor Mammedov wrote: > > > On Fri, 24 Jan 2020 20:17:48 +0100 > > Halil Pasic wrote: > > > > > On Thu, 23 Jan 2020 12:38:43 +0100 > > > Igor Mammedov wrote: > > > > > > > With main RAM now

Re: [PATCH] qxl: introduce hardware revision 5

On 1/28/20 12:28 PM, Gerd Hoffmann wrote: The only difference to hardware revision 4 is that the device doesn't switch to VGA mode in case someone happens to touch a VGA register, which should make things more robust in configurations with multiple vga devices. Swtiching back to VGA mode

Re: [PATCH qemu v5] spapr: Kill SLOF

On Thu, 2020-01-23 at 16:11 +1100, David Gibson wrote: > On Wed, Jan 22, 2020 at 06:14:37PM +1100, Alexey Kardashevskiy wrote: > > On 22/01/2020 17:32, David Gibson wrote: > > > I'm not thinking of "grub" as a separate option - that would be the > > > same as "vof". Using vof + no -kernel we'd

[PATCH] qxl: introduce hardware revision 5

The only difference to hardware revision 4 is that the device doesn't switch to VGA mode in case someone happens to touch a VGA register, which should make things more robust in configurations with multiple vga devices. Swtiching back to VGA mode happens on reset, either full machine reset or qxl

Re: [PATCH v6 2/2] s390: diagnose 318 info reset and migration support

On Mon, 27 Jan 2020 18:05:36 -0500 Collin Walling wrote: > On 1/27/20 12:35 PM, Cornelia Huck wrote: > > On Mon, 27 Jan 2020 11:39:02 -0500 > > Collin Walling wrote: > > > >> On 1/27/20 6:47 AM, Cornelia Huck wrote: > >>> On Fri, 24 Jan 2020 17:14:04 -0500 > >>> Collin Walling wrote: >

Re: [PATCH v3 01/14] dp8393x: Mask EOL bit from descriptor addresses

Hi Finn, On 1/19/20 11:59 PM, Finn Thain wrote: The Least Significant bit of a descriptor address register is used as an EOL flag. It has to be masked when the register value is to be used as an actual address for copying memory around. But when the registers are to be updated the EOL bit

Re: [PATCH v6 2/2] s390: diagnose 318 info reset and migration support

On Mon, 27 Jan 2020 13:52:48 -0500 Collin Walling wrote: > On 1/27/20 1:21 PM, Collin Walling wrote: > > On 1/27/20 12:55 PM, David Hildenbrand wrote: > >> On 27.01.20 18:29, Cornelia Huck wrote: > >>> On Mon, 27 Jan 2020 18:09:11 +0100 > >>> David Hildenbrand wrote: > ... I actually

Re: [RFC PATCH] hw/arm/virt: Support NMI injection

On 2020-01-28 10:59, Peter Maydell wrote: On Tue, 28 Jan 2020 at 10:56, Auger Eric wrote: On 1/28/20 10:25 AM, Marc Zyngier wrote: > You *could* try something like SDEI [1], but that's a pretty terrible > interface too. Thank you for the pointer. There was a patchset recently that had an

Re: [PATCH v3 09/14] dp8393x: Use long-word-aligned RRA pointers in 32-bit mode

On 1/19/20 11:59 PM, Finn Thain wrote: Section 3.4.1 of the datasheet says, The alignment of the RRA is confined to either word or long word boundaries, depending upon the data width mode. In 16-bit mode, the RRA must be aligned to a word boundary (A0 is always zero) and in

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

On 28.01.2020 10:10, Sergio Lopez wrote: > On Tue, Jan 07, 2020 at 01:53:57PM +0100, Kamil Rytarowski wrote: >> Hello QEMU Community! >> >> Over the past year the NetBSD team has been working hard on a new user-mode >> API >> for our hypervisor that will be released as part of the upcoming NetBSD

[Bug 1859656] Re: [2.6] Unable to reboot s390x KVM machine after initial deploy

It took some time (due to travel), but I was now able to do a setup based on the old 2.6.0 version [2.6.0 (7803-g6fc5f26eb- 0ubuntu1~18.04.1)] for testing. And with the combination: $ apt-cache policy maas maas: Installed: 2.6.0-7803-g6fc5f26eb-0ubuntu1~18.04.1 Candidate:

Re: [PATCH v3 13/14] dp8393x: Don't reset Silicon Revision register

On 1/19/20 11:59 PM, Finn Thain wrote: The jazzsonic driver in Linux uses the Silicon Revision register value to probe the chip. The driver fails unless the SR register contains 4. Unfortunately, reading this register in QEMU usually returns 0 because the s->regs[] array gets wiped after a

Re: [PATCH v3 04/14] dp8393x: Have dp8393x_receive() return the packet size

On Tue, Jan 28, 2020 at 12:03 PM Philippe Mathieu-Daudé wrote: > On 1/19/20 11:59 PM, Finn Thain wrote: > > This function re-uses its 'size' argument as a scratch variable. > > Instead, declare a local 'size' variable for that purpose so that the > > function result doesn't get messed up. > > > >

Re: [PATCH v4 2/7] Acceptance tests: Extract _console_interaction()

On 20/01/2020 23:51, Philippe Mathieu-Daudé wrote: Since we are going to re-use the code shared between wait_for_console_pattern() and exec_command_and_wait_for_pattern(), extract the common part into a local function. Tested-by: Niek Linnenbank Signed-off-by: Philippe Mathieu-Daudé

Re: [PATCH v3 04/14] dp8393x: Have dp8393x_receive() return the packet size

On 1/19/20 11:59 PM, Finn Thain wrote: This function re-uses its 'size' argument as a scratch variable. Instead, declare a local 'size' variable for that purpose so that the function result doesn't get messed up. Signed-off-by: Finn Thain Reviewed-by: Philippe Mathieu-Daudé Tested-by: Laurent

Re: [PATCH v3 03/14] dp8393x: Clean up endianness hacks

On 1/19/20 11:59 PM, Finn Thain wrote: According to the datasheet, section 3.4.4, "in 32-bit mode ... the SONIC always writes long words". Therefore, use the same technique for the 'in_use' field that is used everywhere else, and write the full long word. Signed-off-by: Finn Thain Tested-by:

Re: [PATCH v3 10/14] dp8393x: Pad frames to word or long word boundary

On 1/19/20 11:59 PM, Finn Thain wrote: The existing code has a bug where the Remaining Buffer Word Count (RBWC) is calculated with a truncating division, which gives the wrong result for odd-sized packets. Section 1.4.1 of the datasheet says, Once the end of the packet has been reached,

Re: [PATCH v4 3/7] Acceptance tests: Add interrupt_interactive_console_until_pattern()

On 20/01/2020 23:51, Philippe Mathieu-Daudé wrote: We need a function to interrupt interactive consoles. Example: Interrupt U-Boot to set different environment values. Tested-by: Niek Linnenbank Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Liam Merwick Tested-by: Liam Merwick

Re: [PATCH v4 4/7] python/qemu/machine: Allow to use other serial consoles than default

On 21/01/2020 08:20, Philippe Mathieu-Daudé wrote: On 1/21/20 12:51 AM, Philippe Mathieu-Daudé wrote: Currently the QEMU Python module limits the QEMUMachine class to use the first serial console. Some machines/guest might use another console than the first one as the 'boot console'. For

Re: Making QEMU easier for management tools and applications

Am 27.01.2020 um 21:29 hat Dr. David Alan Gilbert geschrieben: > * Kevin Wolf (kw...@redhat.com) wrote: > > A command history already exists for qmp-shell. It's better than bash > > because it doesn't mix QMP history with whatever else I do on my > > computer. > > > > Autocomplete in qmp-shell

Re: [RFC PATCH] hw/arm/virt: Support NMI injection

On Tue, 28 Jan 2020 at 10:56, Auger Eric wrote: > On 1/28/20 10:25 AM, Marc Zyngier wrote: > > You *could* try something like SDEI [1], but that's a pretty terrible > > interface too. > > Thank you for the pointer. There was a patchset recently that had an SDEI implementation, but I would

Re: [question] hw/arm/virt: about the default gic-version in accelerated mode

* Peter Maydell (peter.mayd...@linaro.org) wrote: > On Tue, 28 Jan 2020 at 10:47, Auger Eric wrote: > > When arm virt machine is run in accelerated mode with "-cpu host > > -machine virt", the default gic version is 2. > > > > I understand the rationale with TCG where we don't have MSI ITS > >

Re: [question] hw/arm/virt: about the default gic-version in accelerated mode

Hi Peter, On 1/28/20 11:52 AM, Peter Maydell wrote: > On Tue, 28 Jan 2020 at 10:47, Auger Eric wrote: >> When arm virt machine is run in accelerated mode with "-cpu host >> -machine virt", the default gic version is 2. >> >> I understand the rationale with TCG where we don't have MSI ITS >>

Re: [RFC PATCH] hw/arm/virt: Support NMI injection

Hi Marc, On 1/28/20 10:25 AM, Marc Zyngier wrote: > Gavin, Eric, > > On 2020-01-28 08:05, Auger Eric wrote: >> Hi, >> >> On 1/28/20 7:48 AM, Gavin Shan wrote: >>> [including more folks into the discussion] >>> On Fri, 17 Jan 2020 at 14:00, Peter Maydell wrote: > On Thu, 19 Dec 2019

Re: [question] hw/arm/virt: about the default gic-version in accelerated mode

On Tue, 28 Jan 2020 at 10:47, Auger Eric wrote: > When arm virt machine is run in accelerated mode with "-cpu host > -machine virt", the default gic version is 2. > > I understand the rationale with TCG where we don't have MSI ITS > emulation along with GICv3 so we need to choose GICv2 to get

[PATCH v2 2/4] Add the NetBSD Virtual Machine Monitor accelerator.

From: Maxime Villard Introduces the configure support for the new NetBSD Virtual Machine Monitor that allows for hypervisor acceleration from usermode components on the NetBSD platform. Signed-off-by: Maxime Villard Signed-off-by: Kamil Rytarowski --- configure | 36

[question] hw/arm/virt: about the default gic-version in accelerated mode

Hi Peter, When arm virt machine is run in accelerated mode with "-cpu host -machine virt", the default gic version is 2. I understand the rationale with TCG where we don't have MSI ITS emulation along with GICv3 so we need to choose GICv2 to get GICv2M functionality. However in KVM mode, I

[kvm-unit-tests PATCH v3 14/14] arm/arm64: ITS: pending table migration test

Add two new migration tests. One testing the migration of a topology where collection were unmapped. The second test checks the migration of the pending table. Signed-off-by: Eric Auger --- v2 -> v3: - tests belong to both its and migration groups --- arm/gic.c | 150

[kvm-unit-tests PATCH v3 10/14] arm/arm64: ITS: commands

Implement main ITS commands. The code is largely inherited from the ITS driver. Signed-off-by: Eric Auger --- v2 -> v3: - do not use report() anymore - assert if cmd_write exceeds the queue capacity v1 -> v2: - removed its_print_cmd_state --- arm/Makefile.arm64 | 2 +-

[kvm-unit-tests PATCH v3 12/14] arm/run: Allow Migration tests

Let's link getchar.o to use puts and getchar from the tests. Then allow tests belonging to the migration group to trigger the migration from the test code by putting "migrate" into the uart. Then the code can wait for the migration completion by using getchar(). The __getchar implement is

Re: Making QEMU easier for management tools and applications

Am 28.01.2020 um 11:16 hat Daniel P. Berrangé geschrieben: > On Mon, Jan 27, 2020 at 11:38:49PM +0100, Paolo Bonzini wrote: > > Il lun 27 gen 2020, 21:11 John Snow ha scritto: > > > > > > > > > ./qemu-core < > > { > > > "machine": "Q35", > > > "memory": "2GiB", > > > "accel": "kvm" >

[kvm-unit-tests PATCH v3 08/14] arm/arm64: ITS: its_enable_defaults

its_enable_defaults() is the top init function that allocates the command queue and all the requested tables (device, collection, lpi config and pending tables), enable LPIs at distributor level and ITS level. gicv3_enable_defaults must be called before. Signed-off-by: Eric Auger --- v2 ->

<    1   2   3   4   5   >