This pullreq has:

* two arm bug fixes which fix some "Linux fails to boot" bugs

* a docs typo-fixing patch

* a couple of compile failure/warning issues

I think they're all pretty safe and worth having in rc3.

thanks

-- PMM

The following changes since commit a6b1c53e79d08a99a28cc3e67a3e1a7c34102d6b:

Merge tag 'linux-user-for-7.1-pull-request' of https://gitlab.com/laurent_vivier/qemu into staging (2022-08-10 10:26:57 -0700)

https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20220812

for you to fetch changes up to 4311682ea8293f720730f260e8a7601117d79e65:

cutils: Add missing dyld(3) include on macOS (2022-08-12 11:33:52 +0100)

* Don't report Statistical Profiling Extension in ID registers

* virt ACPI tables: Present the GICR structure properly for GICv4

* Fix some typos in documentation

* tests/unit: fix a -Wformat-truncation warning

* cutils: Add missing dyld(3) include on macOS

Marc-André Lureau (1):

tests/unit: fix a -Wformat-truncation warning

Peter Maydell (1):

target/arm: Don't report Statistical Profiling Extension in ID registers

Philippe Mathieu-Daudé (1):

cutils: Add missing dyld(3) include on macOS

Stefan Weil (1):

Fix some typos in documentation (most of them found by codespell)

Zenghui Yu (1):

hw/arm/virt-acpi-build: Present the GICR structure properly for GICv4

docs/about/deprecated.rst | 2 +-

docs/specs/acpi_erst.rst | 4 ++--

docs/system/devices/canokey.rst | 8 ++++----

docs/system/devices/cxl.rst | 12 ++++++------

hw/arm/virt-acpi-build.c | 4 ++--

target/arm/cpu.c | 11 +++++++++++

tests/unit/test-qobject-input-visitor.c | 3 +--

util/cutils.c | 4 ++++

util/oslib-posix.c | 4 ----

9 files changed, 31 insertions(+), 21 deletions(-)

From: Zenghui Yu <yuzenghui@huawei.com>

With the introduction of the new TCG GICv4, build_madt() is badly broken

as we do not present any GIC Redistributor structure in MADT for GICv4

guests, so that they have no idea about where the Redistributor

register frames are. This fixes a Linux guest crash at boot time with

ACPI enabled and '-machine gic-version=4'.

While at it, let's convert the remaining hard coded gic_version into

enumeration VIRT_GIC_VERSION_2 for consistency.

Signed-off-by: Zenghui Yu <yuzenghui@huawei.com>

Message-id: 20220812022018.1069-1-yuzenghui@huawei.com

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

hw/arm/virt-acpi-build.c | 4 ++--

1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c

--- a/hw/arm/virt-acpi-build.c

+++ b/hw/arm/virt-acpi-build.c

@@ -XXX,XX +XXX,XX @@ build_madt(GArray *table_data, BIOSLinker *linker, VirtMachineState *vms)

uint32_t pmu_interrupt = arm_feature(&armcpu->env, ARM_FEATURE_PMU) ?

PPI(VIRTUAL_PMU_IRQ) : 0;

- if (vms->gic_version == 2) {

+ if (vms->gic_version == VIRT_GIC_VERSION_2) {

physical_base_address = memmap[VIRT_GIC_CPU].base;

gicv = memmap[VIRT_GIC_VCPU].base;

gich = memmap[VIRT_GIC_HYP].base;

@@ -XXX,XX +XXX,XX @@ build_madt(GArray *table_data, BIOSLinker *linker, VirtMachineState *vms)

build_append_int_noprefix(table_data, armcpu->mp_affinity, 8);

- if (vms->gic_version == 3) {

+ if (vms->gic_version != VIRT_GIC_VERSION_2) {

build_append_gicr(table_data, memmap[VIRT_GIC_REDIST].base,

memmap[VIRT_GIC_REDIST].size);

if (virt_gicv3_redist_region_count(vms) == 2) {

2.25.1

From: Philippe Mathieu-Daudé <f4bug@amsat.org>

Commit 06680b15b4 moved qemu_*_exec_dir() to cutils but forgot

to move the macOS dyld(3) include, resulting in the following

error (when building with Homebrew GCC on macOS Monterey 12.4):

[313/1197] Compiling C object libqemuutil.a.p/util_cutils.c.o

FAILED: libqemuutil.a.p/util_cutils.c.o

../../util/cutils.c:1039:13: error: implicit declaration of function '_NSGetExecutablePath' [-Werror=implicit-function-declaration]

1039 | if (_NSGetExecutablePath(fpath, &len) == 0) {

| ^~~~~~~~~~~~~~~~~~~~

../../util/cutils.c:1039:13: error: nested extern declaration of '_NSGetExecutablePath' [-Werror=nested-externs]

Fix by moving the include line to cutils.

Fixes: 06680b15b4 ("include: move qemu_*_exec_dir() to cutils")

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

Message-id: 20220809222046.30812-1-f4bug@amsat.org

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

util/cutils.c | 4 ++++

util/oslib-posix.c | 4 ----

2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/util/cutils.c b/util/cutils.c

--- a/util/cutils.c

+++ b/util/cutils.c

@@ -XXX,XX +XXX,XX @@

#include <kernel/image.h>

#endif

+#ifdef __APPLE__

+#include <mach-o/dyld.h>

+#endif

+

#ifdef G_OS_WIN32

#include <pathcch.h>

#include <wchar.h>

diff --git a/util/oslib-posix.c b/util/oslib-posix.c

index XXXXXXX..XXXXXXX 100644

--- a/util/oslib-posix.c

+++ b/util/oslib-posix.c

@@ -XXX,XX +XXX,XX @@

#include <lwp.h>

#endif

-#ifdef __APPLE__

-#include <mach-o/dyld.h>

-#endif

-

#include "qemu/mmap-alloc.h"

#ifdef CONFIG_DEBUG_STACK_USAGE

2.25.1

From: Marc-André Lureau <marcandre.lureau@redhat.com>

../tests/test-qobject-input-visitor.c: In function ‘test_visitor_in_list’:

../tests/test-qobject-input-visitor.c:454:49: warning: ‘%d’ directive output may be truncated writing between 1 and 10 bytes into a region of size 6 [-Wformat-truncation=]

454 | snprintf(string, sizeof(string), "string%d", i);

| ^~

../tests/test-qobject-input-visitor.c:454:42: note: directive argument in the range [0, 2147483606]

454 | snprintf(string, sizeof(string), "string%d", i);

| ^~~~~~~~~~

../tests/test-qobject-input-visitor.c:454:9: note: ‘snprintf’ output between 8 and 17 bytes into a destination of size 12

454 | snprintf(string, sizeof(string), "string%d", i);

| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Rather than trying to be clever, since this is called 3 times during

tests, let's simply use g_strdup_printf().

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

Reviewed-by: Markus Armbruster <armbru@redhat.com>

Message-id: 20220810121513.1356081-1-marcandre.lureau@redhat.com

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

[PMM: fixed commit message typos]

tests/unit/test-qobject-input-visitor.c | 3 +--

1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/tests/unit/test-qobject-input-visitor.c b/tests/unit/test-qobject-input-visitor.c

--- a/tests/unit/test-qobject-input-visitor.c

+++ b/tests/unit/test-qobject-input-visitor.c

@@ -XXX,XX +XXX,XX @@ static void test_visitor_in_list(TestInputVisitorData *data,

g_assert(head != NULL);

for (i = 0, item = head; item; item = item->next, i++) {

- char string[12];

+ g_autofree char *string = g_strdup_printf("string%d", i);

- snprintf(string, sizeof(string), "string%d", i);

g_assert_cmpstr(item->value->string, ==, string);

g_assert_cmpint(item->value->integer, ==, 42 + i);

}

2.25.1

The newly added neoverse-n1 CPU has ID register values which indicate

the presence of the Statistical Profiling Extension, because the real

hardware has this feature. QEMU's TCG emulation does not yet

implement SPE, though (not even as a minimal stub implementation), so

guests will crash if they try to use it because the SPE system

registers don't exist.

Force ID_AA64DFR0_EL1.PMSVer to 0 in CPU realize for TCG, so that

we don't advertise to the guest a feature that doesn't exist.

(We could alternatively do this by editing the value that

aarch64_neoverse_n1_initfn() sets for this ID register, but

suppressing the field in realize means we won't re-introduce this bug

when we add other CPUs that have SPE in hardware, such as the

Neoverse-V1.)

An example of a non-booting guest is current mainline Linux (5.19),

when booting in EL2 on the virt board (ie with -machine

virtualization=on).

Reported-by: Zenghui Yu <yuzenghui@huawei.com>

target/arm/cpu.c | 11 +++++++++++

1 file changed, 11 insertions(+)