mbox series

[RFC,0/4] selftests/binderfs: Fixes to binderfs_test

Message ID 20240515010805.605511-1-xandfury@gmail.com
Headers show
Series selftests/binderfs: Fixes to binderfs_test | expand

Message

Abhinav Saxena May 15, 2024, 1:08 a.m. UTC 
Hi everyone,

My name is Abhinav Saxena. I am a graduate student at the University
of Calgary. This is my first patch series for the Linux kernel. I am
applying for the "Linux kernel Bug Fixing Summer Unpaid
2024". Apologies in advance if I made any trivial mistakes :)

This patch mainly includes issues reported by checkpatch.pl.  The
changes include:
- Running clang-format on `binderfs_test.c` to fix formatting issues.
- Updates the macro close_prot_errno_disarm macro.

Testing: I tested patches on my local machine (ARM64 ubuntu) with
checkpatch.pl and running the selftests.

Best,
Abhinav

Abhinav Saxena (4):
  run clang-format on bindergfs test
  update close_prot_errno_disarm macro to do{...}while(false) structure
    for safety
  Macro argument 'fd' may be better as '(fd)' to avoid precedence issues
  add missing a blank line after declarations; fix alignment formatting

 .../filesystems/binderfs/binderfs_test.c      | 204 +++++++++++-------
 1 file changed, 126 insertions(+), 78 deletions(-)

--
2.34.1

Comments

Abhinav Saxena May 15, 2024, 4:27 a.m. UTC | #1 
Adding Shuah Khan<skhan@linuxfoundation.org> to cc.

-Abhinav.

On Wed, May 15, 2024 at 01:08:01AM +0000, Abhinav Saxena wrote:
> Hi everyone,
> 
> My name is Abhinav Saxena. I am a graduate student at the University
> of Calgary. This is my first patch series for the Linux kernel. I am
> applying for the "Linux kernel Bug Fixing Summer Unpaid
> 2024". Apologies in advance if I made any trivial mistakes :)
> 
> This patch mainly includes issues reported by checkpatch.pl.  The
> changes include:
> - Running clang-format on `binderfs_test.c` to fix formatting issues.
> - Updates the macro close_prot_errno_disarm macro.
> 
> Testing: I tested patches on my local machine (ARM64 ubuntu) with
> checkpatch.pl and running the selftests.
> 
> Best,
> Abhinav
> 
> Abhinav Saxena (4):
>   run clang-format on bindergfs test
>   update close_prot_errno_disarm macro to do{...}while(false) structure
>     for safety
>   Macro argument 'fd' may be better as '(fd)' to avoid precedence issues
>   add missing a blank line after declarations; fix alignment formatting
> 
>  .../filesystems/binderfs/binderfs_test.c      | 204 +++++++++++-------
>  1 file changed, 126 insertions(+), 78 deletions(-)
> 
> --
> 2.34.1
>