> -----Original Message----- > From: Greg KH [mailto:gregkh@xxxxxxxxxxxxxxxxxxx] > Sent: Wednesday, August 31, 2016 4:13 PM > To: KY Srinivasan <kys@xxxxxxxxxxxxx> > Cc: linux-kernel@xxxxxxxxxxxxxxx; devel@xxxxxxxxxxxxxxxxxxxxxx; > olaf@xxxxxxxxx; apw@xxxxxxxxxxxxx; vkuznets@xxxxxxxxxx; > jasowang@xxxxxxxxxx; leann.ogasawara@xxxxxxxxxxxxx > Subject: Re: [PATCH 1/4] Drivers: hv: cleanup vmbus_open() for wrap around > mappings > > On Wed, Jul 06, 2016 at 05:25:02PM -0700, kys@xxxxxxxxxxxxxxxxxxxxxx > wrote: > > From: Vitaly Kuznetsov <vkuznets@xxxxxxxxxx> > > > > In preparation for doing wrap around mappings for ring buffers cleanup > > vmbus_open() function: > > - check that ring sizes are PAGE_SIZE aligned (they are for all in-kernel > > drivers now); > > - kfree(open_info) on error only after we kzalloc() it (not an issue as it > > is valid to call kfree(NULL); > > - rename poorly named labels; > > - use alloc_pages() instead of __get_free_pages() as we need struct page > > pointer for future. > > > > Signed-off-by: Vitaly Kuznetsov <vkuznets@xxxxxxxxxx> > > Signed-off-by: K. Y. Srinivasan <kys@xxxxxxxxxxxxx> > > Tested-by: Dexuan Cui <decui@xxxxxxxxxxxxx> > > --- > > drivers/hv/channel.c | 43 +++++++++++++++++++++++-------------------- > > 1 files changed, 23 insertions(+), 20 deletions(-) > > This patch doesn't apply to the tree due to the other hv patches you > sent. Can you respin this series and resend? Will do. Regards, K. Y > > thanks, > > greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
