|
From: | Richard Henderson |
Subject: | Re: [PATCH v3 2/2] accel/tcg: Register a force_rcu notifier |
Date: | Tue, 9 Nov 2021 09:21:27 +0100 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 |
On 11/9/21 8:54 AM, Richard Henderson wrote:
On 11/8/21 12:33 PM, Greg Kurz wrote:+static void rr_force_rcu(Notifier *notify, void *data) +{ + /* + * Called with rcu_registry_lock held, using async_run_on_cpu() ensures + * that there are no deadlocks. + */ + async_run_on_cpu(first_cpu, do_nothing, RUN_ON_CPU_NULL); +}Should first_cpu really be rr_current_cpu? It's not clear to me that this will work for -smp 2 -cpu thread=single.
Alternately, no async_run_on_cpu at all, just rr_kick_next_cpu(). r~
[Prev in Thread] | Current Thread | [Next in Thread] |