Re: [PATCH 1/2] ceph: protect kick_requests() with mdsc->mutex

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

 



Reviewed-by: Sage Weil <sage@xxxxxxxxxx>

On Thu, 11 Sep 2014, Yan, Zheng wrote:

> From: "Yan, Zheng" <ukernel@xxxxxxxxx>
> 
> Signed-off-by: Yan, Zheng <zyan@xxxxxxxxxx>
> ---
>  fs/ceph/mds_client.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/fs/ceph/mds_client.c b/fs/ceph/mds_client.c
> index f751fea..267ba44 100644
> --- a/fs/ceph/mds_client.c
> +++ b/fs/ceph/mds_client.c
> @@ -2471,9 +2471,8 @@ static void handle_session(struct ceph_mds_session *session,
>  		if (session->s_state == CEPH_MDS_SESSION_RECONNECTING)
>  			pr_info("mds%d reconnect denied\n", session->s_mds);
>  		remove_session_caps(session);
> -		wake = 1; /* for good measure */
> +		wake = 2; /* for good measure */
>  		wake_up_all(&mdsc->session_close_wq);
> -		kick_requests(mdsc, mds);
>  		break;
>  
>  	case CEPH_SESSION_STALE:
> @@ -2503,6 +2502,8 @@ static void handle_session(struct ceph_mds_session *session,
>  	if (wake) {
>  		mutex_lock(&mdsc->mutex);
>  		__wake_requests(mdsc, &session->s_waiting);
> +		if (wake == 2)
> +			kick_requests(mdsc, mds);
>  		mutex_unlock(&mdsc->mutex);
>  	}
>  	return;
> -- 
> 1.9.3
> 
> --
> 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