FW: eCos for MPC55xx

FW: eCos for MPC55xx

[Gerster Jochen-B01096]

Hi,
 
I have ported eCos to the powerpc mpc55xx family, and now I want to add it to the eCos source repository but how?
 
Some details:
I used the ecos snapshot 25. April 06
 
NEW PACKAGES:
 - CAN device driver (Flexcan) for mpc55xx (supports FlexCanA FlexCanB FlexCanC) uses the CAN_IO
 - serial device driver (ESCI) for mpc55xx
 - eth. device driver (FEC) for mpc5553 (with workaround for ERRATA 2049)
 - eth. device driver (FEC) for mpc5567
 - arch for the powerpc e200z6 core
 - variant the mpc55xx
 - plt. the MPC5554DEMO board
 - plt. the MPC5553DEMO board
 - plt. the MPC5567GW board


Some BUGs + fix:
 cpuload: average is no calculated correctly, so the test worked
 
 net/httpd/.../monitor.c creates a exception because of a null pointer
  line 706     if(getifaddrs(&iflist)!=0) Out of Memory ? yes return
                    return 0;
 
 
New feature:
 kernel/.../intr.cxx: The order the DSRs are called is now changeable between normal or reverse
  normal: the DSRs are called in the order the isr happened (=new)
  reverse: the DSRs are called in the reversed order the isr happened (=old)
 kernel/.../interrupt.cdl: the option CYGIMP_KERNEL_INTERRUPTS_DSRS_LIST_REVERSE was added


ERRORS at compile time + FIX:
 packages\net\snmp\lib\current\include\config.h
  change ~347: define HAVE_GETTIMEOFDAY -> #define HAVE_GETTIMEOFDAY
 
 packages\net\bsd_tcpip\current\include\sys\bsdtypes.h
  Add line ~32: #undef int8_t \ #undef int16_t \ #undef int32_t
 

ERRORS at runtime + FIX:
 gcc optimize memcpy fkt. and uses its inline fkt. and generates alignment error, especially  using the TCP/IP Stack
  change: packages\infra\current\src\memcpy.c  every _memcpy -> my_memcpy
  add line 86: packages\isoinfra\current\include\string.h  #define memcpy my_memcpy pherhaps anyone knows how to disable this by a compiler flag, I didn't found it :(
 
All changed and new files are in the attached zip file. 


bye
Jochen


I send now several emails but I did't get a response nor a new thread in the mailing list was created???
So now I try it without attaching the zip file

Re: FW: eCos for MPC55xx

[Ilija Koco]

Hi Jochen

Nice thing
I recently ported eCos to MAC7100 that has same serial port that i
placed in directory dev/serial/freescale. i tried to meke it platform
independent so maybe you coul'd consolidate it with your's.

I haven't checked CAN but it they are same you coul'd place it under
some general dir so it coul'd seamlessly fit with mac7100 (and possibly
other freescale chips).

regards
Ilija


Gerster Jochen-B01096 wrote:
> Hi,
>  
> I have ported eCos to the powerpc mpc55xx family, and now I want to add it to the eCos source repository but how?
>  
> Some details:
> I used the ecos snapshot 25. April 06
>  
> NEW PACKAGES:
>  - CAN device driver (Flexcan) for mpc55xx (supports FlexCanA FlexCanB FlexCanC) uses the CAN_IO
>  - serial device driver (ESCI) for mpc55xx
>  - eth. device driver (FEC) for mpc5553 (with workaround for ERRATA 2049)
>  - eth. device driver (FEC) for mpc5567
>  - arch for the powerpc e200z6 core
>  - variant the mpc55xx
>  - plt. the MPC5554DEMO board
>  - plt. the MPC5553DEMO board
>  - plt. the MPC5567GW board
>
>
> Some BUGs + fix:
>  cpuload: average is no calculated correctly, so the test worked
>  
>  net/httpd/.../monitor.c creates a exception because of a null pointer
>   line 706     if(getifaddrs(&iflist)!=0) Out of Memory ? yes return
>                     return 0;
>  
>  
> New feature:
>  kernel/.../intr.cxx: The order the DSRs are called is now changeable between normal or reverse
>   normal: the DSRs are called in the order the isr happened (=new)
>   reverse: the DSRs are called in the reversed order the isr happened (=old)
>  kernel/.../interrupt.cdl: the option CYGIMP_KERNEL_INTERRUPTS_DSRS_LIST_REVERSE was added
>
>
> ERRORS at compile time + FIX:
>  packages\net\snmp\lib\current\include\config.h
>   change ~347: define HAVE_GETTIMEOFDAY -> #define HAVE_GETTIMEOFDAY
>  
>  packages\net\bsd_tcpip\current\include\sys\bsdtypes.h
>   Add line ~32: #undef int8_t \ #undef int16_t \ #undef int32_t
>  
>
> ERRORS at runtime + FIX:
>  gcc optimize memcpy fkt. and uses its inline fkt. and generates alignment error, especially  using the TCP/IP Stack
>   change: packages\infra\current\src\memcpy.c  every _memcpy -> my_memcpy
>   add line 86: packages\isoinfra\current\include\string.h  #define memcpy my_memcpy pherhaps anyone knows how to disable this by a compiler flag, I didn't found it :(
>  
> All changed and new files are in the attached zip file. 
>
>
> bye
> Jochen
>
>
> I send now several emails but I did't get a response nor a new thread in the mailing list was created???
> So now I try it without attaching the zip file
>
>   

Re: FW: eCos for MPC55xx

[scratch45]

hi,

its been a while, but im dealing with the same problem right now, do you
still have the software code? i searched everywhere for an open source
bootloder for my MPC5567 and you seem to have one. could please post it here
if you still have it. thanks.
-- 
View this message in context: http://old.nabble.com/FW%3A-eCos-for-MPC55xx-tp5176939p31849548.html
Sent from the Sourceware - ecos-patches mailing list archive at Nabble.com.

RE: FW: eCos for MPC55xx

[Christophe Coutand]

Check this contribution, you might find something for you:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1000761

Christophe

-----Original Message-----
From: ecos-patches-owner@ecos.sourceware.org
[mailto:ecos-patches-owner@ecos.sourceware.org] On Behalf Of scratch45
Sent: 15. juni 2011 10:29
To: ecos-patches@ecos.sourceware.org
Subject: Re: FW: eCos for MPC55xx


hi,

its been a while, but im dealing with the same problem right now, do you
still have the software code? i searched everywhere for an open source
bootloder for my MPC5567 and you seem to have one. could please post it
here
if you still have it. thanks.
-- 
View this message in context:
http://old.nabble.com/FW%3A-eCos-for-MPC55xx-tp5176939p31849548.html
Sent from the Sourceware - ecos-patches mailing list archive at
Nabble.com.

Re: FW: eCos for MPC55xx

[Ilija Koco]

Hi

Recently I sent a proposal for consolidation of MAC7100 and MPC55xx
serial driver(s)
http://sourceware.org/ml/ecos-patches/2006-07/msg00003.html
and got no reply yet.
When I created MAC7100 port I had this option in mind so ESCI serial driver
http://ecos.sourceware.org/cgi-bin/cvsweb.cgi/ecos/packages/devs/serial/freescale/esci/current/?cvsroot=ecos
is prepared for port to other architecture (although of course not
tested with).
Having single source is a benefit to all.

Best regards
Ilija Koco


Gerster Jochen-B01096 wrote:
> Sorry this message never reached the ecos mailing list I send it to myself :Z
> I found another solution without renaming the memcpy to _memcpy ...
>
> -----Original Message-----
> From: Gerster Jochen-B01096 
> Sent: Montag, 10. Juli 2006 11:01
> To: Gerster Jochen-B01096
> Subject: RE: eCos for MPC55xx
>
> Hi
>
> Good news! I think I found a way to handle the gcc memcpy inline alignment problem without changing any file of the source code by using the gcc command -mstrict-align.
> I don't know how I could overlook this ?:) Now I will test some configuration but I'm goodwill this has solved the problem.
>
> I closed the download link.
>
> Thanks for your help!
>
> -----Original Message-----
> From: ecos-patches-owner@ecos.sourceware.org [mailto:ecos-patches-owner@ecos.sourceware.org] On Behalf Of Gerster Jochen-B01096
> Sent: Freitag, 7. Juli 2006 09:56
> To: ecos-patches@ecos.sourceware.org
> Subject: RE: eCos for MPC55xx
>
>
> Here is the output of the gcc -v
> C:\cygwin\opt\ecos\gnutools\powerpc-eabi\bin>powerpc-eabi-gcc.exe -v Reading specs from /opt/ecos/gnutools/powerpc-eabi/bin/../lib/gcc-lib/powerpc-ea
> bi/3.2.1/specs
> Configured with: /local/demonweb/tools/ecos-gnutools-v1.4/r2/powerpc-eabi/cygwin
> /tar_bz2/source/gcc-3.2.1/configure --target=powerpc-eabi --prefix=/local/demonw eb/tools/ecos-gnutools-v1.4/r2/powerpc-eabi/cygwin/tar_bz2/opt/ecos/gnutools/pow
> erpc-eabi --enable-languages=c,c++ --with-gnu-as --with-gnu-ld --with-newlib --w ith-gxx-include-dir=/local/demonweb/tools/ecos-gnutools-v1.4/r2/powerpc-eabi/cyg
> win/tar_bz2/opt/ecos/gnutools/powerpc-eabi/powerpc-eabi/include
> Thread model: single
> gcc version 3.2.1
>
> I have download the gnutools from the ecos site
>
> One different is that I use the flag -msoft-float so gcc uses other libs ....
>
> Attached is a picture showing the debugger at the place doing an unaligned access.
> You can see that there is no call to the memcpy function from ecos!
>
> Its in the if_ether.c file - function is in_arpinput
>
> 0x3fc6c0ba is the address where the src ip-address is found and this address is not aligned to 4byte so no word(32bit) access is allowed
>
> With my patch we simple tricks the compiler, by only changing the function label to _memcpy and so the compiler don't uses its inline memcpy function
>
> Zzz
> Ok forget it! NO picture attached, it seems this mailing list doesn't like anything usefull!?!?
>
> So look here
> http://service.gmx.net/mc/MyrAaChazs9GWwfVR7FL3nHTjEC9J4
> Here you can download the screenshot of the debug window showing one place of the memcpy problem The source code for the new port including every new file and all changed file
>
>
> I added a new package to include the GoAhead WebServer source to ecos.
> The source isn't included so you have to download it from their webpage Look at the readme in the services/goahead/.../src directory
>
>
>
>
> -----Original Message-----
> From: Gary Thomas [mailto:gary@mlbassoc.com]
> Sent: Donnerstag, 6. Juli 2006 14:37
> To: Gerster Jochen-B01096
> Cc: ecos-patches@ecos.sourceware.org
> Subject: Re: eCos for MPC55xx
>
> Gerster Jochen-B01096 wrote:
>   
>> As I said before, the IP-addresses(src/dst) has no fixed offset in the several protocol-headers!
>> That means the IP-addresses in the ARP-Header(14) have a different offset as in the IP-Header(12) and so I can't belive the TCP/IP stack regards this.
>>
>> The exception is generated by the TCP/IP stack handling a ARP message! So this is ecos code!
>>
>> Doing my patch(as described before) everything works fine!
>>
>> And my question was?
>> Anybody else have/has the same problem using a powerpc?
>>     
>
> No, I've not seen this (and I've been running the network stack on eCos for *many* years now)
>
> Exactly what version of GCC are you using?  How was it configured?
>
> Can you point at exactly the source line that you are referring to and the assembly code that gets generated?
>
>   
>> And their solutions.
>>
>>
>> Jochen
>>
>> -----Original Message-----
>> From: Andrew Lunn [mailto:andrew@lunn.ch]
>> Sent: Donnerstag, 6. Juli 2006 12:16
>> To: Gerster Jochen-B01096
>> Cc: Andrew Lunn
>> Subject: Re: eCos for MPC55xx
>>
>> On Thu, Jul 06, 2006 at 09:56:07AM +0200, Gerster Jochen-B01096 wrote:
>>     
>>> Hi Andrew
>>>
>>> Ok so lets start with the memcpy problem, while I take care of the Copyright assignment.
>>> Perhaps a new thread?
>>>
>>> The problem is:
>>> The POWERPC gcc compiler (allocated from the ecos site)  optimize 
>>> memcpy calls with small length, instead it uses its own inline 
>>> function without taking care of aligned access. This is a very very 
>>> big problem using the TCP/IP stack because the IP-address is 4byte so 
>>> the gcc optimize and the offset of the IP-address in the different 
>>> protocol-headers varies. This ends in a exception :(
>>>       
>> The network stack is so setup so that IP addresses in packets are always aligned. So there should not be a problem. Is this user code, or eCos code which is causing problems. If you use char *, not int *, memcpy will be careful and not assume word alignment.
>>
>>        Andrew
>>     
>
>
> --
> ------------------------------------------------------------
> Gary Thomas                 |  Consulting for the
> MLB Associates              |    Embedded world
> ------------------------------------------------------------
>
>   

FW: eCos for MPC55xx

[Gerster Jochen-B01096]

Sorry this message never reached the ecos mailing list I send it to myself :Z
I found another solution without renaming the memcpy to _memcpy ...

-----Original Message-----
From: Gerster Jochen-B01096 
Sent: Montag, 10. Juli 2006 11:01
To: Gerster Jochen-B01096
Subject: RE: eCos for MPC55xx

Hi

Good news! I think I found a way to handle the gcc memcpy inline alignment problem without changing any file of the source code by using the gcc command -mstrict-align.
I don't know how I could overlook this ?:) Now I will test some configuration but I'm goodwill this has solved the problem.

I closed the download link.

Thanks for your help!

-----Original Message-----
From: ecos-patches-owner@ecos.sourceware.org [mailto:ecos-patches-owner@ecos.sourceware.org] On Behalf Of Gerster Jochen-B01096
Sent: Freitag, 7. Juli 2006 09:56
To: ecos-patches@ecos.sourceware.org
Subject: RE: eCos for MPC55xx


Here is the output of the gcc -v
C:\cygwin\opt\ecos\gnutools\powerpc-eabi\bin>powerpc-eabi-gcc.exe -v Reading specs from /opt/ecos/gnutools/powerpc-eabi/bin/../lib/gcc-lib/powerpc-ea
bi/3.2.1/specs
Configured with: /local/demonweb/tools/ecos-gnutools-v1.4/r2/powerpc-eabi/cygwin
/tar_bz2/source/gcc-3.2.1/configure --target=powerpc-eabi --prefix=/local/demonw eb/tools/ecos-gnutools-v1.4/r2/powerpc-eabi/cygwin/tar_bz2/opt/ecos/gnutools/pow
erpc-eabi --enable-languages=c,c++ --with-gnu-as --with-gnu-ld --with-newlib --w ith-gxx-include-dir=/local/demonweb/tools/ecos-gnutools-v1.4/r2/powerpc-eabi/cyg
win/tar_bz2/opt/ecos/gnutools/powerpc-eabi/powerpc-eabi/include
Thread model: single
gcc version 3.2.1

I have download the gnutools from the ecos site

One different is that I use the flag -msoft-float so gcc uses other libs ....

Attached is a picture showing the debugger at the place doing an unaligned access.
You can see that there is no call to the memcpy function from ecos!

Its in the if_ether.c file - function is in_arpinput

0x3fc6c0ba is the address where the src ip-address is found and this address is not aligned to 4byte so no word(32bit) access is allowed

With my patch we simple tricks the compiler, by only changing the function label to _memcpy and so the compiler don't uses its inline memcpy function

Zzz
Ok forget it! NO picture attached, it seems this mailing list doesn't like anything usefull!?!?

So look here
http://service.gmx.net/mc/MyrAaChazs9GWwfVR7FL3nHTjEC9J4
Here you can download the screenshot of the debug window showing one place of the memcpy problem The source code for the new port including every new file and all changed file


I added a new package to include the GoAhead WebServer source to ecos.
The source isn't included so you have to download it from their webpage Look at the readme in the services/goahead/.../src directory




-----Original Message-----
From: Gary Thomas [mailto:gary@mlbassoc.com]
Sent: Donnerstag, 6. Juli 2006 14:37
To: Gerster Jochen-B01096
Cc: ecos-patches@ecos.sourceware.org
Subject: Re: eCos for MPC55xx

Gerster Jochen-B01096 wrote:
> As I said before, the IP-addresses(src/dst) has no fixed offset in the several protocol-headers!
> That means the IP-addresses in the ARP-Header(14) have a different offset as in the IP-Header(12) and so I can't belive the TCP/IP stack regards this.
> 
> The exception is generated by the TCP/IP stack handling a ARP message! So this is ecos code!
> 
> Doing my patch(as described before) everything works fine!
> 
> And my question was?
> Anybody else have/has the same problem using a powerpc?

No, I've not seen this (and I've been running the network stack on eCos for *many* years now)

Exactly what version of GCC are you using?  How was it configured?

Can you point at exactly the source line that you are referring to and the assembly code that gets generated?

> And their solutions.
> 
> 
> Jochen
> 
> -----Original Message-----
> From: Andrew Lunn [mailto:andrew@lunn.ch]
> Sent: Donnerstag, 6. Juli 2006 12:16
> To: Gerster Jochen-B01096
> Cc: Andrew Lunn
> Subject: Re: eCos for MPC55xx
> 
> On Thu, Jul 06, 2006 at 09:56:07AM +0200, Gerster Jochen-B01096 wrote:
>> Hi Andrew
>>
>> Ok so lets start with the memcpy problem, while I take care of the Copyright assignment.
>> Perhaps a new thread?
>>
>> The problem is:
>> The POWERPC gcc compiler (allocated from the ecos site)  optimize 
>> memcpy calls with small length, instead it uses its own inline 
>> function without taking care of aligned access. This is a very very 
>> big problem using the TCP/IP stack because the IP-address is 4byte so 
>> the gcc optimize and the offset of the IP-address in the different 
>> protocol-headers varies. This ends in a exception :(
> 
> The network stack is so setup so that IP addresses in packets are always aligned. So there should not be a problem. Is this user code, or eCos code which is causing problems. If you use char *, not int *, memcpy will be careful and not assume word alignment.
> 
>        Andrew


--
------------------------------------------------------------
Gary Thomas                 |  Consulting for the
MLB Associates              |    Embedded world
------------------------------------------------------------

Re: FW: eCos for MPC55xx

[Jonathan Larmour]

Gerster Jochen-B01096 wrote:
> 
> The problem is: The POWERPC gcc compiler (allocated from the ecos site)
> optimize memcpy calls with small length, instead it uses its own inline
> function without taking care of aligned access.

If it did not work for unaligned accesses on PowerPC in general, then this 
would have been discovered years ago.

Perhaps have the semantics of an opcode in the MPC55xx changed so that 
something that used to work for all PowerPC now only works on aligned 
addresses? If so that wouldn't be good, but it's not beyond the realms of 
possibility.

> This is a very very big
> problem using the TCP/IP stack because the IP-address is 4byte so the
> gcc optimize and the offset of the IP-address in the different
> protocol-headers varies. This ends in a exception :(
> 
> So I asking me now: I'm the only one with this problem? How do you solve
> this problem, perhaps a compiler flag?

Theoretically -fno-builtin can be used to disable GCC's optimisations if 
they are genuinely at fault. But I'm not sure. Have you tried a small 
testcase? And if so have you looked at the assembler GCC is producing?

Jifl
-- 
--["No sense being pessimistic, it wouldn't work anyway"]-- Opinions==mine

FW: eCos for MPC55xx

[Gerster Jochen-B01096]

Hi Andrew

Ok so lets start with the memcpy problem, while I take care of the Copyright assignment.
Perhaps a new thread?

The problem is:
The POWERPC gcc compiler (allocated from the ecos site)  optimize memcpy calls with small length, instead it uses its own inline function without taking care of aligned access. This is a very very big problem using the TCP/IP stack because the IP-address is 4byte so the gcc optimize and the offset of the IP-address in the different protocol-headers varies. This ends in a exception :(

So I asking me now:
 I'm the only one with this problem?
 How do you solve this problem, perhaps a compiler flag?

I solved this by renaming the ecos memcpy function to my_memcpy and define a macro in the string.h which link every memcpy to my_memcpy (#define memcpy my_memcpy). So the gcc don't touch this new function and .... No exceptions


Jochen

-----Original Message-----
From: Andrew Lunn [mailto:andrew@lunn.ch]
Sent: Mittwoch, 5. Juli 2006 18:42
To: Gerster Jochen-B01096
Subject: Re: eCos for MPC55xx

On Wed, Jul 05, 2006 at 09:12:07AM +0200, Gerster Jochen-B01096 wrote:
> 
> I can't belive it!! It works!?! :)
> 
> So any idea how to public the source files??
> The mailing list seems to dislike the attached zip file?

Hi Jochen

As suggested send it as a number of packages. When presented with a big patch i tend to put it to the bottom of the pile because i know it will need a lot of effort. Lots of small, independent patches get handled quicker.

We will need a copyright assignment. Please take a look on the eCos website for the document you need to find.

Lastly, im on holiday until the beginning of August, so i don't expect i will be able to take a look at this until then. Maybe one of the other maintainers will handle it before then.

      Andrew