From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jaegeuk Kim Subject: Re: [PATCH 1/4] f2fs: fix to clear private data in page Date: Thu, 28 Apr 2016 16:29:09 -0700 Message-ID: <20160428232909.GA41124@jaegeuk.gateway> References: <572228AC.30109@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <572228AC.30109@kernel.org> Sender: linux-kernel-owner@vger.kernel.org To: Chao Yu Cc: linux-f2fs-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org List-Id: linux-f2fs-devel.lists.sourceforge.net Hi Chao, Could you check your email setting? All the patches are corrupted, as git am says. Thanks, On Thu, Apr 28, 2016 at 11:13:48PM +0800, Chao Yu wrote: > From: Chao Yu > > Private data in page should be removed during ->releasepage or > ->invalidatepage, otherwise garbage data would be remained in that page. > > Signed-off-by: Chao Yu > --- > fs/f2fs/data.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c > index 38ce5d6..888f178 100644 > --- a/fs/f2fs/data.c > +++ b/fs/f2fs/data.c > @@ -1723,6 +1723,7 @@ void f2fs_invalidate_page(struct page *page, unsigned int > offset, > if (IS_ATOMIC_WRITTEN_PAGE(page)) > return; > > + set_page_private(page, 0); > ClearPagePrivate(page); > } > > @@ -1736,6 +1737,7 @@ int f2fs_release_page(struct page *page, gfp_t wait) > if (IS_ATOMIC_WRITTEN_PAGE(page)) > return 0; > > + set_page_private(page, 0); > ClearPagePrivate(page); > return 1; > } > -- > 2.7.2