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 kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4A003D6204E for ; Tue, 19 Nov 2024 09:16:43 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 9DB2B6B0088; Tue, 19 Nov 2024 04:16:42 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 962A66B0089; Tue, 19 Nov 2024 04:16:42 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 78EE16B008A; Tue, 19 Nov 2024 04:16:42 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0010.hostedemail.com [216.40.44.10]) by kanga.kvack.org (Postfix) with ESMTP id 5060D6B0088 for ; Tue, 19 Nov 2024 04:16:42 -0500 (EST) Received: from smtpin11.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay08.hostedemail.com (Postfix) with ESMTP id 3AAA7140305 for ; Tue, 19 Nov 2024 09:10:00 +0000 (UTC) X-FDA: 82802271636.11.E0A308A Received: from smtp-out1.suse.de (smtp-out1.suse.de [195.135.223.130]) by imf20.hostedemail.com (Postfix) with ESMTP id 752DD1C0011 for ; Tue, 19 Nov 2024 09:08:55 +0000 (UTC) Authentication-Results: imf20.hostedemail.com; dkim=pass header.d=suse.cz header.s=susede2_rsa header.b=SIRiPqoh; dkim=pass header.d=suse.cz header.s=susede2_ed25519 header.b=eLQee8Kl; dkim=pass header.d=suse.cz header.s=susede2_rsa header.b=SIRiPqoh; dkim=pass header.d=suse.cz header.s=susede2_ed25519 header.b=eLQee8Kl; spf=pass (imf20.hostedemail.com: domain of mbenes@suse.cz designates 195.135.223.130 as permitted sender) smtp.mailfrom=mbenes@suse.cz; dmarc=none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1732007217; h=from:from:sender: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: in-reply-to:in-reply-to:references:references:dkim-signature; bh=d3ZUiM0v/u9S5PwB6Ydqo1vhiAlHB49FGA/7ikHpEDE=; b=h4mPTeCCm/oCrs1neCrFFlagzQ5o6wQT6owuD36Y2iPmd3UrR6QYj2Uhg9VFU4plxBp56v vPRZxa+751dRtoKghPxg5zUSLnhJcALoDUGQpZuSyUm3il7np7vGnZ3sYcSoAitDPUWC6X u4uFP1xmEY6+Zg0lTBOmvOYWOt9mqP0= ARC-Authentication-Results: i=1; imf20.hostedemail.com; dkim=pass header.d=suse.cz header.s=susede2_rsa header.b=SIRiPqoh; dkim=pass header.d=suse.cz header.s=susede2_ed25519 header.b=eLQee8Kl; dkim=pass header.d=suse.cz header.s=susede2_rsa header.b=SIRiPqoh; dkim=pass header.d=suse.cz header.s=susede2_ed25519 header.b=eLQee8Kl; spf=pass (imf20.hostedemail.com: domain of mbenes@suse.cz designates 195.135.223.130 as permitted sender) smtp.mailfrom=mbenes@suse.cz; dmarc=none ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1732007217; a=rsa-sha256; cv=none; b=FrfVTkLEicaR+mAo9CcwVIAR6otRHWb+WVUumtcg1c4+tn1e5k4dzkJXE3smihb3gHxzJU 7PlhDncp1CuD3HaOasSk4cVMHG2HBQWvXgsSI9HoqL70Foaz/YqQmce1rs06ltjYPHAUG7 Q+Zp4Ezv+pFETtZkY/u237QQK6XynMk= Received: from pobox.suse.cz (unknown [10.100.2.14]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp-out1.suse.de (Postfix) with ESMTPS id D475321901; Tue, 19 Nov 2024 09:09:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.cz; s=susede2_rsa; t=1732007396; h=from:from:reply-to: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=d3ZUiM0v/u9S5PwB6Ydqo1vhiAlHB49FGA/7ikHpEDE=; b=SIRiPqohnfZsfiR8rWfugglV6kBt9WZNkpsbbr+OOaRL3B/gLbNYdP82bjB0S/Sjpyp4Dh 6DaUo1iD8tvsOcOGm8MLnAkvlYyhNoN/BKk+Nu5/yLAfPrF9/Eu//yrmX3tKwCADYf1zxx 7juTShWa4+8m5I6ysJKabU6DZBznQQA= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.cz; s=susede2_ed25519; t=1732007396; h=from:from:reply-to: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=d3ZUiM0v/u9S5PwB6Ydqo1vhiAlHB49FGA/7ikHpEDE=; b=eLQee8Kl//Z+uAJMSQ8O/3uMhn0cIi08Nnfcslit1gjRhYGbaqZ6swdcVqziESyeoUyf0K N0AZng7gJdnO9PAg== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.cz; s=susede2_rsa; t=1732007396; h=from:from:reply-to: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=d3ZUiM0v/u9S5PwB6Ydqo1vhiAlHB49FGA/7ikHpEDE=; b=SIRiPqohnfZsfiR8rWfugglV6kBt9WZNkpsbbr+OOaRL3B/gLbNYdP82bjB0S/Sjpyp4Dh 6DaUo1iD8tvsOcOGm8MLnAkvlYyhNoN/BKk+Nu5/yLAfPrF9/Eu//yrmX3tKwCADYf1zxx 7juTShWa4+8m5I6ysJKabU6DZBznQQA= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.cz; s=susede2_ed25519; t=1732007396; h=from:from:reply-to: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=d3ZUiM0v/u9S5PwB6Ydqo1vhiAlHB49FGA/7ikHpEDE=; b=eLQee8Kl//Z+uAJMSQ8O/3uMhn0cIi08Nnfcslit1gjRhYGbaqZ6swdcVqziESyeoUyf0K N0AZng7gJdnO9PAg== Date: Tue, 19 Nov 2024 10:09:55 +0100 (CET) From: Miroslav Benes To: Petr Mladek cc: Easwar Hariharan , Christophe Leroy , Pablo Neira Ayuso , Jozsef Kadlecsik , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Simon Horman , Julia Lawall , Nicolas Palix , Daniel Mack , Haojian Zhuang , Robert Jarzmik , Russell King , Heiko Carstens , Vasily Gorbik , Alexander Gordeev , Christian Borntraeger , Sven Schnelle , Ofir Bitton , Oded Gabbay , Lucas De Marchi , =?ISO-8859-15?Q?Thomas_Hellstr=F6m?= , Rodrigo Vivi , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Jeroen de Borst , Praveen Kaligineedi , Shailend Chand , Andrew Lunn , James Smart , Dick Kennedy , "James E.J. Bottomley" , "Martin K. Petersen" , =?ISO-8859-15?Q?Roger_Pau_Monn=E9?= , Jens Axboe , Kalle Valo , Jeff Johnson , Catalin Marinas , Andrew Morton , Jack Wang , Marcel Holtmann , Johan Hedberg , Luiz Augusto von Dentz , Greg Kroah-Hartman , Florian Fainelli , Ray Jui , Scott Branden , Broadcom internal kernel review list , Xiubo Li , Ilya Dryomov , Josh Poimboeuf , Jiri Kosina , Joe Lawrence , Jaroslav Kysela , Takashi Iwai , Lucas Stach , Russell King , Christian Gmeiner , Louis Peens , Michael Ellerman , Nicholas Piggin , Naveen N Rao , Madhavan Srinivasan , netfilter-devel@vger.kernel.org, coreteam@netfilter.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, cocci@inria.fr, linux-arm-kernel@lists.infradead.org, linux-s390@vger.kernel.org, dri-devel@lists.freedesktop.org, intel-xe@lists.freedesktop.org, linux-scsi@vger.kernel.org, xen-devel@lists.xenproject.org, linux-block@vger.kernel.org, linux-wireless@vger.kernel.org, ath11k@lists.infradead.org, linux-mm@kvack.org, linux-bluetooth@vger.kernel.org, linux-staging@lists.linux.dev, linux-rpi-kernel@lists.infradead.org, ceph-devel@vger.kernel.org, live-patching@vger.kernel.org, linux-sound@vger.kernel.org, etnaviv@lists.freedesktop.org, oss-drivers@corigine.com, linuxppc-dev@lists.ozlabs.org, Anna-Maria Behnsen Subject: Re: [PATCH v2 19/21] livepatch: Convert timeouts to secs_to_jiffies() In-Reply-To: Message-ID: References: <20241115-converge-secs-to-jiffies-v2-0-911fb7595e79@linux.microsoft.com> <20241115-converge-secs-to-jiffies-v2-19-911fb7595e79@linux.microsoft.com> <718febc4-59ee-4701-ad62-8b7a8fa7a910@csgroup.eu> <96f3b51b-c28c-4ea8-b61e-a4982196215f@linux.microsoft.com> User-Agent: Alpine 2.21 (LSU 202 2017-01-01) MIME-Version: 1.0 Content-Type: multipart/mixed; BOUNDARY="1678380546-1247530484-1732007229=:15289" Content-ID: X-Rspamd-Server: rspam09 X-Rspamd-Queue-Id: 752DD1C0011 X-Stat-Signature: 11erhduzxjxtpxb7ej8bxza7o9ut1edd X-Rspam-User: X-HE-Tag: 1732007335-568340 X-HE-Meta: U2FsdGVkX1/q/I7H2+lveWAIFWuAXKLfmpjseEIqmkP7OoPRb/cdMi5EykW6AMTNB+cbZR0kfUXa8obC/Nm+ZjYcHL9qwqSdpsS25YR8x1r0DxyAueMRLzy3pHRbXfTsf5sshbOvmJYQpnDL/eQKD8xHrPD1KlbHGUiZmTlxEOakMqQt6ZUyr0ctl/kS3Ry/tvQrvFiv89CK6hoXaOA9a1Y7to8CB9cOfWRIVKAjyLwQTGh7bvSjoHsKW1ufgWzKSCi4XxhCrrkqE7LRn/vPeMpq8JY35f/6ur87I+3HpMOX/dG9tBm14Lni/iwye1NYoQY4cjvp34gtvqkYPbZ67e1zGuK9Nq4pCnrjo1qz4xKqgIJNHMBVaVa5DhjB/45YBJ4JySop27yLpwr4vLkWcTB3e4K8iEFf5jRaCVXTDzJChzDjvg2GI53/SVWWG9ctxaAJ4hsbCyrMosYNFrFPYX3wKT4kyZJ1Lg0MKdQtYvBvm6KEmtrCRQbm7epK8qjwt/nHE31opntW2JkhwM9Kjhj0MOADR9tn4post1dxYOwFrIvJFIY/Kh0/ny455fJZ1vchMHNLzfaWN0DpAPlhvsLHhpKnkVknsWkBV38evmR0O0CjQKqadk6DXBjZFpB3iIWkweWQVo8sRvMuKHw7rQW4vvOWLVlZjbEOjqw80qYks9jqGEJrUHvCAR3HF1Jdd3G1ScIgVsEqdSqTBkKRvmhSXp0mA8oqcA71+1btNMkQ13OhS0d+EjsI8AyG8M0ZzGolTiV5/pt6+yGq9w8v3NBZWc0X+DE8ofTH0VYmu73B2NaaVdU2J02WnTeFCsc1WDBrNjgRSg6yPluYt23BYtuJVhUMJfOgFrDquvS6IKgSmh9YkIol6Dq5a13S/jmhv3UseOh23zVSHo+nkZjRR8ydxFuAq6+1v1fKIWdLNHY8nQBT92jr5o4xDXFs7lOyU9eD5CKjcKmcAavEDTS WQPSFQ5a 3ky2CauAwxa1uIk5p+DWPWQb9kNunDR6RdygDFSQH9cX57gEcHdvH1ItiSY7ioYL8fZyk7LQDz563u7xeJlwlz7Xi9CFf0P9O9qMgviRtRpzHsTbdnpSG/kSOVUmlgcqmHDeorv6G3tfnPKTkXXR7k1bdt+jn/xzyTuDyGWBAsHK2L2tvXn/RynDl3nvMj6Ut3rxi29milRNllfKCXXw+NaB3VUhlTKd/g7qPuAbGaChCPOAgnBtDhZbteGmieJ5ksbU2KG1tS9CzUiGzDdM7RlIhEnoSdHYzGKijMP1biRzuZzck7UWow2CMuUKPrWl0l/fW68ptTX7SPbzeH/adHGNBrZ7BLCOJaUoWumXIzhvxnTAoofumQqkbJTyJaZ/bRZ9Q95FkURxF/9Yf2HvQxEpD5lPu+0P6wBKHqFw+TEn6aL8ZaNDwnxerBkwyJZjMWAnwGbn3rmEA2PK5hwi0LitQbqtrlXZtv4+5nQXWU4dmMFPMubhpmGmy1RggeHUV7Y2r X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. --1678380546-1247530484-1732007229=:15289 Content-Type: text/plain; CHARSET=ISO-8859-15 Content-Transfer-Encoding: 8BIT Content-ID: On Tue, 19 Nov 2024, Petr Mladek wrote: > On Mon 2024-11-18 10:18:49, Easwar Hariharan wrote: > > On 11/18/2024 3:06 AM, Petr Mladek wrote: > > > On Sat 2024-11-16 11:10:52, Christophe Leroy wrote: > > >> > > >> > > >> Le 15/11/2024 à 22:26, Easwar Hariharan a écrit : > > >>> [Vous ne recevez pas souvent de courriers de eahariha@linux.microsoft.com. Découvrez pourquoi ceci est important à https://aka.ms/LearnAboutSenderIdentification ] > > >>> > > >>> Changes made with the following Coccinelle rules: > > >>> > > >>> @@ constant C; @@ > > >>> > > >>> - msecs_to_jiffies(C * 1000) > > >>> + secs_to_jiffies(C) > > >>> > > >>> @@ constant C; @@ > > >>> > > >>> - msecs_to_jiffies(C * MSEC_PER_SEC) > > >>> + secs_to_jiffies(C) > > >>> > > >>> Signed-off-by: Easwar Hariharan > > >>> --- > > >>> samples/livepatch/livepatch-callbacks-busymod.c | 2 +- > > >>> samples/livepatch/livepatch-shadow-fix1.c | 2 +- > > >>> samples/livepatch/livepatch-shadow-mod.c | 10 +++++----- > > >>> 3 files changed, 7 insertions(+), 7 deletions(-) > > >>> > > >>> diff --git a/samples/livepatch/livepatch-callbacks-busymod.c b/samples/livepatch/livepatch-callbacks-busymod.c > > >>> index 378e2d40271a9717d09eff51d3d3612c679736fc..d0fd801a7c21b7d7939c29d83f9d993badcc9aba 100644 > > >>> --- a/samples/livepatch/livepatch-callbacks-busymod.c > > >>> +++ b/samples/livepatch/livepatch-callbacks-busymod.c > > >>> @@ -45,7 +45,7 @@ static int livepatch_callbacks_mod_init(void) > > >>> { > > >>> pr_info("%s\n", __func__); > > >>> schedule_delayed_work(&work, > > >>> - msecs_to_jiffies(1000 * 0)); > > >>> + secs_to_jiffies(0)); > > >> > > >> Using secs_to_jiffies() is pointless, 0 is universal, should become > > >> schedule_delayed_work(&work, 0); > > > > > > Yes, schedule_delayed_work(&work, 0) looks like the right solution. > > > > > > Or even better, it seems that the delayed work might get replaced by > > > a normal workqueue work. > > > > > > Anyway, I am working on a patchset which would remove this sample > > > module. There is no need to put much effort into the clean up > > > of this particular module. Do whatever is easiest for you. > > > > > > Best Regards, > > > Petr > > > > If we're removing the module, I'll drop it from the series. Just to > > clarify, do you mean to remove all of samples/livepatch/* or some > > particular file(s)? > > To be precise, I am going to replace: > > samples/livepatch/livepatch-callbacks-demo.c > samples/livepatch/livepatch-callbacks-mod.c > samples/livepatch/livepatch-callbacks-busymod.c > > with a completely different modules because I am reworking the > callbacks API. > > All other sample modules are going to stay. > > Feel free to remove livepatch-callbacks-busymod.c from the patchset. > But also feel free to keep it. The API rework goes slowly. I am not > sure if it would be ready for 6.14. I would propose that Easwar goes on with his work and prepares an updated version of the patch based on Christophe's feedback. That is, disregarding Petr's rework for now. The patch set has a higher chance to be merged sooner. Petr can then easily rebase. If there is a conflict, we will handle it as usual. What do you think? Miroslav --1678380546-1247530484-1732007229=:15289--