Date: Tue, 03 Sep 2019 14:06:38 -0000 From: Hans Petter Selasky <hselasky@FreeBSD.org> To: src-committers@freebsd.org, svn-src-projects@freebsd.org Subject: svn commit: r346091 - projects/hps_callouts/sys/net Message-ID: <201904101812.x3AICVnC076372@repo.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: hselasky Date: Wed Apr 10 18:12:31 2019 New Revision: 346091 URL: https://svnweb.freebsd.org/changeset/base/346091 Log: Callout drain does not have to be followed by a callout stop call. Fix bogus code. Sponsored by: Mellanox Technologies Modified: projects/hps_callouts/sys/net/if_spppsubr.c Modified: projects/hps_callouts/sys/net/if_spppsubr.c ============================================================================== --- projects/hps_callouts/sys/net/if_spppsubr.c Wed Apr 10 17:56:44 2019 (r346090) +++ projects/hps_callouts/sys/net/if_spppsubr.c Wed Apr 10 18:12:31 2019 (r346091) @@ -1060,15 +1060,13 @@ sppp_detach(struct ifnet *ifp) KASSERT(mtx_initialized(&sp->mtx), ("sppp mutex is not initialized")); /* Stop keepalive handler. */ - if (!callout_drain(&sp->keepalive_callout)) - callout_stop(&sp->keepalive_callout); + callout_drain(&sp->keepalive_callout); for (i = 0; i < IDX_COUNT; i++) { - if (!callout_drain(&sp->ch[i])) - callout_stop(&sp->ch[i]); + callout_drain(&sp->ch[i]); } - if (!callout_drain(&sp->pap_my_to_ch)) - callout_stop(&sp->pap_my_to_ch); + callout_drain(&sp->pap_my_to_ch); + mtx_destroy(&sp->pp_cpq.ifq_mtx); mtx_destroy(&sp->pp_fastq.ifq_mtx); mtx_destroy(&sp->mtx);
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201904101812.x3AICVnC076372>