AW: [PATCH v3 3/3] X86: pxeboot: bugfix: Set variable for size of initrd

[Mittelstaedt Thomas (XC-CT/EBV3)]

Hi Bin,

I have never done this. Can you give me some hints:

- Base version of U-boot
- Path and filename for "efi-payload doc"
- Some info about format of doc file and /or example to have a look for

Mit freundlichen Grüßen / Best regards

Thomas Mittelstaedt

Cross-Domain Computing Solutions, Engineering BBM Vehicle Computer 3 (XC-CT/EBV3)
Robert Bosch GmbH | Postfach 10 02 61 | 31102 Hildesheim | GERMANY | www.bosch.com
Tel. +49 5121 49-2965 | Mobil +49 160 90418883 | Telefax +49 711 811-5052965 | Thomas.Mittelstaedt at de.bosch.com

Sitz: Stuttgart, Registergericht: Amtsgericht Stuttgart, HRB 14000;
Aufsichtsratsvorsitzender: Prof. Dr. Stefan Asenkerschbaumer; Geschäftsführung: Dr. Stefan Hartung, 
Dr. Christian Fischer, Dr. Markus Forschner, Stefan Grosch, Dr. Markus Heyn, Dr. Tanja Rückert

> -----Ursprüngliche Nachricht-----
> Von: Bin Meng <bmeng.cn at gmail.com>
> Gesendet: Freitag, 11. August 2023 17:18
> An: Simon Glass <sjg at chromium.org>
> Cc: Mittelstaedt Thomas (XC-CT/EBV3) <thomas.mittelstaedt at de.bosch.com>;
> u-boot at lists.denx.de; Niel Armstrong <neil.armstrong at linaro.org>; Patrick
> Delaunay <patrick.delaunay at foss.st.com>; Ramon Fried
> <rfried.dev at gmail.com>; Marek Vasut <marex at denx.de>; Manuel Traut
> <manuel.traut at mt.com>; Heinrich Schuchardt <xypron.glpk at gmx.de>
> Betreff: Re: [PATCH v3 3/3] X86: pxeboot: bugfix: Set variable for size of initrd
> 
> Hi Thomas, Simon,
> 
> On Fri, Aug 11, 2023 at 8:26 PM Bin Meng <bmeng.cn at gmail.com> wrote:
> >
> > Hi Simon,
> >
> > On Fri, Aug 11, 2023 at 5:33 AM Simon Glass <sjg at chromium.org> wrote:
> > >
> > > Hi Bin,
> > >
> > > On Thu, 4 May 2023 at 18:40, Simon Glass <sjg at chromium.org> wrote:
> > > >
> > > > On Thu, 4 May 2023 at 07:43, <thomas.mittelstaedt at de.bosch.com> wrote:
> > > > >
> > > > > From: Thomas Mittelstaedt <thomas.mittelstaedt at de.bosch.com>
> > > > >
> > > > > The problem was, that zboot() didn't work because of missing
> > > > > ramdisc size.
> > > > >
> > > > > Signed-off-by: Thomas Mittelstaedt
> > > > > <thomas.mittelstaedt at de.bosch.com>
> > > > > ---
> > > > >  boot/pxe_utils.c | 2 +-
> > > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > > >
> > > >
> > > > Fixes: 085cbdafca9 ("pxe: simplify label_boot()")
> > > > Reviewed-by: Simon Glass <sjg at chromium.org>
> > >
> > > This patch and some others seem to have been lost. Can you please check
> here?
> > >
> > > https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpa
> > >
> tchwork.ozlabs.org%2Fproject%2Fuboot%2Flist%2F%3Fseries%3D%26submitt
> > >
> er%3D%26state%3D%26q%3D%26archive%3D%26delegate%3D56520&data=05
> %7C01
> > >
> %7Cthomas.mittelstaedt%40de.bosch.com%7C1dde215fe90f46abcc1008db9a7e
> > >
> 30a0%7C0ae51e1907c84e4bbb6d648ee58410f4%7C0%7C0%7C638273638986309
> 018
> > >
> %7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJ
> BTiI
> > >
> 6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=b%2B9kLZEcwvj5U
> VSlAmQf
> > > 1JGhSP7EVOvnMezV4vUwqG8%3D&reserved=0
> > >
> >
> > Indeed I missed these. Will take a look soon.
> >
> 
> I have rebased this series on top of u-boot/master and pushed this to u-boot-
> x86/master.
> 
> Would you please post a patch to efi-payload doc to describe how to test this with
> Virtual Box? Thanks!
> 
> Regards,
> Bin