On 2019/4/16 14:43, Park Ju Hyung wrote:
> Following extensions are removed:
>  - divx: deprecated video format and it's usually wrapped with avi
>  - asf: deprecated streaming format
>  - asx: redirecting file to asf(small)
>  - wmx: redirecting file to wma/wmv(small)
>  - rm: deprecated media container
>  - video: unused
>  - wv: unpopular audio format from 1998
> 
> The extensions list is limited to 64 and those don't deserve to be
> on this space-precious list.
> 
> Common prefixes are introduced and are checked with
> https://en.wikipedia.org/wiki/List_of_filename_extensions
> to avoid treating possible hot files as cold:
>  - mp: covers mp3, mp4, mpeg, mpg
>  - wm: covers wma, wmb, wmv
>  - og: covers oga, ogg, ogm, ogv
>  - jp: covers jpg, jpeg, jp2
> 
> Following extensions are added:
>  - webm: extremely popular free media container format from Google
>          VP8/VP9/AV1 and Vorbis/Opus is often wrapped with this container
>  - wav: uncompressed audio format, commonly used with voice recorders
>  - svg: vector image format commonly used in web
>  - webp: free lossy image format commonly used in web
>  - jar: Java archive file
>  - deb: Debian software package
>  - iso: disk image file
>  - gz: gzip compressed file, unable to randomly update
>  - xz: xz compressed file, unable to randomly update
>  - zst: zstd compressed file, unable to randomly update
>  - pdf: PDF document
>  - pyc: Python bytecode automatically generated when
>         executing python to run .py files
>  - ttc, ttf: font files
>  - cnt: image alias files commonly used in Android apps
>  - exo: EXO player's cache files, commonly used in Android's YouTube app
>  - odex, vdex: Android RunTime files found in /data/app/*/oat
> 
> Total entries on the list changed from 34 to 36.
> 
> Signed-off-by: Park Ju Hyung <[email protected]>
> ---
>  mkfs/f2fs_format.c | 50 ++++++++++++++++++++++++++--------------------
>  1 file changed, 28 insertions(+), 22 deletions(-)
> 
> diff --git a/mkfs/f2fs_format.c b/mkfs/f2fs_format.c
> index 0ae0df3..4560611 100644
> --- a/mkfs/f2fs_format.c
> +++ b/mkfs/f2fs_format.c
> @@ -38,48 +38,54 @@ struct f2fs_checkpoint *cp;
>  static unsigned int quotatype_bits = 0;
>  
>  const char *media_ext_lists[] = {
> +     /* common prefix */
> +     "mp", // Covers mp3, mp4, mpeg, mpg
> +     "wm", // Covers wma, wmb, wmv
> +     "og", // Covers oga, ogg, ogm, ogv
> +     "jp", // Covers jpg, jpeg, jp2

Actually, kernel will compare each character of extension in list with file's
extension, rather than prefix, could you confirm that? :)

Thanks,

> +
>       /* video */
>       "avi",
> -     "divx",
>       "m4v",
>       "m4p",
> -     "mp4",
> -     "wmv",
> -     "mpeg",
>       "mkv",
>       "mov",
> -     "asx",
> -     "asf",
> -     "wmx",
> -     "svi",
> -     "wvx",
> -     "wm",
> -     "mpg",
> -     "mpe",
> -     "rm",
> -     "video",
> +     "webm",
>  
>       /* audio */
> +     "wav",
>       "m4a",
> -     "mp3",
>       "3gp",
> -     "wma",
> -     "wv",
> -     "ogg",
>       "opus",
>       "flac",
>  
>       /* image */
> -     "jpeg",
> -     "jpg",
>       "gif",
>       "png",
> -
> -     /* other */
> +     "svg",
> +     "webp",
> +
> +     /* archives */
> +     "jar",
> +     "deb",
> +     "iso",
> +     "gz",
> +     "xz",
> +     "zst",
> +
> +     /* others */
> +     "pdf",
> +     "pyc", // Python bytecode
> +     "ttc",
> +     "ttf",
>       "exe",
>  
>       /* android */
>       "apk",
> +     "cnt", // Image alias
> +     "exo", // YouTube
> +     "odex", // Android RunTime
> +     "vdex", // Android RunTime
>       "so",
>  
>       NULL
> 


_______________________________________________
Linux-f2fs-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to