linux-toradex.git/scripts/coccinelle/misc, branch v6.16

Merge tag 'mm-nonmm-stable-2025-03-30-18-23' of git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm

2025-04-01T17:06:52+00:00

Pull non-MM updates from Andrew Morton:

 - The series "powerpc/crash: use generic crashkernel reservation" from
   Sourabh Jain changes powerpc's kexec code to use more of the generic
   layers.

 - The series "get_maintainer: report subsystem status separately" from
   Vlastimil Babka makes some long-requested improvements to the
   get_maintainer output.

 - The series "ucount: Simplify refcounting with rcuref_t" from
   Sebastian Siewior cleans up and optimizing the refcounting in the
   ucount code.

 - The series "reboot: support runtime configuration of emergency
   hw_protection action" from Ahmad Fatoum improves the ability for a
   driver to perform an emergency system shutdown or reboot.

 - The series "Converge on using secs_to_jiffies() part two" from Easwar
   Hariharan performs further migrations from msecs_to_jiffies() to
   secs_to_jiffies().

 - The series "lib/interval_tree: add some test cases and cleanup" from
   Wei Yang permits more userspace testing of kernel library code, adds
   some more tests and performs some cleanups.

 - The series "hung_task: Dump the blocking task stacktrace" from Masami
   Hiramatsu arranges for the hung_task detector to dump the stack of
   the blocking task and not just that of the blocked task.

 - The series "resource: Split and use DEFINE_RES*() macros" from Andy
   Shevchenko provides some cleanups to the resource definition macros.

 - Plus the usual shower of singleton patches - please see the
   individual changelogs for details.

* tag 'mm-nonmm-stable-2025-03-30-18-23' of git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm: (77 commits)
  mailmap: consolidate email addresses of Alexander Sverdlin
  fs/procfs: fix the comment above proc_pid_wchan()
  relay: use kasprintf() instead of fixed buffer formatting
  resource: replace open coded variant of DEFINE_RES()
  resource: replace open coded variants of DEFINE_RES_*_NAMED()
  resource: replace open coded variant of DEFINE_RES_NAMED_DESC()
  resource: split DEFINE_RES_NAMED_DESC() out of DEFINE_RES_NAMED()
  samples: add hung_task detector mutex blocking sample
  hung_task: show the blocker task if the task is hung on mutex
  kexec_core: accept unaccepted kexec segments' destination addresses
  watchdog/perf: optimize bytes copied and remove manual NUL-termination
  lib/interval_tree: fix the comment of interval_tree_span_iter_next_gap()
  lib/interval_tree: skip the check before go to the right subtree
  lib/interval_tree: add test case for span iteration
  lib/interval_tree: add test case for interval_tree_iter_xxx() helpers
  lib/rbtree: add random seed
  lib/rbtree: split tests
  lib/rbtree: enable userland test suite for rbtree related data structure
  checkpatch: describe --min-conf-desc-length
  scripts/gdb/symbols: determine KASLR offset on s390
  ...

coccinelle: misc: secs_to_jiffies: Patch expressions too

2025-03-17T06:24:15+00:00

Patch series "Converge on using secs_to_jiffies() part two", v3.

This is the second series that converts users of msecs_to_jiffies() that
either use the multiply pattern of either of:
- msecs_to_jiffies(N*1000) or
- msecs_to_jiffies(N*MSEC_PER_SEC)

where N is a constant or an expression, to avoid the multiplication.

The conversion is made with Coccinelle with the secs_to_jiffies() script
in scripts/coccinelle/misc.  Attention is paid to what the best change can
be rather than restricting to what the tool provides.


This patch (of 16):

Teach the script to suggest conversions for timeout patterns where the
arguments to msecs_to_jiffies() are expressions as well.

Link: https://lkml.kernel.org/r/20250225-converge-secs-to-jiffies-part-two-v3-0-a43967e36c88@linux.microsoft.com
Link: https://lkml.kernel.org/r/20250225-converge-secs-to-jiffies-part-two-v3-1-a43967e36c88@linux.microsoft.com
Signed-off-by: Easwar Hariharan 
Cc: Mark Brown 
Cc: Carlos Maiolino 
Cc: Chris Mason 
Cc: Christoph Hellwig 
Cc: Damien Le Maol 
Cc: "Darrick J. Wong" 
Cc: David Sterba 
Cc: Dick Kennedy 
Cc: Dongsheng Yang 
Cc: Fabio Estevam 
Cc: Frank Li 
Cc: Hans de Goede 
Cc: Henrique de Moraes Holschuh 
Cc: Ilpo Jarvinen 
Cc: Ilya Dryomov 
Cc: James Bottomley 
Cc: James Smart 
Cc: Jaroslav Kysela 
Cc: Jason Gunthorpe 
Cc: Jens Axboe 
Cc: Josef Bacik 
Cc: Julia Lawall 
Cc: Kalesh Anakkur Purayil 
Cc: Keith Busch 
Cc: Leon Romanovsky 
Cc: "Martin K. Petersen" 
Cc: Nicolas Palix 
Cc: Niklas Cassel 
Cc: Oded Gabbay 
Cc: Takashi Iwai 
Cc: Sagi Grimberg 
Cc: Sascha Hauer 
Cc: Sebastian Reichel 
Cc: Selvin Thyparampil Xavier 
Cc: Shawn Guo 
Cc: Shyam-sundar S-k 
Cc: Takashi Iwai 
Cc: Xiubo Li 
Cc: Carlos Maiolino 
Cc: Marc Kleine-Budde 
Signed-off-by: Andrew Morton

coccinelle: Add missing (GE)NL_SET_ERR_MSG_* to strings ending with newline test

2025-02-28T02:11:37+00:00

Add missing (GE)NL_SET_ERR_MSG_*() variants to the list of macros
checked for strings ending with a newline.

Reviewed-by: Tariq Toukan 
Signed-off-by: Gal Pressman 
Reviewed-by: Michal Swiatkowski 
Link: https://patch.msgid.link/20250226093904.6632-2-gal@nvidia.com
Signed-off-by: Jakub Kicinski

coccinelle: misc: add secs_to_jiffies script

2025-01-13T04:21:01+00:00

This script finds and suggests conversions of timeout patterns that result
in seconds-denominated timeouts to use the new secs_to_jiffies() API in
include/linux/jiffies.h for better readability.

Link: https://lkml.kernel.org/r/20241210-converge-secs-to-jiffies-v3-2-ddfefd7e9f2a@linux.microsoft.com
Signed-off-by: Easwar Hariharan 
Suggested-by: Anna-Maria Behnsen 
Cc: Alexander Gordeev 
Cc: Andrew Lunn 
Cc: Catalin Marinas 
Cc: Christian Borntraeger 
Cc: Christophe Leroy 
Cc: Daniel Mack 
Cc: David Airlie 
Cc: David S. Miller 
Cc: Dick Kennedy 
Cc: Eric Dumazet 
Cc: Florian Fainelli 
Cc: Greg Kroah-Hartman 
Cc: Haojian Zhuang 
Cc: Heiko Carstens 
Cc: Ilya Dryomov 
Cc: Jack Wang 
Cc: Jakub Kicinski 
Cc: James Bottomley 
Cc: James Smart 
Cc: Jaroslav Kysela 
Cc: Jeff Johnson 
Cc: Jeff Johnson 
Cc: Jens Axboe 
Cc: Jeroen de Borst 
Cc: Jiri Kosina 
Cc: Joe Lawrence 
Cc: Johan Hedberg 
Cc: Josh Poimboeuf 
Cc: Jozsef Kadlecsik 
Cc: Julia Lawall 
Cc: Kalle Valo 
Cc: Louis Peens 
Cc: Lucas De Marchi 
Cc: Luiz Augusto von Dentz 
Cc: Maarten Lankhorst 
Cc: Madhavan Srinivasan 
Cc: Marcel Holtmann 
Cc: Martin K. Petersen 
Cc: Maxime Ripard 
Cc: Michael Ellerman 
Cc: Miroslav Benes 
Cc: Naveen N Rao 
Cc: Nicholas Piggin 
Cc: Nicolas Palix 
Cc: Oded Gabbay 
Cc: Ofir Bitton 
Cc: Pablo Neira Ayuso 
Cc: Paolo Abeni 
Cc: Petr Mladek 
Cc: Praveen Kaligineedi 
Cc: Ray Jui 
Cc: Robert Jarzmik 
Cc: Rodrigo Vivi 
Cc: Roger Pau Monné 
Cc: Russell King 
Cc: Scott Branden 
Cc: Shailend Chand 
Cc: Simona Vetter 
Cc: Simon Horman 
Cc: Sven Schnelle 
Cc: Takashi Iwai 
Cc: Thomas Hellström 
Cc: Thomas Zimmermann 
Cc: Vasily Gorbik 
Cc: Xiubo Li 
Signed-off-by: Andrew Morton

scripts/nsdeps: get 'make nsdeps' working again

2024-12-03T16:22:25+00:00

Since commit cdd30ebb1b9f ("module: Convert symbol namespace to string
literal"), when MODULE_IMPORT_NS() is missing, 'make nsdeps' inserts
pointless code:

    MODULE_IMPORT_NS("ns");

Here, "ns" is not a namespace, but the variable in the semantic patch.
It must not be quoted. Instead, a string literal must be passed to
Coccinelle.

Fixes: cdd30ebb1b9f ("module: Convert symbol namespace to string literal")
Signed-off-by: Masahiro Yamada 
Signed-off-by: Linus Torvalds

module: Convert symbol namespace to string literal

2024-12-02T19:34:44+00:00

Clean up the existing export namespace code along the same lines of
commit 33def8498fdd ("treewide: Convert macro and uses of __section(foo)
to __section("foo")") and for the same reason, it is not desired for the
namespace argument to be a macro expansion itself.

Scripted using

  git grep -l -e MODULE_IMPORT_NS -e EXPORT_SYMBOL_NS | while read file;
  do
    awk -i inplace '
      /^#define EXPORT_SYMBOL_NS/ {
        gsub(/__stringify\(ns\)/, "ns");
        print;
        next;
      }
      /^#define MODULE_IMPORT_NS/ {
        gsub(/__stringify\(ns\)/, "ns");
        print;
        next;
      }
      /MODULE_IMPORT_NS/ {
        $0 = gensub(/MODULE_IMPORT_NS\(([^)]*)\)/, "MODULE_IMPORT_NS(\"\\1\")", "g");
      }
      /EXPORT_SYMBOL_NS/ {
        if ($0 ~ /(EXPORT_SYMBOL_NS[^(]*)\(([^,]+),/) {
  	if ($0 !~ /(EXPORT_SYMBOL_NS[^(]*)\(([^,]+), ([^)]+)\)/ &&
  	    $0 !~ /(EXPORT_SYMBOL_NS[^(]*)\(\)/ &&
  	    $0 !~ /^my/) {
  	  getline line;
  	  gsub(/[[:space:]]*\\$/, "");
  	  gsub(/[[:space:]]/, "", line);
  	  $0 = $0 " " line;
  	}

  	$0 = gensub(/(EXPORT_SYMBOL_NS[^(]*)\(([^,]+), ([^)]+)\)/,
  		    "\\1(\\2, \"\\3\")", "g");
        }
      }
      { print }' $file;
  done

Requested-by: Masahiro Yamada 
Signed-off-by: Peter Zijlstra (Intel) 
Link: https://mail.google.com/mail/u/2/#inbox/FMfcgzQXKWgMmjdFwwdsfgxzKpVHWPlc
Acked-by: Greg KH 
Signed-off-by: Linus Torvalds

coccinelle: misc: minmax: Suppress reports for err returns

2024-05-20T19:31:20+00:00

Most of the people prefer:

return ret < 0 ? ret: 0;

than:

return min(ret, 0);

Let's tweak the cocci file to ignore those lines completely.

Signed-off-by: Ricardo Ribalda 
Signed-off-by: Julia Lawall

coccinelle: semantic patch to check for potential struct_size calls

2024-02-29T21:38:01+00:00

include/linux/overflow.h includes helper macros intended for calculating
sizes of allocations. These macros prevent accidental overflow by
saturating at SIZE_MAX.

In general when calculating such sizes use of the macros is preferred. Add
a semantic patch which can detect code patterns which can be replaced by
struct_size.

Note that I set the confidence to medium because this patch doesn't make an
attempt to ensure that the relevant array is actually a flexible array. The
struct_size macro does specifically require a flexible array. In many cases
the detected code could be refactored to a flexible array, but this is not
always possible (such as if there are multiple over-allocations).

Signed-off-by: Jacob Keller 
Link: https://lore.kernel.org/r/20230227202428.3657443-1-jacob.e.keller@intel.com
Signed-off-by: Kees Cook

update Coccinelle URL

2022-08-07T19:30:36+00:00

Signed-off-by: Julia Lawall

coccinelle: Remove script that checks replacing 0/1 with false/true in functions returning bool

2022-07-16T13:31:33+00:00

There is nothing wrong with current code that returns 0 or 1 for a
function returning bool. It is perfectly acceptable by the C standard.

To avoid churn of unwanted patches that are constantly sent to maintainers
who do not care about this change, remove the script that flags it as an
issue. This issue is not worth the burden on maintainers to accept
useless patches.

Link: https://lore.kernel.org/all/20220705073822.7276-1-jiapeng.chong@linux.alibaba.com/
Link: https://lore.kernel.org/all/20220429075201.68581-1-jiapeng.chong@linux.alibaba.com/
Link: https://lore.kernel.org/all/1649236467-29390-1-git-send-email-baihaowen@meizu.com/
Link: https://lore.kernel.org/all/20220317014740.3138-1-jiapeng.chong@linux.alibaba.com/
Link: https://lore.kernel.org/all/190b5c2f2f2fb9cc775fce8daed72bf893be48a4.1642065293.git.davidcomponentone@gmail.com/
Link: https://lore.kernel.org/all/20211214113845.439392-1-deng.changcheng@zte.com.cn/
Link: https://lore.kernel.org/all/20210824065735.60660-1-deng.changcheng@zte.com.cn/
Link: https://lore.kernel.org/all/20210824064305.60081-1-deng.changcheng@zte.com.cn/
Link: https://lore.kernel.org/all/20210824062359.59474-1-deng.changcheng@zte.com.cn/

Cc: Rasmus Villemoes 
Cc: Julia Lawall 
Cc: Michal Marek 
Cc: Peter Zijlstra 
Signed-off-by: Steven Rostedt (Google) 
Signed-off-by: Julia Lawall 
Acked-by: Rasmus Villemoes 
Acked-by: Paul Gortmaker