From: Greg KH <gregkh@linuxfoundation.org>
To: KY Srinivasan <kys@microsoft.com>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"devel@linuxdriverproject.org" <devel@linuxdriverproject.org>,
"olaf@aepfle.de" <olaf@aepfle.de>,
"apw@canonical.com" <apw@canonical.com>,
"ben@decadent.org.uk" <ben@decadent.org.uk>
Subject: Re: [PATCH V3 08/14] Tools: hv: Gather DNS information
Date: Fri, 17 Aug 2012 10:17:28 -0700 [thread overview]
Message-ID: <20120817171728.GA14288@kroah.com> (raw)
In-Reply-To: <426367E2313C2449837CD2DE46E7EAF9236C1E63@SN2PRD0310MB382.namprd03.prod.outlook.com>
On Fri, Aug 17, 2012 at 05:07:22PM +0000, KY Srinivasan wrote:
> > On Thu, Aug 16, 2012 at 06:32:19PM -0700, K. Y. Srinivasan wrote:
> > > Now, gather DNS information. Invoke an external script (that can be
> > > distro dependent) to gather the DNS information.
> >
> > Where is that script? Why a script, this really isn't in a standard
> > method for all Linux distros? If not, why not work to do that like the
> > freedesktop.org people have done for other system configuration items?
>
> We debated this issue on this mailing list for a long time. The consensus was that
> we would invoke external "distro" specific scripts for both configuring the interface
> as well as gathering information that could not be gathered in a distro independent
> fashion. That is what I have implemented here. This code specifies the interface to this
> external script. If these external scripts are not there, it is not fatal - on the SET side,
> the interface will not be configured and on the get side some attributes that are to be
> gathered via these external scripts will not be collected.
I've applied the first 7 patches here, feel free to submit the rest when
you answer the other questions I asked as well.
greg k-h
next prev parent reply other threads:[~2012-08-17 17:17 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-17 1:31 [PATCH V3 00/14] drivers: hv: kvp K. Y. Srinivasan
2012-08-17 1:32 ` [PATCH V3 01/14] Drivers: hv: kvp: Support the new IP injection messages K. Y. Srinivasan
2012-08-17 1:32 ` [PATCH V3 02/14] Tools: hv: Prepare to expand kvp_get_ip_address() functionality K. Y. Srinivasan
2012-08-17 1:32 ` [PATCH V3 03/14] Tools: hv: Further refactor kvp_get_ip_address() K. Y. Srinivasan
2012-08-17 1:32 ` [PATCH V3 04/14] Tools: hv: Gather address family information K. Y. Srinivasan
2012-08-17 1:32 ` [PATCH V3 05/14] Tools: hv: Gather subnet information K. Y. Srinivasan
2012-08-17 1:32 ` [PATCH V3 06/14] Tools: hv: Represent the ipv6 mask using CIDR notation K. Y. Srinivasan
2012-08-17 1:32 ` [PATCH V3 07/14] Tools: hv: Gather ipv[4,6] gateway information K. Y. Srinivasan
2012-08-17 1:32 ` [PATCH V3 08/14] Tools: hv: Gather DNS information K. Y. Srinivasan
2012-08-17 16:51 ` Greg KH
2012-08-17 16:55 ` Greg KH
2012-08-17 17:07 ` KY Srinivasan
2012-08-17 17:17 ` Greg KH [this message]
2012-08-17 21:41 ` KY Srinivasan
2012-08-17 17:32 ` Ben Hutchings
2012-08-17 17:38 ` Greg KH
2012-08-17 17:51 ` Ben Hutchings
2012-08-17 17:55 ` KY Srinivasan
2012-08-17 17:46 ` KY Srinivasan
2012-08-17 18:27 ` Olaf Hering
2012-08-17 18:44 ` Ben Hutchings
2012-08-17 18:53 ` Olaf Hering
2012-08-22 8:50 ` Andy Whitcroft
2012-08-22 15:45 ` KY Srinivasan
2012-08-17 1:32 ` [PATCH V3 09/14] Tools: hv: Gather DHCP information K. Y. Srinivasan
2012-08-17 1:32 ` [PATCH V3 10/14] Tools: hv: Implement the KVP verb - KVP_OP_SET_IP_INFO K. Y. Srinivasan
2012-08-17 1:32 ` [PATCH V3 11/14] Tools: hv: Rename the function kvp_get_ip_address() K. Y. Srinivasan
2012-08-17 1:32 ` [PATCH V3 12/14] Tools: hv: Implement the KVP verb - KVP_OP_GET_IP_INFO K. Y. Srinivasan
2012-08-17 1:32 ` [PATCH V3 13/14] Tools: hv: Get rid of some unused variables K. Y. Srinivasan
2012-08-17 1:32 ` [PATCH V3 14/14] Tools: hv: Correctly type string variables 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=20120817171728.GA14288@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=apw@canonical.com \
--cc=ben@decadent.org.uk \
--cc=devel@linuxdriverproject.org \
--cc=kys@microsoft.com \
--cc=linux-kernel@vger.kernel.org \
--cc=olaf@aepfle.de \
/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