Re: [PATCH 1/3]: Staging: hv: Use native page allocation/free functions

2011-02-11 Thread Greg KH
On Fri, Feb 11, 2011 at 09:59:00AM -0800, K. Y. Srinivasan wrote: --- a/drivers/staging/hv/hv.c +++ b/drivers/staging/hv/hv.c @@ -230,7 +230,12 @@ int hv_init(void) * Allocate the hypercall page memory * virtaddr = osd_page_alloc(1); */ - virtaddr =

Re: [PATCH 1/3]: Staging: hv: Use native page allocation/free functions

2011-02-11 Thread Greg KH
On Fri, Feb 11, 2011 at 08:55:56PM +, KY Srinivasan wrote: -Original Message- From: Greg KH [mailto:gre...@suse.de] Sent: Friday, February 11, 2011 1:30 PM To: KY Srinivasan Cc: linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; virtualizat...@lists.osdl.org;

RE: [PATCH 1/3]: Staging: hv: Use native page allocation/free functions

2011-02-11 Thread Hank Janssen
-Original Message- And then KY Srinivasan spoke on Friday, February 11, 2011 12:56 PM From: Greg KH [mailto:gre...@suse.de] Sent: Friday, February 11, 2011 1:30 PM - virtaddr = osd_virtual_alloc_exec(PAGE_SIZE); +#ifdef __x86_64__ + virtaddr = __vmalloc(PAGE_SIZE,

RE: [PATCH 1/3]: Staging: hv: Use native page allocation/free functions

2011-02-11 Thread Hank Janssen
-Original Message- From: Greg KH [mailto:gre...@suse.de] On Friday, February 11, 2011 1:24 PM On Fri, Feb 11, 2011 at 08:55:56PM +, KY Srinivasan wrote: I'm not saying this patch is wrong at all, but I still don't understand why this is different depending on the

RE: [PATCH 1/3]: Staging: hv: Use native page allocation/free functions

2011-02-11 Thread KY Srinivasan
-Original Message- From: Greg KH [mailto:gre...@suse.de] Sent: Friday, February 11, 2011 1:30 PM To: KY Srinivasan Cc: linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; virtualizat...@lists.osdl.org; Hank Janssen Subject: Re: [PATCH 1/3]: Staging: hv: Use native page

[PATCH 1/3]: Staging: hv: Use native page allocation/free functions

2011-02-11 Thread K. Y. Srinivasan
In preperation for getting rid of the osd.[ch] files; change all page allocation/free functions to use native interfaces. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/channel.c| 12 +++-

[PATCH 2/3]: Staging: hv: Use native wait primitives

2011-02-11 Thread K. Y. Srinivasan
In preperation for getting rid of the osd layer; change the code to use native wait interfaces. As part of this, fixed the buggy implementation in the osd_wait_primitive where the condition was cleared potentially after the condition was signalled. Signed-off-by: K. Y. Srinivasan

[PATCH]: Staging: hv: Cleanup vmalloc calls

2011-02-11 Thread K. Y. Srinivasan
The subject says it all. There is no need to specify different page protection bits based on the architecture. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Hank Janssen hjans...@microsoft.com --- drivers/staging/hv/hv.c |5 - 1 files changed, 0 insertions(+), 5