On Wed, Dec 16, 2015 at 12:16:01PM +0000, Mark Rutland wrote:
> On Wed, Dec 16, 2015 at 12:24:30PM +0100, Ard Biesheuvel wrote:
> > On 16 December 2015 at 12:18, Mark Rutland <[email protected]> wrote:
> > > On Wed, Dec 16, 2015 at 10:37:39AM +0100, Ard Biesheuvel wrote:
> > >> diff --git a/ArmPkg/Include/Chipset/AArch64.h 
> > >> b/ArmPkg/Include/Chipset/AArch64.h
> > >> index 5e1653bcb69d..7be18546799c 100644
> > >> --- a/ArmPkg/Include/Chipset/AArch64.h
> > >> +++ b/ArmPkg/Include/Chipset/AArch64.h
> > >> @@ -119,15 +119,18 @@
> > >>  #define ARM_VECTOR_LOW_A32_SERR 0x780
> > >>
> > >>  #define VECTOR_BASE(tbl)          \
> > >> +  .section .text.##tbl##,"aw";    \
> > >
> > > Shouldn't this have "x", too?
> > >
> > > Do we overwrite this? If not, it could just be "ax" rather than "awx".
> > >
> > 
> > Ah yes, I meant "ax" not "aw"
> 
> Ok. With that:
> 
> Acked-by: Mark Rutland <[email protected]>

If Mark's happy:
Reviewed-by: Leif Lindholm <[email protected]>

/
    Leif
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to