Re: [PATCH 01/50] x86/boot/e820: Introduce arch/x86/include/asm/e820/types.h

2017-02-01 Thread Ingo Molnar
* Sam Ravnborg wrote: > > Firsty, the headers are not maintained by the user-space project, 99.999% > > of > > the maintenance is done by the kernel developers. > > In the inital mail triggering this plan was that the kernel is moving away > from > having uapi headers what-so-ever. No, tha

Re: [PATCH 01/50] x86/boot/e820: Introduce arch/x86/include/asm/e820/types.h

2017-01-31 Thread Sam Ravnborg
Hi Ingo. > > > > In this thread we have covered one rational reason to push thus > > burden to user space - to give the kernel the freedom to repair > > past stupidity (being that in naming or some other sort). > > There's no real "burden" for heaven's sake: it's having to execute a 'cp' > every

Re: [PATCH 01/50] x86/boot/e820: Introduce arch/x86/include/asm/e820/types.h

2017-01-31 Thread Joe Perches
On Tue, 2017-01-31 at 19:00 +0100, Ingo Molnar wrote: > * Sam Ravnborg wrote: > > In this thread we have covered one rational reason to push thus > > burden to user space - to give the kernel the freedom to repair > > past stupidity (being that in naming or some other sort). > > There's no real "

Re: [PATCH 01/50] x86/boot/e820: Introduce arch/x86/include/asm/e820/types.h

2017-01-31 Thread Ingo Molnar
* Sam Ravnborg wrote: > > > > > > So just to repeat - it is an error prone design to let users > > > of the kernel uapi maintain their own copies of the kernel > > > uapi header. It is the job of the kernel. > > > > But "random copies" is not what perf does. Tell me, how is the perf > > mecha

Re: [PATCH 01/50] x86/boot/e820: Introduce arch/x86/include/asm/e820/types.h

2017-01-31 Thread Sam Ravnborg
> > > > So just to repeat - it is an error prone design to let users > > of the kernel uapi maintain their own copies of the kernel > > uapi header. It is the job of the kernel. > > But "random copies" is not what perf does. Tell me, how is the perf mechanism > of > using the headers "error-pro

Re: [PATCH 01/50] x86/boot/e820: Introduce arch/x86/include/asm/e820/types.h

2017-01-31 Thread Ingo Molnar
* Sam Ravnborg wrote: > Obvious when there are two vastly different solutions there are some > middle ground solutions too. > > And perf implements a tool based method to check that the manually > copied/created/maintained headers are in sync with the kernel. which is just > another way to mai

Re: [PATCH 01/50] x86/boot/e820: Introduce arch/x86/include/asm/e820/types.h

2017-01-30 Thread Sam Ravnborg
On Mon, Jan 30, 2017 at 08:58:33AM +0100, Ingo Molnar wrote: > > * Sam Ravnborg wrote: > > > On Sat, Jan 28, 2017 at 11:11:22PM +0100, Ingo Molnar wrote: > > > > > > The plan is to keep the old UAPI header in place but the kernel won't > > > use it anymore - and after some time we'll try to rem

Re: [PATCH 01/50] x86/boot/e820: Introduce arch/x86/include/asm/e820/types.h

2017-01-29 Thread Ingo Molnar
* Sam Ravnborg wrote: > On Sat, Jan 28, 2017 at 11:11:22PM +0100, Ingo Molnar wrote: > > > > The plan is to keep the old UAPI header in place but the kernel won't > > use it anymore - and after some time we'll try to remove it. (User-space > > tools better have local copies of headers anyway, i

Re: [PATCH 01/50] x86/boot/e820: Introduce arch/x86/include/asm/e820/types.h

2017-01-29 Thread Sam Ravnborg
On Sat, Jan 28, 2017 at 11:11:22PM +0100, Ingo Molnar wrote: > > The plan is to keep the old UAPI header in place but the kernel won't > use it anymore - and after some time we'll try to remove it. (User-space > tools better have local copies of headers anyway, instead of relying > on kernel heade