Re: [PATCH 1/3] powerpc: Split user/kernel definitions of struct pt_regs

2018-10-16 Thread Michael Ellerman
Madhavan Srinivasan writes: > On Monday 15 October 2018 04:38 PM, Michael Ellerman wrote: >> Madhavan Srinivasan writes: >> >>> On Saturday 13 October 2018 04:26 PM, Michael Ellerman wrote: ... At the moment they're still identical, and we check that at build time. That's because we hav

Re: [PATCH 1/3] powerpc: Split user/kernel definitions of struct pt_regs

2018-10-15 Thread Madhavan Srinivasan
On Monday 15 October 2018 04:38 PM, Michael Ellerman wrote: Madhavan Srinivasan writes: On Saturday 13 October 2018 04:26 PM, Michael Ellerman wrote: We use a shared definition for struct pt_regs in uapi/asm/ptrace.h. That means the layout of the structure is ABI, ie. we can't change it.

Re: [PATCH 1/3] powerpc: Split user/kernel definitions of struct pt_regs

2018-10-15 Thread Michael Ellerman
Nicholas Piggin writes: > On Sat, 13 Oct 2018 21:56:44 +1100 > Michael Ellerman wrote: > >> We use a shared definition for struct pt_regs in uapi/asm/ptrace.h. >> That means the layout of the structure is ABI, ie. we can't change it. >> >> That would be fine if it was only used to describe the u

Re: [PATCH 1/3] powerpc: Split user/kernel definitions of struct pt_regs

2018-10-15 Thread Michael Ellerman
Madhavan Srinivasan writes: > On Saturday 13 October 2018 04:26 PM, Michael Ellerman wrote: >> We use a shared definition for struct pt_regs in uapi/asm/ptrace.h. >> That means the layout of the structure is ABI, ie. we can't change it. >> >> That would be fine if it was only used to describe the

Re: [PATCH 1/3] powerpc: Split user/kernel definitions of struct pt_regs

2018-10-13 Thread Madhavan Srinivasan
On Saturday 13 October 2018 04:26 PM, Michael Ellerman wrote: We use a shared definition for struct pt_regs in uapi/asm/ptrace.h. That means the layout of the structure is ABI, ie. we can't change it. That would be fine if it was only used to describe the user-visible register state of a proc

Re: [PATCH 1/3] powerpc: Split user/kernel definitions of struct pt_regs

2018-10-13 Thread Nicholas Piggin
On Sat, 13 Oct 2018 21:56:44 +1100 Michael Ellerman wrote: > We use a shared definition for struct pt_regs in uapi/asm/ptrace.h. > That means the layout of the structure is ABI, ie. we can't change it. > > That would be fine if it was only used to describe the user-visible > register state of a