On Mon, 09 Jul 2012 13:49:17 -0700 Xin Li wrote: XL> On 07/09/12 13:38, Mikolaj Golub wrote: >> Author: trociny Date: Mon Jul 9 20:38:18 2012 New Revision: >> 238309 URL: http://svn.freebsd.org/changeset/base/238309 >> >> Log: In epair_clone_destroy(), when destroying the second half, we >> have to switch to its vnet before calling ether_ifdetach(). >> Otherwise if the second half resides in a different vnet, >> if_detach() silently fails leaving a stale pointer in V_ifnet list, >> and the system crashes trying to access this pointer later. >> >> Another solution could be not to allow to destroy epair unless >> both ends are in the home vnet. >> >> Discussed with: bz Tested by: delphij
XL> Thanks! XL> Since this affects RELENG_9 and RELENG_8, could you please also MFC XL> after a settle period? Sure. Just forgot to add the 'MFC after' reminder. I am going to MFC it after stable/9 unfreeze unless someone really wants it in 9.1 and tests it a little. This does not look like a critical issue because of the existing workaround (which can be considered as a best practice): move both ends to the home vnet before destroying the epair. -- Mikolaj Golub _______________________________________________ svn-src-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-all To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
