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

2023-08-13 Thread Chris Johns
On 11/8/2023 1:17 pm, Vijay Kumar Banerjee wrote: > On Wed, Aug 9, 2023 at 8:04 PM Chris Johns wrote: >> >> On 10/8/2023 2:41 am, Joel Sherrill wrote: >>> Reading the EPICS discussion, I wonder if this should be added for all >>> motorola_powerpc BSP variants. >>> >>> Gedare/Chris/anyone with anot

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

2023-08-10 Thread Vijay Kumar Banerjee
On Wed, Aug 9, 2023 at 8:04 PM Chris Johns wrote: > > On 10/8/2023 2:41 am, Joel Sherrill wrote: > > Reading the EPICS discussion, I wonder if this should be added for all > > motorola_powerpc BSP variants. > > > > Gedare/Chris/anyone with another board in the family? What do you think? > > Michae

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

2023-08-09 Thread Chris Johns
On 10/8/2023 2:41 am, Joel Sherrill wrote: > Reading the EPICS discussion, I wonder if this should be added for all > motorola_powerpc BSP variants. > > Gedare/Chris/anyone with another board in the family? What do you think? Michael asked if the setting is present when building libc, libm etc in

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

2023-08-09 Thread Uchenna Ezeobi
On Wed, Aug 9, 2023 at 3:31 PM Vijay Kumar Banerjee wrote: > > On Wed, Aug 9, 2023 at 3:26 PM Vijay Kumar Banerjee wrote: > > > > On Wed, Aug 9, 2023 at 3:05 PM Joel Sherrill wrote: > > > > > > Vijay.. go ahead and push this if you don't mind. > > > > > Sure, I will push this. > > Pushed. Thanks

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

2023-08-09 Thread Vijay Kumar Banerjee
On Wed, Aug 9, 2023 at 3:26 PM Vijay Kumar Banerjee wrote: > > On Wed, Aug 9, 2023 at 3:05 PM Joel Sherrill 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 > >

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

2023-08-09 Thread Vijay Kumar Banerjee
On Wed, Aug 9, 2023 at 3:05 PM Joel Sherrill wrote: > > Vijay.. go ahead and push this if you don't mind. > Sure, I will push this. > 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 err

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

2023-08-09 Thread Joel Sherrill
Vijay.. go ahead and push this if you don't mind. I was just wondering if we could make a blanket statement that all motorola_powerpc variants need this flag. On Wed, Aug 9, 2023 at 12:01 PM Vijay Kumar Banerjee wrote: > On Wed, Aug 9, 2023 at 11:41 AM Joel Sherrill wrote: > > > > Reading the

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

2023-08-09 Thread Vijay Kumar Banerjee
On Wed, Aug 9, 2023 at 11:41 AM Joel Sherrill 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

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

2023-08-09 Thread Joel Sherrill
Reading the EPICS discussion, I wonder if this should be added for all motorola_powerpc BSP variants. Gedare/Chris/anyone with another board in the family? What do you think? --joel On Wed, Aug 9, 2023 at 11:14 AM Uchenna Ezeobi wrote: > On Wed, Aug 9, 2023 at 9:22 AM Joel Sherrill wrote: > >

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

2023-08-09 Thread Uchenna Ezeobi
On Wed, Aug 9, 2023 at 9:22 AM Joel Sherrill 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

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

2023-08-09 Thread Joel Sherrill
I'm ok with this if there is a known case of an alignment exception that causes this. 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 wrote: > Update #3767 > --- > spec/build/bsps/

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

2023-08-09 Thread Uchenna Ezeobi
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_