RE: [PATCH 2/4] Rename the file hv_utils.c

2010-12-08 Thread Hank Janssen
> From: Evgeniy Polyakov [mailto:z...@ioremap.net] > Sent: Wednesday, December 08, 2010 2:55 PM > To: Ky Srinivasan > Cc: de...@driverdev.osuosl.org; virtualizat...@lists.osdl.org; Haiyang Zhang; > Hank Janssen; Greg KH; Stephen Hemminger > Subject: Re: [PATCH 2/4] Rename the file hv_utils.c > >

Re: [PATCH 2/4] Rename the file hv_utils.c

2010-12-08 Thread Ky Srinivasan
>>> On 12/8/2010 at 5:55 PM, in message <20101208225524.ga11...@ioremap.net>, Evgeniy Polyakov wrote: > You must be kidding. > The same in hv_kvp_onchannelcallback() and probably anywhere else. Hank, if it is ok with you, I will fix this issue across the board and submit that as part of thi

Re: [PATCH 2/4] Rename the file hv_utils.c

2010-12-08 Thread Evgeniy Polyakov
You must be kidding. The same in hv_kvp_onchannelcallback() and probably anywhere else. On Wed, Dec 08, 2010 at 03:26:10PM -0700, Ky Srinivasan (ksriniva...@novell.com) wrote: > +static void shutdown_onchannelcallback(void *context) > +{ > + struct vmbus_channel *channel = context; > + u8

Re: [PATCH 1/4] Add a connector Index to support HyperV KVP functionality

2010-12-08 Thread Ky Srinivasan
>>> On 12/8/2010 at 5:32 PM, in message <20101208223259.ga6...@suse.de>, Greg >>> KH wrote: > Please make your subject a bit more descriptive, for this one, it would > be: > Subject: [PATCH 1/4] Connector: add index to support HyperV > > And what is a "KVP"? > > On Wed, Dec 08, 2010 a

Re: [PATCH 2/4] Rename the file hv_utils.c

2010-12-08 Thread Ky Srinivasan
>>> On 12/8/2010 at 5:34 PM, in message <20101208223400.gb6...@suse.de>, Greg >>> KH wrote: > For the subject on the rest of the patches they would look like: > Subject: [PATCH 2/4] Staging: hv: rename hv_utils.c to hv_util.c > > Note the "Staging: hv:" prefix. > > On Wed, Dec 08, 201

Re: [PATCH 4/4] A daemon to gather guest specific information for KVP

2010-12-08 Thread Greg KH
How about: Subject: [PATCH 4/4] Staging: hv: add userspace daemon code to control key/value connection to the kernel On Wed, Dec 08, 2010 at 03:34:15PM -0700, Ky Srinivasan wrote: > >From 718eed47e4c2eb740dc04a6729c8853424ac6965 Mon Sep 17 00:00:00 2001 > From: ksrinivasan > Date: Wed, 8

Re: [PATCH 3/4] An Implementation of KVP functionality

2010-12-08 Thread Greg KH
How about: Subject: [PATCH 3/4] Staging: hv: implement key/value pair functionality On Wed, Dec 08, 2010 at 03:29:58PM -0700, Ky Srinivasan wrote: > >From 2843393e8f50421e81e033806cd121cfb8cc7b6a Mon Sep 17 00:00:00 2001 > From: ksrinivasan > Date: Wed, 8 Dec 2010 13:46:46 -0700 Again, t

Re: [PATCH 2/4] Rename the file hv_utils.c

2010-12-08 Thread Greg KH
For the subject on the rest of the patches they would look like: Subject: [PATCH 2/4] Staging: hv: rename hv_utils.c to hv_util.c Note the "Staging: hv:" prefix. On Wed, Dec 08, 2010 at 03:26:10PM -0700, Ky Srinivasan wrote: > >From 453acddcf55f1e68c5baf9dc5e8f601d18de15b0 Mon Sep 17 00:0

Re: [PATCH 1/4] Add a connector Index to support HyperV KVP functionality

2010-12-08 Thread Greg KH
Please make your subject a bit more descriptive, for this one, it would be: Subject: [PATCH 1/4] Connector: add index to support HyperV And what is a "KVP"? On Wed, Dec 08, 2010 at 03:20:59PM -0700, Ky Srinivasan wrote: > >From 5b7c16baf2f310a1e7e119c3df8fd976ca0e2f57 Mon Sep 17 00:00:00

[PATCH 4/4] A daemon to gather guest specific information for KVP

2010-12-08 Thread Ky Srinivasan
>From 718eed47e4c2eb740dc04a6729c8853424ac6965 Mon Sep 17 00:00:00 2001 From: ksrinivasan Date: Wed, 8 Dec 2010 14:10:35 -0700 This daemon gathers all the guest specific information needed to support the HyperV KVP functionality. This daemon communicates with the kernel component via a netlink ch

[PATCH 3/4] An Implementation of KVP functionality

2010-12-08 Thread Ky Srinivasan
>From 2843393e8f50421e81e033806cd121cfb8cc7b6a Mon Sep 17 00:00:00 2001 From: ksrinivasan Date: Wed, 8 Dec 2010 13:46:46 -0700 This is an implementation of the key/value pair (KVP) functionality for Linux guests hosted on HyperV. All guest specific data gathering for KVP will be implemented i

[PATCH 2/4] Rename the file hv_utils.c

2010-12-08 Thread Ky Srinivasan
>From 453acddcf55f1e68c5baf9dc5e8f601d18de15b0 Mon Sep 17 00:00:00 2001 From: ksrinivasan Date: Wed, 8 Dec 2010 12:24:58 -0700 The hv_utils module will be composed of more than one file. Rename hv_utils.c to accommodate this without changing the module name. Signed-off-by: ksrinivasan --- dri

[PATCH 1/4] Add a connector Index to support HyperV KVP functionality

2010-12-08 Thread Ky Srinivasan
>From 5b7c16baf2f310a1e7e119c3df8fd976ca0e2f57 Mon Sep 17 00:00:00 2001 From: ksrinivasan Date: Wed, 8 Dec 2010 09:35:13 -0700 Added a connector index to support key value/pair (KVP) functionality for Linux guests hosted on a HyperV platform. All KVP related data gathering will be done in a user-

Re: [PATCH 1/3]: An implementation of HyperV KVP functionality

2010-12-08 Thread Evgeniy Polyakov
On Wed, Dec 08, 2010 at 02:31:47PM -0700, Ky Srinivasan (ksriniva...@novell.com) wrote: > > Care to fix this, and address the connector issues, and then resend them > > all? > > Evgeniy has acked the patch for adding a new connector index for > implementing the KVP functionality. The concern Ev

Re: [PATCH 1/3]: An implementation of HyperV KVP functionality

2010-12-08 Thread Ky Srinivasan
> > Care to fix this, and address the connector issues, and then resend them > all? Evgeniy has acked the patch for adding a new connector index for implementing the KVP functionality. The concern Evgeniy had was with regards to the code currently in the upstream tree and Hank is planning

RE: [PATCH 2/3]: An implementation of HyperV KVP functionality

2010-12-08 Thread Hank Janssen
> From: Ky Srinivasan [mailto:ksriniva...@novell.com] > Sent: Tuesday, December 07, 2010 3:19 PM > >>> On 12/7/2010 at 5:29 PM, in message > <20101207222933.ga10...@ioremap.net>, > Evgeniy Polyakov wrote: > > On Tue, Dec 07, 2010 at 03:25:56PM -0700, Ky Srinivasan > > (ksriniva...@novell.com) wr