mbox series

[v2,0/3] exfat: fix and refine exfat_alloc_cluster()

Message ID PUZPR04MB6316DF13B8E9FC79FD477A6C81AC9@PUZPR04MB6316.apcprd04.prod.outlook.com (mailing list archive)
Headers show
Series exfat: fix and refine exfat_alloc_cluster() | expand

Message

Yuezhang.Mo@sony.com Feb. 28, 2023, 6:07 a.m. UTC 
Changes for v2:
  [2/3] do not return error if hint_cluster invalid

Yuezhang Mo (3):
  exfat: remove unneeded code from exfat_alloc_cluster()
  exfat: don't print error log in normal case
  exfat: fix the newly allocated clusters are not freed in error
    handling

 fs/exfat/fatent.c | 31 +++++++++++--------------------
 1 file changed, 11 insertions(+), 20 deletions(-)

Comments

Sungjong Seo Feb. 28, 2023, 6:11 a.m. UTC | #1 
> Changes for v2:
>   [2/3] do not return error if hint_cluster invalid
> 
> Yuezhang Mo (3):
>   exfat: remove unneeded code from exfat_alloc_cluster()
>   exfat: don't print error log in normal case
>   exfat: fix the newly allocated clusters are not freed in error
>     handling

Looks good. Thanks.
Reviewed-by: Sungjong Seo <sj1557.seo@samsung.com>

> 
>  fs/exfat/fatent.c | 31 +++++++++++--------------------
>  1 file changed, 11 insertions(+), 20 deletions(-)
> 
> --
> 2.25.1
Namjae Jeon Feb. 28, 2023, 11:05 a.m. UTC | #2 
2023-02-28 15:07 GMT+09:00, Yuezhang.Mo@sony.com <Yuezhang.Mo@sony.com>:
> Changes for v2:
>   [2/3] do not return error if hint_cluster invalid
>
> Yuezhang Mo (3):
>   exfat: remove unneeded code from exfat_alloc_cluster()
>   exfat: don't print error log in normal case
>   exfat: fix the newly allocated clusters are not freed in error
>     handling
Applied, Thanks for your patches!