Thread (59 messages) 59 messages, 17 authors, 2024-12-09

Re: [PATCH v2 19/21] livepatch: Convert timeouts to secs_to_jiffies()

From: zhang warden <hidden>
Date: 2024-11-19 01:44:13
Also in: ath11k, ceph-devel, dri-devel, intel-xe, linux-block, linux-bluetooth, linux-mm, linux-s390, linux-scsi, linux-sound, linux-wireless, linuxppc-dev, live-patching, lkml, netdev, netfilter-devel, xen-devel

On Nov 19, 2024, at 02:18, Easwar Hariharan [off-list ref] wrote:

On 11/18/2024 3:06 AM, Petr Mladek wrote:
quoted
On Sat 2024-11-16 11:10:52, Christophe Leroy wrote:
quoted

Le 15/11/2024 à 22:26, Easwar Hariharan a écrit :
quoted
[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 <redacted>
---
 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)?

Thanks,
Easwar
Hi Easwar!

I think Petr is going to remove just this module away. Anyway, I don't think
removing all of them is a good idea.

Thanks.
Wardenjohn. 
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help