Re: [PATCH] ceph: reset cap hold timeout only for requeued inode

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jul 30, 2018 at 11:59 PM Chengguang Xu <cgxu519@xxxxxxx> wrote:
>
> __cap_delay_requeue() only requeue inode which does not
> have CEPH_I_FLUSH flag, so avoid reset cap hold timeout
> for that inode.
>
> Signed-off-by: Chengguang Xu <cgxu519@xxxxxxx>
> ---
>  fs/ceph/caps.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/ceph/caps.c b/fs/ceph/caps.c
> index f50cc008632a..3ed77faa6dc6 100644
> --- a/fs/ceph/caps.c
> +++ b/fs/ceph/caps.c
> @@ -535,7 +535,6 @@ static void __cap_set_timeouts(struct ceph_mds_client *mdsc,
>  static void __cap_delay_requeue(struct ceph_mds_client *mdsc,
>                                 struct ceph_inode_info *ci)
>  {
> -       __cap_set_timeouts(mdsc, ci);
>         dout("__cap_delay_requeue %p flags %d at %lu\n", &ci->vfs_inode,
>              ci->i_ceph_flags, ci->i_hold_caps_max);
>         if (!mdsc->stopping) {
> @@ -545,6 +544,7 @@ static void __cap_delay_requeue(struct ceph_mds_client *mdsc,
>                                 goto no_change;
>                         list_del_init(&ci->i_cap_delay_list);
>                 }
> +               __cap_set_timeouts(mdsc, ci);
>                 list_add_tail(&ci->i_cap_delay_list, &mdsc->cap_delay_list);
>  no_change:
>                 spin_unlock(&mdsc->cap_delay_lock);
> --
> 2.17.1
>

Applied, thanks

Yan, Zheng
> --
> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux