Messages in this thread | | | From | Dexuan Cui <> | Subject | RE: [PATCH v13 net-next 1/1] hv_sock: introduce Hyper-V Sockets | Date | Thu, 30 Jun 2016 14:33:19 +0000 |
| |
> From: David Miller [mailto:davem@davemloft.net] > Sent: Thursday, June 30, 2016 20:45 > To: Dexuan Cui <decui@microsoft.com> > Cc: gregkh@linuxfoundation.org; netdev@vger.kernel.org; linux- > kernel@vger.kernel.org; devel@linuxdriverproject.org; olaf@aepfle.de; > apw@canonical.com; jasowang@redhat.com; vkuznets@redhat.com; > cavery@redhat.com; KY Srinivasan <kys@microsoft.com>; Haiyang Zhang > <haiyangz@microsoft.com>; joe@perches.com; rolf.neugebauer@docker.com > Subject: Re: [PATCH v13 net-next 1/1] hv_sock: introduce Hyper-V Sockets > > From: Dexuan Cui <decui@microsoft.com> > Date: Wed, 29 Jun 2016 11:30:40 +0000 > > > @@ -1509,4 +1509,18 @@ static inline void commit_rd_index(struct > vmbus_channel *channel) > > } > > > > > > +struct vmpipe_proto_header { > > + u32 pkt_type; > > It is wasteful to have two empty lines before this structure definition, one > is sufficient. > > ...
Hi David, Thank you for pointing out the issues!
I'll fix all of them, and check all the similar issues in the patch.
Will post a new version ASAP.
Thanks, -- Dexuan
| |