[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v8 3/8] checkpatch: add qemu_bh_new/aio_bh_new checks
From: |
Alexander Bulekov |
Subject: |
[PATCH v8 3/8] checkpatch: add qemu_bh_new/aio_bh_new checks |
Date: |
Fri, 21 Apr 2023 10:27:31 -0400 |
Advise authors to use the _guarded versions of the APIs, instead.
Reviewed-by: Darren Kenny <darren.kenny@oracle.com>
Signed-off-by: Alexander Bulekov <alxndr@bu.edu>
---
scripts/checkpatch.pl | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index d768171dcf..eeaec436eb 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -2865,6 +2865,14 @@ sub process {
if ($line =~ /\bsignal\s*\(/ && !($line =~ /SIG_(?:IGN|DFL)/)) {
ERROR("use sigaction to establish signal handlers;
signal is not portable\n" . $herecurr);
}
+# recommend qemu_bh_new_guarded instead of qemu_bh_new
+ if ($realfile =~ /.*\/hw\/.*/ && $line =~ /\bqemu_bh_new\s*\(/) {
+ ERROR("use qemu_bh_new_guarded() instead of
qemu_bh_new() to avoid reentrancy problems\n" . $herecurr);
+ }
+# recommend aio_bh_new_guarded instead of aio_bh_new
+ if ($realfile =~ /.*\/hw\/.*/ && $line =~ /\baio_bh_new\s*\(/) {
+ ERROR("use aio_bh_new_guarded() instead of aio_bh_new()
to avoid reentrancy problems\n" . $herecurr);
+ }
# check for module_init(), use category-specific init macros explicitly please
if ($line =~ /^module_init\s*\(/) {
ERROR("please use block_init(), type_init() etc.
instead of module_init()\n" . $herecurr);
--
2.39.0
- [PATCH v8 0/8] memory: prevent dma-reentracy issues, Alexander Bulekov, 2023/04/21
- [PATCH v8 1/8] memory: prevent dma-reentracy issues, Alexander Bulekov, 2023/04/21
- [PATCH v8 3/8] checkpatch: add qemu_bh_new/aio_bh_new checks,
Alexander Bulekov <=
- [PATCH v8 2/8] async: Add an optional reentrancy guard to the BH API, Alexander Bulekov, 2023/04/21
- [PATCH v8 5/8] memory: Allow disabling re-entrancy checking per-MR, Alexander Bulekov, 2023/04/21
- [PATCH v8 4/8] hw: replace most qemu_bh_new calls with qemu_bh_new_guarded, Alexander Bulekov, 2023/04/21
- [PATCH v8 6/8] lsi53c895a: disable reentrancy detection for script RAM, Alexander Bulekov, 2023/04/21
- [PATCH v8 7/8] bcm2835_property: disable reentrancy detection for iomem, Alexander Bulekov, 2023/04/21
- [PATCH v8 8/8] memory: abort on re-entrancy in debug builds, Alexander Bulekov, 2023/04/21