Date: Sun, 29 Jan 2017 10:36:43 +0000 (UTC) From: Konstantin Belousov <kib@FreeBSD.org> To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-10@freebsd.org Subject: svn commit: r312946 - stable/10/sys/kern Message-ID: <201701291036.v0TAahwa014497@repo.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: kib Date: Sun Jan 29 10:36:43 2017 New Revision: 312946 URL: https://svnweb.freebsd.org/changeset/base/312946 Log: MFC r312647: Add comments explaining unobvious td_critnest adjustments in critical_exit(). Modified: stable/10/sys/kern/kern_switch.c Directory Properties: stable/10/ (props changed) Modified: stable/10/sys/kern/kern_switch.c ============================================================================== --- stable/10/sys/kern/kern_switch.c Sun Jan 29 10:33:42 2017 (r312945) +++ stable/10/sys/kern/kern_switch.c Sun Jan 29 10:36:43 2017 (r312946) @@ -206,7 +206,22 @@ critical_exit(void) if (td->td_critnest == 1) { td->td_critnest = 0; + + /* + * Interrupt handlers execute critical_exit() on + * leave, and td_owepreempt may be left set by an + * interrupt handler only when td_critnest > 0. If we + * are decrementing td_critnest from 1 to 0, read + * td_owepreempt after decrementing, to not miss the + * preempt. Disallow compiler to reorder operations. + */ + __compiler_membar(); if (td->td_owepreempt && !kdb_active) { + /* + * Microoptimization: we committed to switch, + * disable preemption in interrupt handlers + * while spinning for the thread lock. + */ td->td_critnest = 1; thread_lock(td); td->td_critnest--;
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201701291036.v0TAahwa014497>