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 v10 4/8] fork: Add USER_WORKER flag to ignore signals
Date:   Sun, 19 Jun 2022 20:13:53 -0500
Message-Id: <20220620011357.10646-5-michael.christie@oracle.com>
In-Reply-To: <20220620011357.10646-1-michael.christie@oracle.com>
References: <20220620011357.10646-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?tJVzdXBM6q/i/fpULSFeaIL3Zn9tgqTu0aYGmre3Xs6ASjAKSE07ZdFldxPj?=
 =?us-ascii?Q?0LxocEZWaUZAYjfZyqYsqa/C97y6oJ214ACaKlJQ4ybjO8a8ju3jAghnV+1Q?=
 =?us-ascii?Q?WZM0RwF68x15B35KHVU9uxCtjklqHqEkFzj1fB0aEm+1eObSCF1nozytdd+3?=
 =?us-ascii?Q?uHELzXOSyO+xZCPVohJDt1ka0Eq6nQrwjMmLb0caVA2BDHXYT7MKLHnvU6UM?=
 =?us-ascii?Q?vzYswcrvwpSrjUkr7/b+dAVPOcahFl/sT3RjBGjBKhI/DMkCiy6XOTanst8N?=
 =?us-ascii?Q?Ebq2nYYMH5KQ/sMZPmK8GTzWbjsMzGThtOSjBK7E+R/Vkxw/qK+oXvKdiomq?=
 =?us-ascii?Q?iPDQESnWKQa/8opvudIL8lGK9JPGf7RwG4AnSEpj6KzvEAT2qpw2kW74OdRo?=
 =?us-ascii?Q?V13otiyafKh8pSgMdXxsySTxbldyq7oXTTT7TwHGMyay6E1T0bADBTmI23dc?=
 =?us-ascii?Q?vf9Mo7IAUP3P6htaDgZYPBgYnAsyqB1yCS0+h8Q6QhkARqdIlrhhHNwa0NCN?=
 =?us-ascii?Q?i3zz5UkraBAjE185fiiKk9IWz4tz3crlP0nw6rV0rW+CnXVH/6W5UTpg1owA?=
 =?us-ascii?Q?/uaDFvzgIcgG/6ES++qZqibWdcUJL6mR43OfB9nnmnkqmQJuMELtKo9oXsW3?=
 =?us-ascii?Q?TA3EKNh5iKWY7XNNhSTEPdQgN9npxcvWiJfcc9I63CXA8LHZ1g2oR/qDz53u?=
 =?us-ascii?Q?HOV9Bm4I/YnzLdQKdisSrQsTGWY2ZTom8Hj7fJY5HAxeBq485PMZwIQXuxsY?=
 =?us-ascii?Q?FOZ/Ph1FKLohhMrvK16/HdJW7hCAp7gOyadFQDyptF2Ps5SiQb0+JyzSggDa?=
 =?us-ascii?Q?rQy85w3MwWHL6PxACE+8IqDPVs2enNBshNnRPscGDD2WnkIKh1Fjmtyn0She?=
 =?us-ascii?Q?7Dwlr3vhYNVSk8RKf8LJk+X3Wrq3HfzSSBJ938+2lvWCumDt/tUmZgR7SkL4?=
 =?us-ascii?Q?M4JoV3tlGvgj2Iz2FMTUIiwjxb0JUu03I8VXvbOQee/G5P8pEsF3cX7rT1dL?=
 =?us-ascii?Q?6Om4E/M7Inay9WI0bkGqbDQ2OF06h5T51mE8J2fdGGqwN2yz4mvJWr4aP+/+?=
 =?us-ascii?Q?UQRJ0Ll/ZoWxbyP/WO37OoU7RMoApyU3HhzO9sqp52dVauw9ddkcKXX459yg?=
 =?us-ascii?Q?O1i3Ud1GVAs/LL19p9JrIErSCoOx6nFXW2qknueUHMjVQwNoNNinLZShxA8b?=
 =?us-ascii?Q?xFvV0P7svHDvDJnq01hGTLwK+HnMdWFApyHDGD6+3nT2WhuRe2lvbQ7M7PUF?=
 =?us-ascii?Q?QbpHMZ4LLcQeCY6xQmgqaXLeBYRtuO2QG7s+rSYTApw5eIFzljCdzC+PJwKp?=
 =?us-ascii?Q?Apfpd2UuQrjQZx5+QZuT0/88238sjSJazLp9KDfkb3lh4MTVnsXgpW5S0EwO?=
 =?us-ascii?Q?SPgEz+OMxb/6CWlOLpeHygkcSTuM6KCXaYK0EtRyNOc2TGIXNdMi1smEsG1O?=
 =?us-ascii?Q?9lwhUp++gTPLGFhrgHQUm9pzIU9fd86WENSp4tdHOn4Tu4fVZ4Dz2CHVe64g?=
 =?us-ascii?Q?NHDGX2cR7isGuI1C0BcZtOLuNdEIC9iOEKTtGqGP7isMuP0xh+BWrA1iUnJ1?=
 =?us-ascii?Q?p0nXCHeRLBF535NzMhgjbMH8RvPbKsJ1xTuLlLvXRGUWFsNySwBvT4zdW0Yq?=
 =?us-ascii?Q?AXYQTxjdQcXHNkkMOVxHrgj6X5WMfmlkQ3zDFTzJXl2U34J8qYfQMk4W1omc?=
 =?us-ascii?Q?AZNKkrGRZ11oiXe8Um9jzDUK/vsaQS6dtW/JOd+1MNHLNghvaKFncuL1YbtA?=
 =?us-ascii?Q?5RDk31Af5xqv5nI96gAzDhWmxmftO2E=3D?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 9cf1b84f-03e1-4905-9ead-08da525a2d35
X-MS-Exchange-CrossTenant-AuthSource: DM5PR10MB1466.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Jun 2022 01:14:08.7400
 (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: TroPPveraprr3XoYXcbrJd1IEiCkqGDXpIJaVi6rb8DbnyGWLTarvUrLlPkiZLp+KXX/ip9xW27Zvt6LWfjzFmAr9/9HsIJ2gK8tiJiM7G4=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN6PR10MB1905
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.517,18.0.883
 definitions=2022-06-19_12:2022-06-17,2022-06-19 signatures=0
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 bulkscore=0 spamscore=0 adultscore=0
 mlxscore=0 phishscore=0 suspectscore=0 malwarescore=0 mlxlogscore=999
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2204290000
 definitions=main-2206200004
X-Proofpoint-ORIG-GUID: Zl5mH4Kgd_rOZm-9kjIMOPWOxPjgBkpI
X-Proofpoint-GUID: Zl5mH4Kgd_rOZm-9kjIMOPWOxPjgBkpI
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,
        RCVD_IN_MSPIKE_H2,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

From: Christian Brauner <brauner@kernel.org>

Since:

commit 10ab825bdef8 ("change kernel threads to ignore signals instead of
blocking them")

kthreads have been ignoring signals by default, and the vhost layer has
never had a need to change that. This patch adds an option flag,
USER_WORKER_SIG_IGN, handled in copy_process() after copy_sighand()
and copy_signals() so vhost_tasks added in the next patches can continue
to ignore singals.

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

diff --git a/include/linux/sched/task.h b/include/linux/sched/task.h
index 7f6364e5aa1a..9fbb489b2512 100644
--- a/include/linux/sched/task.h
+++ b/include/linux/sched/task.h
@@ -21,6 +21,7 @@ struct css_set;
 #define USER_WORKER		BIT(0)
 #define USER_WORKER_IO		BIT(1)
 #define USER_WORKER_NO_FILES	BIT(2)
+#define USER_WORKER_SIG_IGN	BIT(3)
 
 struct kernel_clone_args {
 	u64 flags;
diff --git a/kernel/fork.c b/kernel/fork.c
index 0313aa9ead89..c479b8419a6e 100644
--- a/kernel/fork.c
+++ b/kernel/fork.c
@@ -2261,6 +2261,9 @@ static __latent_entropy struct task_struct *copy_process(
 	if (retval)
 		goto bad_fork_cleanup_io;
 
+	if (args->worker_flags & USER_WORKER_SIG_IGN)
+		ignore_signals(p);
+
 	stackleak_task_init(p);
 
 	if (pid != &init_struct_pid) {
-- 
2.25.1