On Fri, Jan 23, 2009 at 11:53:37AM +0300, Oleg Bulyzhin wrote: > On Fri, Jan 23, 2009 at 09:10:28AM +0100, Luigi Rizzo wrote: > > in your logmessage for the q_time change in dummynet > > http://svn.freebsd.org/viewvc/base?view=revision&revision=184414 > > > > mentions an ABI change that would prevent backporting. > > > > However, as far as I can tell, the kernel side of the change is > > fully self-contained in the ip_dummynet module, and the only > > side effect on an out-of-sync userland (/sbin/ipfw) is that > > you would not be able to run "ipfw pipe show" -- but you > > can still configure ipfw rules and pipes and tables without > > problems. > > > > It seems to me that the change is sufficiently useful to deserve > > a MFC, and the side effects are negligible and do not prevent an > > upgrade with the usual sequence (installkernel-reboot-installworld) > > > > Any objection to an MFC ? > > > > cheers > > luigi > > > > _______________________________________________ > > freebsd-stable@freebsd.org mailing list > > http://lists.freebsd.org/mailman/listinfo/freebsd-stable > > To unsubscribe, send any mail to "freebsd-stable-unsubscr...@freebsd.org" > > Have you seen this? > http://svn.freebsd.org/changeset/base/184506
this is also in RELENG_7 but i am not sure whether this workaround has any drawback e.g., when curr_time passes a 32-bit boundary there seems to be an incorrect setting of q->numbytes @@ -1382,7 +1383,7 @@ if (q->head != m) /* Flow was not idle, we are done. */ goto done; - if (q->q_time < curr_time) + if (q->q_time < (uint32_t)curr_time) q->numbytes = io_fast ? fs->pipe->bandwidth : 0; q->q_time = curr_time; cheers luigi _______________________________________________ freebsd-stable@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-stable To unsubscribe, send any mail to "freebsd-stable-unsubscr...@freebsd.org"