d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

hyperv/vmbus: Don't free the bufring if its GPADL can't be disconnected.

Browse Source 
MFC after:	1 week
Sponsored by:	Microsoft
Differential Revision:	https://reviews.freebsd.org/D8570
This commit is contained in:
Sepherosa Ziehau 2016-11-25 07:41:42 +00:00
parent 32ab625a61
commit f2aeeaff6f
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=309136
1 changed files with 16 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
sys/dev/hyperv/vmbus/vmbus_chan.c
View File

@ -814,8 +814,22 @@ vmbus_chan_close_internal(struct vmbus_channel *chan)
/*
* Disconnect the TX+RX bufrings from this channel.
*/
if (chan->ch_bufring_gpadl) {
vmbus_chan_gpadl_disconnect(chan, chan->ch_bufring_gpadl);
if (chan->ch_bufring_gpadl != 0) {
int error1;
error1 = vmbus_chan_gpadl_disconnect(chan,
chan->ch_bufring_gpadl);
if (error1) {
/*
* XXX
* The bufring GPADL is still connected; abandon
* this bufring, instead of having mysterious
* crash or trashed data later on.
*/
vmbus_chan_printf(chan, "chan%u bufring GPADL "
"is still connected after close\n", chan->ch_id);
chan->ch_bufring = NULL;
}
chan->ch_bufring_gpadl = 0;
}