Re: [PATCH v2 2/3] target/arm: Use ARMGranuleSize in ARMVAParameters

2022-10-03 Thread Richard Henderson
On 10/3/22 09:23, Peter Maydell wrote: Now we have an enum for the granule size, use it in the ARMVAParameters struct instead of the using16k/using64k bools. Signed-off-by: Peter Maydell --- target/arm/internals.h | 23 +-- target/arm/helper.c| 39

Re: [PATCH 1/1] qxl: add subsystem_vendor_id property

2022-10-03 Thread Denis V. Lunev
On 9/29/22 09:37, Gerd Hoffmann wrote: On Wed, Sep 28, 2022 at 05:52:44PM +0200, Denis V. Lunev wrote: This property is needed for WHQL/inboxing of Windows drivers. We do need to get drivers to be separated by the hypervisor vendors and that should be done as PCI subvendor ID. This patch adds

[PATCH] docs/nuvoton: Update URL for images

2022-10-03 Thread Joel Stanley
openpower.xyz was retired some time ago. The OpenBMC Jenkins is where images can be found these days. Signed-off-by: Joel Stanley --- docs/system/arm/nuvoton.rst | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/system/arm/nuvoton.rst b/docs/system/arm/nuvoton.rst

Re: [PATCH v2] target/sh4: Fix TB_FLAG_UNALIGN

2022-10-03 Thread Yoshinori Sato
On Mon, 03 Oct 2022 02:23:51 +0900, Richard Henderson wrote: > > Ping, or should I create a PR myself? > > r~ Sorry. I can't work this week, so please submit a PR. > > On 9/1/22 07:15, Yoshinori Sato wrote: > > On Thu, 01 Sep 2022 19:15:09 +0900, > > Richard Henderson wrote: > >> > >> The

[PATCH v2 2/3] target/arm: Use ARMGranuleSize in ARMVAParameters

2022-10-03 Thread Peter Maydell
Now we have an enum for the granule size, use it in the ARMVAParameters struct instead of the using16k/using64k bools. Signed-off-by: Peter Maydell --- target/arm/internals.h | 23 +-- target/arm/helper.c| 39 --- target/arm/ptw.c

[PATCH v2 0/3] target/arm: Enforce implemented granule size limits

2022-10-03 Thread Peter Maydell
Arm CPUs support some subset of the granule (page) sizes 4K, 16K and 64K. The guest selects the one it wants using bits in the TCR_ELx registers. If it tries to program these registers with a value that is either reserved or which requests a size that the CPU does not implement, the architecture

Re: [PATCH v6 00/18] tcg: CPUTLBEntryFull and TARGET_TB_PCREL

2022-10-03 Thread Alex Bennée
Richard Henderson writes: > Changes for v6: > * CPUTLBEntryFull is now completely reviewed. You should try a --disable-tcg build because I saw that failing in CI. -- Alex Bennée

[PATCH v2 1/3] target/arm: Don't allow guest to use unimplemented granule sizes

2022-10-03 Thread Peter Maydell
Arm CPUs support some subset of the granule (page) sizes 4K, 16K and 64K. The guest selects the one it wants using bits in the TCR_ELx registers. If it tries to program these registers with a value that is either reserved or which requests a size that the CPU does not implement, the architecture

[PATCH v2 3/3] docs/system/arm/emulation.rst: Report FEAT_GTG support

2022-10-03 Thread Peter Maydell
FEAT_GTG is a change tho the ID register ID_AA64MMFR0_EL1 so that it can report a different set of supported granule (page) sizes for stage 1 and stage 2 translation tables. As of commit c20281b2a5048 we already report the granule sizes that way for '-cpu max', and now we also correctly make

Re: [RFC PATCH v2 09/29] target/ppc: remove generic architecture checks from p9_deliver_interrupt

2022-10-03 Thread Fabiano Rosas
"Matheus K. Ferst" writes: > On 30/09/2022 15:13, Fabiano Rosas wrote: >> Matheus Ferst writes: >> >>> No functional change intended. >>> >>> Signed-off-by: Matheus Ferst >>> --- >>> target/ppc/excp_helper.c | 9 + >>> 1 file changed, 1 insertion(+), 8 deletions(-) >>> >>> diff

Re: [RFC PATCH v2 11/29] target/ppc: add power-saving interrupt masking logic to p9_next_unmasked_interrupt

2022-10-03 Thread Fabiano Rosas
"Matheus K. Ferst" writes: > On 30/09/2022 15:38, Fabiano Rosas wrote: >> Matheus Ferst writes: >> >>> Export p9_interrupt_powersave and use it in p9_next_unmasked_interrupt. >>> >>> Signed-off-by: Matheus Ferst >>> --- >>> Temporarily putting the prototype in internal.h for lack of a better

[PATCH v3 04/13] mac_{old|new}world: Avoid else branch by setting default value

2022-10-03 Thread BALATON Zoltan
Several variables are set in if-else branches where the else branch can be removed by setting a default value at the variable declaration which leads to simlpler code that is easier to follow. Signed-off-by: BALATON Zoltan --- hw/ppc/mac_newworld.c | 19 ---

[PATCH v2 02/13] hw/gpio/meson: Introduce dedicated config switch for hw/gpio/mpc8xxx

2022-10-03 Thread Bernhard Beschow
Having a dedicated config switch makes dependency handling cleaner. Signed-off-by: Bernhard Beschow Reviewed-by: Bin Meng Reviewed-by: Philippe Mathieu-Daudé --- hw/gpio/Kconfig | 3 +++ hw/gpio/meson.build | 2 +- hw/ppc/Kconfig | 1 + 3 files changed, 5 insertions(+), 1 deletion(-)

Re: [PATCH v2 09/13] hw/ppc/e500: Implement pflash handling

2022-10-03 Thread Philippe Mathieu-Daudé via
On 3/10/22 22:31, Bernhard Beschow wrote: Allows e500 boards to have their root file system reside on flash using only builtin devices located in the eLBC memory region. Note that the flash memory area is only created when a -pflash argument is given, and that the size is determined by the

Re: ublk-qcow2: ublk-qcow2 is available

2022-10-03 Thread Stefan Hajnoczi
On Fri, Sep 30, 2022 at 05:24:11PM +0800, Ming Lei wrote: > ublk-qcow2 is available now. Cool, thanks for sharing! > > So far it provides basic read/write function, and compression and snapshot > aren't supported yet. The target/backend implementation is completely > based on io_uring, and

[PATCH v4 6/6] hw/arm/virt: Add 'compact-highmem' property

2022-10-03 Thread Gavin Shan
After the improvement to high memory region address assignment is applied, the memory layout can be changed, introducing possible migration breakage. For example, VIRT_HIGH_PCIE_MMIO memory region is disabled or enabled when the optimization is applied or not, with the following configuration.

[PATCH v4 4/6] hw/arm/virt: Introduce virt_get_high_memmap_enabled() helper

2022-10-03 Thread Gavin Shan
This introduces virt_get_high_memmap_enabled() helper, which returns the pointer to vms->highmem_{redists, ecam, mmio}. The pointer will be used in the subsequent patches. No functional change intended. Signed-off-by: Gavin Shan --- hw/arm/virt.c | 30 +- 1 file

[PULL v3 1/4] Hexagon (target/hexagon) add instruction attributes from archlib

2022-10-03 Thread Taylor Simpson
The imported files from the architecture library have added some instruction attributes. Some of these will be used in a subsequent patch for determing the size of a store. Signed-off-by: Taylor Simpson Acked-by: Richard Henderson Message-Id: <20220920080746.26791-2-tsimp...@quicinc.com> ---

[PATCH v3 06/13] mac_newworld: Clean up creation of Uninorth devices

2022-10-03 Thread BALATON Zoltan
Map regions in ascending order and reorganise code a bit to avoid some casts and move Uninorth parts together. Signed-off-by: BALATON Zoltan --- hw/ppc/mac_newworld.c | 38 ++ 1 file changed, 18 insertions(+), 20 deletions(-) diff --git

[PATCH v3 00/13] Misc ppc/mac machines clean up

2022-10-03 Thread BALATON Zoltan
This series includes some clean ups to mac_newworld and mac_oldworld to make them a bit simpler and more readable, It also removes the shared mac.h file that turns out was more of a random collection of unrelated things. Getting rid of this mac.h improves the locality of device models and reduces

[PATCH v3 11/13] hw/ppc/mac.h: Move PROM and KERNEL defines to board code

2022-10-03 Thread BALATON Zoltan
The PROM_FILENAME and KERNEL_* defines are used by mac_oldworld and mac_newworld but they don't have to be identical so these could be moved to the individual boards. Signed-off-by: BALATON Zoltan Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Mark Cave-Ayland --- hw/ppc/mac.h | 4

[PATCH v2 12/13] hw/sd/sdhci: Implement Freescale eSDHC device model

2022-10-03 Thread Bernhard Beschow
Will allow e500 boards to access SD cards using just their own devices. Signed-off-by: Bernhard Beschow --- hw/sd/sdhci.c | 147 +- include/hw/sd/sdhci.h | 3 + 2 files changed, 149 insertions(+), 1 deletion(-) diff --git a/hw/sd/sdhci.c

Re: [RFC PATCH v2 00/29] PowerPC interrupt rework

2022-10-03 Thread Cédric Le Goater
(qemu) info pic info pic CPU[]:   QW   NSR CPPR IPB LSMFB ACK# INC AGE PIPR  W2 CPU[]: USER    00   00  00    00   00  00  00   00  CPU[]:   OS    00   00  00    ff   ff  00  ff   ff  CPU[]: POOL    00   00  00    ff   00  00  00   00  CPU[]: PHYS   

[PULL 0/8] Trivial branch for 7.2 patches

2022-10-03 Thread Laurent Vivier
The following changes since commit c8de6ec63d766ca1998c5af468483ce912fdc0c2: Merge tag 'pull-request-2022-09-28' of https://gitlab.com/thuth/qemu into staging (2022-09-28 17:04:11 -0400) are available in the Git repository at: https://gitlab.com/laurent_vivier/qemu.git

Re: [PATCH v2 07/13] hw/ppc/e500: Remove if statement which is now always true

2022-10-03 Thread Philippe Mathieu-Daudé via
On 3/10/22 22:31, Bernhard Beschow wrote: Now that the MPC8544DS board also has a platform bus, the if statement is always true. Signed-off-by: Bernhard Beschow Reviewed-by: Bin Meng Reviewed-by: Philippe Mathieu-Daudé --- hw/ppc/e500.c | 30 ++

[PULL 1/8] hw/virtio/vhost-shadow-virtqueue: Silence GCC error "maybe-uninitialized"

2022-10-03 Thread Laurent Vivier
From: Bernhard Beschow GCC issues a false positive warning, resulting in build failure with -Werror: In file included from /usr/include/glib-2.0/glib.h:114, from src/include/glib-compat.h:32, from src/include/qemu/osdep.h:144, from

Re: [PATCH v3] virtio-scsi: Send "REPORTED LUNS CHANGED" sense data upon disk hotplug events.

2022-10-03 Thread Venu Busireddy
On 2022-10-03 18:13:06 -0500, Venu Busireddy wrote: > On 2022-09-30 18:25:48 +0200, Paolo Bonzini wrote: > > On Fri, Sep 30, 2022 at 4:42 PM Venu Busireddy > > wrote: > > > > > Immediately after a hotunplug event, qemu (without any action from > > > > > the guest) processes a REPORT_LUNS command

[PATCH v4 5/6] hw/arm/virt: Improve high memory region address

2022-10-03 Thread Gavin Shan
There are three high memory regions, which are VIRT_HIGH_REDIST2, VIRT_HIGH_PCIE_ECAM and VIRT_HIGH_PCIE_MMIO. Their base addresses are floating on highest RAM address. However, they can be disabled in several cases. (1) One specific high memory region is disabled by developer by toggling

Re: [PATCH v10 3/7] block: add block layer APIs resembling Linux ZonedBlockDevice ioctls

2022-10-03 Thread Stefan Hajnoczi
On Thu, Sep 29, 2022 at 04:36:27PM +0800, Sam Li wrote: > Add a new zoned_host_device BlockDriver. The zoned_host_device option > accepts only zoned host block devices. By adding zone management > operations in this new BlockDriver, users can use the new block > layer APIs including Report Zone

[PULL v3 0/4] Hexagon (target/hexagon) improve store handling and gen_tcg_funcs.py

2022-10-03 Thread Taylor Simpson
The following changes since commit c8de6ec63d766ca1998c5af468483ce912fdc0c2: Merge tag 'pull-request-2022-09-28' of https://gitlab.com/thuth/qemu into staging (2022-09-28 17:04:11 -0400) are available in the Git repository at: https://github.com/quic/qemu tags/pull-hex-20221003 for you

[PATCH v3 02/13] mac_oldworld: Drop some more variables

2022-10-03 Thread BALATON Zoltan
Drop some more local variables additionally to commit b8df32555ce5 to match clean ups done to mac_newwold in previous patch. Signed-off-by: BALATON Zoltan Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Mark Cave-Ayland --- hw/ppc/mac_oldworld.c | 43

[PATCH v2 10/13] hw/sd/sdhci-internal: Unexport ESDHC defines

2022-10-03 Thread Bernhard Beschow
These defines aren't used outside of sdhci.c, so can be defined there. Signed-off-by: Bernhard Beschow Reviewed-by: Bin Meng Reviewed-by: Philippe Mathieu-Daudé --- hw/sd/sdhci-internal.h | 20 hw/sd/sdhci.c | 19 +++ 2 files changed, 19

Re: [Virtio-fs] virtiofsd: Any reason why there's not an "openat2" sandbox mode?

2022-10-03 Thread Colin Walters
On Thu, Sep 29, 2022, at 1:03 PM, Vivek Goyal wrote: > > So rust version of virtiofsd, already supports running unprivileged > (inside a user namespace). I know, but as I already said, the use case here is running inside an OpenShift unprivileged pod where *we are already in a container*. >

Re: [PATCH v3 5/5] hw/arm/virt: Add 'highmem-compact' property

2022-10-03 Thread Gavin Shan
Hi Eric, On 10/3/22 4:49 PM, Eric Auger wrote: On 9/29/22 01:49, Gavin Shan wrote: On 9/28/22 10:22 PM, Eric Auger wrote: On 9/22/22 01:13, Gavin Shan wrote: After the improvement to high memory region address assignment is applied, the memory layout is changed. For example,

[PATCH v4 3/6] hw/arm/virt: Introduce variable region_base in virt_set_high_memmap()

2022-10-03 Thread Gavin Shan
This introduces variable 'region_base' for the base address of the specific high memory region. It's the preparatory work to optimize high memory region address assignment. No functional change intended. Signed-off-by: Gavin Shan Reviewed-by: Eric Auger --- hw/arm/virt.c | 12 ++-- 1

[PATCH v3 13/13] mac_nvram: Use NVRAM_SIZE constant

2022-10-03 Thread BALATON Zoltan
The NVRAM_SIZE constant was defined but not used. Rename it to MACIO_NVRAM_SIZE to match the device model and use it where appropriate. Signed-off-by: BALATON Zoltan Reviewed-by: Mark Cave-Ayland --- hw/misc/macio/macio.c| 2 +- hw/ppc/mac_newworld.c| 4 ++--

Re: [PATCH 07/11] hw/ppc/e500: Implement pflash handling

2022-10-03 Thread B
Am 16. September 2022 17:05:13 UTC schrieb Bernhard Beschow : >Am 16. September 2022 15:00:06 UTC schrieb Bin Meng : >>On Thu, Sep 15, 2022 at 11:36 PM Bernhard Beschow wrote: >>> >>> Allows e500 boards to have their root file system reside on flash using >>> only builtin devices. >>> >>> Note

[PATCH v2 00/13] ppc/e500: Add support for two types of flash, cleanup

2022-10-03 Thread Bernhard Beschow
Cover letter: ~ This series adds support for -pflash and direct SD card access to the PPC e500 boards. The idea is to increase compatibility with "real" firmware images where only the bare minimum of drivers is compiled in. The series is structured as follows: Patches 1-5 perform

[PATCH v3 01/13] mac_newworld: Drop some variables

2022-10-03 Thread BALATON Zoltan
Values not used frequently enough may not worth putting in a local variable, especially with names almost as long as the original value because that does not improve readability, to the contrary it makes it harder to see what value is used. Drop a few such variables. This is the same clean up that

[PATCH v2 09/13] hw/ppc/e500: Implement pflash handling

2022-10-03 Thread Bernhard Beschow
Allows e500 boards to have their root file system reside on flash using only builtin devices located in the eLBC memory region. Note that the flash memory area is only created when a -pflash argument is given, and that the size is determined by the given file. The idea is to put users into

[PATCH v2 03/13] docs/system/ppc/ppce500: Add heading for networking chapter

2022-10-03 Thread Bernhard Beschow
The sudden change of topics is slightly confusing and makes the networking information less visible. So separate the networking chapter to improve comprehensibility. Signed-off-by: Bernhard Beschow Reviewed-by: Bin Meng Reviewed-by: Philippe Mathieu-Daudé --- docs/system/ppc/ppce500.rst | 3

Re: [PATCH v2 04/13] hw/ppc/e500: Reduce usage of sysbus API

2022-10-03 Thread Philippe Mathieu-Daudé via
On 3/10/22 22:31, Bernhard Beschow wrote: PlatformBusDevice has an mmio attribute which gets aliased to SysBusDevice::mmio[0]. So PlatformbusDevice::mmio can be used directly, avoiding the sysbus API. Signed-off-by: Bernhard Beschow --- hw/ppc/e500.c | 2 +- 1 file changed, 1 insertion(+),

Re: [PATCH v2] mips/malta: pass RNG seed to to kernel via env var

2022-10-03 Thread Philippe Mathieu-Daudé via
Hi Jason, Per https://www.qemu.org/docs/master/devel/submitting-a-patch.html#when-resending-patches-add-a-version-tag: Send each new revision as a new top-level thread, rather than burying it in-reply-to an earlier revision, as many reviewers are not looking inside deep threads for new

Re: [PULL 0/8] chardev patches

2022-10-03 Thread Stefan Hajnoczi
Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/7.2 for any user-visible changes. signature.asc Description: PGP signature

[PATCH v3 07/13] mac_{old|new}world: Reduce number of QOM casts

2022-10-03 Thread BALATON Zoltan
By storing the device pointers in a variable with the right type the number of QOM casts can be reduced which also makes the code more readable. Signed-off-by: BALATON Zoltan Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Mark Cave-Ayland --- hw/ppc/mac_newworld.c | 61

[PATCH v3 12/13] hw/ppc/mac.h: Rename to include/hw/nvram/mac_nvram.h

2022-10-03 Thread BALATON Zoltan
All that is left in mac.h now belongs to the nvram emulation so rename it accordingly and only include it where it is really used. Signed-off-by: BALATON Zoltan Reviewed-by: Mark Cave-Ayland --- MAINTAINERS | 1 + hw/ide/macio.c |

[PATCH v2 06/13] hw/ppc/mpc8544ds: Add platform bus

2022-10-03 Thread Bernhard Beschow
Models the real device more closely. Address and size values are taken from mpc8544.dts from the linux-5.17.7 tree. The IRQ range is taken from e500plat.c. Signed-off-by: Bernhard Beschow --- hw/ppc/mpc8544ds.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/hw/ppc/mpc8544ds.c

[PATCH v2 08/13] hw/block/pflash_cfi01: Error out if device length isn't a power of two

2022-10-03 Thread Bernhard Beschow
According to the JEDEC standard the device length is communicated to an OS as an exponent (power of two). Signed-off-by: Bernhard Beschow Reviewed-by: Bin Meng --- hw/block/pflash_cfi01.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/hw/block/pflash_cfi01.c

[PATCH v2 13/13] hw/ppc/e500: Add Freescale eSDHC to e500 boards

2022-10-03 Thread Bernhard Beschow
Adds missing functionality to emulated e500 SOCs which increases the chance of given "real" firmware images to access SD cards. Signed-off-by: Bernhard Beschow --- docs/system/ppc/ppce500.rst | 13 + hw/ppc/Kconfig | 1 + hw/ppc/e500.c | 31

Re: [PATCH v2] mips/malta: pass RNG seed to to kernel via env var

2022-10-03 Thread Jason A. Donenfeld
Hi Philippe, On Tue, Oct 4, 2022 at 12:36 AM Philippe Mathieu-Daudé wrote: > Send each new revision as a new top-level thread, rather than burying it > in-reply-to an earlier revision, as many reviewers are not looking > inside deep threads for new patches. Will do. > You seem to justify this

[PULL v3 3/4] Hexagon (target/hexagon) move store size tracking to translation

2022-10-03 Thread Taylor Simpson
The store width is needed for packet commit, so it is stored in ctx->store_width. Currently, it is set when a store has a TCG override instead of a QEMU helper. In the QEMU helper case, the ctx->store_width is not set, we invoke a helper during packet commit that uses the runtime store width.

[PULL 09/15] meson: -display dbus and CFI are incompatible

2022-10-03 Thread Paolo Bonzini
The generated skeletons for DBus call the finalize method of the parent type using code like G_OBJECT_CLASS (qemu_dbus_display1_chardev_skeleton_parent_class)->finalize (object); However, the finalize method is defined in a shared library that is not compiled with CFI. Do not enable

[PATCH v3 05/13] mac_{old|new}world: Simplify cmdline_base calculation

2022-10-03 Thread BALATON Zoltan
By slight reorganisation we can avoid an else branch and some code duplication which makes it easier to follow the code. Signed-off-by: BALATON Zoltan --- hw/ppc/mac_newworld.c | 6 +++--- hw/ppc/mac_oldworld.c | 7 +++ 2 files changed, 6 insertions(+), 7 deletions(-) diff --git

[PATCH v3 09/13] hw/ppc/mac.h: Move macio specific parts out from shared header

2022-10-03 Thread BALATON Zoltan
Move the parts specific to and only used by macio out from the shared mac.h into macio.c where they better belong. Signed-off-by: BALATON Zoltan Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Mark Cave-Ayland --- hw/misc/macio/macio.c | 5 +++-- hw/ppc/mac.h | 23

[PATCH v2 04/13] hw/ppc/e500: Reduce usage of sysbus API

2022-10-03 Thread Bernhard Beschow
PlatformBusDevice has an mmio attribute which gets aliased to SysBusDevice::mmio[0]. So PlatformbusDevice::mmio can be used directly, avoiding the sysbus API. Signed-off-by: Bernhard Beschow --- hw/ppc/e500.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/ppc/e500.c

Re: [PATCH v2 05/13] hw/ppc/mpc8544ds: Rename wrongly named method

2022-10-03 Thread Philippe Mathieu-Daudé via
On 3/10/22 22:31, Bernhard Beschow wrote: Signed-off-by: Bernhard Beschow --- hw/ppc/mpc8544ds.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) Reviewed-by: Philippe Mathieu-Daudé

Re: [PATCH v2 05/13] mac_oldworld: Do not open code sysbus_mmio_map()

2022-10-03 Thread BALATON Zoltan
On Mon, 3 Oct 2022, Mark Cave-Ayland wrote: On 29/09/2022 12:32, BALATON Zoltan wrote: On Thu, 29 Sep 2022, Mark Cave-Ayland wrote: On 25/09/2022 13:38, BALATON Zoltan wrote: Signed-off-by: BALATON Zoltan ---   hw/ppc/mac_oldworld.c | 8 ++--   1 file changed, 2 insertions(+), 6

Re: [PATCH v2 08/13] hw/block/pflash_cfi01: Error out if device length isn't a power of two

2022-10-03 Thread Philippe Mathieu-Daudé via
On 3/10/22 22:31, Bernhard Beschow wrote: According to the JEDEC standard the device length is communicated to an OS as an exponent (power of two). Signed-off-by: Bernhard Beschow Reviewed-by: Bin Meng --- hw/block/pflash_cfi01.c | 8 ++-- 1 file changed, 6 insertions(+), 2

Re: [PATCH v2 00/13] ppc/e500: Add support for two types of flash, cleanup

2022-10-03 Thread Philippe Mathieu-Daudé via
Hi Daniel, On 3/10/22 22:31, Bernhard Beschow wrote: Cover letter: ~ This series adds support for -pflash and direct SD card access to the PPC e500 boards. The idea is to increase compatibility with "real" firmware images where only the bare minimum of drivers is compiled in.

Re: [PATCH 03/10] mac_{old|new}world: Set default values for some local variables

2022-10-03 Thread BALATON Zoltan
On Thu, 29 Sep 2022, Mark Cave-Ayland wrote: On 25/09/2022 10:16, BALATON Zoltan wrote: On Sun, 25 Sep 2022, Mark Cave-Ayland wrote: On 17/09/2022 00:07, BALATON Zoltan wrote: Some lines can be dropped making the code flow simpler and easier to follow by setting default values at variable

Re: [PATCH v3] virtio-scsi: Send "REPORTED LUNS CHANGED" sense data upon disk hotplug events.

2022-10-03 Thread Venu Busireddy
On 2022-09-30 18:25:48 +0200, Paolo Bonzini wrote: > On Fri, Sep 30, 2022 at 4:42 PM Venu Busireddy > wrote: > > > > Immediately after a hotunplug event, qemu (without any action from > > > > the guest) processes a REPORT_LUNS command on the lun 0 of the device > > > > (haven't figured out what

[PATCH v4 0/6] hw/arm/virt: Improve address assignment for high memory regions

2022-10-03 Thread Gavin Shan
There are three high memory regions, which are VIRT_HIGH_REDIST2, VIRT_HIGH_PCIE_ECAM and VIRT_HIGH_PCIE_MMIO. Their base addresses are floating on highest RAM address. However, they can be disabled in several cases. (1) One specific high memory region is disabled by developer by toggling

[PULL v3 4/4] Hexagon (gen_tcg_funcs.py): avoid duplicated tcg code on A_CVI_NEW

2022-10-03 Thread Taylor Simpson
From: Matheus Tavares Bernardino Hexagon instructions with the A_CVI_NEW attribute produce a vector value that can be used in the same packet. The python function responsible for generating code for such instructions has a typo ("if" instead of "elif"), which makes genptr_dst_write_ext() be

[PULL v3 2/4] Hexagon (target/hexagon) Change decision to set pkt_has_store_s[01]

2022-10-03 Thread Taylor Simpson
We have found cases where pkt_has_store_s[01] is set incorrectly. This leads to generating an unnecessary store that is left over from a previous packet. Add an attribute to determine if an instruction is a scalar store The attribute is attached to the fSTORE macro (hex_common.py) Update the

[PULL v3 00/15] x86 + misc changes for 2022-09-29

2022-10-03 Thread Paolo Bonzini
The following changes since commit 99d6b11b5b44d7dd64f4cb1973184e40a4a174f8: Merge tag 'pull-target-arm-20220922' of https://git.linaro.org/people/pmaydell/qemu-arm into staging (2022-09-26 13:38:26 -0400) are available in the Git repository at: https://gitlab.com/bonzini/qemu.git

[PATCH v3 10/13] hw/ppc/mac.h: Move grackle-pcihost type declaration out to a header

2022-10-03 Thread BALATON Zoltan
Signed-off-by: BALATON Zoltan --- MAINTAINERS | 1 + hw/pci-host/grackle.c | 14 +-- hw/ppc/mac.h | 3 --- hw/ppc/mac_oldworld.c | 1 + include/hw/pci-host/grackle.h | 44 +++ 5 files changed, 47

[PATCH v2 05/13] hw/ppc/mpc8544ds: Rename wrongly named method

2022-10-03 Thread Bernhard Beschow
Signed-off-by: Bernhard Beschow --- hw/ppc/mpc8544ds.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/ppc/mpc8544ds.c b/hw/ppc/mpc8544ds.c index 81177505f0..8e674ad195 100644 --- a/hw/ppc/mpc8544ds.c +++ b/hw/ppc/mpc8544ds.c @@ -36,7 +36,7 @@ static void

[PATCH v3 03/13] mac_{old|new}world: Set tbfreq at declaration

2022-10-03 Thread BALATON Zoltan
The tbfreq variable is only set once in an if-else which can be done at the variable declaration saving some lines of code and making it simpler. Signed-off-by: BALATON Zoltan Reviewed-by: Mark Cave-Ayland --- hw/ppc/mac_newworld.c | 9 + hw/ppc/mac_oldworld.c | 9 + 2 files

[PATCH v2 01/13] hw/ppc/meson: Allow e500 boards to be enabled separately

2022-10-03 Thread Bernhard Beschow
Gives users more fine-grained control over what should be compiled into QEMU. Signed-off-by: Bernhard Beschow Reviewed-by: Bin Meng Reviewed-by: Philippe Mathieu-Daudé --- configs/devices/ppc-softmmu/default.mak | 3 ++- hw/ppc/Kconfig | 8 hw/ppc/meson.build

[PATCH v3 08/13] hw/ppc/mac.h: Move newworld specific parts out from shared header

2022-10-03 Thread BALATON Zoltan
Move the parts specific to and only used by mac99 out from the shared mac.h into mac_newworld.c where they better belong. Signed-off-by: BALATON Zoltan Reviewed-by: Mark Cave-Ayland --- hw/ppc/mac.h | 24 hw/ppc/mac_newworld.c | 19 +++

[PATCH v2 07/13] hw/ppc/e500: Remove if statement which is now always true

2022-10-03 Thread Bernhard Beschow
Now that the MPC8544DS board also has a platform bus, the if statement is always true. Signed-off-by: Bernhard Beschow Reviewed-by: Bin Meng Reviewed-by: Philippe Mathieu-Daudé --- hw/ppc/e500.c | 30 ++ hw/ppc/e500.h | 1 - hw/ppc/e500plat.c | 1 -

Re: [RFC 3/7] migration: Block migration comment or code is wrong

2022-10-03 Thread Stefan Hajnoczi
On Mon, Oct 03, 2022 at 05:15:56AM +0200, Juan Quintela wrote: > And it appears that what is wrong is the code. During bulk stage we > need to make sure that some block is dirty, but no games with > max_size at all. > > Signed-off-by: Juan Quintela > --- > migration/block.c | 4 ++-- > 1 file

Re: [PATCH v2 13/13] hw/ppc/e500: Add Freescale eSDHC to e500 boards

2022-10-03 Thread Philippe Mathieu-Daudé via
On 3/10/22 22:31, Bernhard Beschow wrote: Adds missing functionality to emulated e500 SOCs which increases the chance of given "real" firmware images to access SD cards. Signed-off-by: Bernhard Beschow --- docs/system/ppc/ppce500.rst | 13 + hw/ppc/Kconfig | 1 +

Re: [PULL 0/8] Trivial branch for 7.2 patches

2022-10-03 Thread Laurent Vivier
Le 03/10/2022 à 21:04, Stefan Hajnoczi a écrit : On Fri, 30 Sept 2022 at 16:22, Laurent Vivier wrote: Philippe Mathieu-Daudé via (1): block/qcow2-bitmap: Add missing cast to silent GCC error Hi Laurent, This commit uses a mailing list email, probably due to DKIM/SPF issues: Author:

Re: [PATCH v10 3/7] block: add block layer APIs resembling Linux ZonedBlockDevice ioctls

2022-10-03 Thread Damien Le Moal
On 2022/10/04 2:47, Stefan Hajnoczi wrote: > On Thu, Sep 29, 2022 at 04:36:27PM +0800, Sam Li wrote: >> Add a new zoned_host_device BlockDriver. The zoned_host_device option >> accepts only zoned host block devices. By adding zone management >> operations in this new BlockDriver, users can use the

[PATCH v4 1/6] hw/arm/virt: Introduce virt_set_high_memmap() helper

2022-10-03 Thread Gavin Shan
This introduces virt_set_high_memmap() helper. The logic of high memory region address assignment is moved to the helper. The intention is to make the subsequent optimization for high memory region address assignment easier. No functional change intended. Signed-off-by: Gavin Shan Reviewed-by:

[PATCH v4 2/6] hw/arm/virt: Rename variable size to region_size in virt_set_high_memmap()

2022-10-03 Thread Gavin Shan
This renames variable 'size' to 'region_size' in virt_set_high_memmap(). Its counterpart ('region_base') will be introduced in next patch. No functional change intended. Signed-off-by: Gavin Shan Reviewed-by: Eric Auger --- hw/arm/virt.c | 15 --- 1 file changed, 8 insertions(+),

[PATCH v2] hw/net: npcm7xx_emc: set MAC in register space

2022-10-03 Thread Patrick Venture
The MAC address set from Qemu wasn't being saved into the register space. Reviewed-by: Hao Wu Signed-off-by: Patrick Venture --- v2: only set the registers from qemu on reset once registers set, only read and write to them --- hw/net/npcm7xx_emc.c | 30 +++--- 1

Conserving GitLab CI minutes - check your repo is a true fork

2022-10-03 Thread Daniel P . Berrangé
Last week the /qemu-project namespace got added to the GitLab Open Source Program. This gives repos under that namespace extra CI minutes (50,000) and a cost factor of 0.5, meaning in effect 100,000 minutes per month. Primarily this applies to CI pipelines jobs the merge coordinator runs for pull

Re: [PATCH v2] mips/malta: pass RNG seed to to kernel via env var

2022-10-03 Thread Jason A. Donenfeld
On Mon, Oct 03, 2022 at 12:36:27PM +0200, Jason A. Donenfeld wrote: > As of the kernel commit linked below, Linux ingests an RNG seed > passed from the hypervisor. So, pass this for the Malta platform, and > reinitialize it on reboot too, so that it's always fresh. > > Cc: Philippe Mathieu-Daudé

[PATCH v2 11/13] hw/sd/sdhci: Rename ESDHC_* defines to USDHC_*

2022-10-03 Thread Bernhard Beschow
The device model's functions start with "usdhc_", so rename the defines accordingly for consistency. Signed-off-by: Bernhard Beschow Reviewed-by: Bin Meng --- hw/sd/sdhci.c | 66 +-- 1 file changed, 33 insertions(+), 33 deletions(-) diff --git

Re: [PATCH v2 12/13] hw/sd/sdhci: Implement Freescale eSDHC device model

2022-10-03 Thread Philippe Mathieu-Daudé via
On 3/10/22 22:31, Bernhard Beschow wrote: Will allow e500 boards to access SD cards using just their own devices. Signed-off-by: Bernhard Beschow --- hw/sd/sdhci.c | 147 +- include/hw/sd/sdhci.h | 3 + 2 files changed, 149 insertions(+), 1

Re: [PATCH v3 4/5] hw/arm/virt: Improve high memory region address assignment

2022-10-03 Thread Gavin Shan
Hi Eric, On 10/3/22 4:44 PM, Eric Auger wrote: On 9/29/22 01:37, Gavin Shan wrote: On 9/28/22 10:51 PM, Eric Auger wrote: On 9/22/22 01:13, Gavin Shan wrote: There are three high memory regions, which are VIRT_HIGH_REDIST2, VIRT_HIGH_PCIE_ECAM and VIRT_HIGH_PCIE_MMIO. Their base addresses

Re: [PULL 00/10] target-arm queue

2022-10-03 Thread Stefan Hajnoczi
Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/7.2 for any user-visible changes. signature.asc Description: PGP signature

Re: ublk-qcow2: ublk-qcow2 is available

2022-10-03 Thread Denis V. Lunev
On 10/3/22 21:53, Stefan Hajnoczi wrote: On Fri, Sep 30, 2022 at 05:24:11PM +0800, Ming Lei wrote: ublk-qcow2 is available now. Cool, thanks for sharing! yep So far it provides basic read/write function, and compression and snapshot aren't supported yet. The target/backend implementation is

Re: [PATCH v3] tcg/ppc: Optimize 26-bit jumps

2022-10-03 Thread Leandro Lupori
On 9/19/22 14:56, Leandro Lupori wrote: PowerPC64 processors handle direct branches better than indirect ones, resulting in less stalled cycles and branch misses. However, PPC's tb_target_set_jmp_target() was only using direct branches for 16-bit jumps, while PowerPC64's unconditional branch

Re: [PULL 0/8] Trivial branch for 7.2 patches

2022-10-03 Thread Stefan Hajnoczi
On Fri, 30 Sept 2022 at 16:22, Laurent Vivier wrote: > Philippe Mathieu-Daudé via (1): > block/qcow2-bitmap: Add missing cast to silent GCC error Hi Laurent, This commit uses a mailing list email, probably due to DKIM/SPF issues: Author: Philippe Mathieu-Daudé via I think the policy is to

Re: [PULL 00/18] Block layer patches

2022-10-03 Thread Stefan Hajnoczi
Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/7.2 for any user-visible changes. signature.asc Description: PGP signature

A few QEMU questiosn

2022-10-03 Thread a b
Hello, there, I have a few newbie QEMU questions. I found that mmu_idx in aarch64-softmmu falls in 8, 10 and 12. I need some help to understand what they are for. I cannot find which macros are for mmu-idx 8, 10 and 12 at

Re: x86, pflash, unassigned memory access

2022-10-03 Thread Alexey Kardashevskiy
Anyone, ping? On 27/09/2022 12:35, Alexey Kardashevskiy wrote: Hi! I am trying qemu-system-x86_64 with OVMF with the q35 machine, the complete command line is below. It works fine (including SEV on AMD EPYC), but these 2 parameters make me wonder if I miss something: -drive

Re: [PATCH v3 2/2] hw/intc: sifive_plic: change interrupt priority register to WARL field

2022-10-03 Thread Clément Chigot
On Mon, Oct 3, 2022 at 6:14 AM Jim Shu wrote: > > PLIC spec [1] requires interrupt source priority registers are WARL > field and the number of supported priority is power-of-2 to simplify SW > discovery. > > Existing QEMU RISC-V machine (e.g. shakti_c) don't strictly follow PLIC > spec, whose

[PATCH] build-sys: error when slirp is not found and not disabled

2022-10-03 Thread marcandre . lureau
From: Marc-André Lureau This is an alternative configure-time solution to "[PATCH] net: print a more actionable error when slirp is not found". See also "If your networking is failing after updating to the latest git version of QEMU..." ML thread. Signed-off-by: Marc-André Lureau ---

Re: [PATCH v2 10/13] hw/ppc/mac.h: Move grackle-pcihost declaration out from shared header

2022-10-03 Thread Mark Cave-Ayland
On 29/09/2022 12:42, BALATON Zoltan wrote: On Thu, 29 Sep 2022, Mark Cave-Ayland wrote: On 25/09/2022 13:38, BALATON Zoltan wrote: It is only used by mac_oldworld anyway and it already instantiates a few devices by name so this allows reducing the shared header further. Signed-off-by:

Re: [PATCH v2] hw/smbios: support for type 8 (port connector)

2022-10-03 Thread Hal Martin
Hello, Any update on merging this? Kind regards, Hal On Fri, Aug 12, 2022 at 5:04 PM Michael S. Tsirkin wrote: > > On Fri, Aug 12, 2022 at 03:51:53PM +0200, Hal Martin wrote: > > PATCH v1: add support for SMBIOS type 8 to qemu > > PATCH v2: incorporate patch v1 feedback and add smbios type=8

Re: [PATCH v3 2/5] hw/arm/virt: Rename variable size to region_size in virt_set_high_memmap()

2022-10-03 Thread Eric Auger
On 9/22/22 01:13, Gavin Shan wrote: > This renames variable 'size' to 'region_size' in virt_set_high_memmap(). > Its counterpart ('region_base') will be introduced in next patch. > > No functional change intended. > > Signed-off-by: Gavin Shan Reviewed-by: Eric Auger Eric > --- >

Re: [PATCH v3 3/5] hw/arm/virt: Introduce variable region_base in virt_set_high_memmap()

2022-10-03 Thread Eric Auger
On 9/22/22 01:13, Gavin Shan wrote: > This introduces variable 'region_base' for the base address of the > specific high memory region. It's the preparatory work to optimize > high memory region address assignment. > > No functional change intended. > > Signed-off-by: Gavin Shan Reviewed-by:

Re: [PATCH for 7.1] linux-user: fix compat with glibc >= 2.36 sys/mount.h

2022-10-03 Thread Daniel P . Berrangé
On Sat, Oct 01, 2022 at 11:34:20PM +0200, Andreas Schwab wrote: > On Aug 02 2022, Daniel P. Berrangé wrote: > > > This patch removes linux/fs.h, meaning we have to define > > various FS_IOC constants that are now unavailable. > > This breaks a lot of ioctl emulations, as it lacks their

[PATCH v3 1/7] qga: Add initial FreeBSD support

2022-10-03 Thread Alexander Ivanov
- Fix device path. - Fix virtio-serial channel initialization. - Make the code buildable in FreeBSD. Signed-off-by: Alexander Ivanov --- meson.build | 2 +- qga/channel-posix.c | 19 +++ qga/commands-posix.c | 8 qga/main.c | 6 +- 4 files

[PATCH v3 3/7] qga: Add UFS freeze/thaw support for FreeBSD

2022-10-03 Thread Alexander Ivanov
UFS supports FS freezing through ioctl UFSSUSPEND on /dev/ufssuspend. Frozen FS can be thawed by closing /dev/ufssuspend file descriptior. Use getmntinfo to get a list of mounted FS. Reviewed-by: Marc-André Lureau Signed-off-by: Alexander Ivanov --- qga/commands-bsd.c| 169

[PATCH v3 6/7] qga: Move HW address getting to a separate function

2022-10-03 Thread Alexander Ivanov
In the next patch FreeBSD support for guest-network-get-interfaces will be added. Previously move Linux-specific code of HW address getting to a separate functions and add a dumb function to commands-bsd.c. Signed-off-by: Alexander Ivanov --- qga/commands-bsd.c| 16 +++

  1   2   >