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

Albert ARIBAUD albert.u.boot at aribaud.net
Wed Feb 19 11:08:03 CET 2014


Hi Lukasz,

On Wed, 19 Feb 2014 09:19:04 +0100, Lukasz Majewski
<l.majewski at samsung.com> wrote:

> 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.

Does not seem to be, so I will apply V2.

Amicalement,
-- 
Albert.


More information about the U-Boot mailing list