Received: by 2002:a05:7412:8d10:b0:f3:1519:9f41 with SMTP id bj16csp849566rdb; Wed, 6 Dec 2023 01:25:03 -0800 (PST) X-Google-Smtp-Source: AGHT+IF3KcP45AbLYtupepsE9xmNnfu27d0Q24zEu0qcozkRDCFMFmOwU5V94qgKoDsT9GRWFnmE X-Received: by 2002:a05:6a00:1402:b0:6ce:8180:158d with SMTP id l2-20020a056a00140200b006ce8180158dmr431124pfu.62.1701854703286; Wed, 06 Dec 2023 01:25:03 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1701854703; cv=none; d=google.com; s=arc-20160816; b=DEmzaJpHDnQ9yoeJPRUd6On2jq29gDguIxtCgCK9kGNcbyndzo9c97650+WWxvGpe6 RAtPWP28BTeLbY2c8n4GeeZc/Z1vOoA57VBN6CjLhqeXDQtto8LoTJedK4I2i8kiRnQP FIOHQvCB6DXMVyWismGHx0odbtkKhVTHdosDTfJnXkiQYh/eOaS4R/9wOzY3PyIDVSV+ 4Tc6Tfy/iF75/Byf8cp5Jfub/TJVLDZU0+pRXapm/xqdDz96keqJjHcBatYOgMWAKTb6 DoyzV77oE4/yFpgMrLl7OP4Nxp7gYp2LpxU9u6UxWss7mwspHXhSKt1V6ijbH09+acb+ QJWA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:message-id:date:references :in-reply-to:subject:cc:to:dkim-signature:dkim-signature:from; bh=93v4gttHfsOj4z7ueaDWrNz1D5HKd3pZTasSdZAFavo=; fh=zngDPpHc9IOT4XyfACYkaIQBkMO1o2QUewR15pJChdQ=; b=eVXQisa5ffrX/um/CwRc/SZC7FMSBeQaAeKa5wgYpgcpWS3dyqbcMAJZuqXO/daKQD Vs5Z63Ldtvqau98cCEDKCtUin5YBwL9/X9qbUs+e+iDq2OksB2FkLWb07k3uf0nSi53B LGWBb43HeaFJGfzw/mXgKVlUotHBZvWDWufrlXSxwCidx6HiyJ9dLSVGqv9uQ9UggECa WEhPyCe707/4NTi9G0M3R31NIcyx1aDnNVoFBFpdd8LJUoK/lQJSvsItV2GBaq6j/PhN Yt2v0UBZD+QunscQRTcmKE7wzdvcIO5duoCQ1aHfUl8Hq5Dwul6ppTZp4dZK2raUhVnl rsYA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linutronix.de header.s=2020 header.b=v209QlOl; dkim=neutral (no key) header.i=@linutronix.de header.s=2020e; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.34 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=linutronix.de Return-Path: Received: from howler.vger.email (howler.vger.email. [23.128.96.34]) by mx.google.com with ESMTPS id l6-20020a056a00140600b006cb63513372si10988006pfu.12.2023.12.06.01.25.02 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 06 Dec 2023 01:25:03 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.34 as permitted sender) client-ip=23.128.96.34; Authentication-Results: mx.google.com; dkim=pass header.i=@linutronix.de header.s=2020 header.b=v209QlOl; dkim=neutral (no key) header.i=@linutronix.de header.s=2020e; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.34 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=linutronix.de Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by howler.vger.email (Postfix) with ESMTP id 9ECEF82DFD30; Wed, 6 Dec 2023 01:25:00 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at howler.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1346732AbjLFJYo (ORCPT + 99 others); Wed, 6 Dec 2023 04:24:44 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37770 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1346728AbjLFJYn (ORCPT ); Wed, 6 Dec 2023 04:24:43 -0500 Received: from galois.linutronix.de (Galois.linutronix.de [IPv6:2a0a:51c0:0:12e:550::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 07F77D49 for ; Wed, 6 Dec 2023 01:24:48 -0800 (PST) From: Anna-Maria Behnsen DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020; t=1701854686; 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: in-reply-to:in-reply-to:references:references; bh=93v4gttHfsOj4z7ueaDWrNz1D5HKd3pZTasSdZAFavo=; b=v209QlOlMT9g82bMBVx7adDQvqsF6KeALigEAbFmYNlflxkBRwMeEljUW8D39KRfA2nrkG BY7ExinGmsxF0bX5AZCPxCh072aDnzUEztBs0tLWUNof2arEHm6i6fV4enA81n74+JusXe qJ6Z8hh+fYe/rZA658f8Xm/3m+LnxvYqZdDxc9ggTRZwd4WmIiCtSLKJgkPZleHWtV4qua px2YiPT+unZ2u4d4Zvf/NNSeGIAZ9EHIAn3wfuuXI5lzAYJagTUZpFbt/+UF7f/Q2PBHtX uY1O722r5MMGInTm9N94lrnSE6c0KQbiLceviFnKG7nKHmq1yFgIzW9J8rwYfA== DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020e; t=1701854686; 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: in-reply-to:in-reply-to:references:references; bh=93v4gttHfsOj4z7ueaDWrNz1D5HKd3pZTasSdZAFavo=; b=r7SKlq9CJN8pk20mL/YsZXgad23ibFNr9vRc0T7/dZrfYqAEvM21rwHYg42JOepCCxUrFQ HacYlD2hWkIac6AA== To: Sebastian Siewior Cc: linux-kernel@vger.kernel.org, Peter Zijlstra , John Stultz , Thomas Gleixner , Eric Dumazet , "Rafael J . Wysocki" , Arjan van de Ven , "Paul E . McKenney" , Frederic Weisbecker , Rik van Riel , Steven Rostedt , Giovanni Gherdovich , Lukasz Luba , "Gautham R . Shenoy" , Srinivas Pandruvada , K Prateek Nayak Subject: Re: [PATCH v9 17/32] timers: Introduce add_timer() variants which modify timer flags In-Reply-To: <20231205182835.DNCNokiI@linutronix.de> References: <20231201092654.34614-1-anna-maria@linutronix.de> <20231201092654.34614-18-anna-maria@linutronix.de> <20231205182835.DNCNokiI@linutronix.de> Date: Wed, 06 Dec 2023 10:24:45 +0100 Message-ID: <87r0jzel9e.fsf@somnus> MIME-Version: 1.0 Content-Type: text/plain X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on howler.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (howler.vger.email [0.0.0.0]); Wed, 06 Dec 2023 01:25:00 -0800 (PST) Sebastian Siewior writes: > On 2023-12-01 10:26:39 [+0100], Anna-Maria Behnsen wrote: >> Timer might be used as pinned timer (using add_timer_on()) and later on as >> non pinned timers using add_timer(). When the NOHZ timer pull at expiry >> model is in place, TIMER_PINNED flag is required to be used whenever a >> timer needs to expire on a dedicated CPU. Flag must no be set, if >> expiration on a dedicated CPU is not required. > > Slightly reworded. > > | A timer might be used as a pinned timer (using add_timer_on()) and later > | on as non-pinned timer using add_timer(). When the "NOHZ timer pull at > | expiry model" is in place, the TIMER_PINNED flag is required to be used > | whenever a timer needs to expire on a dedicated CPU. Otherwise the flag > | must not be set if expiration on a dedicated CPU is not required. > >> add_timer_on()'s behavior will be changed during the preparation patches >> for the NOHZ timer pull at expiry model to unconditionally set TIMER_PINNED >> flag. To be able to reset/set the flag when queueing a timer, two variants >> of add_timer() are introduced. > > and here. > > | add_timer_on()'s behavior will be changed during the preparation patches > | for the "NOHZ timer pull at expiry model" to unconditionally set > | TIMER_PINNED flag. To be able to clear/ set the flag when queueing a > | timer, two variants of add_timer() are introduced. > > I let you be judge of this. I will take your reworded version. Thanks, Anna-Maria