]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
fuse: rearrange and resize fuse_args fields
authorMiklos Szeredi <mszeredi@redhat.com>
Tue, 10 Sep 2019 13:04:08 +0000 (15:04 +0200)
committerMiklos Szeredi <mszeredi@redhat.com>
Tue, 10 Sep 2019 14:29:48 +0000 (16:29 +0200)
This makes the structure better packed.

Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>
fs/fuse/fuse_i.h
fs/fuse/xattr.c

index 835c0671320c2b27ec9c94730ac2daf6fcd3f791..a89362ee46d932665bf470105ce588a73e987b03 100644 (file)
@@ -287,12 +287,12 @@ struct fuse_page_desc {
 };
 
 struct fuse_args {
-       uint32_t opcode;
        uint64_t nodeid;
-       unsigned int in_numargs;
+       uint32_t opcode;
+       unsigned short in_numargs;
+       unsigned short out_numargs;
+       bool out_argvar:1;
        struct fuse_in_arg in_args[3];
-       unsigned int out_argvar:1;
-       unsigned int out_numargs;
        struct fuse_arg out_args[2];
 };
 
index 2e02486e46e6b519c1f6f4a3e99338d85c46dc07..20d052e08b3be1fcd30415696bd20e34e29e874c 100644 (file)
@@ -70,7 +70,7 @@ ssize_t fuse_getxattr(struct inode *inode, const char *name, void *value,
        /* This is really two different operations rolled into one */
        args.out_numargs = 1;
        if (size) {
-               args.out_argvar = 1;
+               args.out_argvar = true;
                args.out_args[0].size = size;
                args.out_args[0].value = value;
        } else {
@@ -129,7 +129,7 @@ ssize_t fuse_listxattr(struct dentry *entry, char *list, size_t size)
        /* This is really two different operations rolled into one */
        args.out_numargs = 1;
        if (size) {
-               args.out_argvar = 1;
+               args.out_argvar = true;
                args.out_args[0].size = size;
                args.out_args[0].value = list;
        } else {