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;
From:   Mike Christie <michael.christie@oracle.com>
To:     hch@infradead.org, stefanha@redhat.com, jasowang@redhat.com,
        mst@redhat.com, sgarzare@redhat.com,
        virtualization@lists.linux-foundation.org, brauner@kernel.org,
        ebiederm@xmission.com, linux-kernel@vger.kernel.org
Cc:     Mike Christie <michael.christie@oracle.com>,
        Christoph Hellwig <hch@lst.de>
Subject: [PATCH V9 1/8] fork: Make IO worker options flag based
Date:   Thu, 12 May 2022 16:46:57 -0500
Message-Id: <20220512214704.104472-2-michael.christie@oracle.com>
In-Reply-To: <20220512214704.104472-1-michael.christie@oracle.com>
References: <20220512214704.104472-1-michael.christie@oracle.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?rnjESe9JvHanIITKI8ufoXJIkTlqLz3e0fKDGpjWDnU9EzvuSp02v5A5geYw?=
 =?us-ascii?Q?fnJKzKnKgcR5ozZ1FnqR68cJtMgM0VJQ0anzPjkPDJPZyf+s6vqwrgmgWVBf?=
 =?us-ascii?Q?xGgA2Ner8WHJEOMfsJUkJhOGFWYZxJFPIDsTL7ekV90iIGieD1cl9dzfoLUZ?=
 =?us-ascii?Q?AqoJ/1l3Mo6eecodxFhcvPpx4+QCm+UcKt6SNh/3eHk6rvCFJTaMZNpzZf08?=
 =?us-ascii?Q?1V2b2hz3xqsoOKDDpzb87TEZ2gRz6UeX1Sv+5lGVvUSrHKFsxkJ1f9Lvupmw?=
 =?us-ascii?Q?5SbWTGq+U9XZgXxxajxWEoXMEv6zoCazO2k+nJzAcUPbkMabDsE0g7U5UQpS?=
 =?us-ascii?Q?EjH2n+5uvw4fw1rNm7rzCEb4mJ1DyxDNZuFZYX0WbaYhkMoDxdcye27nIo6y?=
 =?us-ascii?Q?nV8cPE38837KXIWWUhADezPUaeM1RH4LIF9pWriSoPwwzciY5yAvAYIayfAr?=
 =?us-ascii?Q?0jaHYKt1AYBnmIx6EjGJ5O1FZLcFz4NvsGrBnK95yzlty5jdgVGS9kZTi4jG?=
 =?us-ascii?Q?4z3layEYujtc+UH8NcaITIxszrM8+RduClVHrOkrsoKOnJ0bccqUIorWmeB5?=
 =?us-ascii?Q?+NumPEyYZ1n3reL5rsn+BeshjnkVHWzIbG1mSbH1i67oXNiuewfgOVWu06EP?=
 =?us-ascii?Q?ntHpOEnkKVA+SbChEIieRLOsuY3/Bh8XmuPaaS1ZuBZrIDv42cf8Tettg7DJ?=
 =?us-ascii?Q?OwKnfrTXgrvcDU77jGwUurGiwIeJcqi7QHgqKAd42KJDgdy8q97Vmw1JX2e3?=
 =?us-ascii?Q?rMnZ+1eG/UVbzsLqxm7pWmTc8G3tRNu2Phs8utouhSRnIDs2c2zOFCMsMbnq?=
 =?us-ascii?Q?siAO5qSVMU5gx7N+qp78LeWFt9q1fw7aQNXg5anP2dVDvmrTcyP+EiMsatBh?=
 =?us-ascii?Q?/MZzdS5Cl7Q9lst8pAZt037Ex6p6gYwPfeo0NVka+P1nsVjExnfqs1MsO/1p?=
 =?us-ascii?Q?OZGpM3+YAsvMUCjWCwuX0ManzgIfvo4D5hfENgt+a3hAYPMha8JTxxXJi8+v?=
 =?us-ascii?Q?d6MZM7hp8wYF/z0kPvCKpaodaer3/ZEKuBxt9pY3JKb68dyPdf5piPQ/WqVE?=
 =?us-ascii?Q?ES7TclJdofeIX/4xwdw13fWjs+pyCkvTWg7ymLRmwsdyEdORsSWjziLK8T/Z?=
 =?us-ascii?Q?V1LwG1CLhQIcliR6ccCHbZXLTMbUm+y75h0HwjM1yetQNvD7AKvdtthRHTbu?=
 =?us-ascii?Q?fh1HCqG8reP8N57xuLC5dd/wP8giN5asbgn4kmC1vNNMNSPZG2ENuXrk1ZQ2?=
 =?us-ascii?Q?mfH8OwcuMoOBU9eBtqVbWhOaa12B2zFvJARVbLNLzP8EXmlkhHpkiAErAAj6?=
 =?us-ascii?Q?DmzfMNcoarqnKKU1sacSJj42mfCxcBmAhDl0NsO6OapsB84CT9gRvoDSLYWS?=
 =?us-ascii?Q?NwxZG47qg9dH6kXFZZxdm8oO4CF+jJb151aAzTUsBvkDNpfHQiFsMSyKCTLd?=
 =?us-ascii?Q?L0ueMgFrFgPL0X1JTEKU/1f5QUKEEKzyyNZ5wcTeUDd/ciRrCrQSed6d1noC?=
 =?us-ascii?Q?ctBoDXupGlZU9qjWZhb9r64LHGWUaZeFAJQGzoYSk1XWDdPsP57Z4mNHqIPs?=
 =?us-ascii?Q?O6GrbM8Tds/Nl7ZuUQ/LR++XhHt8z6zlPSMQu0weR6tgyUil6UYEMSJgsfAN?=
 =?us-ascii?Q?+4t5nIbRECsNYmxV4pYQHLo35So0GnaobeLZj8gZhwBDeZ8hYJCfvO7Lz2E9?=
 =?us-ascii?Q?efzewCy730MMwzUt2Lu1DjAwWLoYjMzbQ2rOb2drO3GDeHJoTNS8SerVRwuX?=
 =?us-ascii?Q?hVYVObGdtxCZl2N8k9dzhkG9EiL2uXY=3D?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 5cbd3e59-6761-4003-9f14-08da3460f8d2
X-MS-Exchange-CrossTenant-AuthSource: DM5PR10MB1466.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 May 2022 21:47:12.3966
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: a9ph7daGRX8c0hX7eKbx6p/RyIGfjpGeVJ1C5lrVAXlnd7qwzJhmiVir7yLu7CO4zEmbspO9XUefJbjatf8VnnVCkMpLALWRiHOaHAD8NYo=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR10MB3311
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.486,18.0.858
 definitions=2022-05-12_13:2022-05-12,2022-05-12 signatures=0
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 malwarescore=0 mlxlogscore=694
 spamscore=0 bulkscore=0 adultscore=0 suspectscore=0 mlxscore=0
 phishscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2202240000 definitions=main-2205120091
X-Proofpoint-GUID: qy8JzMb1rkoe2dCUtBKNFVHWzyiHjnZ6
X-Proofpoint-ORIG-GUID: qy8JzMb1rkoe2dCUtBKNFVHWzyiHjnZ6
X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,SPF_HELO_NONE,
        SPF_NONE,T_SCC_BODY_TEXT_LINE 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

This patchset adds a couple new options to kernel_clone_args for the vhost
layer which is going to work like PF_IO_WORKER but will differ enough that
we will need to add several fields to kernel_clone_args. This patch moves
us to a flags based approach for these types of users.

Signed-off-by: Mike Christie <michael.christie@oracle.com>
Suggested-by: Christian Brauner <brauner@kernel.org>
Acked-by: Christian Brauner <brauner@kernel.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
---
 include/linux/sched/task.h | 4 +++-
 kernel/fork.c              | 4 ++--
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/include/linux/sched/task.h b/include/linux/sched/task.h
index 505aaf9fe477..b0a9d6c75bcc 100644
--- a/include/linux/sched/task.h
+++ b/include/linux/sched/task.h
@@ -18,8 +18,11 @@ struct css_set;
 /* All the bits taken by the old clone syscall. */
 #define CLONE_LEGACY_FLAGS 0xffffffffULL
 
+#define USER_WORKER_IO		BIT(0)
+
 struct kernel_clone_args {
 	u64 flags;
+	u32 worker_flags;
 	int __user *pidfd;
 	int __user *child_tid;
 	int __user *parent_tid;
@@ -31,7 +34,6 @@ struct kernel_clone_args {
 	/* Number of elements in *set_tid */
 	size_t set_tid_size;
 	int cgroup;
-	int io_thread;
 	int kthread;
 	int idle;
 	int (*fn)(void *);
diff --git a/kernel/fork.c b/kernel/fork.c
index 35aa89ed6cf4..f4f2c88d9a2a 100644
--- a/kernel/fork.c
+++ b/kernel/fork.c
@@ -2071,7 +2071,7 @@ static __latent_entropy struct task_struct *copy_process(
 	p->flags &= ~PF_KTHREAD;
 	if (args->kthread)
 		p->flags |= PF_KTHREAD;
-	if (args->io_thread) {
+	if (args->worker_flags & USER_WORKER_IO) {
 		/*
 		 * Mark us an IO worker, and block any signal that isn't
 		 * fatal or STOP
@@ -2594,7 +2594,7 @@ struct task_struct *create_io_thread(int (*fn)(void *), void *arg, int node)
 		.exit_signal	= (lower_32_bits(flags) & CSIGNAL),
 		.fn		= fn,
 		.fn_arg		= arg,
-		.io_thread	= 1,
+		.worker_flags	= USER_WORKER_IO,
 	};
 
 	return copy_process(NULL, 0, node, &args);
-- 
2.25.1