summaryrefslogtreecommitdiffstats
path: root/lib/libc/stdlib/system.c
diff options
context:
space:
mode:
authorattilio <attilio@FreeBSD.org>2010-04-19 23:27:54 +0000
committerattilio <attilio@FreeBSD.org>2010-04-19 23:27:54 +0000
commitfca97c8d7ae8d2555f0a54225f8fd30cf0f63a21 (patch)
tree530d3837d4447fbbc961cd05737bf4aa42408c0e /lib/libc/stdlib/system.c
parent66db61c8c0e978eac926b1b47c6c81a108bdaa98 (diff)
downloadFreeBSD-src-fca97c8d7ae8d2555f0a54225f8fd30cf0f63a21.zip
FreeBSD-src-fca97c8d7ae8d2555f0a54225f8fd30cf0f63a21.tar.gz
Fix a deadlock in the shutdown code:
When performing a smp_rendezvous() or more likely, on amd64 and i386, a smp_tlb_shootdown() the caller will end up with the smp_ipi_mtx spinlock held, busy-waiting for other CPUs to acknowledge the operation. As long as CPUs are suspended (via cpu_reset()) between the active mask read and IPI sending there can be a deadlock where the caller will wait forever for a dead CPU to acknowledge the operation. Please note that on CPU0 that is going to be someway heavier because of the spinlocks being disabled earlier than quitting the machine. Fix this bug by calling cpu_reset() with the smp_ipi_mtx held. Note that it is very likely that a saner offline/online CPUs mechanism will help heavilly in fixing similar cases as it is likely more bugs of this type may arise in the future. Reported by: rwatson Discussed with: jhb Tested by: rnoland, Giovanni Trematerra <giovanni dot trematerra at gmail dot com> MFC: 2 weeks Special deciation to: anyone who made possible to have 16-ways machines in Netperf
Diffstat (limited to 'lib/libc/stdlib/system.c')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud