Re: [PATCH] staging: exfat: rebase to sdFAT v2.2.0

Greg KH
2019-09-18T20:12:04Z

On Thu, Sep 19, 2019 at 04:59:20AM +0900, Park Ju Hyung wrote:
> The new sdFAT driver base brings many improvements to the table.
> Quoting Namjae Jeon:
>
> ======================================================================
> 1. sdfat has been refactored to improve compatibility, readability and
> to be linux friendly.(included support mass storages larger than 2TB.)
>
> 2. sdfat has been optimized for the performance of SD-cards.
> - Support SD-card friendly block allocation and delayed allocation
> for vfat-fs only.
> - Support aligned_mpage_write for both vfat-fs and exfat-fs
>
> 3. sdfat has been optimized for the performance of general operations
> like create,lookup and readdir.
>
> 4. Fix many critical and minor bugs
> - Handle many kinds of error conditions gracefully to prevent panic.
> - Fix critical bugs related to rmdir, truncate behavior and so on...
>
> 5. Fix NLS functions
> ======================================================================
>
> sdFAT v2.2.0 (from N970FXXU1ASHE) was forked and then cleaned up to better
> suit mainline's standards:
>
> * Remove fat12/16/32 handlings and rename to exFAT.
> * Remove older kernel compatibilities.
> * Remove Samsung's userspace-specific code (e.g. defrag).
> * Fix compilation warnings on modern compiler.
> * Remove unused functions.
> * Rename non-static functions for avoiding global symbol table pollutions.
> * Declare functions as static whenever possible.
> * Fix checkpatch.pl errors.
> * Remove aligned mpage write for portability.
>
> Full rebase history can be found on:
> https://github.com/arter97/exfat-linux/commits/for-next
>
> Signed-off-by: Park Ju Hyung <qkrwngud825@gmail.com>
> ---
> drivers/staging/exfat/Kconfig | 88 +-
> drivers/staging/exfat/Makefile | 11 +-
> drivers/staging/exfat/TODO | 15 +-
> drivers/staging/exfat/api.h | 265 ++
> drivers/staging/exfat/blkdev.c | 330 +++
> drivers/staging/exfat/cache.c | 785 +++++
> drivers/staging/exfat/config.h | 32 +
> drivers/staging/exfat/core.c | 3169 ++++++++++++++++++++
> drivers/staging/exfat/core.h | 149 +
> drivers/staging/exfat/core_exfat.c | 1485 ++++++++++
> drivers/staging/exfat/exfat.h | 1261 +++-----
> drivers/staging/exfat/exfat_blkdev.c | 136 -
> drivers/staging/exfat/exfat_cache.c | 724 -----
> drivers/staging/exfat/exfat_core.c | 3701 -----------------------
> drivers/staging/exfat/exfat_fs.h | 398 +++
> drivers/staging/exfat/exfat_nls.c | 404 ---
> drivers/staging/exfat/exfat_super.c | 4049 --------------------------
> drivers/staging/exfat/exfat_upcase.c | 740 -----
> drivers/staging/exfat/extent.c | 329 +++
> drivers/staging/exfat/fatent.c | 113 +
> drivers/staging/exfat/misc.c | 356 +++
> drivers/staging/exfat/nls.c | 323 ++
> drivers/staging/exfat/super.c | 3168 ++++++++++++++++++++
> drivers/staging/exfat/upcase.h | 394 +++
> drivers/staging/exfat/xattr.c | 76 +
> 25 files changed, 11777 insertions(+), 10724 deletions(-)

That's a lot of rewriting :(

How about at least keeping the file names the same to make it easier to
see what happened here?

Then send a follow-on patch that just does the rename?

And by taking something like this, are you agreeing that Samsung will
help out with the development of this code to clean it up and get it
into "real" mergable shape?

Also, I can't take this patch for this simple reason alone:

> --- a/drivers/staging/exfat/Makefile
> +++ b/drivers/staging/exfat/Makefile
> @@ -1,10 +1,5 @@
> -# SPDX-License-Identifier: GPL-2.0
> -

Don't delete SPDX lines :)

thanks,

greg k-h