Messages in this thread | | | From | KY Srinivasan <> | Subject | RE: [PATCH 08/17] Tools: hv: Gather subnet information | Date | Wed, 25 Jul 2012 14:10:51 +0000 |
| |
> -----Original Message----- > From: Ben Hutchings [mailto:ben@decadent.org.uk] > Sent: Tuesday, July 24, 2012 9:14 PM > To: KY Srinivasan > Cc: gregkh@linuxfoundation.org; linux-kernel@vger.kernel.org; > devel@linuxdriverproject.org; virtualization@lists.osdl.org; olaf@aepfle.de; > apw@canonical.com; netdev@vger.kernel.org > Subject: Re: [PATCH 08/17] Tools: hv: Gather subnet information > > On Tue, 2012-07-24 at 09:01 -0700, K. Y. Srinivasan wrote: > > Now gather sub-net information for the specified interface. > > > > Signed-off-by: K. Y. Srinivasan <kys@microsoft.com> > > Reviewed-by: Haiyang Zhang <haiyangz@microsoft.com> > > --- > > tools/hv/hv_kvp_daemon.c | 31 +++++++++++++++++++++++++++++-- > > 1 files changed, 29 insertions(+), 2 deletions(-) > > > > diff --git a/tools/hv/hv_kvp_daemon.c b/tools/hv/hv_kvp_daemon.c > > index 79eb130..2c24ebf 100644 > > --- a/tools/hv/hv_kvp_daemon.c > > +++ b/tools/hv/hv_kvp_daemon.c > > @@ -534,6 +534,7 @@ kvp_get_ip_address(int family, char *if_name, int op, > > struct ifaddrs *ifap; > > struct ifaddrs *curp; > > int offset = 0; > > + int sn_offset = 0; > > const char *str; > > int error = 0; > > char *buffer; > > @@ -594,12 +595,38 @@ kvp_get_ip_address(int family, char *if_name, int op, > > * Gather info other than the IP address. > > * IP address info will be gathered later. > > */ > > - if (curp->ifa_addr->sa_family == AF_INET) > > + if (curp->ifa_addr->sa_family == AF_INET) { > > ip_buffer->addr_family |= ADDR_FAMILY_IPV4; > > - else > > + /* > > + * Get subnet info. > > + */ > > + error = kvp_process_ip_address( > > + curp->ifa_netmask, > > + AF_INET, > > + (char *) > > + ip_buffer->sub_net, > > + length, > > + &sn_offset); > [...] > > This is barely readable; why don't you indent the arguments by just one > extra tab?
Will do.
Regards,
K. Y
| |