Skip to content

Commit

Permalink
Staging: hv: vmbus: Cleanup the code in process_chn_event()
Browse files Browse the repository at this point in the history
A channel in Hyper-V is equivalent to a device. Thus, a channel is
persistent once it is presented to the guest, even if the driver
managing this device is unloaded. By checking and invoking the driver
specific callback function under the protection of the channel
inbound_lock, we can properly deal with racing driver unloads since an
unloading driver sets the callback to NULL under the protection of this
inbound_lock.

Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
Signed-off-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
  • Loading branch information
K. Y. Srinivasan authored and Greg Kroah-Hartman committed Sep 6, 2011
1 parent 7341d90 commit 2432603
Showing 1 changed file with 16 additions and 2 deletions.
18 changes: 16 additions & 2 deletions drivers/staging/hv/connection.c
Original file line number Diff line number Diff line change
Expand Up @@ -219,11 +219,25 @@ static void process_chn_event(u32 relid)
*/
channel = relid2channel(relid);

if (!channel) {
pr_err("channel not found for relid - %u\n", relid);
return;
}

/*
* A channel once created is persistent even when there
* is no driver handling the device. An unloading driver
* sets the onchannel_callback to NULL under the
* protection of the channel inbound_lock. Thus, checking
* and invoking the driver specific callback takes care of
* orderly unloading of the driver.
*/

spin_lock_irqsave(&channel->inbound_lock, flags);
if (channel && (channel->onchannel_callback != NULL))
if (channel->onchannel_callback != NULL)
channel->onchannel_callback(channel->channel_callback_context);
else
pr_err("channel not found for relid - %u\n", relid);
pr_err("no channel callback for relid - %u\n", relid);

spin_unlock_irqrestore(&channel->inbound_lock, flags);
}
Expand Down

0 comments on commit 2432603

Please sign in to comment.