Commit df52699e authored by Trond Myklebust's avatar Trond Myklebust

NFSv4.1: Don't cache deviceids that have no notifications

The spec says that once all layouts that reference a given deviceid
have been returned, then we are only allowed to continue to cache
the deviceid if the metadata server supports notifications.
Signed-off-by: default avatarTrond Myklebust <trond.myklebust@primarydata.com>
parent 4e590803
...@@ -7962,6 +7962,8 @@ _nfs4_proc_getdeviceinfo(struct nfs_server *server, ...@@ -7962,6 +7962,8 @@ _nfs4_proc_getdeviceinfo(struct nfs_server *server,
status = nfs4_call_sync(server->client, server, &msg, &args.seq_args, &res.seq_res, 0); status = nfs4_call_sync(server->client, server, &msg, &args.seq_args, &res.seq_res, 0);
if (res.notification & ~args.notify_types) if (res.notification & ~args.notify_types)
dprintk("%s: unsupported notification\n", __func__); dprintk("%s: unsupported notification\n", __func__);
if (res.notification != args.notify_types)
pdev->nocache = 1;
dprintk("<-- %s status=%d\n", __func__, status); dprintk("<-- %s status=%d\n", __func__, status);
......
...@@ -203,6 +203,7 @@ struct pnfs_device { ...@@ -203,6 +203,7 @@ struct pnfs_device {
struct page **pages; struct page **pages;
unsigned int pgbase; unsigned int pgbase;
unsigned int pglen; /* reply buffer length */ unsigned int pglen; /* reply buffer length */
unsigned char nocache : 1;/* May not be cached */
}; };
#define NFS4_PNFS_GETDEVLIST_MAXNUM 16 #define NFS4_PNFS_GETDEVLIST_MAXNUM 16
...@@ -291,6 +292,7 @@ void pnfs_error_mark_layout_for_return(struct inode *inode, ...@@ -291,6 +292,7 @@ void pnfs_error_mark_layout_for_return(struct inode *inode,
enum { enum {
NFS_DEVICEID_INVALID = 0, /* set when MDS clientid recalled */ NFS_DEVICEID_INVALID = 0, /* set when MDS clientid recalled */
NFS_DEVICEID_UNAVAILABLE, /* device temporarily unavailable */ NFS_DEVICEID_UNAVAILABLE, /* device temporarily unavailable */
NFS_DEVICEID_NOCACHE, /* device may not be cached */
}; };
/* pnfs_dev.c */ /* pnfs_dev.c */
......
...@@ -149,6 +149,8 @@ nfs4_get_device_info(struct nfs_server *server, ...@@ -149,6 +149,8 @@ nfs4_get_device_info(struct nfs_server *server,
*/ */
d = server->pnfs_curr_ld->alloc_deviceid_node(server, pdev, d = server->pnfs_curr_ld->alloc_deviceid_node(server, pdev,
gfp_flags); gfp_flags);
if (d && pdev->nocache)
set_bit(NFS_DEVICEID_NOCACHE, &d->flags);
out_free_pages: out_free_pages:
for (i = 0; i < max_pages; i++) for (i = 0; i < max_pages; i++)
...@@ -235,6 +237,7 @@ nfs4_delete_deviceid(const struct pnfs_layoutdriver_type *ld, ...@@ -235,6 +237,7 @@ nfs4_delete_deviceid(const struct pnfs_layoutdriver_type *ld,
return; return;
} }
hlist_del_init_rcu(&d->node); hlist_del_init_rcu(&d->node);
clear_bit(NFS_DEVICEID_NOCACHE, &d->flags);
spin_unlock(&nfs4_deviceid_lock); spin_unlock(&nfs4_deviceid_lock);
/* balance the initial ref set in pnfs_insert_deviceid */ /* balance the initial ref set in pnfs_insert_deviceid */
...@@ -269,6 +272,11 @@ EXPORT_SYMBOL_GPL(nfs4_init_deviceid_node); ...@@ -269,6 +272,11 @@ EXPORT_SYMBOL_GPL(nfs4_init_deviceid_node);
bool bool
nfs4_put_deviceid_node(struct nfs4_deviceid_node *d) nfs4_put_deviceid_node(struct nfs4_deviceid_node *d)
{ {
if (test_bit(NFS_DEVICEID_NOCACHE, &d->flags)) {
if (atomic_add_unless(&d->ref, -1, 2))
return false;
nfs4_delete_deviceid(d->ld, d->nfs_client, &d->deviceid);
}
if (!atomic_dec_and_test(&d->ref)) if (!atomic_dec_and_test(&d->ref))
return false; return false;
d->ld->free_deviceid_node(d); d->ld->free_deviceid_node(d);
...@@ -312,6 +320,7 @@ _deviceid_purge_client(const struct nfs_client *clp, long hash) ...@@ -312,6 +320,7 @@ _deviceid_purge_client(const struct nfs_client *clp, long hash)
if (d->nfs_client == clp && atomic_read(&d->ref)) { if (d->nfs_client == clp && atomic_read(&d->ref)) {
hlist_del_init_rcu(&d->node); hlist_del_init_rcu(&d->node);
hlist_add_head(&d->tmpnode, &tmp); hlist_add_head(&d->tmpnode, &tmp);
clear_bit(NFS_DEVICEID_NOCACHE, &d->flags);
} }
rcu_read_unlock(); rcu_read_unlock();
spin_unlock(&nfs4_deviceid_lock); spin_unlock(&nfs4_deviceid_lock);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment