diff options
| author |   Jeff Layton <jlayton@redhat.com> | 2017-02-01 13:49:09 -0500 |
|---|---|---|
| committer |   Ilya Dryomov <idryomov@gmail.com> | 2017-02-20 12:16:08 +0100 |
| commit | bc2de10dc4da5036ada3381775bd966f0c21c603 (patch) | |
| tree | 5da0f735f7d5c87b23f7fef5e8fd135a39be12f7 /fs/ceph/debugfs.c | |
| parent | ceph: drop session argument to ceph_fill_trace (diff) | |
| download | linux-dev-bc2de10dc4da5036ada3381775bd966f0c21c603.tar.xz linux-dev-bc2de10dc4da5036ada3381775bd966f0c21c603.zip | |
ceph: convert bools in ceph_mds_request to a new r_req_flags field
Currently, we have a bunch of bool flags in struct ceph_mds_request. We
need more flags though, but each bool takes (at least) a byte. Those
add up over time.
Merge all of the existing bools in this struct into a single unsigned
long, and use the set/test/clear_bit macros to manipulate them. These
are atomic operations, but that is required here to prevent
load/modify/store races. The existing flags are protected by different
locks, so we can't rely on them for that purpose.
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Reviewed-by: Yan, Zheng <zyan@redhat.com>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Diffstat (limited to 'fs/ceph/debugfs.c')
| -rw-r--r-- | fs/ceph/debugfs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/ceph/debugfs.c b/fs/ceph/debugfs.c index 39ff678e567f..f2ae393e2c31 100644 --- a/fs/ceph/debugfs.c +++ b/fs/ceph/debugfs.c @@ -70,7 +70,7 @@ static int mdsc_show(struct seq_file *s, void *p) seq_printf(s, "%s", ceph_mds_op_name(req->r_op)); - if (req->r_got_unsafe) + if (test_bit(CEPH_MDS_R_GOT_UNSAFE, &req->r_req_flags)) seq_puts(s, "\t(unsafe)"); else seq_puts(s, "\t"); |