Re: [PATCH] Drivers: hv: vmbus: Release cpu lock in error case

Wei Liu
2022-06-10T08:40:41Z

On Thu, Jun 09, 2022 at 05:39:06PM +0000, Michael Kelley (LINUX) wrote:
> From: Saurabh Sengar <ssengar@linux.microsoft.com> Sent: Thursday, June 9, 2022 10:17 AM
> >
> > In case of invalid sub channel, release cpu lock before returning.
> >
> > Signed-off-by: Saurabh Sengar <ssengar@linux.microsoft.com>
> > ---
> > drivers/hv/channel_mgmt.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/hv/channel_mgmt.c b/drivers/hv/channel_mgmt.c
> > index 280b529..5b12040 100644
> > --- a/drivers/hv/channel_mgmt.c
> > +++ b/drivers/hv/channel_mgmt.c
> > @@ -639,6 +639,7 @@ static void vmbus_process_offer(struct vmbus_channel
> > *newchannel)
> > */
> > if (newchannel->offermsg.offer.sub_channel_index == 0) {
> > mutex_unlock(&vmbus_connection.channel_mutex);
> > + cpus_read_unlock();
> > /*
> > * Don't call free_channel(), because newchannel->kobj
> > * is not initialized yet.
> > --
> > 1.8.3.1
>
> Reviewed-by: Michael Kelley <mikelley@microsoft.com>
>

The bug comes from a949e86c0d780.

Added a Fixes tag and applied to hyperv-fixes. Thanks.