From mboxrd@z Thu Jan 1 00:00:00 1970 From: "HUANG Weller (CM/ESW12-CN)" Subject: RE: ext4 out of order when use cfq scheduler Date: Fri, 8 Jan 2016 00:46:42 +0000 Message-ID: References: <697280a570654ae0aa1723fb7d11f51e@SGPMBX1004.APAC.bosch.com> <20151222150037.GB18178@quack.suse.cz> <20160105153050.GF14464@quack.suse.cz> <20160106100621.GA24046@quack.suse.cz> <3ab48fa47e434455b101251730e69bd2@SGPMBX1004.APAC.bosch.com> <20160107102420.GB8380@quack.suse.cz> <20160107114736.GC8380@quack.suse.cz> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT Cc: "linux-ext4@vger.kernel.org" , "Li, Michael" To: Jan Kara Return-path: Received: from smtp6-v.fe.bosch.de ([139.15.237.11]:59287 "EHLO smtp6-v.fe.bosch.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753264AbcAHAqr convert rfc822-to-8bit (ORCPT ); Thu, 7 Jan 2016 19:46:47 -0500 In-Reply-To: <20160107114736.GC8380@quack.suse.cz> Content-Language: en-US Sender: linux-ext4-owner@vger.kernel.org List-ID: > > > > > > > > OK, so I was looking into the code and indeed, reality is correct > > > and my mental model was wrong! ;) I thought that inode gets added to > > > the list of inodes for which we need to wait for data IO completion > > > during transaction commit during block allocation. And I was wrong. > > > It used to happen in > > > mpage_da_map_and_submit() until commit f3b59291a69d (ext4: remove > > > calls to > > > ext4_jbd2_file_inode() from delalloc write path) where it got > > > removed. And that was wrong because although we submit data writes > > > before dropping handle for allocating transaction and updating > > > i_size, nobody guarantees that data IO is not delayed in the block layer until > transaction commit. > > > Which seems to happen in your case. I'll send a fix. Thanks for your > > > report and persistence! > > > > > > > Thanks a lot for your feedback :-) > > Because I am not familiar with the detail of the ext4 internal code. I will try to > understand your explanation which you describe above. And have a look on > related funcations. > > Could you send the fix in this mail ? > > And whether the kernel 3.14 also have such issue, right ? > > The problem is in all kernels starting with 3.8. Attached is a patch which should fix > the issue. Can you test whether it fixes the problem for you? > > Honza > -- Yes, of course I will redo the test with the patch. And also give you feedback. Thanks. Huang weller > Jan Kara > SUSE Labs, CR