Scratch this. btrfs_releasepage() is called from btrfs_invalidatepage()
as well. Sorry for the noise.
On 13:15 18/10, Goldwyn Rodrigues wrote:
> From: Goldwyn Rodrigues <rgoldwyn@xxxxxxxx>
>
> We check for PageWriteback in try_to_release_page(), the
> sole caller of address_space->releasepage(). We don't need
> to check it again in btrfs_releasepage()
>
> Signed-off-by: Goldwyn Rodrigues <rgoldwyn@xxxxxxxx>
> ---
> fs/btrfs/inode.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c
> index 0f2754eaa05b..10303c2379a9 100644
> --- a/fs/btrfs/inode.c
> +++ b/fs/btrfs/inode.c
> @@ -8793,7 +8793,7 @@ static int __btrfs_releasepage(struct page *page, gfp_t gfp_flags)
>
> static int btrfs_releasepage(struct page *page, gfp_t gfp_flags)
> {
> - if (PageWriteback(page) || PageDirty(page))
> + if (PageDirty(page))
> return 0;
> return __btrfs_releasepage(page, gfp_flags);
> }
> --
> 2.16.4
>
--
Goldwyn
