mbox series

[v2,0/5] exfat: move exfat_entry_set_cache from heap to stack

Message ID PUZPR04MB631661A405BDD1987B969597810F9@PUZPR04MB6316.apcprd04.prod.outlook.com
Headers
Series exfat: move exfat_entry_set_cache from heap to stack |

Message

Yuezhang.Mo@sony.com Nov. 24, 2022, 6:40 a.m. UTC 
  This patchset reduces the size of exfat_entry_set_cache and moves
it from heap to stack.

Changes for v2:
  * [1/5] [5/5]
    - Rename ES_*_ENTRY to ES_IDX_*
    - Fix ES_IDX_LAST_FILENAME() to return the index of the last
      filename entry
    - Add ES_ENTRY_NUM() MACRO

Yuezhang Mo (5):
  exfat: reduce the size of exfat_entry_set_cache
  exfat: support dynamic allocate bh for exfat_entry_set_cache
  exfat: move exfat_entry_set_cache from heap to stack
  exfat: rename exfat_free_dentry_set() to exfat_put_dentry_set()
  exfat: replace magic numbers with Macros

 fs/exfat/dir.c      | 70 ++++++++++++++++++++++++++-------------------
 fs/exfat/exfat_fs.h | 37 ++++++++++++++++++------
 fs/exfat/inode.c    | 13 ++++-----
 fs/exfat/namei.c    | 11 ++++---
 4 files changed, 80 insertions(+), 51 deletions(-)

Comments

Sungjong Seo Nov. 24, 2022, 1:01 p.m. UTC | #1 
Looks good.
Thanks for your patchset.

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

> This patchset reduces the size of exfat_entry_set_cache and moves
> it from heap to stack.
> 
> Changes for v2:
>   * [1/5] [5/5]
>     - Rename ES_*_ENTRY to ES_IDX_*
>     - Fix ES_IDX_LAST_FILENAME() to return the index of the last
>       filename entry
>     - Add ES_ENTRY_NUM() MACRO
> 
> Yuezhang Mo (5):
>   exfat: reduce the size of exfat_entry_set_cache
>   exfat: support dynamic allocate bh for exfat_entry_set_cache
>   exfat: move exfat_entry_set_cache from heap to stack
>   exfat: rename exfat_free_dentry_set() to exfat_put_dentry_set()
>   exfat: replace magic numbers with Macros
> 
>  fs/exfat/dir.c      | 70 ++++++++++++++++++++++++++-------------------
>  fs/exfat/exfat_fs.h | 37 ++++++++++++++++++------
>  fs/exfat/inode.c    | 13 ++++-----
>  fs/exfat/namei.c    | 11 ++++---
>  4 files changed, 80 insertions(+), 51 deletions(-)
Namjae Jeon Nov. 28, 2022, 12:46 p.m. UTC | #2 
2022-11-24 15:40 GMT+09:00, Yuezhang.Mo@sony.com <Yuezhang.Mo@sony.com>:
> This patchset reduces the size of exfat_entry_set_cache and moves
> it from heap to stack.
>
> Changes for v2:
>   * [1/5] [5/5]
>     - Rename ES_*_ENTRY to ES_IDX_*
>     - Fix ES_IDX_LAST_FILENAME() to return the index of the last
>       filename entry
>     - Add ES_ENTRY_NUM() MACRO
>
> Yuezhang Mo (5):
>   exfat: reduce the size of exfat_entry_set_cache
>   exfat: support dynamic allocate bh for exfat_entry_set_cache
>   exfat: move exfat_entry_set_cache from heap to stack
>   exfat: rename exfat_free_dentry_set() to exfat_put_dentry_set()
>   exfat: replace magic numbers with Macros
Applied, Thanks for your patches!