Message ID | 20230315013128.3911115-1-chengzhihao1@huawei.com |
---|---|
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:5915:0:0:0:0:0 with SMTP id v21csp2084422wrd; Tue, 14 Mar 2023 18:47:20 -0700 (PDT) X-Google-Smtp-Source: AK7set9JDIKXEO6uAO4Mqt4B7gp1+8ZMj54+k+SbDC0ILLzfDvLt47jiMcLOS3zFRzqfg7ohkcMi X-Received: by 2002:a05:6a21:788b:b0:cb:77f0:9a27 with SMTP id bf11-20020a056a21788b00b000cb77f09a27mr45012828pzc.24.1678844840085; Tue, 14 Mar 2023 18:47:20 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1678844840; cv=none; d=google.com; s=arc-20160816; b=gq3CLpH45SskNJu6aHPrw6O1hjg7m53lT5DCvHf+/o3cAnHd+7EgRImluHmurlxYie 5MhHFKZ1KcO/3OcC7+g38ajxD+eJDQOL+XVX4sxKQ4CXOxfdXEgfEV/sNo001ytHn9Gs vdTB7v+zMZ457StCZoPfN6xhardJ1Eavjxjkev9EMFPUvZlJb9uk551S3Z1A3bPN82J3 lx5O21K3tD8eDWRq/+Vckm3zfzAo9HqXPcwJ48AkHAYpReP3Sie2h9B1jx6UTlfpve9A 93SNFEltP3nPSXbqY+/bmPtwOmmvC8d8ejAoUxymriqgixhd+CqF++fDTg0QPzz4liTh tufw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=uaPuuhuo9VNjtEyXnJpeuZ8b5EbIx4iWuQWO955XB5Q=; b=SNMA/SiInyiCvDtdcJ1QJmT1Ipu1yzSL7piZuYGp0pIAZuEtRleozQ7mrs53toqhgI Xyc0oN47FtDkn87npkYPP6vyj/tF8JDFJ81ktr35RovgIKX0jwIRM08YeRVk0bUeOw8h b/dTzWs0fvuFMFb3UU1rq8Vrcvt+mANcHjQ7Lxum6Nm0sCDIvo/Qyfi+/T28KeWnyleO ELnyC/zpTFAHSIdPT2WfoLdu3nWqTKTAXV23avwQBlhL6JE+ZGjYB29gZkPIPaGAvNxl iEUDR/wqPxQwUyZpcX7SYixuEQrM2p/OATslKhU/YY3GYOYk7Z9N0q+lwCg69NOo6mbY Nd1g== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=huawei.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id s1-20020a63dc01000000b00502fe0f38f4si3619738pgg.676.2023.03.14.18.47.05; Tue, 14 Mar 2023 18:47:20 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=huawei.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230166AbjCOBcz (ORCPT <rfc822;realc9580@gmail.com> + 99 others); Tue, 14 Mar 2023 21:32:55 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43458 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230110AbjCOBcv (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Tue, 14 Mar 2023 21:32:51 -0400 Received: from szxga02-in.huawei.com (szxga02-in.huawei.com [45.249.212.188]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D359E1F5EB; Tue, 14 Mar 2023 18:32:17 -0700 (PDT) Received: from kwepemm600013.china.huawei.com (unknown [172.30.72.57]) by szxga02-in.huawei.com (SkyGuard) with ESMTP id 4Pbt9m5msqzHwwG; Wed, 15 Mar 2023 09:30:04 +0800 (CST) Received: from huawei.com (10.175.127.227) by kwepemm600013.china.huawei.com (7.193.23.68) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.21; Wed, 15 Mar 2023 09:32:14 +0800 From: Zhihao Cheng <chengzhihao1@huawei.com> To: <tytso@mit.edu>, <adilger.kernel@dilger.ca>, <jack@suse.com>, <tudor.ambarus@linaro.org> CC: <linux-ext4@vger.kernel.org>, <linux-kernel@vger.kernel.org>, <chengzhihao1@huawei.com>, <yi.zhang@huawei.com> Subject: [PATCH v3 0/6] ext4: Fix stale buffer loading from last failed Date: Wed, 15 Mar 2023 09:31:22 +0800 Message-ID: <20230315013128.3911115-1-chengzhihao1@huawei.com> X-Mailer: git-send-email 2.31.1 MIME-Version: 1.0 Content-Transfer-Encoding: 7BIT Content-Type: text/plain; charset=US-ASCII X-Originating-IP: [10.175.127.227] X-ClientProxiedBy: dggems705-chm.china.huawei.com (10.3.19.182) To kwepemm600013.china.huawei.com (7.193.23.68) X-CFilter-Loop: Reflected X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_MED, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1760396407049633975?= X-GMAIL-MSGID: =?utf-8?q?1760396407049633975?= |
Series |
ext4: Fix stale buffer loading from last failed
|
|
Message
Zhihao Cheng
March 15, 2023, 1:31 a.m. UTC
Patch 1 fixes reusing stale buffer heads from last failed mounting. Patch 2~4 reconstructs 'j_format_version' initialization and checking in loading process. v1->v2: Adopt suggestions from Tudor, add fix tag and corrupt 'stable' field in patch 1. Reserve empty lines in patch 4. v2->v3: Split block device checking cleanup into a new patch (2th). Add 'Reviewed-by' tag in patch 3-6. Zhang Yi (4): jbd2: remove unused feature macros jbd2: switch to check format version in superblock directly jbd2: factor out journal initialization from journal_get_superblock() jbd2: remove j_format_version Zhihao Cheng (2): ext4: Fix reusing stale buffer heads from last failed mounting ext4: ext4_put_super: Remove redundant checking for 'sbi->s_journal_bdev' fs/ext4/super.c | 15 +++++++------ fs/jbd2/journal.c | 53 +++++++++++++++++--------------------------- include/linux/jbd2.h | 33 ++++++++++++--------------- 3 files changed, 42 insertions(+), 59 deletions(-)
Comments
在 2023/3/15 9:31, Zhihao Cheng 写道: > Patch 1 fixes reusing stale buffer heads from last failed mounting. > Patch 2~4 reconstructs 'j_format_version' initialization and checking > in loading process. > > v1->v2: > Adopt suggestions from Tudor, add fix tag and corrupt 'stable' field > in patch 1. > Reserve empty lines in patch 4. > v2->v3: > Split block device checking cleanup into a new patch (2th). > Add 'Reviewed-by' tag in patch 3-6. > > > Zhang Yi (4): > jbd2: remove unused feature macros > jbd2: switch to check format version in superblock directly > jbd2: factor out journal initialization from journal_get_superblock() > jbd2: remove j_format_version > > Zhihao Cheng (2): > ext4: Fix reusing stale buffer heads from last failed mounting > ext4: ext4_put_super: Remove redundant checking for > 'sbi->s_journal_bdev' > > fs/ext4/super.c | 15 +++++++------ > fs/jbd2/journal.c | 53 +++++++++++++++++--------------------------- > include/linux/jbd2.h | 33 ++++++++++++--------------- > 3 files changed, 42 insertions(+), 59 deletions(-) > Hi Ted, will this patchset be merged in next window?
On Fri, Jun 09, 2023 at 04:04:47PM +0800, Zhihao Cheng wrote: > > Hi Ted, will this patchset be merged in next window? It's currently in the dev branch. I haven't set the ack out for it yet because I'm still seeing some test failures, including some test hangs in my regression tests. There are a number of patches series submission that I'm currently working through, so we'll have to see what the "guilty" patch set might be, and whether there is an obvious fix for it or not. (I've found one such problem that was missed by code review[1], and unfortunately, there is at least one more issue that I'm currently trying to pin down.) [1] https://lore.kernel.org/r/20230610190319.GB1436857@mit.edu Cheers, - Ted
在 2023/6/11 12:42, Theodore Ts'o 写道: > On Fri, Jun 09, 2023 at 04:04:47PM +0800, Zhihao Cheng wrote: >> >> Hi Ted, will this patchset be merged in next window? > > It's currently in the dev branch. I haven't set the ack out for it > yet because I'm still seeing some test failures, including some test > hangs in my regression tests. There are a number of patches series > submission that I'm currently working through, so we'll have to see > what the "guilty" patch set might be, and whether there is an obvious > fix for it or not. (I've found one such problem that was missed by > code review[1], and unfortunately, there is at least one more issue > that I'm currently trying to pin down.) > > [1] https://lore.kernel.org/r/20230610190319.GB1436857@mit.edu > > Cheers, > > - Ted > > . > Thanks for reminding. now I'm sure this patchset is tracked. Please follow your process.
On Wed, 15 Mar 2023 09:31:22 +0800, Zhihao Cheng wrote: > Patch 1 fixes reusing stale buffer heads from last failed mounting. > Patch 2~4 reconstructs 'j_format_version' initialization and checking > in loading process. > > v1->v2: > Adopt suggestions from Tudor, add fix tag and corrupt 'stable' field > in patch 1. > Reserve empty lines in patch 4. > v2->v3: > Split block device checking cleanup into a new patch (2th). > Add 'Reviewed-by' tag in patch 3-6. > > [...] Applied, thanks! [1/6] ext4: Fix reusing stale buffer heads from last failed mounting commit: ffea255f4052e6416a4b5738925337afbccd795a [2/6] ext4: ext4_put_super: Remove redundant checking for 'sbi->s_journal_bdev' commit: a8f17d78525adf325c80f9dd1db469d337a5ce49 [3/6] jbd2: remove unused feature macros commit: 870a42846c1055c4ff9dfd492a0929c52a367d63 [4/6] jbd2: switch to check format version in superblock directly commit: 6014c2204f10b1199e15ab61aa30274a14999b1d [5/6] jbd2: factor out journal initialization from journal_get_superblock() commit: 51bacdba23d85af2a9a145d97bfb77e6e85c98ad [6/6] jbd2: remove j_format_version commit: 1f15ee267c0498016cc4aee2cdcc18e56ff42bae Best regards,