Received-SPF: pass (google.com: domain of linux-kernel+bounces-180730-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
From: Yang Yang <yang.yang@vivo.com>
To: Alasdair Kergon <agk@redhat.com>,
	Mike Snitzer <snitzer@kernel.org>,
	Mikulas Patocka <mpatocka@redhat.com>,
	dm-devel@lists.linux.dev,
	linux-kernel@vger.kernel.org
Cc: Yang Yang <yang.yang@vivo.com>
Subject: [PATCH v4 1/5] dm: introduce flush_pass_around flag
Date: Thu, 16 May 2024 16:03:14 +0800
Message-Id: <20240516080318.120231-2-yang.yang@vivo.com>
In-Reply-To: <20240516080318.120231-1-yang.yang@vivo.com>
References: <20240516080318.120231-1-yang.yang@vivo.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?NKYRJyYBCHVSJvAM85MDrdRKGSn8BV1JMeJQzRUKuLrs2s21+ixD7naNenkM?=
 =?us-ascii?Q?eqgQjeAFyDjeWCDHQCbM+dN85y68jaL5mftJyrGqvcZbwSNl6tfhWZJDO9CA?=
 =?us-ascii?Q?yrpszIY38NPEGAEfPdHNGJly5DvYY35C0d0nMCRYg3dIl8niYWXX4EMJ0nLE?=
 =?us-ascii?Q?8LrPN3cmoPes0U3kzqh58ULQtjvlHkYn0RMzJK25mMWyZ7PiFnVyGRIYutYe?=
 =?us-ascii?Q?zFEIyJEg5W7B0Uf//+KbL9YvbOBWmHTzXjyIAbPBFEaudAA8fzQ+lAWN/Ly8?=
 =?us-ascii?Q?zZR2FFQSrpWaHz7AYMsCNpM37iumERL9GuZhtJKtE6pSM1SgrTo/i0pDpq0M?=
 =?us-ascii?Q?yFCXSyCUcVtZ+eUZpjW1FnbNUn0Jh86kQjXPluS2OorkDAX2XBiHm5+0OSLk?=
 =?us-ascii?Q?lO6ClYnR4iNMegb8Ll2n0tdCTq2irrvb0mPCNxs1F2qi2ZG3Q1uJwqxmN/Ge?=
 =?us-ascii?Q?MtG9gCWN1HzfIb+M7KwAvCAS2oaZV34o7Aj+r9IxbMDtJo5vs7OxtjEHa/Wp?=
 =?us-ascii?Q?FUnkH7ByWo+3xxSe2OfS/KgXnOpVaxPVXWIeRVp0cYoBAzumVDXHeBFBOvsd?=
 =?us-ascii?Q?REJ0UN5vmRnKlbY8/sxStXq2ZqKiJ0bt/Tn7HdLrhFI1m/KQhKGzAMgQ+InI?=
 =?us-ascii?Q?DXASmGKF/CPYM6FB/kYvlKTHjK/bAG9K7Fcco/TBZXiM3lnb7Kb3I8t5P2iD?=
 =?us-ascii?Q?nIe73VfCBA8a2m1D6PYvnuIvu6FBF7VuNJGcMqBMrL5RHiqWuu+ZAHPMvc4v?=
 =?us-ascii?Q?lusv8lyBB7wndFwKSjY4J1TEYWm3tdMMe5um16IqHyZVQamAWU9Vey7s3wbg?=
 =?us-ascii?Q?+YlOVFD5uVBnP/v/Dg+GSbnGlwyxdLiYHmuV9u80mq2AQ8vBMz9UIExHNJYT?=
 =?us-ascii?Q?sUeHW5M5AwxFvY/1xiTEeWM+R/vqeyospaFHjSW3XTj9zQ4sluCj/POwvIfg?=
 =?us-ascii?Q?kk9aes4UOfWGL4pdeL7DxXzKum7nM0KDXyh4diPnfd4VJKxdh+p25wxVbaXC?=
 =?us-ascii?Q?l7g5IB73dGDv2V5iSmXPb2nf7E38Pru5P7CfxOntJ1ROHuo+P62ZjL2GW4Sw?=
 =?us-ascii?Q?Esa81pXzrCT7OSy2Fdh1AnFksYglqnNc/89q0+PdeOeowtgxykzrq9pmaAKw?=
 =?us-ascii?Q?ehIcSNsuYlzHqRqsr/tb9/KNrDpQrtdI5zB4nENTXotJJ859vt2WIYvHWyz8?=
 =?us-ascii?Q?3LSL92SFZmObfFD1jFTu92JdhwLOuY0gxGzpxnRMNwqUrLAo7B+gYnz7IRlr?=
 =?us-ascii?Q?6ZqlMO5oZE12OC5NgTZa1U130eP/Fsti8MlYzXmilFVlZ5SuWiiaSafOh+FT?=
 =?us-ascii?Q?S2s6EPQYuZQDPk6u8tYj3VuoGDakx/rf+oq2F9MlKhmQ/KxcztWmW+SUB+VL?=
 =?us-ascii?Q?fErgBKBrLl3M4Iv3smEbQCj2n3O0KWnwWMyUIVpS1dbd8fTdxEkrJCFIlLir?=
 =?us-ascii?Q?MMzEKUaxrBcU4wkLF3p9oo1wFlvXwLyvBRo99WW1+o64B1iw8QFE8uWOBQHX?=
 =?us-ascii?Q?upPKAMPybx5Gmo0xZigTis8h3Pr7ySl7KGi6u+x+W8jhYS4JeQHGtwyT0Iqy?=
 =?us-ascii?Q?9NrHrAwkItbjB1hxzihwuJ+CkAGGFK+8X0s3fSR7?=
X-OriginatorOrg: vivo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 9ff2380c-6319-4787-ebc2-08dc757ec1cf
X-MS-Exchange-CrossTenant-AuthSource: TYSPR06MB6411.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 May 2024 08:04:05.9398
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 923e42dc-48d5-4cbe-b582-1a797a6412ed
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: yayOpf+LkBvum35PH2m9v3mMdfqSGrbCj/549s66K7vix4QKt4faClkT1zrj8V+J4bPG8l2C1UZf+OkGcZp3RQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SEYPR06MB6830

introduce a per-target bit "flush_pass_around" and means that the
target supports flush optimization.
set a per-table "flush_pass_around" bit if all the targets in the
table have "flush_pass_around" set.

Signed-off-by: Yang Yang <yang.yang@vivo.com>
---
 drivers/md/dm-core.h          | 3 +++
 drivers/md/dm-ioctl.c         | 4 ++++
 drivers/md/dm-table.c         | 3 +++
 include/linux/device-mapper.h | 5 +++++
 4 files changed, 15 insertions(+)

diff --git a/drivers/md/dm-core.h b/drivers/md/dm-core.h
index e6757a30dcca..b273f25b634d 100644
--- a/drivers/md/dm-core.h
+++ b/drivers/md/dm-core.h
@@ -208,6 +208,9 @@ struct dm_table {
 	bool singleton:1;
 	unsigned integrity_added:1;
 
+	/* set if all the targets in the table have "flush_pass_around" set */
+	bool flush_pass_around:1;
+
 	/*
 	 * Indicates the rw permissions for the new logical device.  This
 	 * should be a combination of BLK_OPEN_READ and BLK_OPEN_WRITE.
diff --git a/drivers/md/dm-ioctl.c b/drivers/md/dm-ioctl.c
index c2c07bfa6471..bb178df2a340 100644
--- a/drivers/md/dm-ioctl.c
+++ b/drivers/md/dm-ioctl.c
@@ -1445,6 +1445,8 @@ static int populate_table(struct dm_table *table,
 		return -EINVAL;
 	}
 
+	table->flush_pass_around = 1;
+
 	for (i = 0; i < param->target_count; i++) {
 		const char *nul_terminator;
 
@@ -2279,6 +2281,8 @@ int __init dm_early_create(struct dm_ioctl *dmi,
 	if (r)
 		goto err_hash_remove;
 
+	t->flush_pass_around = 1;
+
 	/* add targets */
 	for (i = 0; i < dmi->target_count; i++) {
 		r = dm_table_add_target(t, spec_array[i]->target_type,
diff --git a/drivers/md/dm-table.c b/drivers/md/dm-table.c
index 41f1d731ae5a..bd68af10afed 100644
--- a/drivers/md/dm-table.c
+++ b/drivers/md/dm-table.c
@@ -738,6 +738,9 @@ int dm_table_add_target(struct dm_table *t, const char *type,
 	if (ti->limit_swap_bios && !static_key_enabled(&swap_bios_enabled.key))
 		static_branch_enable(&swap_bios_enabled);
 
+	if (ti->flush_pass_around == 0)
+		t->flush_pass_around = 0;
+
 	return 0;
 
  bad:
diff --git a/include/linux/device-mapper.h b/include/linux/device-mapper.h
index 82b2195efaca..0893ff8c01b6 100644
--- a/include/linux/device-mapper.h
+++ b/include/linux/device-mapper.h
@@ -397,6 +397,11 @@ struct dm_target {
 	 * bio_set_dev(). NOTE: ideally a target should _not_ need this.
 	 */
 	bool needs_bio_set_dev:1;
+
+	/*
+	 * Set if the target supports flush optimization
+	 */
+	bool flush_pass_around:1;
 };
 
 void *dm_per_bio_data(struct bio *bio, size_t data_size);
-- 
2.34.1