Re: [PATCH 3/5] ARM: trusted_foundations: do not use naked function

2018-03-22 Thread Dmitry Osipenko
On 22.03.2018 15:43, Stefan Agner wrote: > On 22.03.2018 12:48, Robin Murphy wrote: >> On 21/03/18 21:41, Stefan Agner wrote: >>> On 21.03.2018 18:16, Robin Murphy wrote: On 21/03/18 16:40, Stephen Warren wrote: > On 03/21/2018 09:26 AM, Dmitry Osipenko wrote: >> On 21.03.2018 17:09, S

Re: [PATCH 3/5] ARM: trusted_foundations: do not use naked function

2018-03-22 Thread Stefan Agner
On 22.03.2018 12:48, Robin Murphy wrote: > On 21/03/18 21:41, Stefan Agner wrote: >> On 21.03.2018 18:16, Robin Murphy wrote: >>> On 21/03/18 16:40, Stephen Warren wrote: On 03/21/2018 09:26 AM, Dmitry Osipenko wrote: > On 21.03.2018 17:09, Stefan Agner wrote: >> On 21.03.2018 13:13, R

Re: [PATCH 3/5] ARM: trusted_foundations: do not use naked function

2018-03-22 Thread Robin Murphy
On 21/03/18 21:41, Stefan Agner wrote: On 21.03.2018 18:16, Robin Murphy wrote: On 21/03/18 16:40, Stephen Warren wrote: On 03/21/2018 09:26 AM, Dmitry Osipenko wrote: On 21.03.2018 17:09, Stefan Agner wrote: On 21.03.2018 13:13, Robin Murphy wrote: On 20/03/18 23:02, Stefan Agner wrote: As

Re: [PATCH 3/5] ARM: trusted_foundations: do not use naked function

2018-03-21 Thread Stefan Agner
On 21.03.2018 18:16, Robin Murphy wrote: > On 21/03/18 16:40, Stephen Warren wrote: >> On 03/21/2018 09:26 AM, Dmitry Osipenko wrote: >>> On 21.03.2018 17:09, Stefan Agner wrote: On 21.03.2018 13:13, Robin Murphy wrote: > On 20/03/18 23:02, Stefan Agner wrote: >> As documented in GCC n

Re: [PATCH 3/5] ARM: trusted_foundations: do not use naked function

2018-03-21 Thread Robin Murphy
On 21/03/18 16:40, Stephen Warren wrote: On 03/21/2018 09:26 AM, Dmitry Osipenko wrote: On 21.03.2018 17:09, Stefan Agner wrote: On 21.03.2018 13:13, Robin Murphy wrote: On 20/03/18 23:02, Stefan Agner wrote: As documented in GCC naked functions should only use Basic asm syntax. The Extended

Re: [PATCH 3/5] ARM: trusted_foundations: do not use naked function

2018-03-21 Thread Stephen Warren
On 03/21/2018 09:26 AM, Dmitry Osipenko wrote: On 21.03.2018 17:09, Stefan Agner wrote: On 21.03.2018 13:13, Robin Murphy wrote: On 20/03/18 23:02, Stefan Agner wrote: As documented in GCC naked functions should only use Basic asm syntax. The Extended asm or mixture of Basic asm and "C" code i

Re: [PATCH 3/5] ARM: trusted_foundations: do not use naked function

2018-03-21 Thread Dmitry Osipenko
On 21.03.2018 17:09, Stefan Agner wrote: > On 21.03.2018 13:13, Robin Murphy wrote: >> On 20/03/18 23:02, Stefan Agner wrote: >>> As documented in GCC naked functions should only use Basic asm >>> syntax. The Extended asm or mixture of Basic asm and "C" code is >>> not guaranteed. Currently this wo

Re: [PATCH 3/5] ARM: trusted_foundations: do not use naked function

2018-03-21 Thread Stefan Agner
On 21.03.2018 13:13, Robin Murphy wrote: > On 20/03/18 23:02, Stefan Agner wrote: >> As documented in GCC naked functions should only use Basic asm >> syntax. The Extended asm or mixture of Basic asm and "C" code is >> not guaranteed. Currently this works because it was hard coded >> to follow and

Re: [PATCH 3/5] ARM: trusted_foundations: do not use naked function

2018-03-21 Thread Robin Murphy
On 20/03/18 23:02, Stefan Agner wrote: As documented in GCC naked functions should only use Basic asm syntax. The Extended asm or mixture of Basic asm and "C" code is not guaranteed. Currently this works because it was hard coded to follow and check GCC behavior for arguments and register placeme

Re: [PATCH 3/5] ARM: trusted_foundations: do not use naked function

2018-03-21 Thread Stefan Agner
On 21.03.2018 00:13, Russell King - ARM Linux wrote: > On Wed, Mar 21, 2018 at 12:02:04AM +0100, Stefan Agner wrote: >> As documented in GCC naked functions should only use Basic asm >> syntax. The Extended asm or mixture of Basic asm and "C" code is >> not guaranteed. Currently this works because

Re: [PATCH 3/5] ARM: trusted_foundations: do not use naked function

2018-03-20 Thread Russell King - ARM Linux
On Wed, Mar 21, 2018 at 12:02:04AM +0100, Stefan Agner wrote: > As documented in GCC naked functions should only use Basic asm > syntax. The Extended asm or mixture of Basic asm and "C" code is > not guaranteed. Currently this works because it was hard coded > to follow and check GCC behavior for a