On Fri, Oct 18, 2024 at 04:58:11AM -0700, Naman Jain wrote: > From: John Starks <jostarks@xxxxxxxxxxxxx> > > When resuming from hibernation, log any channels that were present > before hibernation but now are gone. > > Signed-off-by: John Starks <jostarks@xxxxxxxxxxxxx> > Co-developed-by: Naman Jain <namjain@xxxxxxxxxxxxxxxxxxx> > Signed-off-by: Naman Jain <namjain@xxxxxxxxxxxxxxxxxxx> > --- > drivers/hv/vmbus_drv.c | 16 ++++++++++++++++ > 1 file changed, 16 insertions(+) > > diff --git a/drivers/hv/vmbus_drv.c b/drivers/hv/vmbus_drv.c > index bd3fc41dc06b..1f56d138210e 100644 > --- a/drivers/hv/vmbus_drv.c > +++ b/drivers/hv/vmbus_drv.c > @@ -2462,6 +2462,7 @@ static int vmbus_bus_suspend(struct device *dev) > > static int vmbus_bus_resume(struct device *dev) > { > + struct vmbus_channel *channel; > struct vmbus_channel_msginfo *msginfo; > size_t msgsize; > int ret; > @@ -2494,6 +2495,21 @@ static int vmbus_bus_resume(struct device *dev) > > vmbus_request_offers(); > > + mutex_lock(&vmbus_connection.channel_mutex); > + list_for_each_entry(channel, &vmbus_connection.chn_list, listentry) { > + if (channel->offermsg.child_relid != INVALID_RELID) > + continue; > + > + /* hvsock channels are not expected to be present. */ > + if (is_hvsock_channel(channel)) > + continue; > + > + pr_err("channel %pUl/%pUl not present after resume.\n", > + &channel->offermsg.offer.if_type, > + &channel->offermsg.offer.if_instance); Do we want to put a TODO here, so as to do cleanup of these channels like force rescind etc later ? - Saurabh