| Message ID | 20231106181144.117188-1-atulpant.linux@gmail.com |
|---|---|
| State | New |
| Headers | show |
| Series | [v1] selftests: cgroup: Fixes code style errors | expand |
On 11/6/23 11:11, Atul Kumar Pant wrote: > Fixes following checkpatch.pl issues: > ERROR: do not use assignment in if condition > ERROR: Macros starting with if should be enclosed by a do - while > > Signed-off-by: Atul Kumar Pant <atulpant.linux@gmail.com> > --- > tools/testing/selftests/cgroup/cgroup_util.c | 14 ++++++++------ > tools/testing/selftests/cgroup/test_core.c | 3 ++- > 2 files changed, 10 insertions(+), 7 deletions(-) > > diff --git a/tools/testing/selftests/cgroup/cgroup_util.c b/tools/testing/selftests/cgroup/cgroup_util.c > index 0340d4ca8f51..e165c4a703a6 100644 > --- a/tools/testing/selftests/cgroup/cgroup_util.c > +++ b/tools/testing/selftests/cgroup/cgroup_util.c > @@ -411,12 +411,14 @@ int dirfd_open_opath(const char *dir) > return open(dir, O_DIRECTORY | O_CLOEXEC | O_NOFOLLOW | O_PATH); > } > > -#define close_prot_errno(fd) \ > - if (fd >= 0) { \ > - int _e_ = errno; \ > - close(fd); \ > - errno = _e_; \ > - } > +#define close_prot_errno(fd) \ > + do { \ > + if (fd >= 0) { \ > + int _e_ = errno; \ > + close(fd); \ > + errno = _e_; \ > + } \ > + } while (0); Did you run checkpatch on this patch? You are fixing checkpatch errors - :) thanks, -- Shuah
diff --git a/tools/testing/selftests/cgroup/cgroup_util.c b/tools/testing/selftests/cgroup/cgroup_util.c index 0340d4ca8f51..e165c4a703a6 100644 --- a/tools/testing/selftests/cgroup/cgroup_util.c +++ b/tools/testing/selftests/cgroup/cgroup_util.c @@ -411,12 +411,14 @@ int dirfd_open_opath(const char *dir) return open(dir, O_DIRECTORY | O_CLOEXEC | O_NOFOLLOW | O_PATH); } -#define close_prot_errno(fd) \ - if (fd >= 0) { \ - int _e_ = errno; \ - close(fd); \ - errno = _e_; \ - } +#define close_prot_errno(fd) \ + do { \ + if (fd >= 0) { \ + int _e_ = errno; \ + close(fd); \ + errno = _e_; \ + } \ + } while (0); static int clone_into_cgroup_run_nowait(const char *cgroup, int (*fn)(const char *cgroup, void *arg), diff --git a/tools/testing/selftests/cgroup/test_core.c b/tools/testing/selftests/cgroup/test_core.c index 80aa6b2373b9..ad758e510174 100644 --- a/tools/testing/selftests/cgroup/test_core.c +++ b/tools/testing/selftests/cgroup/test_core.c @@ -817,7 +817,8 @@ static int test_cgcore_lesser_ns_open(const char *root) if (cg_enter_current(cg_test_a)) goto cleanup; - if ((status = write(cg_test_b_procs_fd, "0", 1)) >= 0 || errno != ENOENT) + status = write(cg_test_b_procs_fd, "0", 1); + if (status >= 0 || errno != ENOENT) goto cleanup; ret = KSFT_PASS;
Fixes following checkpatch.pl issues: ERROR: do not use assignment in if condition ERROR: Macros starting with if should be enclosed by a do - while Signed-off-by: Atul Kumar Pant <atulpant.linux@gmail.com> --- tools/testing/selftests/cgroup/cgroup_util.c | 14 ++++++++------ tools/testing/selftests/cgroup/test_core.c | 3 ++- 2 files changed, 10 insertions(+), 7 deletions(-)