| Submitter | Jeff Layton |
|---|---|
| Date | June 1, 2010, 2:54 p.m. |
| Message ID | <1275404092-8400-2-git-send-email-jlayton@redhat.com> |
| Download | mbox | patch |
| Permalink | /patch/54217/ |
| State | New |
| Headers | show |
Comments
merged. Will wait a few days before request upstream. Reviewing and checking the others. On Tue, Jun 1, 2010 at 9:54 AM, Jeff Layton <jlayton@redhat.com> wrote: > Commit 315e995c63a15cb4d4efdbfd70fe2db191917f7a is causing OOM kills > when stress-testing a CIFS filesystem. The VFS readpages operation takes > a page reference. The older code just handed this reference off to the > page cache, but the new code takes an extra one. The simplest fix is to > put the new reference after add_to_page_cache_lru. > > Signed-off-by: Jeff Layton <jlayton@redhat.com> > Acked-by: Nick Piggin <npiggin@suse.de> > --- > fs/cifs/file.c | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/fs/cifs/file.c b/fs/cifs/file.c > index f1ff785..75541af 100644 > --- a/fs/cifs/file.c > +++ b/fs/cifs/file.c > @@ -1952,6 +1952,7 @@ static void cifs_copy_cache_pages(struct address_space *mapping, > bytes_read -= PAGE_CACHE_SIZE; > continue; > } > + page_cache_release(page); > > target = kmap_atomic(page, KM_USER0); > > -- > 1.6.6.1 > >
Patch
diff --git a/fs/cifs/file.c b/fs/cifs/file.c index f1ff785..75541af 100644 --- a/fs/cifs/file.c +++ b/fs/cifs/file.c @@ -1952,6 +1952,7 @@ static void cifs_copy_cache_pages(struct address_space *mapping, bytes_read -= PAGE_CACHE_SIZE; continue; } + page_cache_release(page); target = kmap_atomic(page, KM_USER0);