On Fri, Jun 07, 2013 at 02:33:08PM +0400, Gleb Smirnoff wrote:
T> On Fri, Jun 07, 2013 at 10:27:51AM +0000, Mikolaj Golub wrote:
T> M> Author: trociny
T> M> Date: Fri Jun  7 10:27:50 2013
T> M> New Revision: 251490
T> M> URL: http://svnweb.freebsd.org/changeset/base/251490
T> M> 
T> M> Log:
T> M>   Properly set curvnet context in lagg_port_setlladdr() task handler.
T> M>   
T> M>   Reported by:    Nikos Vassiliadis <nvass gmx.com>
T> M>   Submitted by:   zec
T> M>   Tested by:      Nikos Vassiliadis <nvass gmx.com>
T> M>   MFC after:      1 week
T> M> 
T> M> Modified:
T> M>   head/sys/net/if_lagg.c
T> M> 
T> M> Modified: head/sys/net/if_lagg.c
T> M> 
==============================================================================
T> M> --- head/sys/net/if_lagg.c        Fri Jun  7 09:06:50 2013        
(r251489)
T> M> +++ head/sys/net/if_lagg.c        Fri Jun  7 10:27:50 2013        
(r251490)
T> M> @@ -505,7 +505,9 @@ lagg_port_setlladdr(void *arg, int pendi
T> M>           ifp = llq->llq_ifp;
T> M>  
T> M>           /* Set the link layer address */
T> M> +         CURVNET_SET(ifp->if_vnet);
T> M>           error = if_setlladdr(ifp, llq->llq_lladdr, ETHER_ADDR_LEN);
T> M> +         CURVNET_RESTORE();
T> M>           if (error)
T> M>                   printf("%s: setlladdr failed on %s\n", __func__,
T> M>                       ifp->if_xname);
T> 
T> IMHO, the entire task function should be embraced into VNET context.

Sorry, I'm wrong. :(

-- 
Totus tuus, Glebius.
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to