Re: disk/ahci: Unaligned pointer with old commit as coreboot payload

[Paul Menzel]

Dear Andrei,


Am Sonntag, den 24.05.2015, 13:15 +0300 schrieb Andrei Borzenkov:
> В Sun, 24 May 2015 09:03:40 +0200 Paul Menzel пишет:
>
> > further trying to bisect the AHCI regression on, that the GRUB payload
> > does not detect the AHCI drive [1], I tried to build GRUB from commits
> > changing `grub-core/disk/ahci.c`, I built commit dc2a6c8b (Improve AHCI
> > detection and command issuing.), but it it fails to boot with
> > 
> >         Unaligned pointer …
> >         Aborted.
> > 
> > The same happens, when I just checkout that revision for `ahci.c` and
> > leave the rest as is.
> > 
> >         $ git checkout dc2a6c8b0e3dfe3bce78dbe0febcd9d4d9038411 -- 
> > grub-core/disk/ahci.c
> >         $ ./autogen.sh
> >         $ ./configure --with-plattform=coreboot
> >         $ make -j
> >         $ make default_payload.elf
> > 
> > I’ll attach that version of `ahci.c` to this message. Do you see what
> > could be causing that unaligned pointer?
> > 
> 
> It is not clear - do you mean that this specific revision has this
> problem and others not? Or that this problem happens since this
> revision?

It’s happening whenever `ahci.c` from commit dc2a6c8b is used (with
commit dc2a6c8b and master).


Thanks,

Paul


> > [1] https://lists.gnu.org/archive/html/grub-devel/2014-11/msg00026.html

signature.asc
Description: This is a digitally signed message part