The global variable errno is not recommended to be assigned,
and rc in the function does not check its return value, so it is fixed
Signed-off-by: Liu Jing
---
tools/testing/selftests/kvm/s390x/cmma_test.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/tools
Signed-off-by: Liu Jing
---
V1 -> V2: Delete more unused parameters, such as err, v1 only deleted rplnlh
parameter
Signed-off-by: Liu Jing
---
.../testing/selftests/net/netfilter/conntrack_dump_flush.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/tools/test
There is no va_end after va_copy, just add it.
Signed-off-by: Liu Jing
Signed-off-by: Liu Jing
---
tools/testing/selftests/net/tcp_ao/lib/aolib.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/tools/testing/selftests/net/tcp_ao/lib/aolib.h
b/tools/testing/selftests/net/tcp_ao/lib/aolib.h
The rplnlh parameter is not used in many functions, so delete it.
Signed-off-by: Liu Jing
---
tools/testing/selftests/net/netfilter/conntrack_dump_flush.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/tools/testing/selftests/net/netfilter/conntrack_dump_flush.c
b
devm_ioremap_resource already contains error message, so remove
the redundant dev_err message
Signed-off-by: Liu Jing
diff --git a/drivers/remoteproc/da8xx_remoteproc.c
b/drivers/remoteproc/da8xx_remoteproc.c
index 9041a0e07fb2..289d5ce9f7af 100644
--- a/drivers/remoteproc/da8xx_remoteproc.c
devm_ioremap_resource already contains error message, so remove
the redundant dev_err message
Signed-off-by: Liu Jing
diff --git a/drivers/remoteproc/st_slim_rproc.c
b/drivers/remoteproc/st_slim_rproc.c
index d17719384c16..a6e50f51c794 100644
--- a/drivers/remoteproc/st_slim_rproc.c
+++ b
If the file fails to open, not only return return1 but also close the file
descriptor,otherwise resource
leak will occur
Signed-off-by: Liu Jing
---
.../selftests/iommu/iommufd_fail_nth.c| 143 ++
1 file changed, 83 insertions(+), 60 deletions(-)
diff --git a/tools
in the do_setcpu, this function does not need to have a return value,
which is meaningless
Signed-off-by: Liu Jing
---
tools/testing/selftests/net/msg_zerocopy.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/tools/testing/selftests/net/msg_zerocopy.c
b/tools/testing
8 matches
Mail list logo