Received: by 2002:a05:6358:11c7:b0:104:8066:f915 with SMTP id i7csp5715712rwl; Tue, 4 Apr 2023 02:31:36 -0700 (PDT) X-Google-Smtp-Source: AKy350ailBa0PlABU96Qg9LPoHQIjIal8bVvk8GB/BRLSIBbSg1Kv+QmYTaFNpE0RYU7+OakOZDS X-Received: by 2002:a17:906:dc2:b0:8a9:f870:d259 with SMTP id p2-20020a1709060dc200b008a9f870d259mr1560919eji.48.1680600696390; Tue, 04 Apr 2023 02:31:36 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1680600696; cv=none; d=google.com; s=arc-20160816; b=h+3ObsR9jBermsMN+KeOaCejOhwaa9KCfpvzZCof/LQkpEWrST5zrAmZfUyrJ3m+e1 sPseE2mVVnm5BIVPZgQOfzuKziOMPxA3Ekp7Uuy7BuNit2kH/1ar5AMSVLZ0UZ9sLoi3 TopP6WRM+0Eww+NTWr5YvahShbnWhgvDR6AKCEB2pK9YqbO/lxRD3Q86ykCfRNQP6PUM Ches/cWXzA6CUKdsxd/ExTuhkN5XtmY375FVurTmbzCv1RqREc4LXNiado3SZixPGYz5 PgH9st5sMcg3dHalCyKDxa5BAKjm+8mUZH0aRabMfJ+Uw4qnZkeDX/0QbCXNxhjhvWAl W7lA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=i67LZvndle0FXns2qvwNfT1XOL/ofgh7nRXwodyjXPs=; b=MJA6W5r+3xPpUecm8alHJF76PfOnoZow94zdvV1DrU8SOt7eBnuEA4C3XJW16Ooudo x1kE6R4lVElofs0DTKCUQlfAeL48L4yY+rBHOjnHZ8ZCiwfhuWDrEgVsHOvr1ejkR54X yi9K2BNBINBL+MJTv/hhoxsK+d5dU5+5uqPdZsh6L/2KvQTlPTbz47DSmuzmlYKIR63R +GFoinAz14siSTDQh2bGNeyhcWiTuT0VJVwlZOFnFrVIaQCqeR48TyOO34iuNOSEMmWe lxJRDZwXzTY17RRDwMI9Z1kbaHIAidsH9FlYHnOobRPWJx/IfddWbiuu0j51iSCt2VmB FEFg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@suse.com header.s=susede1 header.b=Oh3tIDqu; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=suse.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id m21-20020a170906581500b00931d3414a47si1875745ejq.200.2023.04.04.02.31.11; Tue, 04 Apr 2023 02:31:36 -0700 (PDT) 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; Authentication-Results: mx.google.com; dkim=pass header.i=@suse.com header.s=susede1 header.b=Oh3tIDqu; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=suse.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234256AbjDDJUI (ORCPT + 99 others); Tue, 4 Apr 2023 05:20:08 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53722 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234241AbjDDJUG (ORCPT ); Tue, 4 Apr 2023 05:20:06 -0400 Received: from smtp-out2.suse.de (smtp-out2.suse.de [IPv6:2001:67c:2178:6::1d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 039EA19BF; Tue, 4 Apr 2023 02:19:56 -0700 (PDT) Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by smtp-out2.suse.de (Postfix) with ESMTPS id 339F7204BB; Tue, 4 Apr 2023 09:19:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=susede1; t=1680599995; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=i67LZvndle0FXns2qvwNfT1XOL/ofgh7nRXwodyjXPs=; b=Oh3tIDquq5GAp/daRsZLPe8+keWjgn/xo26/bDtxYuh/NyIXC8eUV2ajlDvD5PdX6e3Bxc LeWQwHKJ2/4tDCgTl+My4ZLnXLjN4po1o8GhZWAhU+UW6Yq397WVXZdN3DwwLA3IcuWvqN CCHm9q+hpAiowzhA58WooTHxjRuiAGo= Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id BB5451391A; Tue, 4 Apr 2023 09:19:54 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id Z/cuK7rrK2Q+NQAAMHmgww (envelope-from ); Tue, 04 Apr 2023 09:19:54 +0000 Date: Tue, 4 Apr 2023 11:19:53 +0200 From: Michal =?utf-8?Q?Koutn=C3=BD?= To: Waiman Long Cc: Tejun Heo , Zefan Li , Johannes Weiner , Christian Brauner , cgroups@vger.kernel.org, linux-kernel@vger.kernel.org, Juri Lelli , Dietmar Eggemann , gscrivan@redhat.com Subject: Re: [PATCH 1/3] cgroup/cpuset: Make cpuset_fork() handle CLONE_INTO_CGROUP properly Message-ID: <20230404091953.tcu3zg7npstk3ztc@blackpad> References: <20230331145045.2251683-1-longman@redhat.com> <20230331145045.2251683-2-longman@redhat.com> <20230403165523.aphsec2epqi72k27@blackpad> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="t44yctniyorb4iao" Content-Disposition: inline In-Reply-To: X-Spam-Status: No, score=-2.5 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,SPF_HELO_NONE,SPF_PASS autolearn=unavailable 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: X-Mailing-List: linux-kernel@vger.kernel.org --t44yctniyorb4iao Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Mon, Apr 03, 2023 at 01:18:42PM -0400, Waiman Long wrote: > 1) PF_NO_SETAFFINITY flag - which won't be set in the case of fork() as it > is for kthread only. > 2) DL bandwidth - Juri has a cpuset outstanding to modify the way this check > is being done. I want to wait until it is settled before tackling this, if > necessary. BTW what about CLONE_INTO_CGROUP where the target cpuset has empty effective cpuset? > 3) security_task_setscheduler() - the CLONE_INTO_CGROUP code has already > checked that, we don't need to duplicate the check. Not sure what this refers to. > So we don't need a can_fork() check for now. Anyway, good breakdown. Could you please add it to the commit message too? Regards, Michal --t44yctniyorb4iao Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYKAB0WIQTrXXag4J0QvXXBmkMkDQmsBEOquQUCZCvrtQAKCRAkDQmsBEOq uXeGAQCpgXtPsXDXcfOTHX/nDug4IiTdeZK6ER7w5mOHPkUAxgD/cum39WzeVSGZ V0835/NthpeochRAaHAX/FNgF63xlAw= =b803 -----END PGP SIGNATURE----- --t44yctniyorb4iao--