-----Original Message-----
From: Olaf Hering [mailto:olaf@aepfle.de]
Sent: Monday, July 30, 2012 8:39 AM
To: Haiyang Zhang
Cc: davem@davemloft.net; netdev@vger.kernel.org; KY Srinivasan; linux-
kernel@vger.kernel.org; devel@linuxdriverproject.org
Subject: Re: [PATCH net,1/1] hyperv: Add support for setting MAC from
within guests

On Tue, Jul 10, Haiyang Zhang wrote:

quoted

diff --git a/drivers/net/hyperv/rndis_filter.c

b/drivers/net/hyperv/rndis_filter.c
index 981ebb1..fbf5394 100644

--- a/drivers/net/hyperv/rndis_filter.c
+++ b/drivers/net/hyperv/rndis_filter.c

@@ -47,6 +48,7 @@ struct rndis_request {
 	struct hv_page_buffer buf;
 	/* FIXME: We assumed a fixed size request here. */
 	struct rndis_message request_msg;
+	u8 ext[100];

This array is not referenced in the patch.
Please add a comment to the code what the purpose of this array is, and why
its size is 100 bytes.

It's a buffer for the extended info after the RNDIS message. It's referenced based
on the data offset in the RNDIS message. 100 byte size is enough for current needs, 
and should be sufficient for the near future.

I will add a comment to the code.

Thanks,
- Haiyang