Re: [PATCH 1/4] Drivers: hv: cleanup vmbus_open() for wrap around mappings

From: Greg KH
Date: Wed Aug 31 2016 - 06:42:36 EST


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?

thanks,

greg k-h