diff options
author | Ilya Dryomov <idryomov@gmail.com> | 2017-06-22 19:44:05 +0200 |
---|---|---|
committer | Ilya Dryomov <idryomov@gmail.com> | 2017-07-07 17:25:19 +0200 |
commit | b88ed8d84fbd1e652cc7a1f6e03550d2b4edf653 (patch) | |
tree | e6c75686594a16874cd1edce8bd50b5638dcb69d /net/ceph | |
parent | libceph, crush: per-pool crush_choose_arg_map for crush_do_rule() (diff) | |
download | linux-b88ed8d84fbd1e652cc7a1f6e03550d2b4edf653.tar.xz linux-b88ed8d84fbd1e652cc7a1f6e03550d2b4edf653.zip |
crush: crush_init_workspace starts with struct crush_work
It is not just a pointer to crush_work, it is the whole structure.
That is not a problem since it only contains a pointer. But it will
be a problem if new data members are added to crush_work.
Reflects ceph.git commit ee957dd431bfbeb6dadaf77764db8e0757417328.
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Diffstat (limited to 'net/ceph')
-rw-r--r-- | net/ceph/crush/mapper.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ceph/crush/mapper.c b/net/ceph/crush/mapper.c index 0b2646a9cc50..9887fce04219 100644 --- a/net/ceph/crush/mapper.c +++ b/net/ceph/crush/mapper.c @@ -858,7 +858,7 @@ void crush_init_workspace(const struct crush_map *map, void *v) * set the pointer first and then reserve the space for it to * point to by incrementing the point. */ - v += sizeof(struct crush_work *); + v += sizeof(struct crush_work); w->work = v; v += map->max_buckets * sizeof(struct crush_work_bucket *); for (b = 0; b < map->max_buckets; ++b) { |