2021-09-07 14:48:20

by Kari Argillander

Subject: [PATCH 0/4] Fix error path and use kernel macros for min/max/calmp

First fix some error path fault which I found during when I did this.
It should go to 5.15.

For better code readability we can use kernel defined macros. Patch
2/4 also removes some dead code so that it would be more clear why
do we do what we do in patch 3/4.

Series is tested with xfs-tests. No recression in my tests.

Kari Argillander (4):
fs/ntfs3: Fix ntfs_look_for_free_space() does only report -ENOSPC
fs/ntfs3: Remove always false condition check
fs/ntfs3: Use clamp/max macros instead of comparisons
fs/ntfs3: Use min/max macros instated of ternary operators

fs/ntfs3/attrib.c | 3 ++-
fs/ntfs3/bitmap.c | 11 ++++----
fs/ntfs3/fsntfs.c | 68 +++++++++++++++++++++++------------------------
3 files changed, 41 insertions(+), 41 deletions(-)


base-commit: 2e3a51b59ea26544303e168de8a0479915f09aa3
--
2.25.1

2021-09-16 18:03:01

by Konstantin Komarov

Subject: Re: [PATCH 0/4] Fix error path and use kernel macros for min/max/calmp

On 07.09.2021 17:28, Kari Argillander wrote:
> First fix some error path fault which I found during when I did this.
> It should go to 5.15.
>
> For better code readability we can use kernel defined macros. Patch
> 2/4 also removes some dead code so that it would be more clear why
> do we do what we do in patch 3/4.
>
> Series is tested with xfs-tests. No recression in my tests.
>
> Kari Argillander (4):
> fs/ntfs3: Fix ntfs_look_for_free_space() does only report -ENOSPC
> fs/ntfs3: Remove always false condition check
> fs/ntfs3: Use clamp/max macros instead of comparisons
> fs/ntfs3: Use min/max macros instated of ternary operators
>
> fs/ntfs3/attrib.c | 3 ++-
> fs/ntfs3/bitmap.c | 11 ++++----
> fs/ntfs3/fsntfs.c | 68 +++++++++++++++++++++++------------------------
> 3 files changed, 41 insertions(+), 41 deletions(-)
>
>
> base-commit: 2e3a51b59ea26544303e168de8a0479915f09aa3
>

Thanks for patches, applied!