Re: [PATCH] Adding makefile for tools/hv

2014-01-20 Thread Bjarke Istrup Pedersen
Adding Greg to CC list. /Bjarke 2014/1/19 Bjarke Istrup Pedersen : > Currently, there is no makefile for the Hyper-V tools. > This patch adds the missing makefile, and adds it to the main tools makefile. > > Signed-off-by: Bjarke Istrup Pedersen > --- > tool

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

2014-01-20 Thread Bjarke Istrup Pedersen
2014/1/20 KY Srinivasan : > > >> -Original Message- >> From: gurlige...@gmail.com [mailto:gurlige...@gmail.com] On Behalf Of Bjarke >> Istrup Pedersen >> Sent: Sunday, January 19, 2014 11:46 AM >> To: KY Srinivasan >> Cc: Linux Kernel Mailing Lis

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

2014-01-20 Thread Bjarke Istrup Pedersen
2014/1/20 KY Srinivasan k...@microsoft.com: -Original Message- From: gurlige...@gmail.com [mailto:gurlige...@gmail.com] On Behalf Of Bjarke Istrup Pedersen Sent: Sunday, January 19, 2014 11:46 AM To: KY Srinivasan Cc: Linux Kernel Mailing List; Haiyang Zhang Subject: Re: [PATCH v2

Re: [PATCH] Adding makefile for tools/hv

2014-01-20 Thread Bjarke Istrup Pedersen
Adding Greg to CC list. /Bjarke 2014/1/19 Bjarke Istrup Pedersen gurlige...@gentoo.org: Currently, there is no makefile for the Hyper-V tools. This patch adds the missing makefile, and adds it to the main tools makefile. Signed-off-by: Bjarke Istrup Pedersen gurlige...@gentoo.org

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

2014-01-19 Thread Bjarke Istrup Pedersen
2014/1/19 KY Srinivasan : > > >> -Original Message- >> From: gurlige...@gmail.com [mailto:gurlige...@gmail.com] On Behalf Of Bjarke >> Istrup Pedersen >> Sent: Sunday, January 19, 2014 2:15 AM >> To: KY Srinivasan >> Cc: Linux Kernel Mailing Lis

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

[PATCH] Adding makefile for tools/hv

2014-01-19 Thread Bjarke Istrup Pedersen
Currently, there is no makefile for the Hyper-V tools. This patch adds the missing makefile, and adds it to the main tools makefile. Signed-off-by: Bjarke Istrup Pedersen --- tools/Makefile| 11 ++- tools/hv/Makefile | 13 + 2 files changed, 19 insertions(+), 5 deletions

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,

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

2014-01-19 Thread Bjarke Istrup Pedersen
2014/1/19 KY Srinivasan : > > >> -Original Message----- >> From: Bjarke Istrup Pedersen [mailto:gurlige...@gmail.com] On Behalf Of >> Bjarke >> Istrup Pedersen >> Sent: Saturday, January 18, 2014 12:48 PM >> To: linux-kernel@vger.kernel.org >>

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

2014-01-19 Thread Bjarke Istrup Pedersen
2014/1/19 KY Srinivasan : > > >> -Original Message- >> From: gurlige...@gmail.com [mailto:gurlige...@gmail.com] On Behalf Of Bjarke >> Istrup Pedersen >> Sent: Saturday, January 18, 2014 4:23 PM >> To: Linux Kernel Mailing List >> Cc: Haiyang Zh

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

2014-01-19 Thread Bjarke Istrup Pedersen
2014/1/19 KY Srinivasan k...@microsoft.com: -Original Message- From: gurlige...@gmail.com [mailto:gurlige...@gmail.com] On Behalf Of Bjarke Istrup Pedersen Sent: Saturday, January 18, 2014 4:23 PM To: Linux Kernel Mailing List Cc: Haiyang Zhang; KY Srinivasan; Bjarke Istrup

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

2014-01-19 Thread Bjarke Istrup Pedersen
2014/1/19 KY Srinivasan k...@microsoft.com: -Original Message- From: Bjarke Istrup Pedersen [mailto:gurlige...@gmail.com] On Behalf Of Bjarke Istrup Pedersen Sent: Saturday, January 18, 2014 12:48 PM To: linux-kernel@vger.kernel.org Cc: Haiyang Zhang; Hank Janssen; KY Srinivasan

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

2014-01-19 Thread Bjarke Istrup Pedersen
2014/1/19 Borislav Petkov b...@alien8.de: 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

[PATCH] Adding makefile for tools/hv

2014-01-19 Thread Bjarke Istrup Pedersen
Currently, there is no makefile for the Hyper-V tools. This patch adds the missing makefile, and adds it to the main tools makefile. Signed-off-by: Bjarke Istrup Pedersen gurlige...@gentoo.org --- tools/Makefile| 11 ++- tools/hv/Makefile | 13 + 2 files changed, 19

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

2014-01-19 Thread Bjarke Istrup Pedersen
2014/1/19 Borislav Petkov b...@alien8.de: 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

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

2014-01-19 Thread Bjarke Istrup Pedersen
2014/1/19 KY Srinivasan k...@microsoft.com: -Original Message- From: gurlige...@gmail.com [mailto:gurlige...@gmail.com] On Behalf Of Bjarke Istrup Pedersen Sent: Sunday, January 19, 2014 2:15 AM To: KY Srinivasan Cc: Linux Kernel Mailing List; Haiyang Zhang Subject: Re: [PATCH v2

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

2014-01-18 Thread Bjarke Istrup Pedersen
Could any of you Hyper-V developers take a look at this, and see if this split makes sense to you? Thanks :) /Bjarke 2014/1/19 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 userspac

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/u

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 understoo

[PATCH v2] 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. Version 2: Split UAPI parts into new header, instead of duplicating. Signed-off-by: Bjarke Istrup Pedersen --- include

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 understoo

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 fro

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 > &g

[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

[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 gurlige...@gentoo.org --- include/uapi/linux/Kbuild |1 + include/uapi/linux

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

2014-01-18 Thread Bjarke Istrup Pedersen
2014/1/18 Paul Bolle pebo...@tiscali.nl 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

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

2014-01-18 Thread Bjarke Istrup Pedersen
2014/1/18 Borislav Petkov b...@alien8.de: 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

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

2014-01-18 Thread Bjarke Istrup Pedersen
2014/1/18 Borislav Petkov b...@alien8.de: 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

[PATCH v2] 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. Version 2: Split UAPI parts into new header, instead of duplicating. Signed-off-by: Bjarke Istrup Pedersen gurlige

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

2014-01-18 Thread Bjarke Istrup Pedersen
2014/1/18 Borislav Petkov b...@alien8.de: 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

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

2014-01-18 Thread Bjarke Istrup Pedersen
2014/1/19 Borislav Petkov b...@alien8.de: 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

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

2014-01-18 Thread Bjarke Istrup Pedersen
Could any of you Hyper-V developers take a look at this, and see if this split makes sense to you? Thanks :) /Bjarke 2014/1/19 Bjarke Istrup Pedersen gurlige...@gentoo.org: This patch adds the hyperv.h header to the uapi folder, and adds it to the Kbuild file. Doing this enables compiling

Re: Commit 607ca46e97a1b6594b29647d98a32d545c24bdff breaks building tools/hv/hv_kvp_daemon.c

2013-01-25 Thread Bjarke Istrup Pedersen
2012/12/12 Bjarke Istrup Pedersen : > Hey, > > It seems like this commit breaks building hv_kvp_daemon. > > Here is the bisect log: > > # good: [a0d271cbfed1dd50278c6b06bead3d00ba0a88f9] Linux 3.6 > git bisect good a0d271cbfed1dd50278c6b06

Re: Commit 607ca46e97a1b6594b29647d98a32d545c24bdff breaks building tools/hv/hv_kvp_daemon.c

2013-01-25 Thread Bjarke Istrup Pedersen
2012/12/12 Bjarke Istrup Pedersen gurlige...@gentoo.org: Hey, It seems like this commit breaks building hv_kvp_daemon. Here is the bisect log: # good: [a0d271cbfed1dd50278c6b06bead3d00ba0a88f9] Linux 3.6 git bisect good a0d271cbfed1dd50278c6b06bead3d00ba0a88f9 # bad

Commit 607ca46e97a1b6594b29647d98a32d545c24bdff breaks building tools/hv/hv_kvp_daemon.c

2012-12-12 Thread Bjarke Istrup Pedersen
Hey, It seems like this commit breaks building hv_kvp_daemon. Here is the bisect log: # good: [a0d271cbfed1dd50278c6b06bead3d00ba0a88f9] Linux 3.6 git bisect good a0d271cbfed1dd50278c6b06bead3d00ba0a88f9 # bad: [29594404d7fe73cd80eaa4ee8c43dcc53970c60e] Linux 3.7 git bisect bad

Commit 607ca46e97a1b6594b29647d98a32d545c24bdff breaks building tools/hv/hv_kvp_daemon.c

2012-12-12 Thread Bjarke Istrup Pedersen
Hey, It seems like this commit breaks building hv_kvp_daemon. Here is the bisect log: # good: [a0d271cbfed1dd50278c6b06bead3d00ba0a88f9] Linux 3.6 git bisect good a0d271cbfed1dd50278c6b06bead3d00ba0a88f9 # bad: [29594404d7fe73cd80eaa4ee8c43dcc53970c60e] Linux 3.7 git bisect bad