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

[Vijay Kumar Banerjee]

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