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, torvalds@linux-foundation.org,
        konrad.wilk@oracle.com, linux-kernel@vger.kernel.org
Cc:     Mike Christie <michael.christie@oracle.com>,
        Christoph Hellwig <hch@lst.de>
Subject: [PATCH v11 4/8] fork: Add USER_WORKER flag to ignore signals
Date:   Thu,  2 Feb 2023 17:25:13 -0600
Message-Id: <20230202232517.8695-5-michael.christie@oracle.com>
In-Reply-To: <20230202232517.8695-1-michael.christie@oracle.com>
References: <20230202232517.8695-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?xYFPeBEhHsTfzxZB1uFDwK3cRt/NJZY8typZvlBh7/bqAbGob4Ft146vVls/?=
 =?us-ascii?Q?3d88gBNOlleGfe5OUlFT419OiRpOuf+WHTGQYKIcwWz272pomBIujgwxa462?=
 =?us-ascii?Q?0bFSfb2Hm+7GfLrT4wdRNogdQqzdxflBbGLquOQ0rmYIylRGXEaj7TWkeDdl?=
 =?us-ascii?Q?sYIgygJyGMhy1AGTquWKvtTj1xhg4++MBIYEdRCWDd+CFQKEpNTb6ghIDsc9?=
 =?us-ascii?Q?jpnXatMAGd7E+PSnM3JilJ6rTNh4/m5VjsTex6uHxhiTVVuNQ2npuOXJUd+D?=
 =?us-ascii?Q?CfZc1N8P19Y3mZij1RfuNrL3hvxWzhTq422rKAVU9kkP7svg53hDj2jIvS6I?=
 =?us-ascii?Q?0d8axZD74S0iHLvYDvi78RqAoTyatOCFsjl74uNAdsyk5hCafIwRTa7hTZpC?=
 =?us-ascii?Q?dA4PSMNVCEYfWhDi3z0uWcDfR2SngvdSdYgVX9R/hkmV+TvZX+/now2P24gE?=
 =?us-ascii?Q?ZSc275SrCcnbPYC9oRuFozMCNwHPT4dCpYrS6RkkdNWp3uwShxu/hC1HAZPn?=
 =?us-ascii?Q?K6TeDVk2qwv3h+u73ExKkP/lyuckaWK6L/yaH32+mOy4bILgPe0MMTnjRa6X?=
 =?us-ascii?Q?IhQ6SHaoGUj5D5fLvuHWjsmUdiPljeOie0R6BVjPrFBDOwOQUY474crOECPv?=
 =?us-ascii?Q?w0SqGTem8Pja0nm/7pWyWV/9c5mbmIZ3wB7E2XI6iLViqMzk20ygIp+B/ejK?=
 =?us-ascii?Q?BTQD/jrF+zYKoxuLru9idmg1Y5KxgepLEzg1jEneT8hUGWb+YX/1qyQaqTYf?=
 =?us-ascii?Q?d6Lf8GqC1k/FGU7vI8bLrf3ulIP66+ifA7zWXOZ+6gq2biOUCPAwHWSgKLbw?=
 =?us-ascii?Q?nWLgKvgAkgcssX0LkbfejHZLHR1kLXFphugStS6K/dNGX9v4wf0W3IaLK7vZ?=
 =?us-ascii?Q?EMdMUQIzawF1H+ypR89b/8pd7Vu/rF7iFfpWkphtGjOeL9HUtGcwzzqFmiae?=
 =?us-ascii?Q?zz0OVZVKlMeYlzIYfpC55OAt+JHUjsGxYnyWlDLG/lHVMBAzV9guAZJTn2by?=
 =?us-ascii?Q?YL7Rqq/LJSMmA/r7FI0BmuImt4ZSW4pLBQi7eJ/vikl/wQINFhTkbFknatuP?=
 =?us-ascii?Q?HinGDNBbgfXAcrk3wcb792OGQR4pkXj8eCVthXQAUDl3ebrR+mvvf5b+ohXz?=
 =?us-ascii?Q?H8T8Y4aWrsE/g01LC2XPnHng0xY4tS9mKgypmo8CijpPZZp0vGHx8fHxGwWL?=
 =?us-ascii?Q?t72i1S1xoheT2Cedsk+UygfkMg4KDbKytxdeCPqNAfUyL5BxAFGhISSravQB?=
 =?us-ascii?Q?hA6SIjUZymOW+TYjr/WAsSf5XhSFqjZgIXcloZ5vP6TQgQgtWquNqd6AgqLG?=
 =?us-ascii?Q?4e2RDsK/XFk1KtL1y4OhW8TmJkVRWC6Jxx7sYcOR8P6M1vBpetMusrCeClcr?=
 =?us-ascii?Q?f3DyoqiWuzOn65ATyNdSG31z6Z2ZykF4Qquh2lUknq56lsVSGYCtSQs/d4Bg?=
 =?us-ascii?Q?iNikli7t/M0uLV9amfEfKvQXRmMQ/Q8PfFoZZSxrtdkfbnZAv2BRtxiM+fPC?=
 =?us-ascii?Q?XkIWJ3DXUOyg7IsTdZw5ra8m0T6zf5C/AdEqNDEVUxfiXCoO2yTL6SVJvfSV?=
 =?us-ascii?Q?sEuQE8GztIpcn+Wxp4ivk7dCnr7wOAHrTImbKAQWpLtUpKpmaP2JXh1ysjkC?=
 =?us-ascii?Q?bA=3D=3D?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: =?us-ascii?Q?n+S8VVckaX8JywLItpLwuuu4ykMauADpEKCfUne6ru3XXDxh6O81h68Dhgc4?=
 =?us-ascii?Q?O7z72c6D0BkiMeQJuriRYRd61ypkqZ1P4jrRYBi56rTweo4Ge/ssKhaWj4zj?=
 =?us-ascii?Q?XNvUDNKPB2Fvg+fuFeAb6LRcSlrwdsjP8A8uDTYAE6hx7OEkm83fd8PnvQYy?=
 =?us-ascii?Q?kIw6PiVm5got1oCjm3s3Y8VNxqjF2Wpo3tB2fhAAhjsFyW8Escvudb5ixQGY?=
 =?us-ascii?Q?5ZK+xne9EtW9ZU638HEchvitFfFsEoYGhwziU5Jxje7txwPXWqoj27uiJqwT?=
 =?us-ascii?Q?52XEw3u5Zsj664GqaP3kv/1RzC5oV6tR6WijnJdlkuFZkxHZBzWuHucMciNe?=
 =?us-ascii?Q?Nkh46/xPhHMPpmffM3AzdwUg9Yv0uJ3mvmgDmLhzhC4ReIqImnYraDCgAsHO?=
 =?us-ascii?Q?d+2LVH1LkE2PGW5txjLYgLyV9r03BTIJu/SzApVRMWvkyI5OCYxyzY+9XtZ8?=
 =?us-ascii?Q?TY/hubBU6pHtkWIowdVUrSOJMEW4qnb2nFHArsdxo6C9Fn15+SMSqzVJA0/6?=
 =?us-ascii?Q?tus2r2MNoV6HhWsQ9qtiGSEcbAXCUHTPnrYDbYr2MjIChFRu3SXEDse5oUoj?=
 =?us-ascii?Q?vHCEHVPLAqFwz+/N8DAXTbVNDOOibHSqybdK+HeaOha7UOryUZhzWrxmqnwW?=
 =?us-ascii?Q?qCROJRIKqKjw6yH9pqNmFPbqVkBo//7H9Aw79ChoeflXyziDSuY7Ouw6h+lC?=
 =?us-ascii?Q?jmtWrTdrVziArE8uKEFC/QDbMi70BZO9fLbL9DrbQGKR/iqDvdkxsOmrE31j?=
 =?us-ascii?Q?+rLye+nTWExRmyO78EkDakUrIotHAU0rinEmFRu1JehhCeCpt2mSBAhHwqq7?=
 =?us-ascii?Q?etbGD4toh+dgpSSj2DjPKM5TLKPqP+cJtDVx7zCyzdWS6fox2p15prZ2JdyP?=
 =?us-ascii?Q?mEij/cDJZrK6rGrHSeSjYqbpfu/4XtMkh69othrhNzIcDM3LCbygicV176sX?=
 =?us-ascii?Q?5VBG1uD8N3Hy1OWgrSlRBAWPxX22jQtfF5q+gFeCCfnpBrd0UBmhuEWvAcuP?=
 =?us-ascii?Q?txOq9A6gANYWZWk72asZCdVhUEhviVGX1R9J9gFoOV8ifCk=3D?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 231f0840-8cc5-48e6-35f3-08db0574c572
X-MS-Exchange-CrossTenant-AuthSource: DM5PR10MB1466.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Feb 2023 23:25:29.1404
 (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: 9xmQ6ftIpB3wIasWsGnitQRmMvJTfzr4gPjk6F8gpibbOpybj7OOw3Q0NILdjgEQu1JSboG2IiDIW40Jkz/wHhTfCO/Y6alogi56ZvfoXwY=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR10MB6643
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.219,Aquarius:18.0.930,Hydra:6.0.562,FMLib:17.11.122.1
 definitions=2023-02-02_14,2023-02-02_01,2022-06-22_01
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxscore=0 adultscore=0 malwarescore=0
 spamscore=0 phishscore=0 mlxlogscore=999 suspectscore=0 bulkscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2212070000
 definitions=main-2302020207
X-Proofpoint-ORIG-GUID: I8NtYgJlzj251c2bGSMwR8SnTnA9KWEg
X-Proofpoint-GUID: I8NtYgJlzj251c2bGSMwR8SnTnA9KWEg
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 18e614591c24..ce6240a006cf 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 bb98b48bc35c..55c77de45271 100644
--- a/kernel/fork.c
+++ b/kernel/fork.c
@@ -2287,6 +2287,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