Re: [PATCH] Adding hyperv.h to uapi headers

2014-01-19 Thread Bjarke Istrup Pedersen
2014/1/19 Borislav Petkov : > On Sun, Jan 19, 2014 at 03:37:24PM +0100, Bjarke Istrup Pedersen wrote: >> > Oh, and some build infra so that they can be buildable from within the >> > kernel tree. >> >> Indeed, but first lets get it building :-) > > Oh, I didn't mean you with that, rather the other

Re: [PATCH] Adding hyperv.h to uapi headers

2014-01-19 Thread Borislav Petkov
On Sun, Jan 19, 2014 at 03:37:24PM +0100, Bjarke Istrup Pedersen wrote: > > Oh, and some build infra so that they can be buildable from within the > > kernel tree. > > Indeed, but first lets get it building :-) Oh, I didn't mean you with that, rather the other readers on this thread. Unless you r

Re: [PATCH] Adding hyperv.h to uapi headers

2014-01-19 Thread Bjarke Istrup Pedersen
2014/1/19 Borislav Petkov : > On Sun, Jan 19, 2014 at 12:57:11AM +0100, Bjarke Istrup Pedersen wrote: >> To be able to build the kvp deamon, which has been broken since the >> UAPI split a few versions ago. > > Right. > >> The tool located in tools/hv. > > Ah, I see. > > Those things there could us

Re: [PATCH] Adding hyperv.h to uapi headers

2014-01-19 Thread Borislav Petkov
On Sun, Jan 19, 2014 at 12:57:11AM +0100, Bjarke Istrup Pedersen wrote: > To be able to build the kvp deamon, which has been broken since the > UAPI split a few versions ago. Right. > The tool located in tools/hv. Ah, I see. Those things there could use a Makefile, btw. $ gcc -Wall -o hv_kvp_d

Re: [PATCH] Adding hyperv.h to uapi headers

2014-01-19 Thread Bjarke Istrup Pedersen
; Cc: Haiyang Zhang; Hank Janssen; KY Srinivasan; Bjarke Istrup Pedersen >> Subject: [PATCH] Adding hyperv.h to uapi headers >> >> This patch adds the hyperv.h header to the uapi folder, and adds it to the >> Kbuild >> file. >> Doing this enables compiling use

RE: [PATCH] Adding hyperv.h to uapi headers

2014-01-18 Thread KY Srinivasan
Pedersen > Subject: [PATCH] Adding hyperv.h to uapi headers > > This patch adds the hyperv.h header to the uapi folder, and adds it to the > Kbuild > file. > Doing this enables compiling userspace Hyper-V tools using the installed > headers. > Not all of the contents hyper

Re: [PATCH] Adding hyperv.h to uapi headers

2014-01-18 Thread Bjarke Istrup Pedersen
2014/1/19 Borislav Petkov : > On Sun, Jan 19, 2014 at 12:24:10AM +0100, Bjarke Istrup Pedersen wrote: >> I have submitted a v2 patch - does it look better? > > If you mean this: > > include/linux/hyperv.h | 321 + > include/uapi/linux/Kbuild | 1 + >

Re: [PATCH] Adding hyperv.h to uapi headers

2014-01-18 Thread Borislav Petkov
On Sun, Jan 19, 2014 at 12:24:10AM +0100, Bjarke Istrup Pedersen wrote: > I have submitted a v2 patch - does it look better? If you mean this: include/linux/hyperv.h | 321 + include/uapi/linux/Kbuild | 1 + include/uapi/linux/hyperv.h | 344 +

Re: [PATCH] Adding hyperv.h to uapi headers

2014-01-18 Thread Bjarke Istrup Pedersen
2014/1/18 Borislav Petkov : > On Sat, Jan 18, 2014 at 11:24:53PM +0100, Bjarke Istrup Pedersen wrote: >> I should take all the parts the is not guarded by __KERNEL__, and move >> them to a uapi header, and then include it at the top of the normal >> header. Correct understood? :) > > Yes, that's ba

Re: [PATCH] Adding hyperv.h to uapi headers

2014-01-18 Thread Bjarke Istrup Pedersen
2014/1/18 Borislav Petkov : > On Sat, Jan 18, 2014 at 11:24:53PM +0100, Bjarke Istrup Pedersen wrote: >> I should take all the parts the is not guarded by __KERNEL__, and move >> them to a uapi header, and then include it at the top of the normal >> header. Correct understood? :) > > Yes, that's ba

Re: [PATCH] Adding hyperv.h to uapi headers

2014-01-18 Thread Borislav Petkov
On Sat, Jan 18, 2014 at 11:24:53PM +0100, Bjarke Istrup Pedersen wrote: > I should take all the parts the is not guarded by __KERNEL__, and move > them to a uapi header, and then include it at the top of the normal > header. Correct understood? :) Yes, that's basically the approach but be conserva

Re: [PATCH] Adding hyperv.h to uapi headers

2014-01-18 Thread Bjarke Istrup Pedersen
2014/1/18 Borislav Petkov : > On Sat, Jan 18, 2014 at 11:07:10PM +0100, Bjarke Istrup Pedersen wrote: >> I have been thinking about other solutions, but so far, I haven't been >> able to find one that solves it, so the tools build. (An option might >> be to strip it from the __KERNEL__ part, to mak

Re: [PATCH] Adding hyperv.h to uapi headers

2014-01-18 Thread Borislav Petkov
On Sat, Jan 18, 2014 at 11:07:10PM +0100, Bjarke Istrup Pedersen wrote: > I have been thinking about other solutions, but so far, I haven't been > able to find one that solves it, so the tools build. (An option might > be to strip it from the __KERNEL__ part, to make it smaller, but I > don't know

Re: [PATCH] Adding hyperv.h to uapi headers

2014-01-18 Thread Bjarke Istrup Pedersen
2014/1/18 Paul Bolle > > Bjarke Istrup Pedersen schreef op za 18-01-2014 om 20:48 [+]: > > This patch adds the hyperv.h header to the uapi folder, and adds it to the > > Kbuild file. > > Doing this enables compiling userspace Hyper-V tools using the installed > > headers. > > > > Signed-off-

Re: [PATCH] Adding hyperv.h to uapi headers

2014-01-18 Thread Paul Bolle
Bjarke Istrup Pedersen schreef op za 18-01-2014 om 20:48 [+]: > This patch adds the hyperv.h header to the uapi folder, and adds it to the > Kbuild file. > Doing this enables compiling userspace Hyper-V tools using the installed > headers. > > Signed-off-by: Bjarke Istrup Pedersen > --- >

[PATCH] Adding hyperv.h to uapi headers

2014-01-18 Thread Bjarke Istrup Pedersen
This patch adds the hyperv.h header to the uapi folder, and adds it to the Kbuild file. Doing this enables compiling userspace Hyper-V tools using the installed headers. Signed-off-by: Bjarke Istrup Pedersen --- include/uapi/linux/Kbuild |1 + include/uapi/linux/hyperv.h | 1469 +