[PATCH 04/18] x86: qemu: Enable dhrystone

2024-11-12 Thread Simon Glass
Provide the 'dhry' command, which helps to check that kvm is being used properly with QEMU. Signed-off-by: Simon Glass --- configs/qemu-x86_64_defconfig | 1 + configs/qemu-x86_defconfig| 1 + 2 files changed, 2 insertions(+) diff --git a/configs/qemu-x86_64_defconfig b/configs/qemu-x86_64

[PATCH 05/18] x86: qemu: Avoid accessing BSS too early

2024-11-12 Thread Simon Glass
BSS is placed in DRAM which is actually available early with QEMU. But it is cleared by the init sequence, so values stored there are lost. Move the system-type flag into a function, instead. Signed-off-by: Simon Glass --- arch/x86/cpu/qemu/qemu.c | 20 ++-- 1 file changed, 14

Re: [PATCH v2 1/4] tests/functional: Introduce _console_read()

2024-11-12 Thread Daniel P . Berrangé
On Tue, Nov 12, 2024 at 02:02:43PM +0100, Cédric Le Goater wrote: > Interaction with the console has been a problem in our avocado > tests. In some cases, the expected string does not match in the > output, causing the test to fail with a timeout. These were worked > around by sleeping before readi

[PATCH v3 10/15] acpi/ghes: don't crash QEMU if ghes GED is not found

2024-11-12 Thread Mauro Carvalho Chehab
Make error handling within ghes_record_cper_errors() consistent, i.e. instead abort just print a error in case ghes GED is not found. Signed-off-by: Mauro Carvalho Chehab --- hw/acpi/ghes.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/hw/acpi/ghes.c b/hw/acpi/ghes.c in

[PATCH v3 09/15] acpi/ghes: better name GHES memory error function

2024-11-12 Thread Mauro Carvalho Chehab
The current function used to generate GHES data is specific for memory errors. Give a better name for it, as we now have a generic function as well. Reviewed-by: Igor Mammedov Signed-off-by: Mauro Carvalho Chehab --- hw/acpi/ghes-stub.c| 2 +- hw/acpi/ghes.c | 2 +- include/hw/acpi/

[PATCH v3 08/15] acpi/ghes: make the GHES record generation more generic

2024-11-12 Thread Mauro Carvalho Chehab
Split the code into separate functions to allow using the common CPER filling code by different error sources. The generic code was moved to ghes_record_cper_errors(), and ghes_gen_err_data_uncorrectable_recoverable() now contains only a logic to fill GEGB part of the record. The remaining code t

[PATCH v3 11/15] acpi/ghes: rename etc/hardware_error file macros

2024-11-12 Thread Mauro Carvalho Chehab
Now that we have also have a file to store HEST data location, which is part of GHES, better name the file where CPER records are stored. No functional changes. Signed-off-by: Mauro Carvalho Chehab Reviewed-by: Igor Mammedov Reviewed-by: Jonathan Cameron --- hw/acpi/ghes.c | 38 ++

[PATCH v3 03/15] acpi/ghes: simplify the per-arch caller to build HEST table

2024-11-12 Thread Mauro Carvalho Chehab
The GHES driver requires not only a HEST table, but also a separate firmware file to store Error Structure records. It can't do one without the other. Simplify the caller logic for it to require one function. No functional changes. Signed-off-by: Mauro Carvalho Chehab Reviewed-by: Jonathan Came

[PATCH v3 14/15] acpi/ghes: Change ghes fill logic to work with only one source

2024-11-12 Thread Mauro Carvalho Chehab
Extending to multiple sources require a BIOS pointer to the beginning of the HEST table, which in turn requires a backward-compatible code. So, the current code supports only one source. Ensure that and simplify the code. Signed-off-by: Mauro Carvalho Chehab --- hw/acpi/ghes.c | 11 +++

[PATCH v3 06/15] acpi/ghes: Remove a duplicated out of bounds check

2024-11-12 Thread Mauro Carvalho Chehab
acpi_ghes_record_errors() has an assert() at the beginning to ensure that source_id will be lower than ACPI_GHES_ERROR_SOURCE_COUNT. Remove a duplicated check. Signed-off-by: Mauro Carvalho Chehab Reviewed-by: Igor Mammedov Reviewed-by: Jonathan Cameron --- hw/acpi/ghes.c | 4 +--- 1 file chan

Re: [PATCH] MAINTAINERS: add myself as the maintainer for LoongArch VirtMachine

2024-11-12 Thread Philippe Mathieu-Daudé
On 12/11/24 07:37, Bibo Mao wrote: Song Gao is will be sick leave for a long time, I apply for maintainer for LoongArch VirtMachien during this period, LoongArch TCG keeps unchanged since I am not familiar with it. The maintainer duty will transfer to him after he comes back to work. Signed-off-

Re: [PATCH v2 1/7] accel/kvm: Keep track of the HWPoisonPage page_size

2024-11-12 Thread David Hildenbrand
On 07.11.24 11:21, “William Roche wrote: From: William Roche When a memory page is added to the hwpoison_page_list, include the page size information. This size is the backend real page size. To better deal with hugepages, we create a single entry for the entire page. Signed-off-by: William R

Re: [PATCH] MAINTAINERS: add myself as the maintainer for LoongArch VirtMachine

2024-11-12 Thread Philippe Mathieu-Daudé
On 12/11/24 07:37, Bibo Mao wrote: Song Gao is will be sick leave for a long time, I apply for maintainer for LoongArch VirtMachien during this period, LoongArch TCG keeps unchanged "Virt Machine" s/keeps/stays/ I suppose, but I'm not a native English speaker. since I am not familiar with it

Re: [PATCH] firmware: Switch back to struct platform_driver::remove()

2024-11-12 Thread Greg Kroah-Hartman
On Tue, Nov 12, 2024 at 09:35:20AM +0100, Uwe Kleine-König wrote: > After commit 0edb555a65d1 ("platform: Make platform_driver::remove() > return void") .remove() is (again) the right callback to implement for > platform drivers. > > Convert all platform drivers below drivers/firmware to use .remo

[PATCH] docs/devel/testing/functional: Clarify that we have to use the build folder

2024-11-12 Thread Thomas Huth
Make it clear that the commands have to be run from the folder with the build, and use the python3 from our pyvenv to make sure that the pycotap module is available. Signed-off-by: Thomas Huth --- docs/devel/testing/functional.rst | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff

Re: [PATCH v2 4/4] tests/functional: Remove sleep workarounds from Aspeed tests

2024-11-12 Thread Thomas Huth
On 12/11/2024 14.02, Cédric Le Goater wrote: These were introduced in the avocado tests to workaround read issues when interacting with console. They are no longer necessary and we can use the expected "login:" string now. Signed-off-by: Cédric Le Goater --- tests/functional/test_arm_aspeed.

Re: [Stable-7.2.15 17/33] target/i386: Walk NPT in guest real mode

2024-11-12 Thread Michael Tokarev
On 12.11.2024 15:16, Alexander Graf wrote: Hey Michael, Does this queue also include the fix for the fix ("target/i386: Fix legacy page table walk")? https://lore.kernel.org/qemu-devel/20241106154329.67218-1-g...@amazon.com/ Not yet, I'm waiting for it to land in master first. Yes, I'm awar

Re: [PATCH v2 4/4] tests/functional: Remove sleep workarounds from Aspeed tests

2024-11-12 Thread Cédric Le Goater
On 11/12/24 14:36, Thomas Huth wrote: On 12/11/2024 14.02, Cédric Le Goater wrote: These were introduced in the avocado tests to workaround read issues when interacting with console. They are no longer necessary and we can use the expected  "login:" string now. Signed-off-by: Cédric Le Goater

Re: [PATCH v2 6/7] hostmem: Handle remapping of RAM

2024-11-12 Thread David Hildenbrand
On 07.11.24 11:21, “William Roche wrote: From: David Hildenbrand Let's register a RAM block notifier and react on remap notifications. Simply re-apply the settings. Warn only when something goes wrong. Note: qemu_ram_remap() will not remap when RAM_PREALLOC is set. Could be that hostmem is sti

Re: [PATCH v2 1/4] tests/functional: Introduce _console_read()

2024-11-12 Thread Thomas Huth
On 12/11/2024 14.02, Cédric Le Goater wrote: Interaction with the console has been a problem in our avocado tests. In some cases, the expected string does not match in the output, causing the test to fail with a timeout. These were worked around by sleeping before reading the console and even wit

Re: [PATCH v2 1/4] tests/functional: Introduce _console_read()

2024-11-12 Thread Cédric Le Goater
Maybe we could use some funky unicode character instead? For example:     console_logger.debug('🔎 %s', msg) or:     console_logger.debug('✅ %s', msg) How's that ? https://paste.debian.net/1335303/ Thanks, C.

Re: [PATCH v4 7/7] target/i386: Add EPYC-Genoa model to support Zen 4 processor series

2024-11-12 Thread Moger, Babu
Hi Maksim, On 11/12/24 04:09, Maksim Davydov wrote: > > > On 11/8/24 23:56, Moger, Babu wrote: >> Hi Maxim, >> >> Thanks for looking into this. I will fix the bits I mentioned below in >> upcoming Genoa/Turin model update. >> >> I have few comments below. >> >> On 11/8/2024 12:15 PM, Maksim Davy

<    1   2   3