Message ID | 20240625152139.16412-1-jain.abhinav177@gmail.com (mailing list archive) |
---|---|
State | New |
Headers | show |
Series | selftests/proc: fix unused result warning during test compilation | expand |
On Tue, 25 Jun 2024 15:21:39 +0000 Abhinav Jain <jain.abhinav177@gmail.com> wrote: > Check the return value from write function to get rid of the warning > during test compilation, shared below. > Tested by compiling after the change, the warning disappears. > > proc-empty-vm.c:385:17: warning: ignoring return value of ‘write’ > declared with attribute ‘warn_unused_result’ [-Wunused-result] > 385 | write(1, buf, rv); > > ... > > --- a/tools/testing/selftests/proc/proc-empty-vm.c > +++ b/tools/testing/selftests/proc/proc-empty-vm.c > @@ -382,7 +382,12 @@ static int test_proc_pid_statm(pid_t pid) > assert(rv >= 0); > assert(rv <= sizeof(buf)); > if (0) { > - write(1, buf, rv); > + ssize_t bytes_written = write(1, buf, rv); > + > + if (bytes_written != rv) { > + perror("write"); > + return EXIT_FAILURE; > + } > } Thanks. There's a patch queued which simply deletes this code. https://lkml.kernel.org/r/20240603124220.33778-1-amer.shanawany@gmail.com
On Tue, 25 Jun 2024 11:05:26 -0700, Andrew Morton wrote: > Thanks. There's a patch queued which simply deletes this code. > > https://lkml.kernel.org/r/20240603124220.33778-1-amer.shanawany@gmail.com Thank you for sharing the queued patch Andrew. There has been no update/revert on it, may I know what would be the next step here? After reading the patch above, I believe my change is "more" right. Is that correct? Thanks and Regards - Abhinav
diff --git a/tools/testing/selftests/proc/proc-empty-vm.c b/tools/testing/selftests/proc/proc-empty-vm.c index 56198d4ca2bf..510ab4a5330a 100644 --- a/tools/testing/selftests/proc/proc-empty-vm.c +++ b/tools/testing/selftests/proc/proc-empty-vm.c @@ -382,7 +382,12 @@ static int test_proc_pid_statm(pid_t pid) assert(rv >= 0); assert(rv <= sizeof(buf)); if (0) { - write(1, buf, rv); + ssize_t bytes_written = write(1, buf, rv); + + if (bytes_written != rv) { + perror("write"); + return EXIT_FAILURE; + } } const char *p = buf;
Check the return value from write function to get rid of the warning during test compilation, shared below. Tested by compiling after the change, the warning disappears. proc-empty-vm.c:385:17: warning: ignoring return value of ‘write’ declared with attribute ‘warn_unused_result’ [-Wunused-result] 385 | write(1, buf, rv); Signed-off-by: Abhinav Jain <jain.abhinav177@gmail.com> --- tools/testing/selftests/proc/proc-empty-vm.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-)