All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sheng Yong <shengyong@oppo.com>
To: jaegeuk@kernel.org, chao@kernel.org
Cc: linux-f2fs-devel@lists.sourceforge.net,
	linux-kernel@vger.kernel.org, ebiggers@kernel.org,
	Sheng Yong <shengyong@oppo.com>
Subject: [PATCH v4 0/6] f2fs: add f2fs_ioc_[get|set]_extra_attr
Date: Mon, 12 Jun 2023 11:01:15 +0800	[thread overview]
Message-ID: <20230612030121.2393541-1-shengyong@oppo.com> (raw)

This patchset introduces two ioctls to get or modify values in
f2fs_inode's extra attribute area:
  * f2fs_ioc_get_extra_attr
  * f2fs_ioc_set_extra_attr

The argument of these two ioctls is `struct f2fs_extra_attr', which has
three members:
  * field: indicates which field in extra attribute area is handled
  * attr: value or userspace pointer
  * attr_size: size of `attr'

The `field' member could help extend functionality of these two ioctls
without modify or add new interfaces, if more fields are added into
extra attributes ares in the feture.

In order to reuse existed functions, several helpers are added to:
  * read inode chksum from inode page
  * get compress blocks
  * verify compression level
And, a compress_option v2 is added to access compress level and flags.

Thanks,
shengyong

v4:
  * split into small commits
v3:
  * setting lz4(hc) level correctly
v2:
  * fix compiling error if CONFIG_F2FS_FS_ZSTD is disabled by adding a
    helper f2fs_is_compress_level_valid()
  * fix compiling warning for casting unsinged long long to pointer

Sheng Yong (6):
  f2fs: add helper to check compression level
  f2fs: cleanup MIN_INLINE_XATTR_SIZE
  f2fs: add helper to get inode chksum from inode page
  f2fs: add f2fs_ioc_get_compress_blocks
  f2fs: add f2fs_ioc_[get|set]_extra_attr
  f2fs: access compression level and flags by extra attr ioctls

 fs/f2fs/compress.c        |  31 +++++
 fs/f2fs/f2fs.h            |   4 +
 fs/f2fs/file.c            | 280 ++++++++++++++++++++++++++++++++++++--
 fs/f2fs/inode.c           |  21 +++
 fs/f2fs/super.c           |   6 +-
 fs/f2fs/xattr.h           |   1 +
 include/uapi/linux/f2fs.h |  33 +++++
 7 files changed, 359 insertions(+), 17 deletions(-)

-- 
2.40.1


WARNING: multiple messages have this Message-ID (diff)
From: Sheng Yong via Linux-f2fs-devel <linux-f2fs-devel@lists.sourceforge.net>
To: jaegeuk@kernel.org, chao@kernel.org
Cc: ebiggers@kernel.org, linux-kernel@vger.kernel.org,
	linux-f2fs-devel@lists.sourceforge.net
Subject: [f2fs-dev] [PATCH v4 0/6] f2fs: add f2fs_ioc_[get|set]_extra_attr
Date: Mon, 12 Jun 2023 11:01:15 +0800	[thread overview]
Message-ID: <20230612030121.2393541-1-shengyong@oppo.com> (raw)

This patchset introduces two ioctls to get or modify values in
f2fs_inode's extra attribute area:
  * f2fs_ioc_get_extra_attr
  * f2fs_ioc_set_extra_attr

The argument of these two ioctls is `struct f2fs_extra_attr', which has
three members:
  * field: indicates which field in extra attribute area is handled
  * attr: value or userspace pointer
  * attr_size: size of `attr'

The `field' member could help extend functionality of these two ioctls
without modify or add new interfaces, if more fields are added into
extra attributes ares in the feture.

In order to reuse existed functions, several helpers are added to:
  * read inode chksum from inode page
  * get compress blocks
  * verify compression level
And, a compress_option v2 is added to access compress level and flags.

Thanks,
shengyong

v4:
  * split into small commits
v3:
  * setting lz4(hc) level correctly
v2:
  * fix compiling error if CONFIG_F2FS_FS_ZSTD is disabled by adding a
    helper f2fs_is_compress_level_valid()
  * fix compiling warning for casting unsinged long long to pointer

Sheng Yong (6):
  f2fs: add helper to check compression level
  f2fs: cleanup MIN_INLINE_XATTR_SIZE
  f2fs: add helper to get inode chksum from inode page
  f2fs: add f2fs_ioc_get_compress_blocks
  f2fs: add f2fs_ioc_[get|set]_extra_attr
  f2fs: access compression level and flags by extra attr ioctls

 fs/f2fs/compress.c        |  31 +++++
 fs/f2fs/f2fs.h            |   4 +
 fs/f2fs/file.c            | 280 ++++++++++++++++++++++++++++++++++++--
 fs/f2fs/inode.c           |  21 +++
 fs/f2fs/super.c           |   6 +-
 fs/f2fs/xattr.h           |   1 +
 include/uapi/linux/f2fs.h |  33 +++++
 7 files changed, 359 insertions(+), 17 deletions(-)

-- 
2.40.1



_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

             reply	other threads:[~2023-06-12  3:02 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-06-12  3:01 Sheng Yong [this message]
2023-06-12  3:01 ` [f2fs-dev] [PATCH v4 0/6] f2fs: add f2fs_ioc_[get|set]_extra_attr Sheng Yong via Linux-f2fs-devel
2023-06-12  3:01 ` [PATCH v4 1/6] f2fs: add helper to check compression level Sheng Yong
2023-06-12  3:01   ` [f2fs-dev] " Sheng Yong via Linux-f2fs-devel
2023-06-12 22:14   ` Jaegeuk Kim
2023-06-12 22:14     ` [f2fs-dev] " Jaegeuk Kim
2023-06-13  1:48     ` Chao Yu
2023-06-13  1:48       ` [f2fs-dev] " Chao Yu
2023-06-13  2:28     ` Sheng Yong
2023-06-13  2:28       ` [f2fs-dev] " Sheng Yong via Linux-f2fs-devel
2023-06-12  3:01 ` [PATCH v4 2/6] f2fs: cleanup MIN_INLINE_XATTR_SIZE Sheng Yong
2023-06-12  3:01   ` [f2fs-dev] " Sheng Yong via Linux-f2fs-devel
2023-06-13  1:48   ` Chao Yu
2023-06-13  1:48     ` Chao Yu
2023-06-12  3:01 ` [PATCH v4 3/6] f2fs: add helper to get inode chksum from inode page Sheng Yong
2023-06-12  3:01   ` [f2fs-dev] " Sheng Yong via Linux-f2fs-devel
2023-06-13  1:52   ` Chao Yu
2023-06-13  1:52     ` [f2fs-dev] " Chao Yu
2023-06-12  3:01 ` [PATCH v4 4/6] f2fs: add f2fs_ioc_get_compress_blocks Sheng Yong
2023-06-12  3:01   ` [f2fs-dev] " Sheng Yong via Linux-f2fs-devel
2023-06-13  1:53   ` Chao Yu
2023-06-13  1:53     ` [f2fs-dev] " Chao Yu
2023-06-12  3:01 ` [PATCH v4 5/6] f2fs: add f2fs_ioc_[get|set]_extra_attr Sheng Yong
2023-06-12  3:01   ` [f2fs-dev] " Sheng Yong via Linux-f2fs-devel
2023-06-12 15:47   ` Jaegeuk Kim
2023-06-12 15:47     ` [f2fs-dev] " Jaegeuk Kim
2023-06-13  1:53     ` Sheng Yong
2023-06-13  1:53       ` [f2fs-dev] " Sheng Yong via Linux-f2fs-devel
2023-06-12  3:01 ` [PATCH v4 6/6] f2fs: access compression level and flags by extra attr ioctls Sheng Yong
2023-06-12  3:01   ` [f2fs-dev] " Sheng Yong via Linux-f2fs-devel
2023-06-15 18:00 ` [f2fs-dev] [PATCH v4 0/6] f2fs: add f2fs_ioc_[get|set]_extra_attr patchwork-bot+f2fs
2023-06-15 18:00   ` patchwork-bot+f2fs

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230612030121.2393541-1-shengyong@oppo.com \
    --to=shengyong@oppo.com \
    --cc=chao@kernel.org \
    --cc=ebiggers@kernel.org \
    --cc=jaegeuk@kernel.org \
    --cc=linux-f2fs-devel@lists.sourceforge.net \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.