rrs added a comment. Moving the tests to a sub-dir sounds like a good idea as well.. Thanks Hans..
INLINE COMMENTS sys/kern/kern_timeout.c:674 Hans, no *you must* set it to false here in the soft clock so you can recognize if it is requested to be stopped (if possible). The return codes are all correct if the c_lock is set or not. sys/kern/kern_timeout.c:1059 Sure I can do that.. I will ALT-Q it and see if I can get it better formatted sys/modules/callout_test/callout_test.c:58 Sure thats a good ideas since they are only used in the module. sys/modules/callout_test/callout_test.c:69 ahh yes, it surely is. I had been testing it in a branch that did not have the define in callout.h and it must have slipped in.. good catch! sys/modules/callout_test/callout_test.c:254 Yes it is, since the MOD_QUIESCE may unload the model.. or it may not. sys/sys/callout_test.h:1 Good catch I will fix it ;-) sys/sys/kern_testfrwk.h:43 That sounds fine.. I was just copying how the callout code did it (which was without typedef) but we can do that too ;-) REVISION DETAIL https://reviews.freebsd.org/D1711 To: rrs, gnn, rwatson, imp, adrian, sbruno, lstewart, hselasky Cc: neel, erj, freebsd-net _______________________________________________ freebsd-net@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-net To unsubscribe, send any mail to "freebsd-net-unsubscr...@freebsd.org"
