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 | |
| 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 '')
| -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) { |