Commit a19b89ca authored by Jason Uhlenkott's avatar Jason Uhlenkott Committed by Trond Myklebust

NFS: Clean up nfs_create_request comments

Remove some stale comments about hard limits which went away in 2.5.
Signed-off-by: default avatarJason Uhlenkott <juhlenko@akamai.com>
Signed-off-by: default avatarTrond Myklebust <Trond.Myklebust@netapp.com>
parent 61322b30
...@@ -51,9 +51,7 @@ nfs_page_free(struct nfs_page *p) ...@@ -51,9 +51,7 @@ nfs_page_free(struct nfs_page *p)
* @count: number of bytes to read/write * @count: number of bytes to read/write
* *
* The page must be locked by the caller. This makes sure we never * The page must be locked by the caller. This makes sure we never
* create two different requests for the same page, and avoids * create two different requests for the same page.
* a possible deadlock when we reach the hard limit on the number
* of dirty pages.
* User should ensure it is safe to sleep in this function. * User should ensure it is safe to sleep in this function.
*/ */
struct nfs_page * struct nfs_page *
...@@ -64,16 +62,12 @@ nfs_create_request(struct nfs_open_context *ctx, struct inode *inode, ...@@ -64,16 +62,12 @@ nfs_create_request(struct nfs_open_context *ctx, struct inode *inode,
struct nfs_server *server = NFS_SERVER(inode); struct nfs_server *server = NFS_SERVER(inode);
struct nfs_page *req; struct nfs_page *req;
/* Deal with hard limits. */
for (;;) { for (;;) {
/* try to allocate the request struct */ /* try to allocate the request struct */
req = nfs_page_alloc(); req = nfs_page_alloc();
if (req != NULL) if (req != NULL)
break; break;
/* Try to free up at least one request in order to stay
* below the hard limit
*/
if (signalled() && (server->flags & NFS_MOUNT_INTR)) if (signalled() && (server->flags & NFS_MOUNT_INTR))
return ERR_PTR(-ERESTARTSYS); return ERR_PTR(-ERESTARTSYS);
yield(); yield();
......
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