diff options
| author | Saurabh Sengar <ssengar@linux.microsoft.com> | 2022-06-09 10:16:36 -0700 |
|---|---|---|
| committer | Wei Liu <wei.liu@kernel.org> | 2022-06-10 08:41:28 +0000 |
| commit | 656c5ba50b7172a0ea25dc1b37606bd51d01fe8d (patch) | |
| tree | aa2dc5c0a31545a18ee6a7a21ac684438d4f25d8 /drivers/hv | |
| parent | f5f93d7f5a5cbfef02609dead21e7056e83f4fab (diff) | |
| download | lwn-656c5ba50b7172a0ea25dc1b37606bd51d01fe8d.tar.gz lwn-656c5ba50b7172a0ea25dc1b37606bd51d01fe8d.zip | |
Drivers: hv: vmbus: Release cpu lock in error case
In case of invalid sub channel, release cpu lock before returning.
Fixes: a949e86c0d780 ("Drivers: hv: vmbus: Resolve race between init_vp_index() and CPU hotplug")
Signed-off-by: Saurabh Sengar <ssengar@linux.microsoft.com>
Reviewed-by: Michael Kelley <mikelley@microsoft.com>
Link: https://lore.kernel.org/r/1654794996-13244-1-git-send-email-ssengar@linux.microsoft.com
Signed-off-by: Wei Liu <wei.liu@kernel.org>
Diffstat (limited to 'drivers/hv')
| -rw-r--r-- | drivers/hv/channel_mgmt.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/hv/channel_mgmt.c b/drivers/hv/channel_mgmt.c index 280b52927758..5b120402d405 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. |