[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH qemu v11] spapr: Implement Open Firmware client interface

From: BALATON Zoltan
Subject: Re: [PATCH qemu v11] spapr: Implement Open Firmware client interface
Date: Tue, 8 Dec 2020 12:23:43 +0100 (CET)

On Tue, 8 Dec 2020, Alexey Kardashevskiy wrote:
On 07/12/2020 22:48, BALATON Zoltan wrote:
diff --git a/include/hw/ppc/spapr.h b/include/hw/ppc/spapr.h
index 2e89e36cfbdc..048bf49592aa 100644
--- a/include/hw/ppc/spapr.h
+++ b/include/hw/ppc/spapr.h
@@ -175,6 +175,13 @@ struct SpaprMachineState {
    long kernel_size;
    bool kernel_le;
    uint64_t kernel_addr;
+    bool vof; /* Virtual Open Firmware */
+    uint32_t rtas_base;
+    GArray *claimed; /* array of SpaprOfClaimed */
+    uint64_t claimed_base;
+    GHashTable *of_instances; /* ihandle -> SpaprOfInstance */
+    uint32_t of_instance_last;
+    char *bootargs;

Are these really state for vof so is it better to place them in a separate of_state struct instead of adding to the machine state? I'm not interested in spapr but interested in using vof as a replacement firmware for other machines so clear separation of what is spapr specific and what is vof specific would help me (and maybe also other reviewers to tell how much impact this really has on spapr which seems to be a concern of Greg).

This is a very good point, I'll separate VOF from the rest, may be even at QOM level. I was also thinking of adding a pseries-vof machine type but this is probably an overkill.

Out of curiosity - how are you going to use this VOF anyway, for what machine type?


It is basically working now, can boot MorphOS (and also AmigaOS but that has no display driver for VGA so can't be seen such as some Linux versions for the machine too which have the same problem) but to be able to upstream it I'll need to clean it up and have some firmware to avoid needing a non-distributable ROM image. VOF might be the simplest way for this to just get the Amiga like bootloaders and Linux start which only need some CI fuctions.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]