> -----Original Message----- > From: David Miller [mailto:[email protected]] > Sent: Thursday, August 14, 2014 5:29 PM > To: Haiyang Zhang > Cc: [email protected]; KY Srinivasan; [email protected]; > [email protected]; [email protected]; driverdev- > [email protected] > Subject: Re: [PATCH net-next,v2] hyperv: Increase the buffer length for > netvsc_channel_cb() > > From: Haiyang Zhang <[email protected]> > Date: Wed, 13 Aug 2014 18:03:44 +0000 > > > When the buffer is too small for a packet from VMBus, a bigger buffer > will be > > allocated in netvsc_channel_cb() and retry reading the packet from > VMBus. > > Increasing this buffer size will reduce the retry overhead. > > > > Signed-off-by: Haiyang Zhang <[email protected]> > > Reviewed-by: Dexuan Cui <[email protected]> > ... > > - net_device = kzalloc(sizeof(struct netvsc_device), GFP_KERNEL); > > + net_device = vzalloc(sizeof(*net_device)); > > This isn't what I suggested that you do. > > I said that the buffer inside of netvsc_device should be made an > indirect pointer and thus allocated seperately. > > Thus you're still kzalloc() net_device, but net_device->cb_buffer > becomes "unsigned char *" and another allocation is made for it.
I will change the patch to this way. Thanks, - Haiyang _______________________________________________ devel mailing list [email protected] http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
