Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753996AbbGOWmx (ORCPT ); Wed, 15 Jul 2015 18:42:53 -0400 Received: from mail.kernel.org ([198.145.29.136]:34103 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753340AbbGOWmv (ORCPT ); Wed, 15 Jul 2015 18:42:51 -0400 Date: Wed, 15 Jul 2015 15:42:48 -0700 From: Jaegeuk Kim To: Chao Yu Cc: linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-f2fs-devel@lists.sourceforge.net Subject: Re: [f2fs-dev] [PATCH 1/2] f2fs: handle error cases in move_encrypted_block Message-ID: <20150715224248.GA86133@jaegeuk-mac02.mot.com> References: <1436901502-78825-1-git-send-email-jaegeuk@kernel.org> <012501d0be9f$3ca52dc0$b5ef8940$@samsung.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <012501d0be9f$3ca52dc0$b5ef8940$@samsung.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2467 Lines: 77 On Wed, Jul 15, 2015 at 09:39:47AM +0800, Chao Yu wrote: > Hi Jaegeuk, > > > -----Original Message----- > > From: Jaegeuk Kim [mailto:jaegeuk@kernel.org] > > Sent: Wednesday, July 15, 2015 3:18 AM > > To: linux-kernel@vger.kernel.org; linux-fsdevel@vger.kernel.org; > > linux-f2fs-devel@lists.sourceforge.net > > Cc: Jaegeuk Kim > > Subject: [f2fs-dev] [PATCH 1/2] f2fs: handle error cases in move_encrypted_block > > > > This patch fixes some missing error handlers. > > > > Signed-off-by: Jaegeuk Kim > > --- > > fs/f2fs/gc.c | 25 ++++++++++++++++--------- > > 1 file changed, 16 insertions(+), 9 deletions(-) > > > > diff --git a/fs/f2fs/gc.c b/fs/f2fs/gc.c > > index 11046db..db11861 100644 > > --- a/fs/f2fs/gc.c > > +++ b/fs/f2fs/gc.c > > @@ -556,27 +556,34 @@ static void move_encrypted_block(struct inode *inode, block_t bidx) > > if (!fio.encrypted_page) > > goto put_out; > > > > - f2fs_submit_page_bio(&fio); > > - > > - /* allocate block address */ > > - f2fs_wait_on_page_writeback(dn.node_page, NODE); > > - > > - allocate_data_block(fio.sbi, NULL, fio.blk_addr, > > - &fio.blk_addr, &sum, CURSEG_COLD_DATA); > > - dn.data_blkaddr = fio.blk_addr; > > + err = f2fs_submit_page_bio(&fio); > > + if (err) > > + goto put_page_out; > > f2fs_submit_page_bio will put the page when failed. > > So goto put_out is enough? Right. BTW, I realized that this is somewhat wrong, since this function needs to take care of its page for error cases too. I wrote a patch dealing with errors of f2fs_submit_page_bio, and submitted it right ago. > > > > > /* write page */ > > lock_page(fio.encrypted_page); > > + > > + if (unlikely(!PageUptodate(fio.encrypted_page))) > > + goto put_page_out; > > + if (unlikely(fio.encrypted_page->mapping != META_MAPPING(fio.sbi))) > > + goto put_page_out; > > + > > set_page_writeback(fio.encrypted_page); > > fio.rw = WRITE_SYNC; > > f2fs_submit_page_mbio(&fio); > > > > + /* allocate block address */ > > + f2fs_wait_on_page_writeback(dn.node_page, NODE); > > + allocate_data_block(fio.sbi, NULL, fio.blk_addr, > > + &fio.blk_addr, &sum, CURSEG_COLD_DATA); > > move above f2fs_submit_page_mbio(&fio) to here? Fixed. Thanks, -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/