Re: [PATCH 0/5] s390: rework compat wrapper generation

2019-01-18 Thread Martin Schwidefsky
On Thu, 17 Jan 2019 21:19:20 +0100 Heiko Carstens wrote: > On Thu, Jan 17, 2019 at 05:21:50PM +0100, Arnd Bergmann wrote: > > On Thu, Jan 17, 2019 at 2:36 PM Heiko Carstens > > wrote: > > > > > > On Wed, Jan 16, 2019 at 02:15:18PM +0100, Arnd Bergmann wrote: > > > > > > I did not test

Re: [PATCH 0/5] s390: rework compat wrapper generation

2019-01-17 Thread Heiko Carstens
On Thu, Jan 17, 2019 at 05:21:50PM +0100, Arnd Bergmann wrote: > On Thu, Jan 17, 2019 at 2:36 PM Heiko Carstens > wrote: > > > > On Wed, Jan 16, 2019 at 02:15:18PM +0100, Arnd Bergmann wrote: > > > > I did not test the changes at runtime, but I looked at the > > > generated object code, which

Re: [PATCH 0/5] s390: rework compat wrapper generation

2019-01-17 Thread Arnd Bergmann
On Thu, Jan 17, 2019 at 2:36 PM Heiko Carstens wrote: > > On Wed, Jan 16, 2019 at 02:15:18PM +0100, Arnd Bergmann wrote: > > I did not test the changes at runtime, but I looked at the > > generated object code, which seems fine here and includes > > the same conversions as before. > > All looks

Re: [PATCH 0/5] s390: rework compat wrapper generation

2019-01-17 Thread Heiko Carstens
On Wed, Jan 16, 2019 at 02:15:18PM +0100, Arnd Bergmann wrote: > Hi Heiko and Martin, > > As promised, I gave this a go and changed the SYSCALL_DEFINEx() > infrastructure to always include the wrappers for doing the > 31-bit argument conversion on s390 compat mode. > > This does three main

Re: [PATCH 0/5] s390: rework compat wrapper generation

2019-01-16 Thread Heiko Carstens
On Wed, Jan 16, 2019 at 02:15:18PM +0100, Arnd Bergmann wrote: > Hi Heiko and Martin, > > As promised, I gave this a go and changed the SYSCALL_DEFINEx() > infrastructure to always include the wrappers for doing the > 31-bit argument conversion on s390 compat mode. > > This does three main

[PATCH 0/5] s390: rework compat wrapper generation

2019-01-16 Thread Arnd Bergmann
Hi Heiko and Martin, As promised, I gave this a go and changed the SYSCALL_DEFINEx() infrastructure to always include the wrappers for doing the 31-bit argument conversion on s390 compat mode. This does three main things: - The UID16 rework saved a lot of duplicated code, and would probably