lkml.org 
[lkml]   [2009]   [Nov]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[84/99] NFSv4: Kill nfs4_renewd_prepare_shutdown()
    2.6.31-stable review patch.  If anyone has any objections, please let us know.

    ------------------
    From: Trond Myklebust <Trond.Myklebust@netapp.com>

    commit 3050141bae57984dd660e6861632ccf9b8bca77e upstream.

    The NFSv4 renew daemon is shared between all active super blocks that refer
    to a particular NFS server, so it is wrong to be shutting it down in
    nfs4_kill_super every time a super block is destroyed.

    This patch therefore kills nfs4_renewd_prepare_shutdown altogether, and
    leaves it up to nfs4_shutdown_client() to also shut down the renew daemon
    by means of the existing call to nfs4_kill_renewd().

    Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

    ---
    fs/nfs/nfs4renewd.c | 6 ------
    fs/nfs/super.c | 1 -
    2 files changed, 7 deletions(-)

    --- a/fs/nfs/nfs4renewd.c
    +++ b/fs/nfs/nfs4renewd.c
    @@ -127,12 +127,6 @@ nfs4_schedule_state_renewal(struct nfs_c
    }

    void
    -nfs4_renewd_prepare_shutdown(struct nfs_server *server)
    -{
    - cancel_delayed_work(&server->nfs_client->cl_renewd);
    -}
    -
    -void
    nfs4_kill_renewd(struct nfs_client *clp)
    {
    cancel_delayed_work_sync(&clp->cl_renewd);
    --- a/fs/nfs/super.c
    +++ b/fs/nfs/super.c
    @@ -2670,7 +2670,6 @@ static void nfs4_kill_super(struct super
    dprintk("--> %s\n", __func__);
    nfs_super_return_all_delegations(sb);
    kill_anon_super(sb);
    - nfs4_renewd_prepare_shutdown(server);
    nfs_fscache_release_super_cookie(sb);
    nfs_free_server(server);
    dprintk("<-- %s\n", __func__);



    \
     
     \ /
      Last update: 2009-11-06 23:33    [W:4.306 / U:0.288 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site