From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chao Yu Subject: Re: mount fail with option fault_injection < 20000 && >0 Date: Tue, 18 Sep 2018 20:21:28 +0800 Message-ID: References: <61afbde5-f05e-9bd5-8dc2-31932c13cd51@gmx.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from [172.30.20.202] (helo=mx.sourceforge.net) by sfs-ml-1.v29.lw.sourceforge.com with esmtps (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.90_1) (envelope-from ) id 1g2F0k-0001OL-5K for linux-f2fs-devel@lists.sourceforge.net; Tue, 18 Sep 2018 12:21:46 +0000 Received: from mail.kernel.org ([198.145.29.99]) by sfi-mx-2.v28.lw.sourceforge.com with esmtps (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.90_1) id 1g2F0i-00ErEn-3a for linux-f2fs-devel@lists.sourceforge.net; Tue, 18 Sep 2018 12:21:45 +0000 In-Reply-To: <61afbde5-f05e-9bd5-8dc2-31932c13cd51@gmx.com> Content-Language: en-US List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: linux-f2fs-devel-bounces@lists.sourceforge.net To: cgxu519 , linux-f2fs-devel@lists.sourceforge.net Cc: jaegeuk@kernel.org Hi Chengguang, On 2018/9/18 19:02, cgxu519 wrote: > HI guys, > > I found mount with option fault_injection < 20000 && >0 will fail with error > -ENOMEM. > I understand fault_injection function is explicitly mocking some errors, but If we skip injecting error into memory allocation functions during mount, we may miss checking most error paths in fill_super(). So I think we really need this during mount. But if you want to inject error after mount, you can reenable error injection by configuring fault_injection/fault_rate sysfs entry. Thanks, > isn't it better > mocking errors after successful mount? Is it really allocating memory so many times > during mount process? > > Thanks, > Chengguang > > > > > > > > _______________________________________________ > Linux-f2fs-devel mailing list > Linux-f2fs-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel