Re: Still did not succeed to boot on Lenovo Yoga C630

2022年12月24日(土) 10:35 Hiroo Ono (小野寛生) <hiroo.ono+freebsd@gmail.com>:
>
> > I run other arm64 machines w/o issue with the current code.
>
> Yes, Qualcomm's snapdragon is weird. I saw Linux people complain about
> it somewhere... Wanted to know  before I bought this Yoga C630.
>
> 2022年12月24日(土) 10:03 Warner Losh <imp@bsdimp.com>:
> >
> >
> >
> > On Fri, Dec 23, 2022 at 5:49 PM Hiroo Ono (小野寛生) <hiroo.ono+freebsd@gmail.com> wrote:
> >>
> >> The current status of FreeBSD 14-current on Lenovo Yoga C630 is as follows:
> >>
> >>  1) Merging from OpenBSD's loader code made the loader boot apart from
> >> 3 points (#2 to 4 ).
> >>  2) when comconsole->c_init() runs the 2nd time, it seems to freeze.
> >> (might be C630 specific)
> >>  3) SetVirtualAddressMap() in efi_do_vmap() freezes. (might also
> >> affect other snapdragon systems like Microsoft Arm Developer Kit)
> >>  4) The kernel is kicked but does not start.
> >>
> >> 1) is quite straightforward. What needs to be changed is
> >> stand/efi/loader/arch/arm64/start.S.
> >
> >
> > Can you share what needs to be done? To my eye, we don't need any changes, so it would be good to know what you've had to do exactly.
>
> Attached is the diff to start.S. There are 3 points.
> 1) The loader has to be aligned to 4kb.
> 2) Proper characteristic value should be in the PE header.
> 3) .text and .data segment have to be separate.
>
> It is from OpenBSD:
> https://github.com/openbsd/src/blob/master/sys/arch/arm64/stand/efiboot/start.S

Sorry patch to ldscript.arm64 was missing.
I am going to test your serial patch now.


> >> For 2), I do not know what to do. Currently, I commented out
> >> comconsole from struct console *consoles[] in stand/efi/loader/conf.c
> >> as a workaround. Maybe, I should write a fault handler that helps
> >> returning from the fault.
> >
> >
> > There were problems with this with HyperV on aarch64 too.
> >
> > Something like
> > diff --git a/stand/efi/loader/efiserialio.c b/stand/efi/loader/efiserialio.c
> > index 8b3f8e83e0b3..54ee39096685 100644
> > --- a/stand/efi/loader/efiserialio.c
> > +++ b/stand/efi/loader/efiserialio.c
> > @@ -261,11 +261,11 @@ comc_probe(struct console *sc)
> >                 if (comc_port == NULL)
> >                         return;
> >         }
> > -       comc_port->baudrate = COMSPEED;
> > +       comc_port->baudrate = 0;
> >         comc_port->ioaddr = 0;                  /* default port */
> > -       comc_port->databits = 8;                /* 8,n,1 */
> > -       comc_port->parity = NoParity;           /* 8,n,1 */
> > -       comc_port->stopbits = OneStopBit;       /* 8,n,1 */
> > +       comc_port->databits = 0;                /* 8,n,1 */
> > +       comc_port->parity = 0;          /* 8,n,1 */
> > +       comc_port->stopbits = 0;        /* 8,n,1 */
> >         comc_port->ignore_cd = 1;               /* ignore cd */
> >         comc_port->rtsdtr_off = 0;              /* rts-dtr is on */
> >         comc_port->sio = NULL;
> >
> > was needed.  Possibly the following would be better:
> >
> > diff --git a/stand/efi/loader/efiserialio.c b/stand/efi/loader/efiserialio.c
> > index 8b3f8e83e0b3..54ee39096685 100644
> > --- a/stand/efi/loader/efiserialio.c
> > +++ b/stand/efi/loader/efiserialio.c
> > @@ -494,8 +494,7 @@ comc_setup(void)
> >                 return (false);
> >
> >         status = comc_port->sio->SetAttributes(comc_port->sio,
> > -           comc_port->baudrate, 0, 0, comc_port->parity,
> > -           comc_port->databits, comc_port->stopbits);
> > +           0, 0, 0, 0, 0, 0);
> >         if (EFI_ERROR(status))
> >                 return (false);
> >