> From: David Miller [mailto:da...@davemloft.net] > Sent: Thursday, June 30, 2016 20:45 > To: Dexuan Cui <de...@microsoft.com> > Cc: gre...@linuxfoundation.org; net...@vger.kernel.org; linux- > ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de; > a...@canonical.com; jasow...@redhat.com; vkuzn...@redhat.com; > cav...@redhat.com; KY Srinivasan <k...@microsoft.com>; Haiyang Zhang > <haiya...@microsoft.com>; j...@perches.com; rolf.neugeba...@docker.com > Subject: Re: [PATCH v13 net-next 1/1] hv_sock: introduce Hyper-V Sockets > > From: Dexuan Cui <de...@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 _______________________________________________ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
