btrfs: send: remove unused type parameter to iterate_inode_ref_t
authorOmar Sandoval <osandov@fb.com>
Fri, 5 Nov 2021 00:00:13 +0000 (17:00 -0700)
committerDavid Sterba <dsterba@suse.com>
Mon, 3 Jan 2022 14:09:44 +0000 (15:09 +0100)
Again, I don't think this was ever used since iterate_dir_item() is only
used for xattrs. No functional change.

Reviewed-by: Josef Bacik <josef@toxicpanda.com>
Reviewed-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: Omar Sandoval <osandov@fb.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/send.c

index 877276f34d72ce66a0f3dc1fb8649d9355e648fb..6bdcb9d481d52c3fd4d7fab2f5ccb3d5319f1839 100644 (file)
@@ -1002,7 +1002,7 @@ out:
 typedef int (*iterate_dir_item_t)(int num, struct btrfs_key *di_key,
                                  const char *name, int name_len,
                                  const char *data, int data_len,
-                                 u8 type, void *ctx);
+                                 void *ctx);
 
 /*
  * Helper function to iterate the entries in ONE btrfs_dir_item.
@@ -1027,7 +1027,6 @@ static int iterate_dir_item(struct btrfs_root *root, struct btrfs_path *path,
        u32 total;
        int slot;
        int num;
-       u8 type;
 
        /*
         * Start with a small buffer (1 page). If later we end up needing more
@@ -1053,10 +1052,9 @@ static int iterate_dir_item(struct btrfs_root *root, struct btrfs_path *path,
        while (cur < total) {
                name_len = btrfs_dir_name_len(eb, di);
                data_len = btrfs_dir_data_len(eb, di);
-               type = btrfs_dir_type(eb, di);
                btrfs_dir_item_key_to_cpu(eb, di, &di_key);
 
-               if (type == BTRFS_FT_XATTR) {
+               if (btrfs_dir_type(eb, di) == BTRFS_FT_XATTR) {
                        if (name_len > XATTR_NAME_MAX) {
                                ret = -ENAMETOOLONG;
                                goto out;
@@ -1106,7 +1104,7 @@ static int iterate_dir_item(struct btrfs_root *root, struct btrfs_path *path,
                cur += len;
 
                ret = iterate(num, &di_key, buf, name_len, buf + name_len,
-                               data_len, type, ctx);
+                             data_len, ctx);
                if (ret < 0)
                        goto out;
                if (ret) {
@@ -4643,9 +4641,8 @@ out:
 }
 
 static int __process_new_xattr(int num, struct btrfs_key *di_key,
-                              const char *name, int name_len,
-                              const char *data, int data_len,
-                              u8 type, void *ctx)
+                              const char *name, int name_len, const char *data,
+                              int data_len, void *ctx)
 {
        int ret;
        struct send_ctx *sctx = ctx;
@@ -4689,8 +4686,7 @@ out:
 
 static int __process_deleted_xattr(int num, struct btrfs_key *di_key,
                                   const char *name, int name_len,
-                                  const char *data, int data_len,
-                                  u8 type, void *ctx)
+                                  const char *data, int data_len, void *ctx)
 {
        int ret;
        struct send_ctx *sctx = ctx;
@@ -4735,10 +4731,8 @@ struct find_xattr_ctx {
        int found_data_len;
 };
 
-static int __find_xattr(int num, struct btrfs_key *di_key,
-                       const char *name, int name_len,
-                       const char *data, int data_len,
-                       u8 type, void *vctx)
+static int __find_xattr(int num, struct btrfs_key *di_key, const char *name,
+                       int name_len, const char *data, int data_len, void *vctx)
 {
        struct find_xattr_ctx *ctx = vctx;
 
@@ -4788,7 +4782,7 @@ static int find_xattr(struct btrfs_root *root,
 static int __process_changed_new_xattr(int num, struct btrfs_key *di_key,
                                       const char *name, int name_len,
                                       const char *data, int data_len,
-                                      u8 type, void *ctx)
+                                      void *ctx)
 {
        int ret;
        struct send_ctx *sctx = ctx;
@@ -4800,12 +4794,12 @@ static int __process_changed_new_xattr(int num, struct btrfs_key *di_key,
                         &found_data_len);
        if (ret == -ENOENT) {
                ret = __process_new_xattr(num, di_key, name, name_len, data,
-                               data_len, type, ctx);
+                                         data_len, ctx);
        } else if (ret >= 0) {
                if (data_len != found_data_len ||
                    memcmp(data, found_data, data_len)) {
                        ret = __process_new_xattr(num, di_key, name, name_len,
-                                       data, data_len, type, ctx);
+                                                 data, data_len, ctx);
                } else {
                        ret = 0;
                }
@@ -4818,7 +4812,7 @@ static int __process_changed_new_xattr(int num, struct btrfs_key *di_key,
 static int __process_changed_deleted_xattr(int num, struct btrfs_key *di_key,
                                           const char *name, int name_len,
                                           const char *data, int data_len,
-                                          u8 type, void *ctx)
+                                          void *ctx)
 {
        int ret;
        struct send_ctx *sctx = ctx;
@@ -4827,7 +4821,7 @@ static int __process_changed_deleted_xattr(int num, struct btrfs_key *di_key,
                         name, name_len, NULL, NULL);
        if (ret == -ENOENT)
                ret = __process_deleted_xattr(num, di_key, name, name_len, data,
-                               data_len, type, ctx);
+                                             data_len, ctx);
        else if (ret >= 0)
                ret = 0;