Commit cfcbfb13 authored by Josef Bacik's avatar Josef Bacik Committed by Linus Torvalds

mm/filemap.c: enable error injection at add_to_page_cache()

Recently I messed up the error handling in filemap_fault() because of an
unexpected ENOMEM (related to cgroup memory limits) in add_to_page_cache.
Enable error injection at this point so I can add a testcase to xfstests
to verify I don't mess this up again.

[akpm@linux-foundation.org: include linux/error-injection.h]
Link: http://lkml.kernel.org/r/20190403152604.14008-1-josef@toxicpanda.comSigned-off-by: default avatarJosef Bacik <josef@toxicpanda.com>
Reviewed-by: default avatarWilliam Kucharski <william.kucharski@oracle.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent c6d23413
...@@ -24,6 +24,7 @@ ...@@ -24,6 +24,7 @@
#include <linux/pagemap.h> #include <linux/pagemap.h>
#include <linux/file.h> #include <linux/file.h>
#include <linux/uio.h> #include <linux/uio.h>
#include <linux/error-injection.h>
#include <linux/hash.h> #include <linux/hash.h>
#include <linux/writeback.h> #include <linux/writeback.h>
#include <linux/backing-dev.h> #include <linux/backing-dev.h>
...@@ -882,6 +883,7 @@ static int __add_to_page_cache_locked(struct page *page, ...@@ -882,6 +883,7 @@ static int __add_to_page_cache_locked(struct page *page,
put_page(page); put_page(page);
return xas_error(&xas); return xas_error(&xas);
} }
ALLOW_ERROR_INJECTION(__add_to_page_cache_locked, ERRNO);
/** /**
* add_to_page_cache_locked - add a locked page to the pagecache * add_to_page_cache_locked - add a locked page to the pagecache
......
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