Received-SPF: pass (google.com: domain of linux-kernel+bounces-195222-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
From: Sheng Yong <shengyong@oppo.com>
To: jaegeuk@kernel.org,
	chao@kernel.org
Cc: linux-f2fs-devel@lists.sourceforge.net,
	linux-kernel@vger.kernel.org,
	Sheng Yong <shengyong@oppo.com>
Subject: [PATCH v2] f2fs: alloc new section if curseg is not the first seg in its zone
Date: Thu, 30 May 2024 18:01:58 +0800
Message-Id: <20240530100158.762954-1-shengyong@oppo.com>
In-Reply-To: <20240529112411.380453-1-shengyong@oppo.com>
References: <20240529112411.380453-1-shengyong@oppo.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?GHDX/IJokzw1R0umXM6QrEUMmaNnPMGaHefRWG9Aq4OFGFaFETCeWhEaftGu?=
 =?us-ascii?Q?fiFrKUbb/XO0cbodQ9hziY5V4VznVSxt/gPvhM39hG40FHm6I4GNZ9Trz3Wp?=
 =?us-ascii?Q?e4bSarfI9QPrqJ/2xXjGcnKwLgp/9bqxkVFk7rMbpdV+mE76zh0w0yzvdm22?=
 =?us-ascii?Q?m7eBDB9KdXC4mW0s1pLrqR9ZX1HmMY8FmCIkSNtjIYe/OXl2VDLc/6i0EofW?=
 =?us-ascii?Q?SNsYu8TTdUs2ForuFMBZqDSyodo4RJ2Hh/NVmF5sNGzAmU8DRwLvVPJJvk0c?=
 =?us-ascii?Q?1WcYiNctGDn6kTUhQAn59MPULlyFMLitVrmupUefKmFHMNy6vwVPoL+vWizc?=
 =?us-ascii?Q?XcsDae72bNVj7yWpjfslJPH12vtjjq2NJHBDiNa/oPR4UpO5bd2yxIwlTZr7?=
 =?us-ascii?Q?RqEWwoM7JlyZxbV/u9Dyhd59f/V3khk6iLTtWMBmbZrYVvchH7Nw5G5L2GLc?=
 =?us-ascii?Q?rd4LdVNBV0teO7754yRJOilvZoLEzGRXJvChKuy/Bu79GD+oKKO3wYOiZB0G?=
 =?us-ascii?Q?m/ANyv66HnPjowV8jpjpDvOVxp1jlXhrpbkTEoO0IymtwCm8gacrzYbYVJC4?=
 =?us-ascii?Q?pzTuHmNnyFveh4e2z5/Ot99RNs7o/X7ZHz+S/F+8GABezdis9iBFREQVKwXE?=
 =?us-ascii?Q?/gjC2AQReR/bAMdK2jC10gw8ezF5u/9Ojv660o2vfbOnkyyc273kKf/tSSEp?=
 =?us-ascii?Q?gYNPEXTYMlovz6i2mY2QkPk2s6Hz6gwMeqVhcut4bTNnw+cl0ByfQfRp8YBm?=
 =?us-ascii?Q?22vcf/Slla+2qfxeDLrHSQ4IVBt90c2bbx4Mwy9klsr0UHVVWxP7Zll3ExPl?=
 =?us-ascii?Q?IFaHHcm+ng4Ug0n5hY67+IQDOiD0gkuDGpBmXW39xaK3qmFvT2jWaB6a8bbz?=
 =?us-ascii?Q?54Qvw8rdU5C9CkjT3K8/zH0cpSRrAPYQlKzD/MkURP9JjOVlrba7OjRLqNCS?=
 =?us-ascii?Q?Qqyma3JhDTcSxZL1xhQc/pDf8Tb3eD7jHWT1gk+lE4QaMzsEt0/RXOiw0AMt?=
 =?us-ascii?Q?MmJ4/HTloGE3L3hAvFTZ8HWDRf9lq2dzntH4cQKMp0fKuz2QEDYDfxLgYJg8?=
 =?us-ascii?Q?8TVhqs0xU2ov1wjNMkotxk7qZaklii8l0wXukuMxpTk1qiyhng65nShZMYBd?=
 =?us-ascii?Q?5hdaNWCzzgMTDcmJ3xTNr4WS7oB4gHOWSzijDVyDl+8Mlatt20JuI6PDRsbG?=
 =?us-ascii?Q?VH6nOtgeeYVoyT5ONZrLzeDPpbsP9NVa21gfMOwidGxWQ2tHdnHuhHt6woX5?=
 =?us-ascii?Q?haqi1POkguDfPXKRF8E3O6UdJQKXJmEjPYYR8Bz/JtBJJCYJ4U8UgxPKKvqk?=
 =?us-ascii?Q?qg6yxUSs8rtH8NheB65P3/3af9sWRVnlJJnDrlYLh3c/70muuwW7RjMxJIX1?=
 =?us-ascii?Q?IUMCM7ATd3lntkzpYWhrmp+JoDZA/I11HIzwxxBPaJwO2NSCgwv3vW+IHx1x?=
 =?us-ascii?Q?zlG71l9Gl5i4CBkVRmwNKmwGuO/XVrsZRmDrAYqCNcuSMpi/6QjuyWRqmIba?=
 =?us-ascii?Q?hKY/8HUEmEj+Fy7en3LN2oF+ccekbgujvS+WXrt7JCmtMr+Ov1gri1S6BEgH?=
 =?us-ascii?Q?f/cdb04cNmMUmLd/FIYAzDEVHB7Xii96hdcJ64K6?=
X-OriginatorOrg: oppo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 0b6e9608-3213-4f19-d752-08dc808f9cb0
X-MS-Exchange-CrossTenant-AuthSource: SEYPR02MB6014.apcprd02.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 May 2024 10:02:28.0515
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: f1905eb1-c353-41c5-9516-62b4a54b5ee6
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: X+KKWsoW9zjuMbBJMB1zypXkW1GEq2H/GOl7M1mnPLgRCMROFGYRXlHCbIQFq/I07jxxP9K2/thLttYo2S17bg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYSPR02MB7389

If curseg is not the first segment in its zone, the zone is not empty.
A new section should be allocated and avoid resetting the old zone.

Reviewed-by: Chao Yu <chao@kernel.org>
Signed-off-by: Sheng Yong <shengyong@oppo.com>
---
v2: remove and update inaccurate commit msg
---
 fs/f2fs/segment.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c
index 4c8836ded90fc..50b38cbe33401 100644
--- a/fs/f2fs/segment.c
+++ b/fs/f2fs/segment.c
@@ -5004,7 +5004,8 @@ static int fix_curseg_write_pointer(struct f2fs_sb_info *sbi, int type)
 	}
 
 	/* Allocate a new section if it's not new. */
-	if (cs->next_blkoff) {
+	if (cs->next_blkoff ||
+	    cs->segno != GET_SEG_FROM_SEC(sbi, GET_ZONE_FROM_SEC(sbi, cs_section))) {
 		unsigned int old_segno = cs->segno, old_blkoff = cs->next_blkoff;
 
 		f2fs_allocate_new_section(sbi, type, true);
-- 
2.40.1