From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 61210C4332F for ; Wed, 13 Dec 2023 02:28:13 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1378303AbjLMC2F (ORCPT ); Tue, 12 Dec 2023 21:28:05 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40540 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232721AbjLMC2D (ORCPT ); Tue, 12 Dec 2023 21:28:03 -0500 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5FDC7A0 for ; Tue, 12 Dec 2023 18:28:09 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1702434488; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=q7ogPeVXGwrtgOAoSCz/LwV9L8KL6rZODfc0Bz4jg5I=; b=MaTpY4hWQVDu012XjFagR7j3NRuJR7Us8PGRA89qj37v4KSFCZ913lIsb0YpMxgyAO4VMI aVPlcjcFdb6Ho8qZeL1LYgJRzeUuFBrb1MQTywX1yEQpIQzWTVesEmny2pVSrqPl6T07Hj JUbzeiypVFHUkSRWirWnwYOtofD9bhY= Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-38-XpbZrWEdMHG6dIbEnA-3Kw-1; Tue, 12 Dec 2023 21:28:06 -0500 X-MC-Unique: XpbZrWEdMHG6dIbEnA-3Kw-1 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.rdu2.redhat.com [10.11.54.7]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id C1D9D87B2A1; Wed, 13 Dec 2023 02:28:05 +0000 (UTC) Received: from [10.22.16.51] (unknown [10.22.16.51]) by smtp.corp.redhat.com (Postfix) with ESMTP id 81AB51C060AF; Wed, 13 Dec 2023 02:28:04 +0000 (UTC) Message-ID: Date: Tue, 12 Dec 2023 21:28:04 -0500 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: Modifying isolcpus, nohz_full, and rcu_nocb kernel parameters at runtime Content-Language: en-US To: Frederic Weisbecker Cc: Gianfranco Dutka , Tejun Heo , Vincent Guittot , linux-kernel@vger.kernel.org, vincent.guittot@linaro.com, Ingo Molnar , Peter Zijlstra , Juri Lelli , Dietmar Eggemann , Steven Rostedt , Ben Segall , Mel Gorman , Daniel Bristot de Oliveira , Valentin Schneider , "Pandruvada, Srinivas" , Phil Auld , Cestmir Kalina References: <76587DD3-2A77-41A3-9807-6AEE4398EBA6@arista.com> <25E6E1E4-DC16-490E-B907-A3236FB9317A@arista.com> <7e3bf653-d3ea-48b0-b808-d92a3c5f2c5b@redhat.com> From: Waiman Long In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Scanned-By: MIMEDefang 3.4.1 on 10.11.54.7 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 12/12/23 18:57, Frederic Weisbecker wrote: > On Tue, Dec 12, 2023 at 03:18:43PM -0500, Waiman Long wrote: >> On 12/12/23 08:27, Frederic Weisbecker wrote: >>> On Fri, Dec 08, 2023 at 09:18:53AM -0500, Gianfranco Dutka wrote: >>>>> The isolcpus, nohz_full and rcu_nocbs are boot-time kernel parameters. I am in the process of improving dynamic CPU isolation at runtime. Right now, we are able to do isolcpus=domain with the isolated cpuset partition functionality. Other aspects of CPU isolation are being looked at with the goal of reducing the gap of what one can do at boot time versus what can be done at run time. It will certain take time to reach that goal. >>>>> >>>>> Cheers, >>>>> Longman >>>>> >>>> Thank you Waiman for the response. It would seem that getting similar >>>> functionality through cgroups/cpusets is the only option at the moment. Is it >>>> completely out of the question to possibly patch the kernel to modify these >>>> parameters at runtime? Or would that entail a significant change that might >>>> not be so trivial to accomplish? For instance, the solution wouldn’t be as >>>> simple as patching the kernel to make these writeable and then calling the >>>> same functions which run at boot-time when these parameters are originally >>>> written? >>> As for nohz_full (which implies rcu_nocb), it's certainly possible to make it >>> tunable at runtime via cpusets. If people really want it, I'm willing to help. >> As said by Phil, your help in in enabling dynamic rcu_nocb will be greatly >> appreciated. > rcu_nocb is already ready for that. The not yet ready part is nohz_full and its > several components (tick, remote tick, [hr-]timers affinity, workqueues affinity, kthreads > affinity, vmstat, buffer head, etc...). Last debate on plumbers suggested that > nohz_full should be dynamically turned on/off only on offline CPUs. That will > indeed simplify the problem. So rcu_nocb is ready for dynamically changing it without too much additional work. That is good to know as I haven't looked into that myself. The other pieces will still need additional work. I already have a patch in the cgroup tree that updates the unbound workqueue affinity to exclude isolated cpuset CPUs, though there may still be some further fine tuning that can be done. > >> My current thought is to have a root level >> cpuset.cpus.isolation_control file to enable additional CPU isolation like >> rcu_nocb to be applied to CPUs in isolated partitions. > Last time I tried that, Peter Zijlstra was more in favour of an isolate all or nothing > switch by default for nohz_full that would include rcu_nocb. And then if people > are interested in something more finegrained, introduce such a file to control > individual features (see > https://lore.kernel.org/lkml/YpIwsiaY2IPK96WO@hirez.programming.kicks-ass.net/ ) > > But so far I never heard about the need for such a finegrained isolation. Users of > nohz_full= seem to want to isolate everything out. Yes, I recall some of the discussion now. I am fine with a single on/off switch. That will likely simplify the process as we can add additional isolation features over time once the code is ready, may be a cpuset.cpus.isolation_full boolean flag. Cheers, Longman > > Thanks. >