Re: [PATCH] nfs: remove roc_rpcwaitq

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

 



Please ignore this. We'll need the roc_rpcwaitq back when adding ROC
waiters on layoutreturn.

Cheers,
Tao

On Wed, Sep 9, 2015 at 5:17 PM, Peng Tao <tao.peng@xxxxxxxxxxxxxxx> wrote:
>
> There are no users of it after we remove pnfs_roc_drain.
>
> Signed-off-by: Peng Tao <tao.peng@xxxxxxxxxxxxxxx>
> ---
>  fs/nfs/client.c           | 10 ----------
>  fs/nfs/pnfs.c             |  4 ----
>  include/linux/nfs_fs_sb.h |  1 -
>  3 files changed, 15 deletions(-)
>
> diff --git a/fs/nfs/client.c b/fs/nfs/client.c
> index 57c5a02..46ce7bf 100644
> --- a/fs/nfs/client.c
> +++ b/fs/nfs/client.c
> @@ -215,11 +215,6 @@ static void nfs_cb_idr_remove_locked(struct nfs_client *clp)
>                 idr_remove(&nn->cb_ident_idr, clp->cl_cb_ident);
>  }
>
> -static void pnfs_init_server(struct nfs_server *server)
> -{
> -       rpc_init_wait_queue(&server->roc_rpcwaitq, "pNFS ROC");
> -}
> -
>  #else
>  void nfs_cleanup_cb_ident_idr(struct net *net)
>  {
> @@ -229,10 +224,6 @@ static void nfs_cb_idr_remove_locked(struct nfs_client *clp)
>  {
>  }
>
> -static void pnfs_init_server(struct nfs_server *server)
> -{
> -}
> -
>  #endif /* CONFIG_NFS_V4 */
>
>  /*
> @@ -900,7 +891,6 @@ struct nfs_server *nfs_alloc_server(void)
>
>         ida_init(&server->openowner_id);
>         ida_init(&server->lockowner_id);
> -       pnfs_init_server(server);
>
>         return server;
>  }
> diff --git a/fs/nfs/pnfs.c b/fs/nfs/pnfs.c
> index ba12464..dcc3571 100644
> --- a/fs/nfs/pnfs.c
> +++ b/fs/nfs/pnfs.c
> @@ -334,15 +334,12 @@ static void
>  pnfs_layout_remove_lseg(struct pnfs_layout_hdr *lo,
>                 struct pnfs_layout_segment *lseg)
>  {
> -       struct inode *inode = lo->plh_inode;
> -
>         WARN_ON(test_bit(NFS_LSEG_VALID, &lseg->pls_flags));
>         list_del_init(&lseg->pls_list);
>         /* Matched by pnfs_get_layout_hdr in pnfs_layout_insert_lseg */
>         atomic_dec(&lo->plh_refcount);
>         if (list_empty(&lo->plh_segs))
>                 clear_bit(NFS_LAYOUT_BULK_RECALL, &lo->plh_flags);
> -       rpc_wake_up(&NFS_SERVER(inode)->roc_rpcwaitq);
>  }
>
>  /* Return true if layoutreturn is needed */
> @@ -936,7 +933,6 @@ void pnfs_clear_layoutreturn_waitbit(struct pnfs_layout_hdr *lo)
>         clear_bit_unlock(NFS_LAYOUT_RETURN, &lo->plh_flags);
>         smp_mb__after_atomic();
>         wake_up_bit(&lo->plh_flags, NFS_LAYOUT_RETURN);
> -       rpc_wake_up(&NFS_SERVER(lo->plh_inode)->roc_rpcwaitq);
>  }
>
>  static int
> diff --git a/include/linux/nfs_fs_sb.h b/include/linux/nfs_fs_sb.h
> index 570a7df..fad6b61 100644
> --- a/include/linux/nfs_fs_sb.h
> +++ b/include/linux/nfs_fs_sb.h
> @@ -190,7 +190,6 @@ struct nfs_server {
>                                                    handle volatility type for
>                                                    this filesystem */
>         struct pnfs_layoutdriver_type  *pnfs_curr_ld; /* Active layout driver */
> -       struct rpc_wait_queue   roc_rpcwaitq;
>         void                    *pnfs_ld_data;  /* per mount point data */
>
>         /* the following fields are protected by nfs_client->cl_lock */
> --
> 1.8.3.1
>
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux