lkml.org 
[lkml]   [2021]   [Mar]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Subject[PATCH v4 0/5] Make UTF-8 encoding loadable
    Date
    utf8data.h_shipped has a large database table which is an auto-generated
    decodification trie for the unicode normalization functions and it is not
    necessary to carry this large table in the kernel.
    Goal is to make UTF-8 encoding loadable by converting it into a module
    and adding a layer between the filesystems and the utf8 module which will
    load the module whenever any filesystem that needs unicode is mounted.

    1st patch in the series resolves the warning reported by kernel test robot
    and 2nd patch fixes the incorrect use of utf8_unload() in ext4 and
    f2fs filesystems.

    Unicode is the subsystem and utf8 is a charachter encoding for the
    subsystem, hence 3rd and 4th patches in the series are renaming functions
    and file name to unicode for better understanding the difference between
    UTF-8 module and unicode layer.

    Last patch in the series adds the layer and utf8 module and also uses
    static_call() function introducted for preventing speculative execution
    attacks.

    ---
    Changes in v4
    - Return error from the static calls instead of doing nothing and
    succeeding even without loading the module.
    - Remove the complete usage of utf8_ops and use static calls at all
    places.
    - Restore the static calls to default values when module is unloaded.
    - Decrement the reference of module after calling the unload function.
    - Remove spinlock as there will be no race conditions after removing
    utf8_ops.

    Changes in v3
    - Add a patch which checks if utf8 is loaded before calling utf8_unload()
    in ext4 and f2fs filesystems
    - Return error if strscpy() returns value < 0
    - Correct the conditions to prevent NULL pointer dereference while
    accessing functions via utf8_ops variable.
    - Add spinlock to avoid race conditions.
    - Use static_call() for preventing speculative execution attacks.

    Changes in v2
    - Remove the duplicate file from the last patch.
    - Make the wrapper functions inline.
    - Remove msleep and use try_module_get() and module_put()
    for ensuring that module is loaded correctly and also
    doesn't get unloaded while in use.
    - Resolve the warning reported by kernel test robot.
    - Resolve all the checkpatch.pl warnings.

    Shreeya Patel (5):
    fs: unicode: Use strscpy() instead of strncpy()
    fs: Check if utf8 encoding is loaded before calling utf8_unload()
    fs: unicode: Rename function names from utf8 to unicode
    fs: unicode: Rename utf8-core file to unicode-core
    fs: unicode: Add utf8 module and a unicode layer

    fs/ext4/hash.c | 2 +-
    fs/ext4/namei.c | 12 +-
    fs/ext4/super.c | 8 +-
    fs/f2fs/dir.c | 12 +-
    fs/f2fs/super.c | 11 +-
    fs/libfs.c | 6 +-
    fs/unicode/Kconfig | 11 +-
    fs/unicode/Makefile | 5 +-
    fs/unicode/unicode-core.c | 84 +++++++++++++
    fs/unicode/{utf8-core.c => unicode-utf8.c} | 93 +++++++++-----
    fs/unicode/utf8-selftest.c | 8 +-
    include/linux/unicode.h | 133 ++++++++++++++++++---
    12 files changed, 307 insertions(+), 78 deletions(-)
    create mode 100644 fs/unicode/unicode-core.c
    rename fs/unicode/{utf8-core.c => unicode-utf8.c} (59%)

    --
    2.30.1

    \
     
     \ /
      Last update: 2021-03-25 01:10    [W:2.945 / U:0.960 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site