Whamcloud - gitweb
LU-12587 llite: don't check vmpage refcount in ll_releasepage()
We could not use vmpage refcount to check whether page could be
released because it break invalidate_complete_page2():
See comments:
/*
* This is like invalidate_complete_page(), except it ignores the page's
* refcount. We do this because invalidate_inode_pages2() needs stronger
* invalidation guarantees, and cannot afford to leave pages behind because
* shrink_page_list() has a temp ref on them, or because they're transiently
* sitting in the lru_cache_add() pagevecs.
*/
So checking refcount > 3 might be wrong here, one common
case is page might be transiently in lru_cache_add().
Since we have checked whether vmpage is used by cl_page later in the
function, and vmpage will be locked before called, it should be safe
to remove vmpage refcount check.
One of problem currently is following DIO will mostly
fall back to Buffer IO:
$ dd if=/dev/zero of=data bs=1M count=1
$ dd if=/dev/zero of=data bs=1M count=1 oflag=direct conv=notrunc
Which is because DIO will firstly try to writeback and invalidate
clean page which fail because vmpage refcount could be 4 here.
Function calls come from linux-3.10.0-957.1.3.el7.x86_64:
|->generic_file_direct_write()
|->filemap_write_and_wait_range()
|->invalidate_inode_pages2_range()
|->invalidate_complete_page2() If a page can not be invalidated,
return 0 to fall back to buffered write.
|->try_to_release_page()
|->ll_releasepage()
return 0 becuase of vmpage count is 4 > 3
|->generic_file_buffered_write
Change-Id: I14ed0877a789d68dee2de80574688a0e699556a3
Signed-off-by: Wang Shilong <wshilong@ddn.com>
Reviewed-on: https://review.whamcloud.com/35610
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Patrick Farrell <pfarrell@whamcloud.com>
Reviewed-by: Li Dongyang <dongyangli@ddn.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>