[PATCH v2] livepatch: core: Return EOPNOTSUPP instead of ENOSYS

From: Alice Ferrazzi As a result of an unsupported operation is better to use EOPNOTSUPP as error code. ENOSYS is only used for 'invalid syscall nr' and nothing else. Signed-off-by: Alice Ferrazzi --- Changes v1->v2: - Use EOPNOTSUPP instead of ENOTSUPP (Petr) kernel/livepatch/core.c

[PATCH] livepatch: core: Return ENOTSUPP instead of ENOSYS

This patch fixes a checkpatch warning: WARNING: ENOSYS means 'invalid syscall nr' and nothing else Signed-off-by: Alice Ferrazzi --- kernel/livepatch/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/livepatch/core.c b/kernel/livepatch/core.c index

[PATCH 4/7] selftest/bpf: test_offload PEP8 format style fix

Signed-off-by: Alice Ferrazzi --- tools/testing/selftests/bpf/test_offload.py | 68 - 1 file changed, 54 insertions(+), 14 deletions(-) diff --git a/tools/testing/selftests/bpf/test_offload.py b/tools/testing/selftests/bpf/test_offload.py index d59642e70f56..f80c4f13991d

[PATCH 5/7] selftest/bpf: Fix PEP8 ambiguous variable name

Signed-off-by: Alice Ferrazzi --- tools/testing/selftests/bpf/test_offload.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/testing/selftests/bpf/test_offload.py b/tools/testing/selftests/bpf/test_offload.py index f80c4f13991d..0f9130ebfd2c 100755 --- a/tools

[PATCH 3/7] selftest/bpf: PEP 8: multiple statements on one line (colon)

Reformat the file for fixing PEP 8 style Signed-off-by: Alice Ferrazzi --- tools/testing/selftests/bpf/tcp_client.py | 13 + tools/testing/selftests/bpf/tcp_server.py | 16 +++- 2 files changed, 20 insertions(+), 9 deletions(-) diff --git a/tools/testing/selftests/bpf

[PATCH 2/7] selftest/bpf: optimize import

Fix PEP8 warnings Signed-off-by: Alice Ferrazzi --- tools/testing/selftests/bpf/tcp_client.py | 7 +++ tools/testing/selftests/bpf/tcp_server.py | 7 --- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/tools/testing/selftests/bpf/tcp_client.py b/tools/testing/selftests

[PATCH 7/7] selftest/bpf: fix E501 line too long

fix PEP8 style issues Signed-off-by: Alice Ferrazzi --- tools/testing/selftests/bpf/test_offload.py | 35 + 1 file changed, 22 insertions(+), 13 deletions(-) diff --git a/tools/testing/selftests/bpf/test_offload.py b/tools/testing/selftests/bpf/test_offload.py index

[PATCH 6/7] selftest/bpf: remove redundant parenthesis

Signed-off-by: Alice Ferrazzi --- tools/testing/selftests/bpf/test_offload.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/testing/selftests/bpf/test_offload.py b/tools/testing/selftests/bpf/test_offload.py index 0f9130ebfd2c..b06cc0eea0eb 100755 --- a/tools/testing

[PATCH 1/7] selftest/bpf: Fix trailing semicolon in the statement

fix python PEP8 style issue Signed-off-by: Alice Ferrazzi --- tools/testing/selftests/bpf/tcp_client.py | 4 ++-- tools/testing/selftests/bpf/tcp_server.py | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/tools/testing/selftests/bpf/tcp_client.py b/tools/testing

[PATCH 0/7] selftest/bpf fix PEP8 warnings

fixing PEP8 warning in the selftest bpf python files. Alice Ferrazzi (7): selftest/bpf: Fix trailing semicolon in the statement selftest/bpf: optimize import selftest/bpf: PEP 8: multiple statements on one line (colon) selftest/bpf: test_offload PEP8 format style fix selftest/bpf: Fix

Re: [PATCH v14 11/11] selftests/livepatch: introduce tests

atch: test-shadow-vars.sh TEST: basic shadow variable API ... ok ok 1..3 selftests: livepatch: test-shadow-vars.sh [PASS] make[1]: Leaving directory '/root/linux-dev/tools/testing/selftests/livepatch' make: Leaving directory '/root/linux-dev/tools/testing/selftests' ``` Tested-by: Alice Ferrazzi

Re: [PATCH v14 11/11] selftests/livepatch: introduce tests

atch: test-shadow-vars.sh TEST: basic shadow variable API ... ok ok 1..3 selftests: livepatch: test-shadow-vars.sh [PASS] make[1]: Leaving directory '/root/linux-dev/tools/testing/selftests/livepatch' make: Leaving directory '/root/linux-dev/tools/testing/selftests' ``` Tested-by: Alice Ferrazzi

Re: [PATCH v14 01/11] livepatch: Change unsigned long old_addr -> void *old_func in struct klp_func

_func > where we want to assign func->new_func = func->old_addr respectively > func->new_func = func->old_func. > > This patch does not modify the existing behavior. > > Suggested-by: Josh Poimboeuf > Signed-off-by: Petr Mladek Acked-by: Alice Ferrazzi --

Re: [PATCH v14 01/11] livepatch: Change unsigned long old_addr -> void *old_func in struct klp_func

_func > where we want to assign func->new_func = func->old_addr respectively > func->new_func = func->old_func. > > This patch does not modify the existing behavior. > > Suggested-by: Josh Poimboeuf > Signed-off-by: Petr Mladek Acked-by: Alice Ferrazzi --

Re: [PATCH 4.4 00/37] 4.4.110-stable review

tly. Logs: http://kernel1.amd64.dev.gentoo.org:8010/#/builders/5/builds/44 -- Thanks, Alice Ferrazzi Gentoo Kernel Project Leader Gentoo Foundation Board Member Mail: Alice Ferrazzi <ali...@gentoo.org> PGP: 2E4E 0856 461C 0585 1336 F496 5621 A6B2 8638 781A

Re: [PATCH 4.4 00/37] 4.4.110-stable review

o.org:8010/#/builders/5/builds/44 -- Thanks, Alice Ferrazzi Gentoo Kernel Project Leader Gentoo Foundation Board Member Mail: Alice Ferrazzi PGP: 2E4E 0856 461C 0585 1336 F496 5621 A6B2 8638 781A

Re: [PATCH 4.9 00/39] 4.9.75-stable review

d the kernel boot up correctly. https://github.com/gentoo/linux-patches/tree/4.9_001 http://kernel1.amd64.dev.gentoo.org:8010/#/builders/3/builds/47 -- Thanks, Alice Ferrazzi Gentoo Kernel Project Leader Gentoo Foundation Board Member Mail: Alice Ferrazzi <alice.ferra...@gmail.com> PGP: 2E4E 0856 461C 0585 1336 F496 5621 A6B2 8638 781A

Re: [PATCH 4.9 00/39] 4.9.75-stable review

https://github.com/gentoo/linux-patches/tree/4.9_001 http://kernel1.amd64.dev.gentoo.org:8010/#/builders/3/builds/47 -- Thanks, Alice Ferrazzi Gentoo Kernel Project Leader Gentoo Foundation Board Member Mail: Alice Ferrazzi PGP: 2E4E 0856 461C 0585 1336 F496 5621 A6B2 8638 781A

[RFC] ubsan: signed integer overflow in atomc.h atomic_add_return

A Gentoo user reported a UBSAN signed integer overflow in atomic_add_return. /** * atomic_add_return - add integer and return * @i: integer value to add * @v: pointer of type atomic_t * * Atomically adds @i to @v and returns @i + @v */ static __always_inline int atomic_add_return(int i,

[RFC] ubsan: signed integer overflow in atomc.h atomic_add_return

A Gentoo user reported a UBSAN signed integer overflow in atomic_add_return. /** * atomic_add_return - add integer and return * @i: integer value to add * @v: pointer of type atomic_t * * Atomically adds @i to @v and returns @i + @v */ static __always_inline int atomic_add_return(int i,

[RFC] ubsan: signed integer overflow in scsi_partsize

A Gentoo user reported a USBAN signed integer overflow in scsicam.c Shall we change something? kernel: UBSAN: Undefined behaviour in drivers/scsi/scsicam.c:173:29 kernel: signed integer overflow: kernel: 62015235 *

[RFC] ubsan: signed integer overflow in scsi_partsize

A Gentoo user reported a USBAN signed integer overflow in scsicam.c Shall we change something? kernel: UBSAN: Undefined behaviour in drivers/scsi/scsicam.c:173:29 kernel: signed integer overflow: kernel: 62015235 *

[RFC] ubsan: signed integer overflow in mem_cgroup_event_ratelimit

Hello, a user reported a UBSAN signed integer overflow in memcontrol.c Shall we change something in mem_cgroup_event_ratelimit()? kernel: UBSAN: Undefined behaviour in mm/memcontrol.c:661:17 kernel: signed integer

[RFC] ubsan: signed integer overflow in mem_cgroup_event_ratelimit

Hello, a user reported a UBSAN signed integer overflow in memcontrol.c Shall we change something in mem_cgroup_event_ratelimit()? kernel: UBSAN: Undefined behaviour in mm/memcontrol.c:661:17 kernel: signed integer

[PATCH] Infiniband: core: change to

This patch replaces with to comply with the checkpatch.pl hint. Signed-off-by: Alice Ferrazzi --- drivers/infiniband/core/ucm.c |2 +- drivers/infiniband/core/user_mad.c|2 +- drivers/infiniband/core/uverbs_cmd.c |2 +- drivers/infiniband/core/uverbs_main.c |2

[PATCH] Infiniband: core: change asm/uaccess.h to linux/uaccess.h

This patch replaces asm/uaccess.h with linux/uaccess.h to comply with the checkpatch.pl hint. Signed-off-by: Alice Ferrazzi alice.ferra...@gmail.com --- drivers/infiniband/core/ucm.c |2 +- drivers/infiniband/core/user_mad.c|2 +- drivers/infiniband/core/uverbs_cmd.c |2

[PATCH] Block: blk-flush: Fixed indent code style

Fixed code indent should use tabs where possible. Signed-off-by: Alice Ferrazzi --- block/blk-flush.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/block/blk-flush.c b/block/blk-flush.c index db8f1b5..cc2b827 100644 --- a/block/blk-flush.c +++ b/block/blk-flush.c

[PATCH] Staging: media: solo6x10: v4l2-enc: Fixed C99 comment

Fixed do not use C99 // comments. Signed-off-by: Alice Ferrazzi --- drivers/staging/media/solo6x10/v4l2-enc.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/solo6x10/v4l2-enc.c b/drivers/staging/media/solo6x10/v4l2-enc.c index 4977e86..8f4ec1b

[PATCH] Staging: comedi: serial2002: fixed consistent spacing issue

Fixed consistent spacing around '*'. Signed-off-by: Alice Ferrazzi --- drivers/staging/comedi/drivers/serial2002.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/comedi/drivers/serial2002.c b/drivers/staging/comedi/drivers/serial2002.c index e6177b4

[PATCH] Staging: comedi: serial2002: fixed consistent spacing issue

Fixed consistent spacing around '*'. Signed-off-by: Alice Ferrazzi alice.ferra...@gmail.com --- drivers/staging/comedi/drivers/serial2002.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/comedi/drivers/serial2002.c b/drivers/staging/comedi/drivers

[PATCH] Staging: media: solo6x10: v4l2-enc: Fixed C99 comment

Fixed do not use C99 // comments. Signed-off-by: Alice Ferrazzi alice.ferra...@gmail.com --- drivers/staging/media/solo6x10/v4l2-enc.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/solo6x10/v4l2-enc.c b/drivers/staging/media/solo6x10/v4l2-enc.c

[PATCH] Block: blk-flush: Fixed indent code style

Fixed code indent should use tabs where possible. Signed-off-by: Alice Ferrazzi alice.ferra...@gmail.com --- block/blk-flush.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/block/blk-flush.c b/block/blk-flush.c index db8f1b5..cc2b827 100644 --- a/block/blk-flush.c +++ b