Skip to content

Commit

Permalink
Revert "ceph: make members in struct ceph_mds_request_args_ext a union"
Browse files Browse the repository at this point in the history
This reverts commit 3af5ae2.

ceph_mds_request_args_ext was already (and remains to be) a union.  An
additional anonymous union inside is bogus:

    union ceph_mds_request_args_ext {
        union {
            union ceph_mds_request_args old;
            struct { ... } __attribute__ ((packed)) setattr_ext;
        };
    }

Signed-off-by: Ilya Dryomov <[email protected]>
Reviewed-by: Xiubo Li <[email protected]>
  • Loading branch information
idryomov committed Sep 18, 2023
1 parent 2816a09 commit d57125b
Showing 1 changed file with 11 additions and 13 deletions.
24 changes: 11 additions & 13 deletions include/linux/ceph/ceph_fs.h
Original file line number Diff line number Diff line change
Expand Up @@ -467,19 +467,17 @@ union ceph_mds_request_args {
} __attribute__ ((packed));

union ceph_mds_request_args_ext {
union {
union ceph_mds_request_args old;
struct {
__le32 mode;
__le32 uid;
__le32 gid;
struct ceph_timespec mtime;
struct ceph_timespec atime;
__le64 size, old_size; /* old_size needed by truncate */
__le32 mask; /* CEPH_SETATTR_* */
struct ceph_timespec btime;
} __attribute__ ((packed)) setattr_ext;
};
union ceph_mds_request_args old;
struct {
__le32 mode;
__le32 uid;
__le32 gid;
struct ceph_timespec mtime;
struct ceph_timespec atime;
__le64 size, old_size; /* old_size needed by truncate */
__le32 mask; /* CEPH_SETATTR_* */
struct ceph_timespec btime;
} __attribute__ ((packed)) setattr_ext;
};

#define CEPH_MDS_FLAG_REPLAY 1 /* this is a replayed op */
Expand Down

0 comments on commit d57125b

Please sign in to comment.