[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 0/4] X86: Alias isa-bios area and clean up
From: |
Michael S. Tsirkin |
Subject: |
Re: [PATCH 0/4] X86: Alias isa-bios area and clean up |
Date: |
Thu, 25 Apr 2024 06:16:10 -0400 |
On Mon, Apr 22, 2024 at 10:06:21PM +0200, Bernhard Beschow wrote:
> This series changes the "isa-bios" MemoryRegion to be an alias rather than a
> copy in the pflash case. This fixes issuing pflash commands in the isa-bios
> region which matches real hardware and which some real-world legacy bioses I'm
> running rely on. Furthermore, aliasing in the isa-bios area is already the
> current behavior in the bios (a.k.a. ROM) case, so this series consolidates
> behavior.
>
> The consolidateion results in duplicate code which is resolved in the second
> half (patches 3 and 4) in this series.
>
> Question: AFAIU, patch 2 changes the behavior for SEV-enabled guests since the
> isa-bios area is now encrypted. Does this need compat machinery or is it a
> bugfix?
When in doubt, do a compat thing.
> Testing done:
> * `make check` with qemu-system-x86_64 (QEMU 8.2.2) installed. All tests
> including migration tests pass.
> * `make check-avocado`
>
> Best regards,
> Bernhard
>
> Bernhard Beschow (4):
> hw/i386/pc_sysfw: Remove unused parameter from pc_isa_bios_init()
> hw/i386/pc_sysfw: Alias rather than copy isa-bios region
> hw/i386/x86: Eliminate two if statements in x86_bios_rom_init()
> hw/i386: Consolidate isa-bios creation
>
> include/hw/i386/x86.h | 2 ++
> hw/i386/pc_sysfw.c | 38 ++++----------------------------------
> hw/i386/x86.c | 35 +++++++++++++++++++----------------
> 3 files changed, 25 insertions(+), 50 deletions(-)
>
> --
> 2.44.0
>
- Re: [PATCH 1/4] hw/i386/pc_sysfw: Remove unused parameter from pc_isa_bios_init(), (continued)
- [PATCH 2/4] hw/i386/pc_sysfw: Alias rather than copy isa-bios region, Bernhard Beschow, 2024/04/22
- [PATCH 3/4] hw/i386/x86: Eliminate two if statements in x86_bios_rom_init(), Bernhard Beschow, 2024/04/22
- [PATCH 4/4] hw/i386: Consolidate isa-bios creation, Bernhard Beschow, 2024/04/22
- Re: [PATCH 0/4] X86: Alias isa-bios area and clean up, Bernhard Beschow, 2024/04/24
- Re: [PATCH 0/4] X86: Alias isa-bios area and clean up, Philippe Mathieu-Daudé, 2024/04/25
- Re: [PATCH 0/4] X86: Alias isa-bios area and clean up,
Michael S. Tsirkin <=