diff options
author | Yan, Zheng <zheng.z.yan@intel.com> | 2013-08-05 14:10:29 +0800 |
---|---|---|
committer | Sage Weil <sage@inktank.com> | 2013-08-15 11:12:11 -0700 |
commit | 3871cbb9a41b1371dc13fc619e3ab4e0a1e29b4a (patch) | |
tree | 527e72ea247c0bc632a28f021aa4b8149ecf78e8 /net | |
parent | b0d7c2231015b331b942746610a05b6ea72977ab (diff) | |
download | op-kernel-dev-3871cbb9a41b1371dc13fc619e3ab4e0a1e29b4a.zip op-kernel-dev-3871cbb9a41b1371dc13fc619e3ab4e0a1e29b4a.tar.gz |
ceph: fix request max size
ceph_check_caps() requests new max size only when there is Fw cap.
If we call check_max_size() while there is no Fw cap. It updates
i_wanted_max_size and calls ceph_check_caps(), but ceph_check_caps()
does nothing. Later when Fw cap is issued, we call check_max_size()
again. But i_wanted_max_size is equal to 'endoff' at this time, so
check_max_size() doesn't call ceph_check_caps() and we end up with
waiting for the new max size forever.
The fix is duplicate ceph_check_caps()'s "request max size" code in
check_max_size(), and make try_get_cap_refs() wait for the Fw cap
before retry requesting new max size.
This patch also removes the "endoff > (inode->i_size << 1)" check
in check_max_size(). It's useless because there is no corresponding
logic in ceph_check_caps().
Reviewed-by: Sage Weil <sage@inktank.com>
Signed-off-by: Yan, Zheng <zheng.z.yan@intel.com>
Diffstat (limited to 'net')
0 files changed, 0 insertions, 0 deletions