From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751566AbdJVQvP (ORCPT ); Sun, 22 Oct 2017 12:51:15 -0400 Received: from mail.kernel.org ([198.145.29.99]:35216 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751226AbdJVQvN (ORCPT ); Sun, 22 Oct 2017 12:51:13 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 83F1321908 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=jaegeuk@kernel.org Date: Sun, 22 Oct 2017 09:51:09 -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 2/2] f2fs: relax EIO injection for quota file Message-ID: <20171022165109.GA33676@jaegeuk-macbookpro.roam.corp.google.com> References: <20171019165617.79128-1-jaegeuk@kernel.org> <20171019165617.79128-2-jaegeuk@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.8.2 (2017-04-18) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10/22, Chao Yu wrote: > On 2017/10/20 0:56, Jaegeuk Kim wrote: > > This case is not happening easily. > > Actually it can happen, so why not just keep it? Okay, so let me keep this patch for local stress tests only. Thanks, > > Thanks, > > > > > Signed-off-by: Jaegeuk Kim > > --- > > fs/f2fs/f2fs.h | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h > > index e0ef31cb2cc6..6301ccca8888 100644 > > --- a/fs/f2fs/f2fs.h > > +++ b/fs/f2fs/f2fs.h > > @@ -1544,7 +1544,7 @@ static inline int inc_valid_block_count(struct f2fs_sb_info *sbi, > > return ret; > > > > #ifdef CONFIG_F2FS_FAULT_INJECTION > > - if (time_to_inject(sbi, FAULT_BLOCK)) { > > + if (!IS_NOQUOTA(inode) && time_to_inject(sbi, FAULT_BLOCK)) { > > f2fs_show_injection_info(FAULT_BLOCK); > > release = *count; > > goto enospc; > >