[LTP] [PATCH 1/1] checkpatch: Ignore USE_NEGATIVE_ERRNO

[LTP] [PATCH 1/1] checkpatch: Ignore USE_NEGATIVE_ERRNO

[Petr Vorel]

return of an errno should typically be negative is purely kernel
specific, avoid warning like:

epoll_ctl06.c:36: WARNING: return of an errno should typically be negative (ie: return -EPERM)

Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
 .checkpatch.conf | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/.checkpatch.conf b/.checkpatch.conf
index a7b6219ada..2ede8d54c5 100644
--- a/.checkpatch.conf
+++ b/.checkpatch.conf
@@ -12,6 +12,9 @@
 # kernel specific
 --ignore BIT_MACRO,PREFER_DEFINED_ATTRIBUTE_MACRO,PREFER_KERNEL_TYPES,STRCPY,STRLCPY,STRNCPY
 
+# kernel specific (cont)
+--ignore USE_NEGATIVE_ERRNO
+
 # TODO: document reason
 --ignore CONST_STRUCT,VOLATILE,FILE_PATH_CHANGES,LONG_LINE,MACRO_ARG_REUSE,MULTIPLE_ASSIGNMENTS
 
-- 
2.54.0


-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

Re: [LTP] [PATCH 1/1] checkpatch: Ignore USE_NEGATIVE_ERRNO

[Cyril Hrubis]

Hi!
Looks obviously correct.

Reviewed-by: Cyril Hrubis <chrubis@suse.cz>

-- 
Cyril Hrubis
chrubis@suse.cz

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

Re: [LTP] checkpatch: Ignore USE_NEGATIVE_ERRNO

[linuxtestproject.agent]

Hi Petr,

On Fri, Jul 10, 2026 at 08:29:21 +0200, Petr Vorel wrote:
> checkpatch: Ignore USE_NEGATIVE_ERRNO

Verdict - Reviewed

---
Note:

The agent can sometimes produce false positives although often its
findings are genuine. If you find issues with the review, please
comment this email or ignore the suggestions.

Regards,
LTP AI Reviewer

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

Re: [LTP] [PATCH 1/1] checkpatch: Ignore USE_NEGATIVE_ERRNO

[Andrea Cervesato via ltp]

Reviewed-by: Andrea Cervesato <andrea.cervesato@suse.com>

--
Andrea Cervesato
SUSE QE Automation Engineer Linux
andrea.cervesato@suse.com

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp