Re: [U-Boot] Data Abort with gcc 7.1

2017-07-23 Thread Jörg Krause
Hi, On Thu, 2017-07-13 at 01:43 +0100, Måns Rullgård wrote: > Maxime Ripard writes: > > > Hi, > > > > I recently got a gcc 7.1 based toolchain, and it seems like it > > generates unaligned code, specifically in the net_set_ip_header > > function in my case. > > > > Whenever some packet is sent

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-13 Thread Måns Rullgård
Maxime Ripard writes: > On Thu, Jul 13, 2017 at 11:20:34AM +0100, Peter Robinson wrote: >> >>> What hardware did this happen on? If it was on ARMv5, adding the packed >> >>> attribute is probably the correct fix. If it was ARMv6 or later, >> >>> something else is broken as well. >> >> >> >> It

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-13 Thread Maxime Ripard
On Thu, Jul 13, 2017 at 11:20:34AM +0100, Peter Robinson wrote: > >>> What hardware did this happen on? If it was on ARMv5, adding the packed > >>> attribute is probably the correct fix. If it was ARMv6 or later, > >>> something else is broken as well. > >> > >> It does not matter if this was ARM

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-13 Thread Peter Robinson
On Thu, Jul 13, 2017 at 11:11 AM, Måns Rullgård wrote: > Siarhei Siamashka writes: > >> On Thu, 13 Jul 2017 01:43:37 +0100 >> Måns Rullgård wrote: >> >>> Maxime Ripard writes: >>> >>> > Hi, >>> > >>> > I recently got a gcc 7.1 based toolchain, and it seems like it >>> > generates unaligned code

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-13 Thread Måns Rullgård
Siarhei Siamashka writes: > On Thu, 13 Jul 2017 01:43:37 +0100 > Måns Rullgård wrote: > >> Maxime Ripard writes: >> >> > Hi, >> > >> > I recently got a gcc 7.1 based toolchain, and it seems like it >> > generates unaligned code, specifically in the net_set_ip_header >> > function in my case. >

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-12 Thread Siarhei Siamashka
On Thu, 13 Jul 2017 01:43:37 +0100 Måns Rullgård wrote: > Maxime Ripard writes: > > > Hi, > > > > I recently got a gcc 7.1 based toolchain, and it seems like it > > generates unaligned code, specifically in the net_set_ip_header > > function in my case. > > > > Whenever some packet is sent, thi

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-12 Thread Måns Rullgård
Maxime Ripard writes: > Hi, > > I recently got a gcc 7.1 based toolchain, and it seems like it > generates unaligned code, specifically in the net_set_ip_header > function in my case. > > Whenever some packet is sent, this data abort is triggered: > > => setenv ipaddr 10.42.0.1; ping 10.42.0.254

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-12 Thread Mark Kettenis
> From: "Dr. Philipp Tomsich" > Date: Wed, 12 Jul 2017 20:26:13 +0200 > > > On 12 Jul 2017, at 20:07, Siarhei Siamashka > > wrote: > > > > On Wed, 12 Jul 2017 17:48:16 +0200 > > "Dr. Philipp Tomsich" wrote: > > > >> Tom & Maxime, > >> > >>> On 12 Jul 2017, at 16:34, Tom Rini wrote: > >>>

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-12 Thread Dr. Philipp Tomsich
> On 12 Jul 2017, at 20:07, Siarhei Siamashka > wrote: > > On Wed, 12 Jul 2017 17:48:16 +0200 > "Dr. Philipp Tomsich" wrote: > >> Tom & Maxime, >> >>> On 12 Jul 2017, at 16:34, Tom Rini wrote: >>> >>> On Wed, Jul 12, 2017 at 04:20:52PM +0200, Maxime Ripard wrote: On Tue, Jul 11, 201

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-12 Thread Siarhei Siamashka
On Wed, 12 Jul 2017 17:48:16 +0200 "Dr. Philipp Tomsich" wrote: > Tom & Maxime, > > > On 12 Jul 2017, at 16:34, Tom Rini wrote: > > > > On Wed, Jul 12, 2017 at 04:20:52PM +0200, Maxime Ripard wrote: > >> On Tue, Jul 11, 2017 at 07:59:21PM +0200, Dr. Philipp Tomsich wrote: > >>> Maxime, > >

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-12 Thread Siarhei Siamashka
On Tue, 11 Jul 2017 18:54:55 +0200 Maxime Ripard wrote: > Hi, > > I recently got a gcc 7.1 based toolchain, and it seems like it > generates unaligned code, specifically in the net_set_ip_header > function in my case. > > Whenever some packet is sent, this data abort is triggered: > > => seten

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-12 Thread Dr. Philipp Tomsich
> On 12 Jul 2017, at 19:49, Tom Rini wrote: > > On Wed, Jul 12, 2017 at 05:48:16PM +0200, Dr. Philipp Tomsich wrote: >> Tom & Maxime, >> >>> On 12 Jul 2017, at 16:34, Tom Rini wrote: >>> >>> On Wed, Jul 12, 2017 at 04:20:52PM +0200, Maxime Ripard wrote: On Tue, Jul 11, 2017 at 07:59:21PM

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-12 Thread Tom Rini
On Wed, Jul 12, 2017 at 05:48:16PM +0200, Dr. Philipp Tomsich wrote: > Tom & Maxime, > > > On 12 Jul 2017, at 16:34, Tom Rini wrote: > > > > On Wed, Jul 12, 2017 at 04:20:52PM +0200, Maxime Ripard wrote: > >> On Tue, Jul 11, 2017 at 07:59:21PM +0200, Dr. Philipp Tomsich wrote: > >>> Maxime, > >>

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-12 Thread Tom Rini
On Wed, Jul 12, 2017 at 05:30:14PM +0200, Thomas Petazzoni wrote: > Hello, > > On Wed, 12 Jul 2017 10:34:47 -0400, Tom Rini wrote: > > > Please bear in mind that packed should be used carefully. We've had > > some discussions about this before and have > > doc/README.unaligned-memory-access.txt

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-12 Thread Thomas Petazzoni
Hello, On Wed, 12 Jul 2017 10:34:47 -0400, Tom Rini wrote: > Please bear in mind that packed should be used carefully. We've had > some discussions about this before and have > doc/README.unaligned-memory-access.txt which may need a little more > updating now as well, depending on what the final

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-12 Thread Dr. Philipp Tomsich
Tom & Maxime, > On 12 Jul 2017, at 16:34, Tom Rini wrote: > > On Wed, Jul 12, 2017 at 04:20:52PM +0200, Maxime Ripard wrote: >> On Tue, Jul 11, 2017 at 07:59:21PM +0200, Dr. Philipp Tomsich wrote: >>> Maxime, >>> On 11 Jul 2017, at 18:59, Tom Rini wrote: On Tue, Jul 11, 2017 at

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-12 Thread Dr. Philipp Tomsich
> On 12 Jul 2017, at 16:34, Tom Rini wrote: > > On Wed, Jul 12, 2017 at 04:20:52PM +0200, Maxime Ripard wrote: >> On Tue, Jul 11, 2017 at 07:59:21PM +0200, Dr. Philipp Tomsich wrote: >>> Maxime, >>> On 11 Jul 2017, at 18:59, Tom Rini wrote: On Tue, Jul 11, 2017 at 06:54:55PM +0

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-12 Thread Tom Rini
On Wed, Jul 12, 2017 at 04:20:52PM +0200, Maxime Ripard wrote: > On Tue, Jul 11, 2017 at 07:59:21PM +0200, Dr. Philipp Tomsich wrote: > > Maxime, > > > > > On 11 Jul 2017, at 18:59, Tom Rini wrote: > > > > > > On Tue, Jul 11, 2017 at 06:54:55PM +0200, Maxime Ripard wrote: > > >> Hi, > > >> > >

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-12 Thread Maxime Ripard
On Tue, Jul 11, 2017 at 07:59:21PM +0200, Dr. Philipp Tomsich wrote: > Maxime, > > > On 11 Jul 2017, at 18:59, Tom Rini wrote: > > > > On Tue, Jul 11, 2017 at 06:54:55PM +0200, Maxime Ripard wrote: > >> Hi, > >> > >> I recently got a gcc 7.1 based toolchain, and it seems like it > >> generates

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-11 Thread Dr. Philipp Tomsich
Maxime, > On 11 Jul 2017, at 18:59, Tom Rini wrote: > > On Tue, Jul 11, 2017 at 06:54:55PM +0200, Maxime Ripard wrote: >> Hi, >> >> I recently got a gcc 7.1 based toolchain, and it seems like it >> generates unaligned code, specifically in the net_set_ip_header >> function in my case. >> >> Wh

Re: [U-Boot] Data Abort with gcc 7.1

2017-07-11 Thread Tom Rini
On Tue, Jul 11, 2017 at 06:54:55PM +0200, Maxime Ripard wrote: > Hi, > > I recently got a gcc 7.1 based toolchain, and it seems like it > generates unaligned code, specifically in the net_set_ip_header > function in my case. > > Whenever some packet is sent, this data abort is triggered: > > =>

[U-Boot] Data Abort with gcc 7.1

2017-07-11 Thread Maxime Ripard
Hi, I recently got a gcc 7.1 based toolchain, and it seems like it generates unaligned code, specifically in the net_set_ip_header function in my case. Whenever some packet is sent, this data abort is triggered: => setenv ipaddr 10.42.0.1; ping 10.42.0.254 using musb-hdrc, OUT ep1out IN ep1in ST