Add support for fuse2/3 lowlevel API in erofsfuse,
pass the make check test in experimental-test branch.
Conduct performance evaluation, providing higher
performance compared to highlevel API. Compatible with
fuse 2(>=2.6) and 3(>=3.0)

Signed-off-by: Li Yiyan <lyy0...@sjtu.edu.cn>
---
Changes since v2:
- merge lowlevel.c into main.c
- fix typo in autoconf
- optimize naming
- remove redundant code
- avoid global sbi dependencies

Changes since v1:
- remove highlevel fallback path
- remove redundant code
- add static for erofsfuse_ll_func
---
 configure.ac     |  23 +-
 fuse/Makefile.am |   4 +-
 fuse/main.c      | 624 +++++++++++++++++++++++++++++++++++++----------
 3 files changed, 516 insertions(+), 135 deletions(-)

diff --git a/configure.ac b/configure.ac
index a8cecd0..6d08d96 100644
--- a/configure.ac
+++ b/configure.ac
@@ -336,15 +336,26 @@ AS_IF([test "x$with_selinux" != "xno"], [
 
 # Configure fuse
 AS_IF([test "x$enable_fuse" != "xno"], [
-  PKG_CHECK_MODULES([libfuse], [fuse >= 2.6])
   # Paranoia: don't trust the result reported by pkgconfig before trying out
   saved_LIBS="$LIBS"
   saved_CPPFLAGS=${CPPFLAGS}
-  CPPFLAGS="${libfuse_CFLAGS} ${CPPFLAGS}"
-  LIBS="${libfuse_LIBS} $LIBS"
-  AC_CHECK_LIB(fuse, fuse_main, [
-    have_fuse="yes" ], [
-    AC_MSG_ERROR([libfuse (>= 2.6) doesn't work properly])])
+  PKG_CHECK_MODULES([libfuse3], [fuse3 >= 3.0], [
+    AC_DEFINE([FUSE_USE_VERSION], [30], [used FUSE API version])
+    CPPFLAGS="${libfuse3_CFLAGS} ${CPPFLAGS}"
+    LIBS="${libfuse3_LIBS} $LIBS"
+    AC_CHECK_LIB(fuse3, fuse_session_new, [], [
+    AC_MSG_ERROR([libfuse3 (>= 3.0) doesn't work properly for lowlevel api])])
+    have_fuse="yes"
+  ], [
+    PKG_CHECK_MODULES([libfuse2], [fuse >= 2.6], [
+      AC_DEFINE([FUSE_USE_VERSION], [26], [used FUSE API version])
+      CPPFLAGS="${libfuse2_CFLAGS} ${CPPFLAGS}"
+      LIBS="${libfuse2_LIBS} $LIBS"
+      AC_CHECK_LIB(fuse, fuse_lowlevel_new, [], [
+        AC_MSG_ERROR([libfuse (>= 2.6) doesn't work properly for lowlevel 
api])])
+      have_fuse="yes"
+    ], [have_fuse="no"])
+  ])
   LIBS="${saved_LIBS}"
   CPPFLAGS="${saved_CPPFLAGS}"], [have_fuse="no"])
 
diff --git a/fuse/Makefile.am b/fuse/Makefile.am
index 50be783..c63efcd 100644
--- a/fuse/Makefile.am
+++ b/fuse/Makefile.am
@@ -5,6 +5,6 @@ noinst_HEADERS = $(top_srcdir)/fuse/macosx.h
 bin_PROGRAMS     = erofsfuse
 erofsfuse_SOURCES = main.c
 erofsfuse_CFLAGS = -Wall -I$(top_srcdir)/include
-erofsfuse_CFLAGS += -DFUSE_USE_VERSION=26 ${libfuse_CFLAGS} 
${libselinux_CFLAGS}
-erofsfuse_LDADD = $(top_builddir)/lib/liberofs.la ${libfuse_LIBS} 
${liblz4_LIBS} \
+erofsfuse_CFLAGS += ${libfuse2_CFLAGS} ${libfuse3_CFLAGS} ${libselinux_CFLAGS}
+erofsfuse_LDADD = $(top_builddir)/lib/liberofs.la ${libfuse2_LIBS} 
${libfuse3_LIBS} ${liblz4_LIBS} \
        ${libselinux_LIBS} ${liblzma_LIBS} ${zlib_LIBS} ${libdeflate_LIBS}
diff --git a/fuse/main.c b/fuse/main.c
index 821d98c..37bd3ab 100644
--- a/fuse/main.c
+++ b/fuse/main.c
@@ -1,13 +1,12 @@
 // SPDX-License-Identifier: GPL-2.0+
 /*
  * Created by Li Guifu <blucer...@gmail.com>
+ * Lowlevel implemented by Li Yiyan <lyy0...@sjtu.edu.com>
  */
 #include <stdlib.h>
 #include <string.h>
 #include <signal.h>
 #include <libgen.h>
-#include <fuse.h>
-#include <fuse_opt.h>
 #include "macosx.h"
 #include "erofs/config.h"
 #include "erofs/print.h"
@@ -15,180 +14,514 @@
 #include "erofs/dir.h"
 #include "erofs/inode.h"
 
-struct erofsfuse_dir_context {
+#include <float.h>
+#include <fuse.h>
+#include <fuse_lowlevel.h>
+
+/* used in list/getxattr, given buf size is 0 and
+ * expected return val is xattr size
+ */
+#define EROFSFUSE_XATTR_BUF_SIZE 4096
+#define EROFSFUSE_TIMEOUT DBL_MAX
+
+struct erofsfuse_ll_dir_context {
        struct erofs_dir_context ctx;
-       fuse_fill_dir_t filler;
-       struct fuse_file_info *fi;
+
+       fuse_req_t req;
        void *buf;
+       int is_plus;
+       size_t offset;
+       size_t buf_size;
+       size_t start_off;
+       struct fuse_file_info *fi;
+};
+
+struct erofsfuse_ll_dir_search_context {
+       struct erofs_dir_context ctx;
+
+       const char *target_name;
+       struct fuse_entry_param *ent;
 };
 
-static int erofsfuse_fill_dentries(struct erofs_dir_context *ctx)
+static int erofsfuse_ll_fill_dentries(struct erofs_dir_context *ctx)
 {
-       struct erofsfuse_dir_context *fusectx = (void *)ctx;
-       struct stat st = {0};
+       size_t r = 0;
+       struct stat st = { 0 };
        char dname[EROFS_NAME_LEN + 1];
+       struct erofsfuse_ll_dir_context *fusectx =
+               (struct erofsfuse_ll_dir_context *)ctx;
+
+       if (fusectx->offset < fusectx->start_off) {
+               fusectx->offset +=
+                       ctx->de_namelen + sizeof(struct erofs_dirent);
+               return 0;
+       }
 
        strncpy(dname, ctx->dname, ctx->de_namelen);
        dname[ctx->de_namelen] = '\0';
-       st.st_mode = erofs_ftype_to_dtype(ctx->de_ftype) << 12;
-       fusectx->filler(fusectx->buf, dname, &st, 0);
+       fusectx->offset += ctx->de_namelen + sizeof(struct erofs_dirent);
+
+       if (!fusectx->is_plus) { /* fuse 3 still use non-plus readdir */
+               st.st_mode = erofs_ftype_to_dtype(ctx->de_ftype);
+               st.st_ino = ctx->de_nid;
+
+               r = fuse_add_direntry(fusectx->req, fusectx->buf,
+                                     fusectx->buf_size, dname, &st,
+                                     fusectx->offset);
+       } else {
+#if FUSE_MAJOR_VERSION >= 3
+               struct fuse_entry_param param;
+
+               param.ino = ctx->de_nid;
+               param.generation = 0;
+               param.attr.st_ino = ctx->de_nid;
+               param.attr.st_mode = erofs_ftype_to_dtype(ctx->de_ftype);
+               param.attr_timeout = EROFSFUSE_TIMEOUT;
+               param.entry_timeout = EROFSFUSE_TIMEOUT;
+
+               r = fuse_add_direntry_plus(fusectx->req, fusectx->buf,
+                                          fusectx->buf_size, dname, &param,
+                                          fusectx->offset);
+#else
+               erofs_err("fuse 2 readdirplus is not supported\n");
+#endif
+       }
+
+       if (r > fusectx->buf_size) {
+               fusectx->offset -=
+                       ctx->de_namelen + sizeof(struct erofs_dirent);
+               return 1;
+       }
+       fusectx->buf += r;
+       fusectx->buf_size -= r;
        return 0;
 }
 
-int erofsfuse_readdir(const char *path, void *buf, fuse_fill_dir_t filler,
-                     off_t offset, struct fuse_file_info *fi)
+static void erofsfuse_ll_fill_stat(struct erofs_inode *vi, struct stat *stbuf)
 {
-       int ret;
-       struct erofs_inode dir;
-       struct erofsfuse_dir_context ctx = {
-               .ctx.dir = &dir,
-               .ctx.cb = erofsfuse_fill_dentries,
-               .filler = filler,
-               .fi = fi,
-               .buf = buf,
-       };
-       erofs_dbg("readdir:%s offset=%llu", path, (long long)offset);
-
-       dir.sbi = &sbi;
-       ret = erofs_ilookup(path, &dir);
-       if (ret)
-               return ret;
+       stbuf->st_mode = vi->i_mode;
+       stbuf->st_nlink = vi->i_nlink;
 
-       erofs_dbg("path=%s nid = %llu", path, dir.nid | 0ULL);
-       if (!S_ISDIR(dir.i_mode))
-               return -ENOTDIR;
+       if (!S_ISDIR(stbuf->st_mode))
+               stbuf->st_size = vi->i_size;
+       if (S_ISBLK(vi->i_mode) || S_ISCHR(vi->i_mode))
+               stbuf->st_rdev = vi->u.i_rdev;
+
+       stbuf->st_blocks = roundup(vi->i_size, erofs_blksiz(&sbi)) >> 9;
+
+       stbuf->st_uid = vi->i_uid;
+       stbuf->st_gid = vi->i_gid;
+
+       stbuf->st_ctime = vi->i_mtime;
+       stbuf->st_mtime = stbuf->st_ctime;
+       stbuf->st_atime = stbuf->st_ctime;
+}
+
+static int erofsfuse_ll_search_dentries(struct erofs_dir_context *ctx)
+{
+       int r = 0;
+       struct erofs_inode vi = { .sbi = &sbi };
+       struct erofsfuse_ll_dir_search_context *search_ctx =
+               (struct erofsfuse_ll_dir_search_context *)ctx;
+
+       if (search_ctx->ent->ino == 0 &&
+           strlen(search_ctx->target_name) == ctx->de_namelen &&
+           !strncmp(search_ctx->target_name, ctx->dname, ctx->de_namelen)) {
+               search_ctx->ent->ino = ctx->de_nid;
+               search_ctx->ent->attr.st_ino = ctx->de_nid;
+               vi.nid = (erofs_nid_t)ctx->de_nid;
+
+               r = erofs_read_inode_from_disk(&vi);
+               if (r < 0)
+                       return r;
+
+               erofsfuse_ll_fill_stat(&vi, &(search_ctx->ent->attr));
+       }
+
+       return 0;
+}
+
+static inline erofs_nid_t erofsfuse_ll_getnid(fuse_ino_t ino)
+{
+       return ino == FUSE_ROOT_ID ? sbi.root_nid : (erofs_nid_t)ino;
+}
+
+static inline void erofsfuse_readdir_general(fuse_req_t req, fuse_ino_t ino,
+                                            size_t size, off_t off,
+                                            struct fuse_file_info *fi,
+                                            int plus)
+{
+       int err = 0;
+       char *buf = malloc(size);
+       struct erofsfuse_ll_dir_context ctx;
+       struct erofs_inode *vi = (struct erofs_inode *)fi->fh;
+
+       erofs_dbg("readdir, ino: %lu, req: %p, fh: %lu, size: %lu, off: %lu\n",
+                 ino, req, fi->fh, size, off);
+       if (!buf) {
+               fuse_reply_err(req, ENOMEM);
+               return;
+       }
+       ctx.ctx.dir = vi;
+       ctx.ctx.cb = erofsfuse_ll_fill_dentries;
+
+       ctx.fi = fi;
+       ctx.buf = buf;
+       ctx.buf_size = size;
+       ctx.req = req;
+       ctx.offset = 0;
+       ctx.start_off = off;
+       ctx.is_plus = plus;
 
-       if (!dir.i_size)
-               return 0;
 #ifdef NDEBUG
-       return erofs_iterate_dir(&ctx.ctx, false);
+       err = erofs_iterate_dir(&ctx.ctx, false);
 #else
-       return erofs_iterate_dir(&ctx.ctx, true);
+       err = erofs_iterate_dir(&ctx.ctx, true);
 #endif
+
+       if (err < 0) /* if buffer insufficient, return 1 */
+               fuse_reply_err(req, EIO);
+       else
+               fuse_reply_buf(req, buf, size - ctx.buf_size);
+
+       free(buf);
+}
+
+static void erofsfuse_ll_readdir(fuse_req_t req, fuse_ino_t ino, size_t size,
+                                off_t off, struct fuse_file_info *fi)
+{
+       erofsfuse_readdir_general(req, ino, size, off, fi, 0);
 }
 
-static void *erofsfuse_init(struct fuse_conn_info *info)
+#if FUSE_MAJOR_VERSION >= 3
+static void erofsfuse_ll_readdirplus(fuse_req_t req, fuse_ino_t ino,
+                                    size_t size, off_t off,
+                                    struct fuse_file_info *fi)
 {
-       erofs_info("Using FUSE protocol %d.%d", info->proto_major, 
info->proto_minor);
-       return NULL;
+       erofsfuse_readdir_general(req, ino, size, off, fi, 1);
 }
+#endif
 
-static int erofsfuse_open(const char *path, struct fuse_file_info *fi)
+static void erofsfuse_ll_init(void *userdata, struct fuse_conn_info *conn)
 {
-       erofs_dbg("open path=%s", path);
+       erofs_dbg("init fuse lowlevel erofs\n");
+}
 
-       if ((fi->flags & O_ACCMODE) != O_RDONLY)
-               return -EACCES;
+static void erofsfuse_ll_open(fuse_req_t req, fuse_ino_t ino,
+                             struct fuse_file_info *fi)
+{
+       int ret = 0;
+       struct erofs_inode *vi;
 
-       return 0;
+       erofs_dbg("open, ino = %lu, req = %p\n", ino, req);
+       if (fi->flags & (O_WRONLY | O_RDWR)) {
+               fuse_reply_err(req, EROFS);
+               return;
+       }
+
+       if (ino == FUSE_ROOT_ID) {
+               fuse_reply_err(req, EISDIR);
+               return;
+       }
+
+       vi = (struct erofs_inode *)malloc(sizeof(struct erofs_inode));
+       if (!vi) {
+               fuse_reply_err(req, ENOMEM);
+               return;
+       }
+
+       vi->sbi = &sbi;
+       vi->nid = (erofs_nid_t)ino;
+       ret = erofs_read_inode_from_disk(vi);
+       if (ret < 0) {
+               fuse_reply_err(req, EIO);
+               goto out;
+       }
+
+       if (!S_ISREG(vi->i_mode)) {
+               fuse_reply_err(req, EISDIR);
+       } else {
+               fi->fh = (uint64_t)vi;
+               fi->keep_cache = 1;
+               fuse_reply_open(req, fi);
+               return;
+       }
+
+out:
+       free(vi);
 }
 
-static int erofsfuse_getattr(const char *path, struct stat *stbuf)
+static void erofsfuse_ll_getattr(fuse_req_t req, fuse_ino_t ino,
+                                struct fuse_file_info *fi)
 {
-       struct erofs_inode vi = { .sbi = &sbi };
        int ret;
+       struct stat stbuf = { 0 };
+       struct erofs_inode vi = { .sbi = &sbi };
 
-       erofs_dbg("getattr(%s)", path);
-       ret = erofs_ilookup(path, &vi);
-       if (ret)
-               return -ENOENT;
-
-       stbuf->st_mode  = vi.i_mode;
-       stbuf->st_nlink = vi.i_nlink;
-       stbuf->st_size  = vi.i_size;
-       stbuf->st_blocks = roundup(vi.i_size, erofs_blksiz(vi.sbi)) >> 9;
-       stbuf->st_uid = vi.i_uid;
-       stbuf->st_gid = vi.i_gid;
-       if (S_ISBLK(vi.i_mode) || S_ISCHR(vi.i_mode))
-               stbuf->st_rdev = vi.u.i_rdev;
-       stbuf->st_ctime = vi.i_mtime;
-       stbuf->st_mtime = stbuf->st_ctime;
-       stbuf->st_atime = stbuf->st_ctime;
-       return 0;
+       erofs_dbg("getattr, ino: %lu, req: %p\n", ino, req);
+       vi.nid = erofsfuse_ll_getnid(ino);
+       ret = erofs_read_inode_from_disk(&vi);
+       if (ret < 0) {
+               erofs_dbg("read inode from disk failed, nid = %lu\n", vi.nid);
+               fuse_reply_err(req, ENOENT);
+       }
+
+       erofsfuse_ll_fill_stat(&vi, &stbuf);
+       stbuf.st_ino = ino;
+
+       fuse_reply_attr(req, &stbuf, EROFSFUSE_TIMEOUT);
 }
 
-static int erofsfuse_read(const char *path, char *buffer,
-                         size_t size, off_t offset,
-                         struct fuse_file_info *fi)
+static void erofsfuse_ll_opendir(fuse_req_t req, fuse_ino_t ino,
+                                struct fuse_file_info *fi)
 {
        int ret;
-       struct erofs_inode vi;
+       struct erofs_inode *vi;
 
-       erofs_dbg("path:%s size=%zd offset=%llu", path, size, (long 
long)offset);
+       erofs_dbg("opendir, ino = %lu\n", ino);
+       vi = (struct erofs_inode *)malloc(sizeof(struct erofs_inode));
+       if (!vi) {
+               fuse_reply_err(req, ENOMEM);
+               return;
+       }
 
-       vi.sbi = &sbi;
-       ret = erofs_ilookup(path, &vi);
-       if (ret)
-               return ret;
+       vi->sbi = &sbi;
+       vi->nid = erofsfuse_ll_getnid(ino);
+       ret = erofs_read_inode_from_disk(vi);
+       if (ret < 0) {
+               fuse_reply_err(req, EIO);
+               goto out;
+       }
 
-       ret = erofs_pread(&vi, buffer, size, offset);
-       if (ret)
-               return ret;
-       if (offset >= vi.i_size)
-               return 0;
-       if (offset + size > vi.i_size)
-               return vi.i_size - offset;
-       return size;
+       if (!S_ISDIR(vi->i_mode)) {
+               fuse_reply_err(req, ENOTDIR);
+               goto out;
+       }
+
+       fi->fh = (uint64_t)vi;
+       fuse_reply_open(req, fi);
+       return;
+
+out:
+       free(vi);
 }
 
-static int erofsfuse_readlink(const char *path, char *buffer, size_t size)
+static void erofsfuse_ll_release(fuse_req_t req, fuse_ino_t ino,
+                                struct fuse_file_info *fi)
 {
-       int ret = erofsfuse_read(path, buffer, size, 0, NULL);
-
-       if (ret < 0)
-               return ret;
-       DBG_BUGON(ret > size);
-       if (ret == size)
-               buffer[size - 1] = '\0';
-       erofs_dbg("readlink(%s): %s", path, buffer);
-       return 0;
+       free((struct erofs_inode *)fi->fh);
+       fi->fh = 0;
+       fuse_reply_err(req, 0);
 }
 
-static int erofsfuse_getxattr(const char *path, const char *name, char *value,
-                       size_t size
-#ifdef __APPLE__
-                       , uint32_t position)
+static void erofsfuse_ll_lookup(fuse_req_t req, fuse_ino_t parent,
+                               const char *name)
+{
+       int err, ret;
+       struct erofs_inode *vi;
+       struct fuse_entry_param fentry;
+       struct erofsfuse_ll_dir_search_context ctx;
+
+       vi = (struct erofs_inode *)malloc(sizeof(struct erofs_inode));
+       if (!vi) {
+               fuse_reply_err(req, ENOMEM);
+               return;
+       }
+
+       vi->sbi = &sbi;
+       vi->nid = erofsfuse_ll_getnid(parent);
+       ret = erofs_read_inode_from_disk(vi);
+       if (ret < 0) {
+               fuse_reply_err(req, EIO);
+               goto out;
+       }
+       if (!S_ISDIR(vi->i_mode)) {
+               fuse_reply_err(req, ENOTDIR);
+               goto out;
+       }
+
+       memset(&fentry, 0, sizeof(fentry));
+       fentry.ino = 0;
+       fentry.attr_timeout = fentry.entry_timeout = EROFSFUSE_TIMEOUT;
+       ctx.ctx.dir = vi;
+       ctx.ctx.cb = erofsfuse_ll_search_dentries;
+
+       ctx.ent = &fentry;
+       ctx.target_name = name;
+
+#ifdef NDEBUG
+       err = erofs_iterate_dir(&ctx.ctx, false);
 #else
-                       )
+       err = erofs_iterate_dir(&ctx.ctx, true);
 #endif
+
+       if (err < 0) {
+               fuse_reply_err(req, EIO);
+               goto out;
+       }
+       fuse_reply_entry(req, &fentry);
+
+out:
+       free(vi);
+}
+
+static void erofsfuse_ll_read(fuse_req_t req, fuse_ino_t ino, size_t size,
+                             off_t off, struct fuse_file_info *fi)
 {
        int ret;
-       struct erofs_inode vi;
+       struct erofs_inode *vi = (struct erofs_inode *)fi->fh;
+       char *buf = malloc(size);
+
+       erofs_dbg("read, ino = %lu, size = %lu, off = %lu, fh = %lu\n", ino,
+                 size, off, fi->fh);
+       if (!buf) {
+               fuse_reply_err(req, ENOMEM);
+               return;
+       }
 
-       erofs_dbg("getxattr(%s): name=%s size=%llu", path, name, size);
+       if (!S_ISREG(vi->i_mode)) {
+               fuse_reply_err(req, EIO);
+               goto out;
+       }
 
-       vi.sbi = &sbi;
-       ret = erofs_ilookup(path, &vi);
-       if (ret)
-               return ret;
+       ret = erofs_pread(vi, buf, size, off);
+       if (ret) {
+               fuse_reply_err(req, EIO);
+               goto out;
+       }
+       if (off >= vi->i_size)
+               ret = 0;
+       else if (off + size > vi->i_size)
+               ret = vi->i_size - off;
+       else
+               ret = size;
 
-       return erofs_getxattr(&vi, name, value, size);
+       fuse_reply_buf(req, buf, ret);
+
+out:
+       free(buf);
 }
 
-static int erofsfuse_listxattr(const char *path, char *list, size_t size)
+static void erofsfuse_ll_readlink(fuse_req_t req, fuse_ino_t ino)
 {
        int ret;
-       struct erofs_inode vi;
+       char *dst;
+       struct erofs_inode vi = { .sbi = &sbi };
 
-       erofs_dbg("listxattr(%s): size=%llu", path, size);
+       erofs_dbg("read_link, ino = %lu\n", ino);
+       vi.nid = erofsfuse_ll_getnid(ino);
+       ret = erofs_read_inode_from_disk(&vi);
+       if (ret < 0) {
+               fuse_reply_err(req, EIO);
+               return;
+       }
 
-       vi.sbi = &sbi;
-       ret = erofs_ilookup(path, &vi);
-       if (ret)
-               return ret;
+       if (!S_ISLNK(vi.i_mode)) {
+               fuse_reply_err(req, EINVAL);
+               return;
+       }
+
+       dst = malloc(vi.i_size + 1);
+       if (!dst) {
+               fuse_reply_err(req, ENOMEM);
+               return;
+       }
+
+       ret = erofs_pread(&vi, dst, vi.i_size, 0);
+       if (ret < 0) {
+               fuse_reply_err(req, EIO);
+               goto out;
+       }
 
-       return erofs_listxattr(&vi, list, size);
+       dst[vi.i_size] = '\0';
+       fuse_reply_readlink(req, dst);
+
+out:
+       free(dst);
+}
+
+static void erofsfuse_ll_getxattr(fuse_req_t req, fuse_ino_t ino,
+                                 const char *name, size_t size)
+{
+       int ret;
+       char *buf = NULL;
+       struct erofs_inode vi = { .sbi = &sbi };
+       size_t real = size == 0 ? EROFSFUSE_XATTR_BUF_SIZE : size;
+
+       erofs_dbg("getxattr, ino = %lu, name = %s, size = %lu\n", ino, name,
+                 size);
+       vi.nid = erofsfuse_ll_getnid(ino);
+       ret = erofs_read_inode_from_disk(&vi);
+       if (ret < 0) {
+               fuse_reply_err(req, EIO);
+               return;
+       }
+
+       buf = malloc(real);
+       if (!buf) {
+               fuse_reply_err(req, ENOMEM);
+               return;
+       }
+
+       ret = erofs_getxattr(&vi, name, buf, real);
+       if (ret < 0)
+               fuse_reply_err(req, -ret);
+       else if (size == 0)
+               fuse_reply_xattr(req, ret);
+       else
+               fuse_reply_buf(req, buf, ret);
+
+       free(buf);
+}
+
+static void erofsfuse_ll_listxattr(fuse_req_t req, fuse_ino_t ino, size_t size)
+{
+       int ret;
+       char *buf = NULL;
+       struct erofs_inode vi = { .sbi = &sbi };
+       size_t real = size == 0 ? EROFSFUSE_XATTR_BUF_SIZE : size;
+
+       erofs_dbg("listxattr, ino = %lu, size = %lu\n", ino, size);
+       vi.nid = erofsfuse_ll_getnid(ino);
+       ret = erofs_read_inode_from_disk(&vi);
+       if (ret < 0) {
+               fuse_reply_err(req, EIO);
+               return;
+       }
+
+       buf = malloc(real);
+       if (!buf) {
+               fuse_reply_err(req, ENOMEM);
+               return;
+       }
+
+       ret = erofs_listxattr(&vi, buf, real);
+       if (ret < 0)
+               fuse_reply_err(req, -ret);
+       else if (size == 0)
+               fuse_reply_xattr(req, ret);
+       else
+               fuse_reply_buf(req, buf, ret);
+
+       free(buf);
 }
 
-static struct fuse_operations erofs_ops = {
-       .getxattr = erofsfuse_getxattr,
-       .listxattr = erofsfuse_listxattr,
-       .readlink = erofsfuse_readlink,
-       .getattr = erofsfuse_getattr,
-       .readdir = erofsfuse_readdir,
-       .open = erofsfuse_open,
-       .read = erofsfuse_read,
-       .init = erofsfuse_init,
+struct fuse_lowlevel_ops erofsfuse_lops = {
+       .getxattr = erofsfuse_ll_getxattr,
+       .opendir = erofsfuse_ll_opendir,
+       .releasedir = erofsfuse_ll_release,
+       .release = erofsfuse_ll_release,
+       .lookup = erofsfuse_ll_lookup,
+       .listxattr = erofsfuse_ll_listxattr,
+       .readlink = erofsfuse_ll_readlink,
+       .getattr = erofsfuse_ll_getattr,
+       .readdir = erofsfuse_ll_readdir,
+#if FUSE_MAJOR_VERSION >= 3
+       .readdirplus = erofsfuse_ll_readdirplus,
+#endif
+       .open = erofsfuse_ll_open,
+       .read = erofsfuse_ll_read,
+       .init = erofsfuse_ll_init,
 };
 
 static struct options {
@@ -200,18 +533,20 @@ static struct options {
        bool odebug;
 } fusecfg;
 
-#define OPTION(t, p) { t, offsetof(struct options, p), 1 }
+#define OPTION(t, p)                              \
+       {                                         \
+               t, offsetof(struct options, p), 1 \
+       }
 static const struct fuse_opt option_spec[] = {
-       OPTION("--offset=%lu", offset),
-       OPTION("--dbglevel=%u", debug_lvl),
-       OPTION("--help", show_help),
-       FUSE_OPT_KEY("--device=", 1),
-       FUSE_OPT_END
+       OPTION("--offset=%lu", offset), OPTION("--dbglevel=%u", debug_lvl),
+       OPTION("--help", show_help), FUSE_OPT_KEY("--device=", 1), FUSE_OPT_END
 };
 
 static void usage(void)
 {
+#if FUSE_MAJOR_VERSION < 3
        struct fuse_args args = FUSE_ARGS_INIT(0, NULL);
+#endif
 
        fputs("usage: [options] IMAGE MOUNTPOINT\n\n"
              "Options:\n"
@@ -221,7 +556,8 @@ static void usage(void)
 #if FUSE_MAJOR_VERSION < 3
              "    --help                 display this help and exit\n"
 #endif
-             "\n", stderr);
+             "\n",
+             stderr);
 
 #if FUSE_MAJOR_VERSION >= 3
        fuse_cmdline_help();
@@ -261,8 +597,10 @@ static int optional_opt_func(void *data, const char *arg, 
int key,
                        fusecfg.disk = strdup(arg);
                        return 0;
                }
-               if (!fusecfg.mountpoint)
+               if (!fusecfg.mountpoint) {
                        fusecfg.mountpoint = strdup(arg);
+                       return 0;
+               }
        case FUSE_OPT_KEY_OPT:
                if (!strcmp(arg, "-d"))
                        fusecfg.odebug = true;
@@ -301,10 +639,11 @@ static void signal_handle_sigsegv(int signal)
 int main(int argc, char *argv[])
 {
        int ret;
+       struct fuse_session *se;
        struct fuse_args args = FUSE_ARGS_INIT(argc, argv);
 
        erofs_init_configure();
-       printf("%s %s\n", basename(argv[0]), cfg.c_version);
+       erofs_dbg("%s %s\n", basename(argv[0]), cfg.c_version);
 
 #if defined(HAVE_EXECINFO_H) && defined(HAVE_BACKTRACE)
        if (signal(SIGSEGV, signal_handle_sigsegv) == SIG_ERR) {
@@ -341,7 +680,38 @@ int main(int argc, char *argv[])
                goto err_dev_close;
        }
 
-       ret = fuse_main(args.argc, args.argv, &erofs_ops, NULL);
+#if FUSE_MAJOR_VERSION >= 3
+       se = fuse_session_new(&args, &erofsfuse_lops, sizeof(erofsfuse_lops),
+                             NULL);
+       if (se != NULL) {
+               fuse_set_signal_handlers(se);
+               fuse_session_mount(se, fusecfg.mountpoint);
+
+               ret = fuse_session_loop(se);
+
+               fuse_remove_signal_handlers(se);
+               fuse_session_unmount(se);
+               fuse_session_destroy(se);
+       }
+#else
+       struct fuse_chan *ch;
+
+       ch = fuse_mount(fusecfg.mountpoint, &args);
+       if (ch != NULL) {
+               se = fuse_lowlevel_new(&args, &erofsfuse_lops,
+                                      sizeof(erofsfuse_lops), NULL);
+               if (se != NULL) {
+                       if (fuse_set_signal_handlers(se) != -1) {
+                               fuse_session_add_chan(se, ch);
+                               ret = fuse_session_loop(se);
+                               fuse_remove_signal_handlers(se);
+                               fuse_session_remove_chan(ch);
+                       }
+                       fuse_session_destroy(se);
+               }
+               fuse_unmount(fusecfg.mountpoint, ch);
+       }
+#endif
 
        erofs_put_super(&sbi);
 err_dev_close:
-- 
2.34.1

Reply via email to