]> git.baikalelectronics.ru Git - kernel.git/commit
Correct the race condition in aarch64_insn_patch_text_sync()
authorWilliam Cohen <wcohen@redhat.com>
Tue, 11 Nov 2014 14:41:27 +0000 (09:41 -0500)
committerCatalin Marinas <catalin.marinas@arm.com>
Thu, 13 Nov 2014 15:21:39 +0000 (15:21 +0000)
commitfe503364b280f74717a9b98f45575a28dac5ed14
tree16101db15930a5a2c1ce7bd822d02f4f498582c6
parent978b2675a66077027ef73b3c3c5a91238b729932
Correct the race condition in aarch64_insn_patch_text_sync()

When experimenting with patches to provide kprobes support for aarch64
smp machines would hang when inserting breakpoints into kernel code.
The hangs were caused by a race condition in the code called by
aarch64_insn_patch_text_sync().  The first processor in the
aarch64_insn_patch_text_cb() function would patch the code while other
processors were still entering the function and incrementing the
cpu_count field.  This resulted in some processors never observing the
exit condition and exiting the function.  Thus, processors in the
system hung.

The first processor to enter the patching function performs the
patching and signals that the patching is complete with an increment
of the cpu_count field. When all the processors have incremented the
cpu_count field the cpu_count will be num_cpus_online()+1 and they
will return to normal execution.

Fixes: 732fbe69d571 arm64: introduce interfaces to hotpatch kernel and module code
Signed-off-by: William Cohen <wcohen@redhat.com>
Acked-by: Will Deacon <will.deacon@arm.com>
Cc: <stable@vger.kernel.org>
Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
arch/arm64/kernel/insn.c