Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Sungjong Seo <sjdev.seo@gmail.com>
Message-ID: <45e7ecdd-c4a0-5d41-5e32-db53424558da@samsung.com>
Date:   Thu, 24 Nov 2022 22:01:27 +0900
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.4.2
Reply-To: sj1557.seo@samsung.com
Subject: Re: [PATCH v2 0/5] exfat: move exfat_entry_set_cache from heap to
 stack
To:     "Yuezhang.Mo@sony.com" <Yuezhang.Mo@sony.com>,
        "linkinjeon@kernel.org" <linkinjeon@kernel.org>
Cc:     "linux-fsdevel@vger.kernel.org" <linux-fsdevel@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "Andy.Wu@sony.com" <Andy.Wu@sony.com>,
        "Wataru.Aoyama@sony.com" <Wataru.Aoyama@sony.com>
References: <PUZPR04MB631661A405BDD1987B969597810F9@PUZPR04MB6316.apcprd04.prod.outlook.com>
Content-Language: en-US
In-Reply-To: <PUZPR04MB631661A405BDD1987B969597810F9@PUZPR04MB6316.apcprd04.prod.outlook.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

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(-)