On Thu, 2019-01-17 at 14:52 +0800, tien.fong.c...@intel.com wrote:
> From: Tien Fong Chee <tien.fong.c...@intel.com>
>
> Release cluster block immediately when no longer use would help to
> reduce
> 64KiB memory allocated to the memory pool.
>
> Signed-off-by: Tien Fong Chee <tien.fong.c...@intel.com>
> ---
> fs/fat/fat.c | 11 +++++++++--
> 1 files changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/fs/fat/fat.c b/fs/fat/fat.c
> index aa4636d..5574620 100644
> --- a/fs/fat/fat.c
> +++ b/fs/fat/fat.c
> @@ -1153,12 +1153,19 @@ int file_fat_read_at(const char *filename,
> loff_t pos, void *buffer,
> goto out_free_both;
>
> debug("reading %s at pos %llu\n", filename, pos);
> - ret = get_contents(&fsdata, itr->dent, pos, buffer, maxsize,
> actread);
> +
> + /* For saving default max clustersize memory allocated to
> malloc pool */
> + dir_entry *dentptr = itr->dent;
> +
> + free(itr);
I noticed msising of assignning NULL to itr after freeing it. Other
than this, any comment about this implementation to maximise the
reusable on reduce memory pool.
> +
> + ret = get_contents(&fsdata, dentptr, pos, buffer, maxsize,
> actread);
>
> out_free_both:
> free(fsdata.fatbuf);
> out_free_itr:
> - free(itr);
> + if (!itr)
> + free(itr);
> return ret;
> }
>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
