[PATCH v1 4/5] scripts/checkstack.pl: Change min_stack to 512 by default

Tiezhu Yang posted 5 patches 1 year, 12 months ago
[PATCH v1 4/5] scripts/checkstack.pl: Change min_stack to 512 by default
Posted by Tiezhu Yang 1 year, 12 months ago
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@loongson.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