On Fri, 13 Dec 2024 13:06:46 -0600
Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
Reviewed-by: Greg Kurz
> hw/9pfs/virtio-9p-device.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/9pfs/virtio-9p-device.c b/hw/9pfs/virtio-9p-device.c
> index e
Am Thu, 12 Dec 2024 11:46:17 +
schrieb Mark Cave-Ayland :
> Move these functions in next-cube.c so that they are with the rest of the
> next-rtc functions.
>
> Signed-off-by: Mark Cave-Ayland
> ---
> hw/m68k/next-cube.c | 172 ++--
> 1 file changed, 8
Am Thu, 12 Dec 2024 11:45:57 +
schrieb Mark Cave-Ayland :
> Since the ESCC is part of the next-pc device, move the ESCC to be a QOM child
> of the next-pc device.
>
> Signed-off-by: Mark Cave-Ayland
> ---
> hw/m68k/next-cube.c | 54 ++---
> 1 file cha
On 12/13/24 21:29, Richard Henderson wrote:
On 12/13/24 21:44, Pierrick Bouvier wrote:
Hi Richard,
On 12/13/24 13:47, Richard Henderson wrote:
Hi,
Several of the recent contrib/plugins/ patches do not build on e.g. arm32.
All of the issues are related to casting between pointers and uint64_t;
Am Thu, 12 Dec 2024 11:46:11 +
schrieb Mark Cave-Ayland :
> This is to allow the RTC functionality to be maintained within its own
> separate
> device rather than as part of the next-pc device.
>
> Signed-off-by: Mark Cave-Ayland
> ---
> hw/m68k/next-cube.c | 71 +++
Am Thu, 12 Dec 2024 11:45:57 +
schrieb Mark Cave-Ayland :
> Since the ESCC is part of the next-pc device, move the ESCC to be a QOM child
> of the next-pc device.
>
> Signed-off-by: Mark Cave-Ayland
> ---
> hw/m68k/next-cube.c | 54 ++---
> 1 file cha
Am Thu, 12 Dec 2024 11:45:49 +
schrieb Mark Cave-Ayland :
> Move the ESP SCSI and SCSI CSR registers to the new next.scsi container memory
> region.
>
> Signed-off-by: Mark Cave-Ayland
> ---
> hw/m68k/next-cube.c | 8 +++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
Reviewed-by: T
Am Thu, 12 Dec 2024 11:45:54 +
schrieb Mark Cave-Ayland :
> The SCSI 4020/4021 logic refers to the offset of the SCSI CSRs within the
> NeXTCube
> address space. Due to the previously overlapping memory regions, there were
> duplicate MMIO accessors in the next.scr memory region for these reg
On 12/13/24 21:44, Pierrick Bouvier wrote:
Hi Richard,
On 12/13/24 13:47, Richard Henderson wrote:
Hi,
Several of the recent contrib/plugins/ patches do not build on e.g. arm32.
All of the issues are related to casting between pointers and uint64_t; there
is a Werror
generated for casting bet
Am Thu, 12 Dec 2024 11:45:53 +
schrieb Mark Cave-Ayland :
> The SCSI CSRs are located within the SCSI subsystem of the NeXT PC (Peripheral
> Contoller) which is now modelled as a separate QEMU device. Add a new memory
> region subregion to contain the SCSI CSRs that simply store and retrieve t
Hi Richard,
On 12/13/24 13:47, Richard Henderson wrote:
Hi,
Several of the recent contrib/plugins/ patches do not build on e.g. arm32.
All of the issues are related to casting between pointers and uint64_t; there
is a Werror
generated for casting between pointers and integers of different size
On Fri, 2024-12-13 at 10:04 +0100, Mario Fleischmann wrote:
> Hi,
>
> apologies for the delayed review; I've just gotten to it now.
>
> On 06.12.2024 01:14, Yanfeng Liu wrote:
> > This adds virtualization mode (V bit) as bit(2) of register `priv`
> > per RiscV debug spec v1.0.0-rc3. Checked with
On 12/13/24 2:07 PM, Richard Henderson wrote:
Signed-off-by: Richard Henderson
Reviewed-by: Stefan Berger
---
hw/tpm/tpm_crb.c| 2 +-
hw/tpm/tpm_spapr.c | 2 +-
hw/tpm/tpm_tis_i2c.c| 2 +-
hw/tpm/tpm_tis_isa.c| 2 +-
hw/tpm/tpm_tis_sysbus.c | 2 +-
5 files cha
On Fri 13 Dec 2024 01:07:07 PM -06, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
Acked-by: Alberto Garcia
Berto
On Fri 13 Dec 2024 01:06:54 PM -06, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
Acked-by: Alberto Garcia
Berto
From: Bernhard Beschow
In some adhoc profiling booting Linux VMs, it's observed that ahci_irq_lower()
can be a hot path (1+ triggers until login prompt appears). Even though the
parent device never changes, this method re-determines whether the parent device
is a PCI device or not using the r
Commit b9599519a01 ("hw/usb/hcd-xhci: Remove XHCI_FLAG_SS_FIRST
flag") remove the last use of XHCINecState::flags but neglected
to remove it; do that now.
Reported-by: Nicholas Piggin
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Thomas Huth
Message-Id: <20241127122812.89487-1-phi...@linar
The FW_CFG_DATA_GENERATOR interface allows any object to
produce a blob of data consumable by the fw_cfg device.
Implement that for PCI bus objects.
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Daniel P. Berrangé
Message-Id: <20241213133352.10915-5-phi...@linaro.org>
---
hw/pci/pci.c | 37
Currently fw_cfg_add_file_from_generator() is restricted
to command line created objects which reside in the
'/objects' QOM container. In order to extend to other
types of containers, pass the QOM parent by argument.
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Daniel P. Berrangé
Message-I
From: Phil Dennis-Jordan
Fixes number of spaces used for indentation on one line.
Signed-off-by: Phil Dennis-Jordan
Message-ID: <20241208191646.64857-6-p...@philjordan.eu>
Signed-off-by: Philippe Mathieu-Daudé
---
hw/usb/hcd-xhci-pci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
di
From: Nicholas Piggin
This is a convenience change that accepts a status when completing a
packet.
Signed-off-by: Nicholas Piggin
Reviewed-by: Philippe Mathieu-Daudé
Message-ID: <20241110034000.379463-2-npig...@gmail.com>
Signed-off-by: Philippe Mathieu-Daudé
---
hw/usb/dev-storage.c | 17 ++
From: Bernhard Beschow
Implement in dedicated file, just like TYPE_ICH9_AHCI.
Signed-off-by: Bernhard Beschow
Reviewed-by: Philippe Mathieu-Daudé
Message-ID: <20241212110926.23548-3-shen...@gmail.com>
Signed-off-by: Philippe Mathieu-Daudé
---
hw/ide/ahci-sysbus.c | 91 +++
Some files indirectly get "exec/tswap.h" declarations via
"exec/cpu-all.h". Include it directly to be able to remove
the former from the latter, otherwise we get:
hw/mips/malta.c:674:22: error: call to undeclared function 'tswap32'; ISO C99
and later do not support implicit function declaration
fw_cfg_add_from_generator() is adding a 'file' entry,
so rename as fw_cfg_add_file_from_generator() for
clarity. Besides, we might introduce generators for
other entry types.
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Daniel P. Berrangé
Message-Id: <20241206181352.6836-2-phi...@linaro.or
Some files indirectly get "exec/tswap.h" declarations via
"exec/cpu-all.h". Include it directly to be able to remove
the former from the latter, otherwise we get:
hw/xtensa/bootparam.h:40:16: error: call to undeclared function 'tswap16';
ISO C99 and later do not support implicit function declar
pci_bus_add_fw_cfg_extra_pci_roots() calls the fw_cfg
API with PCI bus specific arguments.
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Daniel P. Berrangé
Message-Id: <20241206181352.6836-5-phi...@linaro.org>
---
include/hw/pci/pci.h | 3 +++
hw/pci/pci.c | 16
2
The following changes since commit 83aaec1d5a49f158abaa31797a0f976b3c07e5ca:
Merge tag 'pull-tcg-20241212' of https://gitlab.com/rth7680/qemu into staging
(2024-12-12 18:45:39 -0500)
are available in the Git repository at:
https://github.com/philmd/qemu.git tags/hw-misc-20241214
for you to
r2d.c indirectly get "exec/tswap.h" declarations via
"exec/cpu-all.h". Include it directly to be able to
remove the former from the latter, otherwise we get:
hw/sh4/r2d.c:357:35: error: call to undeclared function 'tswap32'; ISO C99
and later do not support implicit function declarations
[-Wim
Now that all uses of fw_cfg_add_extra_pci_roots() have been
converted to the newer pci_bus_add_fw_cfg_extra_pci_roots(),
we can remove that bogus method. hw/nvram/fw_cfg must
stay generic. Device specific entries have to be implemented
using TYPE_FW_CFG_DATA_GENERATOR_INTERFACE.
This mostly revert
Allow the FW_CFG_DATA_GENERATOR interface get_data() handler to
return NULL when there is nothing to generate. In that case
fw_cfg_add_file_from_generator() will not add any item and
return %true.
Reported-by: Daniel P. Berrangé
Reviewed-by: Daniel P. Berrangé
Signed-off-by: Philippe Mathieu-Dau
We want to remove fw_cfg_add_extra_pci_roots() which introduced
PCI bus knowledge within the generic hw/nvram/fw_cfg.c file.
Replace the calls by the pci_bus_add_fw_cfg_extra_pci_roots()
which is a 1:1 equivalent, but using correct API.
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Daniel P.
No need to QOM-cast twice, since the intermediate value
is not used.
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Thomas Huth
Message-Id: <20241125140535.4526-7-phi...@linaro.org>
---
hw/riscv/virt.c | 20 +---
1 file changed, 9 insertions(+), 11 deletions(-)
diff --git a
From: Alex Bennée
The event_notifier, thread and socket includes look like copy and
paste of standard headers. None of the canbus devices use chardev
although some relied on chardev to bring in bitops and byte swapping
headers. In this case include them directly.
Signed-off-by: Alex Bennée
Acke
From: Dorjoy Chowdhury
Signed-off-by: Dorjoy Chowdhury
Reviewed-by: Alexander Graf
Message-ID: <20241109122844.24057-1-dorjoychy...@gmail.com>
Signed-off-by: Philippe Mathieu-Daudé
---
docs/system/i386/nitro-enclave.rst | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git
From: Dorjoy Chowdhury
NSM device in AWS Nitro Enclaves supports extending with both
bytestring and string data.
Signed-off-by: Dorjoy Chowdhury
Reviewed-by: Alexander Graf
Message-ID: <20241109123208.24281-1-dorjoychy...@gmail.com>
Signed-off-by: Philippe Mathieu-Daudé
---
hw/virtio/virtio-
From: Dorjoy Chowdhury
We were storing the pointers to buffers in a GList due to lack of
stateful crypto apis and instead doing the final hash computation at
the end after we had all the necessary buffers. Now that we have the
stateful qcrypto apis available, we can instead update the hashes
inli
On 13/12/24 14:33, Philippe Mathieu-Daudé wrote:
Philippe Mathieu-Daudé (7):
hw/nvram/fw_cfg: Rename fw_cfg_add_[file]_from_generator()
hw/nvram/fw_cfg: Pass QOM parent to fw_cfg_add_file_from_generator()
hw/nvram/fw_cfg: Skip FW_CFG_DATA_GENERATOR when no data to generate
hw/pci: Ha
On 13/12/24 20:16, Philippe Mathieu-Daudé wrote:
On 9/12/24 19:12, Brian Cain wrote:
Mea culpa, I don't know how I got this wrong in 2dfe93699c. Still
getting used to the new address, I suppose. Somehow I got it right in
the
mailmap, though.
Signed-off-by: Brian Cain
---
MAINTAINERS | 2
Specifying this bit in the guest CLP response indicates that the guest
can optionally choose to skip translation and instead use
identity-mapped operations.
Signed-off-by: Matthew Rosato
---
hw/s390x/s390-pci-vfio.c| 5 -
include/hw/s390x/s390-pci-clp.h | 1 +
2 files changed, 5 inse
When receiving a guest mpcifc(4) or mpcifc(6) instruction without the T
bit set, treat this as a request to perform direct mapping instead of
address translation. In order to facilitate this, pin the entirety of
guest memory into the host iommu.
Subsequent guest DMA operations are all expected to
This series introduces the concept of the relaxed translation requirement
for s390x guests in order to allow bypass of the guest IOMMU for more
efficient PCI passthrough.
With this series, QEMU can indicate to the guest that an IOMMU is not
strictly required for a zPCI device. This would subseque
Some of the info I was asked for in V1 of the patch:
I'm seeing the failure on 2 different machines, the compiler versions are:
- gcc version 11.4.0 (Ubuntu 11.4.0-1ubuntu1~22.04)
- gcc version 14.2.0 (Debian 14.2.0-3+build3)
Both of the outputs below are run on the Debian machine.
Output of the
On 12/10/24 3:58 AM, David Hildenbrand wrote:
> Maybe there is a reason s390x needs to handle this using
> memory_region_notify_iommu() callbacks instead of doing it similar to "struct
> vfio_memory_listener" when registered on &address_space_memory without a
> viommu.
>
Hi David,
I think I
V2: Removed scripts/meson-buildoptions.sh.tmp Extra file that slipped
through the cracks and shouldn't be in this patch
Recent CDMSK Watchdog changes (eff9dc5660fad3a610171c56a5ec3fada245e519)
updated the CDMSK APB Watchdog to not free run out of reset. That led to
this test failing since it never
Hi,
Several of the recent contrib/plugins/ patches do not build on e.g. arm32.
All of the issues are related to casting between pointers and uint64_t; there is a Werror
generated for casting between pointers and integers of different sizes.
I suspect all of the instances will need to use separ
Nabih Estefan writes:
> Recent CDMSK Watchdog changes (eff9dc5660fad3a610171c56a5ec3fada245e519)
> updated the CDMSK APB Watchdog to not free run out of reset. That led to
> this test failing since it never triggers the watchdog to start running.
> No watchdog running means that the timer and cou
On 12/13/24 15:06, Richard Henderson wrote:
Before, via pahole:
Bah. s/by 8 bytes// in subject; moved to the summary at the end.
Afterward there are no holes in either structure.
For arm32, size 48, padding 2, saved 8 bytes.
For arm64, size 72, padding 6, saved 16 bytes.
Also meant to say,
Nabih Estefan writes:
> From what I can tell this is the same issue Thomas was looking at yes.
>
> I saw the failure on the master branch at the v9.2.0 tag (ae35f033) and just
> re-tested it against (83aaec1d) and still see it. I haven't seen it be an
> intermittent failure, it has failed 100% of
Before, via pahole:
arm32, a 32-bit host which aligns uint64_t:
struct Property {
const char * name; /* 0 4 */
const PropertyInfo * info; /* 4 4 */
ptrdiff_t offset; /* 8
On 12/13/24 4:07 PM, Richard Henderson wrote:
Signed-off-by: Richard Henderson
---
For all riscv controllers, pnv controllers and spapr_xive:
Reviewed-by: Daniel Henrique Barboza
hw/intc/apic_common.c | 2 +-
hw/intc/arm_gic_common.c | 2 +-
hw/intc/arm_gicv2m.c
On 12/13/24 4:06 PM, Richard Henderson wrote:
Signed-off-by: Richard Henderson
---
Reviewed-by: Daniel Henrique Barboza
target/riscv/cpu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/target/riscv/cpu.c b/target/riscv/cpu.c
index 80b09952e7..4329015076 100644
-
On 12/13/24 4:07 PM, Richard Henderson wrote:
Signed-off-by: Richard Henderson
---
Reviewed-by: Daniel Henrique Barboza
hw/riscv/opentitan.c | 2 +-
hw/riscv/riscv-iommu-pci.c | 2 +-
hw/riscv/riscv-iommu.c | 2 +-
hw/riscv/riscv_hart.c | 2 +-
hw/riscv/sifive_u.c
ping for review...
On 11/29/24 15:38, del...@kernel.org wrote:
From: Helge Deller
This patchset adds various missing sockopt calls, so that qemu linux-user
is able to successfully build the debian gupnp package in a chroot.
Tested with a 32-bit big-endian hppa linux-user chroot running on a p
Am Fri, 13 Dec 2024 13:07:10 -0600
schrieb Richard Henderson :
> Signed-off-by: Richard Henderson
> ---
> hw/m68k/mcf5206.c | 2 +-
> hw/m68k/mcf_intc.c | 2 +-
> hw/m68k/next-cube.c | 2 +-
> hw/m68k/q800-glue.c | 2 +-
> 4 files changed, 4 insertions(+), 4 deletions(-)
Reviewed-by: Thomas
On 9/11/24 13:28, Dorjoy Chowdhury wrote:
Signed-off-by: Dorjoy Chowdhury
---
docs/system/i386/nitro-enclave.rst | 8
1 file changed, 4 insertions(+), 4 deletions(-)
Queued, thanks!
Signed-off-by: Richard Henderson
---
hw/cxl/switch-mailbox-cci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/cxl/switch-mailbox-cci.c b/hw/cxl/switch-mailbox-cci.c
index 4f419443ab..3fde0f8aae 100644
--- a/hw/cxl/switch-mailbox-cci.c
+++ b/hw/cxl/switch-mailbox-cci.c
@@
On 12/13/2024 1:06 PM, Richard Henderson wrote:
Signed-off-by: Richard Henderson
---
target/hexagon/cpu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/target/hexagon/cpu.c b/target/hexagon/cpu.c
index c9aa9408ec..a70007245e 100644
--- a/target/hexagon/cpu.c
+++ b/targ
Hi Mark!
Am Thu, 12 Dec 2024 11:45:48 +
schrieb Mark Cave-Ayland :
> Change the start of the next.mmio memory region so that it follows on directly
> after the next.dma memory region. Increase the address offsets in
> next_mmio_read() and next_mmio_write(), and reduce the size of the next.
On 13/12/24 20:06, Richard Henderson wrote:
Since d36f165d952 (qdev: make properties array "const"), we can
define our Property structure const. Do this across the entire tree.
There are a few other minor changes:
- Two instances where it was obvious that an empty property list
could be
Yeah, a no brainer.
Acked-by: Corey Minyard
On Fri, Dec 13, 2024 at 01:07:02PM -0600, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> hw/i2c/aspeed_i2c.c | 4 ++--
> hw/i2c/core.c| 2 +-
> hw/i2c/i2c_mux_pca954x.c | 2 +-
> hw/i2c/omap_i2c.c| 2 +-
>
Acked-by: Corey Minyard
On Fri, Dec 13, 2024 at 01:07:08PM -0600, Richard Henderson wrote:
> Signed-off-by: Richard Henderson
> ---
> hw/ipmi/ipmi.c| 2 +-
> hw/ipmi/ipmi_bmc_extern.c | 2 +-
> hw/ipmi/ipmi_bmc_sim.c| 2 +-
> hw/ipmi/isa_ipmi_bt.c | 2 +-
> hw/ipmi/isa_ipmi_
From: Philippe Mathieu-Daudé
Rather than manually copying each register, use
the libc memcpy(), which is well optimized nowadays.
Suggested-by: Pierrick Bouvier
Reviewed-by: Pierrick Bouvier
Reviewed-by: Richard Henderson
Signed-off-by: Philippe Mathieu-Daudé
Message-ID: <20241205205418.6761
Signed-off-by: Richard Henderson
---
tests/unit/test-qdev-global-props.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/unit/test-qdev-global-props.c
b/tests/unit/test-qdev-global-props.c
index c8862cac5f..1eb95d2429 100644
--- a/tests/unit/test-qdev-global-props.c
+++
Signed-off-by: Richard Henderson
---
hw/vfio/ap.c | 2 +-
hw/vfio/ccw.c | 2 +-
hw/vfio/pci.c | 4 ++--
hw/vfio/platform.c | 2 +-
4 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/hw/vfio/ap.c b/hw/vfio/ap.c
index 4013e7b436..2e6ea2dd93 100644
--- a/hw/vfio/ap.c
+++
Signed-off-by: Richard Henderson
---
hw/timer/a9gtimer.c | 2 +-
hw/timer/allwinner-a10-pit.c | 2 +-
hw/timer/arm_mptimer.c | 2 +-
hw/timer/arm_timer.c | 2 +-
hw/timer/aspeed_timer.c | 2 +-
hw/timer/avr_timer16.c | 2 +-
hw/timer/grlib_gptimer.c | 2 +-
h
Signed-off-by: Richard Henderson
---
hw/usb/bus.c | 2 +-
hw/usb/canokey.c | 2 +-
hw/usb/ccid-card-emulated.c | 2 +-
hw/usb/ccid-card-passthru.c | 2 +-
hw/usb/dev-audio.c| 2 +-
hw/usb/dev-hid.c | 6 +++---
hw/usb/dev-hub.c
Signed-off-by: Richard Henderson
---
hw/block/fdc-isa.c| 2 +-
hw/block/fdc-sysbus.c | 4 ++--
hw/block/fdc.c| 2 +-
hw/block/m25p80.c | 2 +-
hw/block/nand.c | 2 +-
hw/block/pflash_cfi01.c | 2 +-
hw/block/pflash_cfi02.c | 2 +-
hw/block/swim.c
Signed-off-by: Richard Henderson
---
hw/xen/xen-bus.c| 2 +-
hw/xen/xen-legacy-backend.c | 4 ++--
hw/xen/xen_pt.c | 2 +-
3 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/hw/xen/xen-bus.c b/hw/xen/xen-bus.c
index 95b207ac8b..0d7defb8cd 100644
--- a/hw/xen/xe
Signed-off-by: Richard Henderson
---
docs/devel/migration/compatibility.rst | 4 ++--
docs/devel/virtio-backends.rst | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/docs/devel/migration/compatibility.rst
b/docs/devel/migration/compatibility.rst
index 5a5417ef06..c7
Signed-off-by: Richard Henderson
---
hw/char/avr_usart.c | 2 +-
hw/char/bcm2835_aux.c | 2 +-
hw/char/cadence_uart.c | 2 +-
hw/char/cmsdk-apb-uart.c| 2 +-
hw/char/debugcon.c | 2 +-
hw/char/digic-uart.c| 2 +-
hw/char/escc.c | 2 +-
hw/char/
Signed-off-by: Richard Henderson
---
hw/watchdog/sbsa_gwdt.c | 2 +-
hw/watchdog/wdt_aspeed.c | 2 +-
hw/watchdog/wdt_imx2.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/hw/watchdog/sbsa_gwdt.c b/hw/watchdog/sbsa_gwdt.c
index 80f9b36e79..2e25d4b4e9 100644
--- a/hw/wat
Signed-off-by: Richard Henderson
---
hw/remote/proxy.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/remote/proxy.c b/hw/remote/proxy.c
index 302a0a4d4d..6f84fdd3fa 100644
--- a/hw/remote/proxy.c
+++ b/hw/remote/proxy.c
@@ -191,7 +191,7 @@ static void pci_proxy_write_conf
On 9/11/24 13:32, Dorjoy Chowdhury wrote:
NSM device in AWS Nitro Enclaves supports extending with both
bytestring and string data.
Signed-off-by: Dorjoy Chowdhury
---
hw/virtio/virtio-nsm.c | 21 ++---
1 file changed, 14 insertions(+), 7 deletions(-)
Queued, thanks!
Signed-off-by: Richard Henderson
---
hw/nvme/ctrl.c | 2 +-
hw/nvme/ns.c | 2 +-
hw/nvme/subsys.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/hw/nvme/ctrl.c b/hw/nvme/ctrl.c
index ec75419566..33a3062466 100644
--- a/hw/nvme/ctrl.c
+++ b/hw/nvme/ctrl.c
@@ -8927,7 +
Use DEFINE_PROP_UNSIGNED instead of DEFINE_PROP_UINT64
so that we can set the PropertyInfo during initialization,
instead of updating within trng_class_init.
Signed-off-by: Richard Henderson
---
hw/misc/xlnx-versal-trng.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/
Signed-off-by: Richard Henderson
---
hw/xen/xen-legacy-backend.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/xen/xen-legacy-backend.c b/hw/xen/xen-legacy-backend.c
index b15393e934..f6ad58fe13 100644
--- a/hw/xen/xen-legacy-backend.c
+++ b/hw/xen/xen-legacy-backend.c
@@
Signed-off-by: Richard Henderson
---
hw/m68k/mcf5206.c | 2 +-
hw/m68k/mcf_intc.c | 2 +-
hw/m68k/next-cube.c | 2 +-
hw/m68k/q800-glue.c | 2 +-
4 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/hw/m68k/mcf5206.c b/hw/m68k/mcf5206.c
index 7247cdbe5e..45e5f74600 100644
--- a/hw/m6
Sigh… apologies for re-sending v13 together with v14 here. Please ignore
v13 of course. git send-email is not my friend…
On Fri, 13 Dec 2024 at 16:24, Phil Dennis-Jordan wrote:
>
> This patch series has been through months of review and
> refinement. It now has end-to-end Reviewed-by: tags and
Signed-off-by: Richard Henderson
---
hw/ufs/lu.c | 2 +-
hw/ufs/ufs.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/ufs/lu.c b/hw/ufs/lu.c
index 81bfff9b4e..74ff52ad09 100644
--- a/hw/ufs/lu.c
+++ b/hw/ufs/lu.c
@@ -274,7 +274,7 @@ static UfsReqResult ufs_process_scsi_
>From what I can tell this is the same issue Thomas was looking at yes.
I saw the failure on the master branch at the v9.2.0 tag (ae35f033) and just
re-tested it against (83aaec1d) and still see it. I haven't seen it be an
intermittent failure, it has failed 100% of the time that I have tested it
Signed-off-by: Richard Henderson
---
hw/virtio/vdpa-dev-pci.c | 2 +-
hw/virtio/vdpa-dev.c | 2 +-
hw/virtio/vhost-scsi-pci.c | 2 +-
hw/virtio/vhost-user-blk-pci.c | 2 +-
hw/virtio/vhost-user-device.c| 2 +-
hw/virtio/vhost-user-fs-pci.c| 2 +-
hw/virtio/vhos
Signed-off-by: Richard Henderson
---
hw/riscv/opentitan.c | 2 +-
hw/riscv/riscv-iommu-pci.c | 2 +-
hw/riscv/riscv-iommu.c | 2 +-
hw/riscv/riscv_hart.c | 2 +-
hw/riscv/sifive_u.c| 2 +-
5 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/hw/riscv/opentitan.c b
Signed-off-by: Richard Henderson
---
hw/intc/apic_common.c | 2 +-
hw/intc/arm_gic_common.c | 2 +-
hw/intc/arm_gicv2m.c | 2 +-
hw/intc/arm_gicv3_common.c| 2 +-
hw/intc/arm_gicv3_its.c | 2 +-
hw/intc/arm_gicv3_its_kvm.c | 2 +-
hw/intc/armv7m_nvic.c |
Signed-off-by: Richard Henderson
---
target/arm/cpu.c | 26 +-
target/arm/cpu64.c | 6 +++---
target/arm/tcg/cpu64.c | 2 +-
3 files changed, 17 insertions(+), 17 deletions(-)
diff --git a/target/arm/cpu.c b/target/arm/cpu.c
index 0cbda76ba0..1afa07511e 100644
Signed-off-by: Richard Henderson
---
hw/nubus/nubus-bridge.c | 2 +-
hw/nubus/nubus-device.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/nubus/nubus-bridge.c b/hw/nubus/nubus-bridge.c
index a42c86080f..83893e5a46 100644
--- a/hw/nubus/nubus-bridge.c
+++ b/hw/nubus/nu
Signed-off-by: Richard Henderson
---
hw/nvram/ds1225y.c | 2 +-
hw/nvram/eeprom_at24c.c| 2 +-
hw/nvram/fw_cfg.c | 6 +++---
hw/nvram/mac_nvram.c | 2 +-
hw/nvram/nrf51_nvm.c | 2 +-
hw/nvram/spapr_nvram.c | 2 +-
Signed-off-by: Richard Henderson
---
hw/ufs/lu.c | 2 +-
hw/ufs/ufs.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/ufs/lu.c b/hw/ufs/lu.c
index 81bfff9b4e..74ff52ad09 100644
--- a/hw/ufs/lu.c
+++ b/hw/ufs/lu.c
@@ -274,7 +274,7 @@ static UfsReqResult ufs_process_scsi_
Signed-off-by: Richard Henderson
---
hw/display/artist.c | 2 +-
hw/display/ati.c | 2 +-
hw/display/bcm2835_fb.c | 2 +-
hw/display/bochs-display.c | 2 +-
hw/display/cg3.c | 2 +-
hw/display/cirrus_vga.c | 2 +-
hw/display/cir
On 9/12/24 19:12, Brian Cain wrote:
Mea culpa, I don't know how I got this wrong in 2dfe93699c. Still
getting used to the new address, I suppose. Somehow I got it right in the
mailmap, though.
Signed-off-by: Brian Cain
---
MAINTAINERS | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Signed-off-by: Richard Henderson
---
hw/ssi/aspeed_smc.c | 4 ++--
hw/ssi/ibex_spi_host.c| 2 +-
hw/ssi/npcm7xx_fiu.c | 2 +-
hw/ssi/pnv_spi.c | 2 +-
hw/ssi/sifive_spi.c | 2 +-
hw/ssi/ssi.c | 2 +-
hw/ssi/xilinx_spi.c | 2 +-
hw/ssi/xilinx_spips.
Signed-off-by: Richard Henderson
---
hw/pci-host/astro.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/hw/pci-host/astro.c b/hw/pci-host/astro.c
index 379095b356..62e9c8acbf 100644
--- a/hw/pci-host/astro.c
+++ b/hw/pci-host/astro.c
@@ -461,10 +461,6 @@ static void elroy_pcihost_init(Ob
Signed-off-by: Richard Henderson
---
hw/hyperv/hv-balloon.c | 2 +-
hw/hyperv/syndbg.c | 2 +-
hw/hyperv/vmbus.c | 4 ++--
3 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/hw/hyperv/hv-balloon.c b/hw/hyperv/hv-balloon.c
index 3a9ef07691..74897b1604 100644
--- a/hw/hyperv/hv
Signed-off-by: Richard Henderson
---
hw/9pfs/virtio-9p-device.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/9pfs/virtio-9p-device.c b/hw/9pfs/virtio-9p-device.c
index efa41cfd73..b764e4cd3d 100644
--- a/hw/9pfs/virtio-9p-device.c
+++ b/hw/9pfs/virtio-9p-device.c
@@ -243
Signed-off-by: Richard Henderson
---
hw/misc/a9scu.c | 2 +-
hw/misc/allwinner-h3-dramc.c | 2 +-
hw/misc/allwinner-r40-dramc.c| 2 +-
hw/misc/allwinner-sid.c | 2 +-
hw/misc/applesmc.c | 2 +-
hw/misc/arm11scu.c | 2 +-
hw/misc/arm_l2
Signed-off-by: Richard Henderson
---
target/hexagon/cpu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/target/hexagon/cpu.c b/target/hexagon/cpu.c
index c9aa9408ec..a70007245e 100644
--- a/target/hexagon/cpu.c
+++ b/target/hexagon/cpu.c
@@ -48,7 +48,7 @@ static ObjectClass
Signed-off-by: Richard Henderson
---
hw/rtc/allwinner-rtc.c | 2 +-
hw/rtc/goldfish_rtc.c | 2 +-
hw/rtc/m48t59-isa.c| 2 +-
hw/rtc/m48t59.c| 2 +-
hw/rtc/mc146818rtc.c | 2 +-
hw/rtc/pl031.c | 2 +-
6 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/hw/rtc/allw
On 13/12/24 19:23, Peter Maydell wrote:
In the GICv3 ITS model, we have a common coding pattern which has a
local C struct like "DTEntry dte", which is a C representation of an
in-guest-memory data structure, and we call a function such as
get_dte() to read guest memory and fill in the C struct.
Signed-off-by: Richard Henderson
---
hw/sparc/sun4m.c | 2 +-
hw/sparc/sun4m_iommu.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/sparc/sun4m.c b/hw/sparc/sun4m.c
index d52e6a7213..b8cce271a7 100644
--- a/hw/sparc/sun4m.c
+++ b/hw/sparc/sun4m.c
@@ -732,7 +732,7
Signed-off-by: Richard Henderson
---
hw/pci-bridge/cxl_downstream.c | 2 +-
hw/pci-bridge/cxl_root_port.c | 2 +-
hw/pci-bridge/cxl_upstream.c| 2 +-
hw/pci-bridge/gen_pcie_root_port.c | 2 +-
hw/pci-bridge/pci_bridge_dev.c | 2 +-
hw/pci-bridge/pci_expander_bridge.c | 4
From: Roman Artemev
On RISC-V to StoreStore barrier corresponds
`fence w, w` not `fence r, r`
Cc: qemu-sta...@nongnu.org
Fixes: efbea94c76b ("tcg/riscv: Add slowpath load and store instructions")
Reviewed-by: Richard Henderson
Signed-off-by: Denis Tomashev
Signed-off-by: Roman Artemev
Message
1 - 100 of 384 matches
Mail list logo
