[v5] hw/core/loader: capture Error from load_image_targphys

[Patch v5 0/6] hw/core/loader: capture Error from load_image_targphys

Vishal Chourasia posted 6 patches 3 weeks, 3 days ago

Diff against v4 v6 v7
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20251021105442.1474602-2-vishalc@linux.ibm.com

Maintainers: Richard Henderson <richard.henderson@linaro.org>, Peter Maydell <peter.maydell@linaro.org>, Antony Pavlov <antonynpavlov@gmail.com>, Rob Herring <robh@kernel.org>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Alistair Francis <alistair@alistair23.me>, "Alex Bennée" <alex.bennee@linaro.org>, Helge Deller <deller@gmx.de>, "Michael S. Tsirkin" <mst@redhat.com>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, Paolo Bonzini <pbonzini@redhat.com>, Eduardo Habkost <eduardo@habkost.net>, Corey Minyard <minyard@acm.org>, Song Gao <gaosong@loongson.cn>, Bibo Mao <maobibo@loongson.cn>, Jiaxun Yang <jiaxun.yang@flygoat.com>, Thomas Huth <huth@tuxfamily.org>, Laurent Vivier <laurent@vivier.eu>, "Edgar E. Iglesias" <edgar.iglesias@gmail.com>, Paul Burton <paulburton@kernel.org>, Aleksandar Rikalo <arikalo@gmail.com>, Huacai Chen <chenhuacai@kernel.org>, "Hervé Poussineau" <hpoussin@reactos.org>, Aurelien Jarno <aurelien@aurel32.net>, Stafford Horne <shorne@gmail.com>, BALATON Zoltan <balaton@eik.bme.hu>, Bernhard Beschow <shentey@gmail.com>, Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>, Nicholas Piggin <npiggin@gmail.com>, Aditya Gupta <adityag@linux.ibm.com>, Glenn Miles <milesg@linux.ibm.com>, Harsh Prateek Bora <harshpb@linux.ibm.com>, Palmer Dabbelt <palmer@dabbelt.com>, Weiwei Li <liwei1518@gmail.com>, Daniel Henrique Barboza <dbarboza@ventanamicro.com>, Liu Zhiwei <zhiwei_liu@linux.alibaba.com>, Yoshinori Sato <yoshinori.sato@nifty.com>, Christian Borntraeger <borntraeger@linux.ibm.com>, Jared Rossi <jrossi@linux.ibm.com>, Zhuoying Cai <zycai@linux.ibm.com>, David Hildenbrand <david@redhat.com>, Ilya Leoshkevich <iii@linux.ibm.com>, Halil Pasic <pasic@linux.ibm.com>, Eric Farman <farman@linux.ibm.com>, Matthew Rosato <mjrosato@linux.ibm.com>, Magnus Damm <magnus.damm@gmail.com>, Igor Mammedov <imammedo@redhat.com>, Ani Sinha <anisinha@redhat.com>, "Clément Chigot" <chigot@adacore.com>, Frederic Konrad <konrad.frederic@yahoo.fr>, Artyom Tarasenko <atar4qemu@gmail.com>, Max Filippov <jcmvbkbc@gmail.com>, David Gibson <david@gibson.dropbear.id.au>

There is a newer version of this series

hw/alpha/dp264.c         |  4 ++--
hw/arm/armv7m.c          |  2 +-
hw/arm/boot.c            |  5 +++--
hw/arm/digic_boards.c    |  2 +-
hw/arm/highbank.c        |  3 ++-
hw/arm/raspi.c           |  2 +-
hw/arm/vexpress.c        |  2 +-
hw/core/generic-loader.c |  8 +++++---
hw/core/guest-loader.c   |  7 ++++---
hw/core/loader.c         | 36 ++++++++++++++++++++++++++++--------
hw/hppa/machine.c        |  5 +++--
hw/i386/multiboot.c      |  2 +-
hw/i386/x86-common.c     |  4 ++--
hw/ipmi/ipmi_bmc_sim.c   |  2 +-
hw/loongarch/boot.c      |  5 ++---
hw/m68k/an5206.c         |  2 +-
hw/m68k/mcf5208.c        |  4 ++--
hw/m68k/next-cube.c      |  2 +-
hw/m68k/q800.c           |  7 ++++---
hw/m68k/virt.c           |  4 ++--
hw/microblaze/boot.c     |  5 +++--
hw/mips/boston.c         |  2 +-
hw/mips/fuloong2e.c      |  9 +++++----
hw/mips/jazz.c           |  2 +-
hw/mips/loongson3_virt.c | 10 ++++++----
hw/mips/malta.c          |  9 +++++----
hw/nubus/nubus-device.c  |  2 +-
hw/openrisc/boot.c       |  5 +++--
hw/pci/pci.c             |  2 +-
hw/ppc/amigaone.c        | 15 ++++++++-------
hw/ppc/e500.c            | 18 ++++++++++--------
hw/ppc/mac_newworld.c    | 28 ++++++++++++++++++----------
hw/ppc/mac_oldworld.c    | 28 ++++++++++++++++++----------
hw/ppc/pegasos2.c        | 18 ++++++++++++------
hw/ppc/pnv.c             | 30 +++++++++++++++---------------
hw/ppc/ppc440_bamboo.c   | 10 ++++++----
hw/ppc/prep.c            | 27 +++++++++++++++++----------
hw/ppc/sam460ex.c        | 10 ++++++----
hw/ppc/spapr.c           | 35 ++++++++++++++++++-----------------
hw/ppc/virtex_ml507.c    | 18 ++++++++++++------
hw/riscv/boot.c          |  7 ++++---
hw/rx/rx-gdbsim.c        |  2 +-
hw/s390x/ipl.c           |  8 +++++---
hw/sh4/r2d.c             |  8 +++++---
hw/smbios/smbios.c       |  2 +-
hw/sparc/leon3.c         |  4 ++--
hw/sparc/sun4m.c         |  8 +++++---
hw/sparc64/sun4u.c       |  7 ++++---
hw/xtensa/xtfpga.c       |  3 ++-
include/hw/loader.h      |  8 +++++---
system/device_tree.c     |  2 +-
51 files changed, 268 insertions(+), 182 deletions(-)

Expand all Fold all

[Patch v5 0/6] hw/core/loader: capture Error from load_image_targphys

Posted by Vishal Chourasia 3 weeks, 3 days ago

Currently, when QEMU fails to load images (kernel, BIOS, initrd) via
load_image_targphys(), the error messages are not descriptive enough
for users to understand why the loading failed. This series adds the
ability to pass Error information through the loader functions to
provide more detailed error reporting.

v5:
- Moved the patch which replaces open() with qemu_open() earlier in the
  series
- Incorporated minor changes in the error messages requested by Balaton
- Moved check for size zero in load_image_targphys_as() into it's
  separate patch 

v4:
- Reject empty files (`size == 0`) with an appropriate error. [1/5]
- Populate error when `rom_add_file_fixed_as()` fails. [1/5]
- Add missing check in hw/ppc/virtex_ml507.c [5/5]
- Fixed checkpatch warnings [5/5]
- Check errp for failures from load_image_targphys/load_image_targphys_as [3,5/5]
- Remove unused variables [5/5]

v3:
- Use qemu_open() instead of open() in get_image_size() (danpb)
- Remove redundant error_setg_file_open() call (danpb)
- Use error_prepend()/error_reportf_err() to preserve underlying
  error details (danpb)

Vishal Chourasia (6):
  hw/core/loader: Use qemu_open() instead of open() in get_image_size()
  hw/core/loader: capture Error from load_image_targphys
  hw/core/loader: add check for zero size in load_image_targphys_as
  hw/core: Pass errp to load_image_targphys_as()
  hw/ppc/spapr: Rename resize_hpt_err to errp
  hw/ppc: Pass errp to load_image_targphys() and report errors

 hw/alpha/dp264.c         |  4 ++--
 hw/arm/armv7m.c          |  2 +-
 hw/arm/boot.c            |  5 +++--
 hw/arm/digic_boards.c    |  2 +-
 hw/arm/highbank.c        |  3 ++-
 hw/arm/raspi.c           |  2 +-
 hw/arm/vexpress.c        |  2 +-
 hw/core/generic-loader.c |  8 +++++---
 hw/core/guest-loader.c   |  7 ++++---
 hw/core/loader.c         | 36 ++++++++++++++++++++++++++++--------
 hw/hppa/machine.c        |  5 +++--
 hw/i386/multiboot.c      |  2 +-
 hw/i386/x86-common.c     |  4 ++--
 hw/ipmi/ipmi_bmc_sim.c   |  2 +-
 hw/loongarch/boot.c      |  5 ++---
 hw/m68k/an5206.c         |  2 +-
 hw/m68k/mcf5208.c        |  4 ++--
 hw/m68k/next-cube.c      |  2 +-
 hw/m68k/q800.c           |  7 ++++---
 hw/m68k/virt.c           |  4 ++--
 hw/microblaze/boot.c     |  5 +++--
 hw/mips/boston.c         |  2 +-
 hw/mips/fuloong2e.c      |  9 +++++----
 hw/mips/jazz.c           |  2 +-
 hw/mips/loongson3_virt.c | 10 ++++++----
 hw/mips/malta.c          |  9 +++++----
 hw/nubus/nubus-device.c  |  2 +-
 hw/openrisc/boot.c       |  5 +++--
 hw/pci/pci.c             |  2 +-
 hw/ppc/amigaone.c        | 15 ++++++++-------
 hw/ppc/e500.c            | 18 ++++++++++--------
 hw/ppc/mac_newworld.c    | 28 ++++++++++++++++++----------
 hw/ppc/mac_oldworld.c    | 28 ++++++++++++++++++----------
 hw/ppc/pegasos2.c        | 18 ++++++++++++------
 hw/ppc/pnv.c             | 30 +++++++++++++++---------------
 hw/ppc/ppc440_bamboo.c   | 10 ++++++----
 hw/ppc/prep.c            | 27 +++++++++++++++++----------
 hw/ppc/sam460ex.c        | 10 ++++++----
 hw/ppc/spapr.c           | 35 ++++++++++++++++++-----------------
 hw/ppc/virtex_ml507.c    | 18 ++++++++++++------
 hw/riscv/boot.c          |  7 ++++---
 hw/rx/rx-gdbsim.c        |  2 +-
 hw/s390x/ipl.c           |  8 +++++---
 hw/sh4/r2d.c             |  8 +++++---
 hw/smbios/smbios.c       |  2 +-
 hw/sparc/leon3.c         |  4 ++--
 hw/sparc/sun4m.c         |  8 +++++---
 hw/sparc64/sun4u.c       |  7 ++++---
 hw/xtensa/xtfpga.c       |  3 ++-
 include/hw/loader.h      |  8 +++++---
 system/device_tree.c     |  2 +-
 51 files changed, 268 insertions(+), 182 deletions(-)

-- 
2.51.0