From: "K. Y. Srinivasan" <kys@microsoft.com>
To: gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org,
devel@linuxdriverproject.org, virtualization@lists.osdl.org,
olaf@aepfle.de, apw@canonical.com, ben@decadent.org.uk
Cc: "K. Y. Srinivasan" <kys@microsoft.com>
Subject: [PATCH V3 00/14] drivers: hv: kvp
Date: Thu, 16 Aug 2012 17:45:11 -0700 [thread overview]
Message-ID: <1345164325-5198-1-git-send-email-kys@microsoft.com> (raw)
In-Reply-To: <1345164300-5146-1-git-send-email-kys@microsoft.com>
This patch set expands the KVP (Key Value Pair) functionality to
implement the mechanism to GET/SET IP addresses in the guest. This
functionality is used in Windows Server 2012 to implement VM
replication functionality. The way IP configuration information
is managed is distro specific. Based on the feedback I have gotten
from Olaf, Greg, Steve, Ben and Mairus, I have chosen to seperate
distro specific code from this patch-set. Most of the GET operation
can be implemented in a way that is completely distro independent and
I have implemented that as such and is included in this patch-set.
Some of the attributes that can only be fetched in a distro
dependent way as well the mechanism for configuring an interface
(the SET operation) that is clearly distro specific is to be
implemented via external scripts that will be invoked via the KVP
code. We define here the interface to these scripts.
Adding support for IP injection resulted in some changes to the
protocol between the user level daemon and the kernel driver.
These changes have been implemented in way that would retain
compatibility with older daemons. I would like to thank Olaf and
Greg for pointing out the compatibility issue.
This version of the patch set addresses all of the comments that I
have received to date from Olaf, Ben and others. Specifically, I have
cleanedup all of the string manipulation code. Furthermore, I have also
simplified the format of the interface configuration file that is
generated by the KVP daemon and gotten rid of some constant strings that
may not be relevant on all distros. I would like to thank Olaf and Ben
for their detailed review.
Following are the changes from V2 of this patch set:
1. Addressed the indentation issue Greg noted in the patch:
0001-Drivers-hv-kvp-Support-the-new-IP-injection-messages.patch
2. Cleaned up the Reviewed-byi line based on input from Ben and Greg.
3. Since Ben had an identical patch for properly closing the open
streams, I have not included that patch in this series.
4. Greg applied the first 3 patches of the V2 patch set. This version
has the remaining patches.
K. Y. Srinivasan (14):
Drivers: hv: kvp: Support the new IP injection messages
Tools: hv: Prepare to expand kvp_get_ip_address() functionality
Tools: hv: Further refactor kvp_get_ip_address()
Tools: hv: Gather address family information
Tools: hv: Gather subnet information
Tools: hv: Represent the ipv6 mask using CIDR notation
Tools: hv: Gather ipv[4,6] gateway information
Tools: hv: Gather DNS information
Tools: hv: Gather DHCP information
Tools: hv: Implement the KVP verb - KVP_OP_SET_IP_INFO
Tools: hv: Rename the function kvp_get_ip_address()
Tools: hv: Implement the KVP verb - KVP_OP_GET_IP_INFO
Tools: hv: Get rid of some unused variables
Tools: hv: Correctly type string variables
drivers/hv/hv_kvp.c | 141 +++++++-
include/linux/hyperv.h | 2 +
tools/hv/hv_kvp_daemon.c | 866 ++++++++++++++++++++++++++++++++++++++++++----
3 files changed, 942 insertions(+), 67 deletions(-)
--
1.7.4.1
next prev parent reply other threads:[~2012-08-17 0:45 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-17 0:45 [PATCH V3 00/14] drivers: hv: kvp K. Y. Srinivasan
2012-08-17 0:45 ` K. Y. Srinivasan [this message]
2012-08-17 0:45 ` [PATCH 01/14] Drivers: hv: kvp: Support the new IP injection messages K. Y. Srinivasan
2012-08-17 0:45 ` [PATCH 02/14] Tools: hv: Prepare to expand kvp_get_ip_address() functionality K. Y. Srinivasan
2012-08-17 0:45 ` [PATCH 03/14] Tools: hv: Further refactor kvp_get_ip_address() K. Y. Srinivasan
2012-08-17 0:45 ` [PATCH 04/14] Tools: hv: Gather address family information K. Y. Srinivasan
2012-08-17 0:45 ` [PATCH 05/14] Tools: hv: Gather subnet information K. Y. Srinivasan
2012-08-17 0:45 ` [PATCH 06/14] Tools: hv: Represent the ipv6 mask using CIDR notation K. Y. Srinivasan
2012-08-17 0:45 ` [PATCH 07/14] Tools: hv: Gather ipv[4,6] gateway information K. Y. Srinivasan
2012-08-17 0:45 ` [PATCH 08/14] Tools: hv: Gather DNS information K. Y. Srinivasan
2012-08-17 0:45 ` [PATCH 09/14] Tools: hv: Gather DHCP information K. Y. Srinivasan
2012-08-17 0:45 ` [PATCH 10/14] Tools: hv: Implement the KVP verb - KVP_OP_SET_IP_INFO K. Y. Srinivasan
2012-08-17 0:45 ` [PATCH 11/14] Tools: hv: Rename the function kvp_get_ip_address() K. Y. Srinivasan
2012-08-17 0:45 ` [PATCH 12/14] Tools: hv: Implement the KVP verb - KVP_OP_GET_IP_INFO K. Y. Srinivasan
2012-08-17 0:45 ` [PATCH 13/14] Tools: hv: Get rid of some unused variables K. Y. Srinivasan
2012-08-17 0:45 ` [PATCH 14/14] Tools: hv: Correctly type string variables K. Y. Srinivasan
2012-08-17 1:12 ` [PATCH V3 00/14] drivers: hv: kvp KY Srinivasan
-- strict thread matches above, loose matches on Subject: below --
2012-08-17 1:31 K. Y. Srinivasan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1345164325-5198-1-git-send-email-kys@microsoft.com \
--to=kys@microsoft.com \
--cc=apw@canonical.com \
--cc=ben@decadent.org.uk \
--cc=devel@linuxdriverproject.org \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=olaf@aepfle.de \
--cc=virtualization@lists.osdl.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox