diff options
| author | Jason Uhlenkott <juhlenko@akamai.com> | 2007-04-26 20:25:51 -0400 |
|---|---|---|
| committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2007-05-02 10:37:29 -0400 |
| commit | a19b89cad51b6f0da8f4bafdfdcfb10264cbcdea (patch) | |
| tree | 65631769b94790d187ef2f556016758ea3bd6f83 | |
| parent | 61322b30139b79ec77170723a3a80043dcc94e87 (diff) | |
NFS: Clean up nfs_create_request comments
Remove some stale comments about hard limits which went away in 2.5.
Signed-off-by: Jason Uhlenkott <juhlenko@akamai.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
| -rw-r--r-- | fs/nfs/pagelist.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/fs/nfs/pagelist.c b/fs/nfs/pagelist.c index d846d39a31ef..fe90130bd7a9 100644 --- a/fs/nfs/pagelist.c +++ b/fs/nfs/pagelist.c | |||
| @@ -51,9 +51,7 @@ nfs_page_free(struct nfs_page *p) | |||
| 51 | * @count: number of bytes to read/write | 51 | * @count: number of bytes to read/write |
| 52 | * | 52 | * |
| 53 | * The page must be locked by the caller. This makes sure we never | 53 | * The page must be locked by the caller. This makes sure we never |
| 54 | * create two different requests for the same page, and avoids | 54 | * create two different requests for the same page. |
| 55 | * a possible deadlock when we reach the hard limit on the number | ||
| 56 | * of dirty pages. | ||
| 57 | * User should ensure it is safe to sleep in this function. | 55 | * User should ensure it is safe to sleep in this function. |
| 58 | */ | 56 | */ |
| 59 | struct nfs_page * | 57 | struct nfs_page * |
| @@ -64,16 +62,12 @@ nfs_create_request(struct nfs_open_context *ctx, struct inode *inode, | |||
| 64 | struct nfs_server *server = NFS_SERVER(inode); | 62 | struct nfs_server *server = NFS_SERVER(inode); |
| 65 | struct nfs_page *req; | 63 | struct nfs_page *req; |
| 66 | 64 | ||
| 67 | /* Deal with hard limits. */ | ||
| 68 | for (;;) { | 65 | for (;;) { |
| 69 | /* try to allocate the request struct */ | 66 | /* try to allocate the request struct */ |
| 70 | req = nfs_page_alloc(); | 67 | req = nfs_page_alloc(); |
| 71 | if (req != NULL) | 68 | if (req != NULL) |
| 72 | break; | 69 | break; |
| 73 | 70 | ||
| 74 | /* Try to free up at least one request in order to stay | ||
| 75 | * below the hard limit | ||
| 76 | */ | ||
| 77 | if (signalled() && (server->flags & NFS_MOUNT_INTR)) | 71 | if (signalled() && (server->flags & NFS_MOUNT_INTR)) |
| 78 | return ERR_PTR(-ERESTARTSYS); | 72 | return ERR_PTR(-ERESTARTSYS); |
| 79 | yield(); | 73 | yield(); |
