[PATCH RTEMS] spec: Add -mstrict-align to mvme2100 default build

[Uchenna Ezeobi]

On Wed, Aug 9, 2023 at 3:31 PM Vijay Kumar Banerjee <vijay at rtems.org> wrote:
>
> On Wed, Aug 9, 2023 at 3:26 PM Vijay Kumar Banerjee <vijay at rtems.org> wrote:
> >
> > On Wed, Aug 9, 2023 at 3:05 PM Joel Sherrill <joel at rtems.org> wrote:
> > >
> > > Vijay.. go ahead and push this if you don't mind.
> > >
> > Sure, I will push this.
>
> Pushed. Thanks.
Great, thank you.

> >
> > > I was just wondering if we could make a blanket statement that all motorola_powerpc variants need this flag.
> >
> > I do not have a test case other than the EPICS error. At least 2 BSPs
> > (2100, 2700/2307) in motorola_powerpc need this, and except mcp750 all
> > other BSPs in motorola_powerpc seem to have it in the spec. It is
> > likely that all motorola_powerpc variants need this.
> >
> > >
> > > On Wed, Aug 9, 2023 at 12:01 PM Vijay Kumar Banerjee <vijay at rtems.org> wrote:
> > >>
> > >> On Wed, Aug 9, 2023 at 11:41 AM Joel Sherrill <joel at rtems.org> wrote:
> > >> >
> > >> > Reading the EPICS discussion, I wonder if this should be added for all motorola_powerpc BSP variants.
> > >> >
> > >> MVME2700 and the qemu variants already add it separately
> > >> https://git.rtems.org/rtems/tree/spec/build/bsps/powerpc/motorola_powerpc/abi.yml#n24
> > >>
> > >> > Gedare/Chris/anyone with another board in the family? What do you think?
> > >>
> > >> 2700 works for me, and it already has the flag added.
> > >>
> > >> >
> > >> > --joel
> > >> >
> > >> > On Wed, Aug 9, 2023 at 11:14 AM Uchenna Ezeobi <uchenna.ezeobi3 at gmail.com> wrote:
> > >> >>
> > >> >> On Wed, Aug 9, 2023 at 9:22 AM Joel Sherrill <joel at rtems.org> wrote:
> > >> >> >
> > >> >> > I'm ok with this if there is a known case of an alignment exception that causes this.
> > >> >> >
> > >> >>  We were getting the error from the
> > >> >> https://github.com/epics-base/epics-base/issues/29 while trying to
> > >> >> build EPICS on RTEMS-MVME2100, doing this fixed the error.
> > >> >>
> > >> >> > We didn't want to force it into every PowerPC BSP.
> > >> >> >
> > >> >> > Do you have a case that failed and was resolved by adding this?
> > >> >> >
> > >> >> > On Wed, Aug 9, 2023 at 7:36 AM Uchenna Ezeobi <uchenna.ezeobi3 at gmail.com> wrote:
> > >> >> >>
> > >> >> >> Update #3767
> > >> >> >> ---
> > >> >> >>  spec/build/bsps/powerpc/motorola_powerpc/abi.yml | 4 ++++
> > >> >> >>  1 file changed, 4 insertions(+)
> > >> >> >>
> > >> >> >> diff --git a/spec/build/bsps/powerpc/motorola_powerpc/abi.yml b/spec/build/bsps/powerpc/motorola_powerpc/abi.yml
> > >> >> >> index 6734e9babf..2438c30f1d 100644
> > >> >> >> --- a/spec/build/bsps/powerpc/motorola_powerpc/abi.yml
> > >> >> >> +++ b/spec/build/bsps/powerpc/motorola_powerpc/abi.yml
> > >> >> >> @@ -17,6 +17,10 @@ default:
> > >> >> >>    - -mcpu=powerpc
> > >> >> >>    - -mmultiple
> > >> >> >>    - -mstrict-align
> > >> >> >> +- enabled-by: [powerpc/mvme2100]
> > >> >> >> +  value:
> > >> >> >> +  - -mcpu=603e
> > >> >> >> +  - -mstrict-align
> > >> >> >>  - enabled-by: [powerpc/mvme2307, powerpc/mvme2700]
> > >> >> >>    value:
> > >> >> >>    - -mcpu=604
> > >> >> >> --
> > >> >> >> 2.39.3
> > >> >> >>
> > >> >> >> _______________________________________________
> > >> >> >> devel mailing list
> > >> >> >> devel at rtems.org
> > >> >> >> http://lists.rtems.org/mailman/listinfo/devel
> > >> >
> > >> > _______________________________________________
> > >> > devel mailing list
> > >> > devel at rtems.org
> > >> > http://lists.rtems.org/mailman/listinfo/devel