[U-Boot] [PATCH] part_efi: fix protective_mbr struct allocation

Lukasz Majewski l.majewski at samsung.com
Wed Feb 19 09:19:04 CET 2014


Hi Albert, Tom

> Hi Lukasz,
> 
> On Wed, 12 Feb 2014 21:45:41 +0100, Lukasz Majewski
> <l.majewski at majess.pl> wrote:
> 
> > I think, that the patch for fixing the unaligned access in this
> > function has already been posted by Piotr Wilczek. We have
> > experienced similar issues with Samsung's Exynos4 based targets.
> > 
> > [PATCH V2] disk:efi: avoid unaligned access on efi partition
> > 
> > Despite its disappearance from patchwork it shall be available at
> > mailing list archive.
> > 
> > The v1 can be found at the following link:
> > http://patchwork.ozlabs.org/patch/282753/
> 
> Neither V1 nor V2 have disappeared from patchwork. A "filter" on the
> subject, selecting "any" state and "both" archived and non-archived
> patches will return both:
> 
> V1: http://patchwork.ozlabs.org/patch/282753/ (New)
> V2: http://patchwork.ozlabs.org/patch/314717/ (Deferred)
> 
> (I think that the patchwork web interface does not make it possible at
> all to remove a patch anyway)

Thanks for pointing out. Now it is perfectly visible :-)

> 
> > Inclusion of v2 has been postponed since there was a discussion if
> > we shall allow unaligned access (-mno-unaligned-access flag) at
> > armv7 (after patches posted by Tom).
> > 
> > As fair as I can tell, we will keep the current approach so, I think
> > that Tom will be willing to pull this patch (v2) now.
> 
> Agreed, but then we should make sure no one has comments on V2 that
> they might have withheld due to the initial rejection of V2.

Any comments? 

This patch do fix unaligned access problem on Trats2 (Exynos4412), when
we restore/create GPT, so I would like to know if there are any new
inquires regarding this patch.


> 
> > Best regards,
> > Lukasz Majewski
> 
> Amicalement,


-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group


More information about the U-Boot mailing list