PERFORCE change 161261 for review
Julian Elischer
julian at elischer.org
Tue Apr 28 17:59:41 UTC 2009
Marko Zec wrote:
> http://perforce.freebsd.org/chv.cgi?CH=161261
>
> Change 161261 by zec at zec_amdx2 on 2009/04/28 17:51:16
>
> Unbreak in_rtqtimo().
>
so, If I'm right,
the plan is:
commit vimage-commit, with one single vimage
leave to simmer for a week while people test
commit vimage-commit2.. in time for BSDCan..
Oh this is so exciting.. :-)
> Affected files ...
>
> .. //depot/projects/vimage-commit2/src/sys/netinet/in_rmx.c#28 edit
>
> Differences ...
>
> ==== //depot/projects/vimage-commit2/src/sys/netinet/in_rmx.c#28 (text+ko) ====
>
> @@ -250,14 +250,13 @@
> static void
> in_rtqtimo(void *rock)
> {
> + CURVNET_SET((struct vnet *) rock);
> INIT_VNET_NET(curvnet);
> INIT_VNET_INET(curvnet);
> int fibnum;
> void *newrock;
> struct timeval atv;
>
> - KASSERT((rock == (void *)V_rt_tables[0][AF_INET]),
> - ("in_rtqtimo: unexpected arg"));
> for (fibnum = 0; fibnum < rt_numfibs; fibnum++) {
> if ((newrock = V_rt_tables[fibnum][AF_INET]) != NULL)
> in_rtqtimo_one(newrock);
> @@ -265,6 +264,7 @@
> atv.tv_usec = 0;
> atv.tv_sec = V_rtq_timeout;
> callout_reset(&V_rtq_timer, tvtohz(&atv), in_rtqtimo, rock);
> + CURVNET_RESTORE();
> }
>
> static void
> @@ -377,7 +377,7 @@
> rnh->rnh_close = in_clsroute;
> if (_in_rt_was_here == 0 ) {
> callout_init(&V_rtq_timer, CALLOUT_MPSAFE);
> - in_rtqtimo(rnh); /* kick off timeout first time */
> + callout_reset(&V_rtq_timer, 1, in_rtqtimo, curvnet);
> _in_rt_was_here = 1;
> }
> return 1;
More information about the p4-projects
mailing list