Date: Thu, 16 Aug 2007 19:09:16 GMT From: Marko Zec <zec@FreeBSD.org> To: Perforce Change Reviews <perforce@FreeBSD.org> Subject: PERFORCE change 125229 for review Message-ID: <200708161909.l7GJ9GFB008915@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=125229 Change 125229 by zec@zec_tca51 on 2007/08/16 19:08:27 Remove a debugging printf() in VNET_LIST_LOCK() that was triggered when acquisition of the exclusive access didn't succeed immediately. Resolving of this condition seems to work fine use the condvar signalling from the thread releasing the shared lock, hence trim down the debugging noise. Affected files ... .. //depot/projects/vimage/src/sys/kern/kern_vimage.c#35 edit Differences ... ==== //depot/projects/vimage/src/sys/kern/kern_vimage.c#35 (text+ko) ==== @@ -115,11 +115,8 @@ #define VNET_LIST_LOCK() \ mtx_lock(&vnet_list_refc_mtx); \ - while (vnet_list_refc != 0) { \ - cv_wait(&vnet_list_condvar, &vnet_list_refc_mtx); \ - printf("XXX vnet_list_refc = %d in %s\n", \ - vnet_list_refc, __FUNCTION__); \ - } + while (vnet_list_refc != 0) \ + cv_wait(&vnet_list_condvar, &vnet_list_refc_mtx); #define VNET_LIST_UNLOCK() \ mtx_unlock(&vnet_list_refc_mtx);
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200708161909.l7GJ9GFB008915>