Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 9 May 2009 13:07:43 GMT
From:      Marko Zec <zec@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 161828 for review
Message-ID:  <200905091307.n49D7hN7057989@repoman.freebsd.org>

next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=161828

Change 161828 by zec@zec_tpx32 on 2009/05/09 13:07:41

	Don't call if_dead() in if_detach() in options VIMAGE builds,
	given that we may wish to reattach the ifnet to another vnet
	after if_detach() is done, and if_dead() would render the
	ifnet useless.
	
	Pointed out by:	rwatson

Affected files ...

.. //depot/projects/vimage-commit2/src/sys/net/if.c#57 edit

Differences ...

==== //depot/projects/vimage-commit2/src/sys/net/if.c#57 (text+ko) ====

@@ -950,8 +950,12 @@
 
 	/*
 	 * Prevent further calls into the device driver via ifnet.
+	 * XXX temporarily disabled in VIMAGE builds as we may need to
+	 * to reattach the ifnet to another vnet.
 	 */
++#ifndef VIMAGE
 	if_dead(ifp);
++#endif
 
 	/*
 	 * Remove link ifaddr pointer and maybe decrement if_index.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200905091307.n49D7hN7057989>