linux-kernel - [PATCH v1 4/5] scripts/checkstack.pl: Change min

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20231219125008.23007-5-yangtiezhu@loongson.cn>
Date: Tue, 19 Dec 2023 20:50:00 +0800
From: Tiezhu Yang <yangtiezhu@...ngson.cn>
To: Andrew Morton <akpm@...ux-foundation.org>,
	Jonathan Corbet <corbet@....net>
Cc: linux-doc@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: [PATCH v1 4/5] scripts/checkstack.pl: Change min_stack to 512 by default

According to Documentation/process/submit-checklist.rst, checkstack
does not point out problems explicitly, but any one function that
uses more than 512 bytes on the stack is a candidate for change,
hence it is better to omit any stack frame sizes smaller than 512
bytes, just change min_stack to 512 by default.

Signed-off-by: Tiezhu Yang <yangtiezhu@...ngson.cn>
---
 scripts/checkstack.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/checkstack.pl b/scripts/checkstack.pl
index 5e8f1c20e31c..40ab29f77b5b 100755
--- a/scripts/checkstack.pl
+++ b/scripts/checkstack.pl
@@ -47,7 +47,7 @@ my (@stack, $re, $dre, $sub, $x, $xs, $funcre, $min_stack);
 
 	$min_stack = shift;
 	if ($min_stack eq "" || $min_stack !~ /^\d+$/) {
-		$min_stack = 100;
+		$min_stack = 512;
 	}
 
 	$x	= "[0-9a-f]";	# hex character
-- 
2.42.0