From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0717F3E463; Thu, 23 Nov 2023 16:24:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="dmnZyFi9" Received: by smtp.kernel.org (Postfix) with ESMTPSA id B7570C433C9; Thu, 23 Nov 2023 16:24:51 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1700756698; bh=8qbTRbdLFdnXb3Xe8uP0IN8YCP7nL2AK2pZi0k8jVmI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=dmnZyFi9UMYT31BH9Q3rAbNfIaVQ0bRDx2OxJp8JC3rGb5UBAa/ZbMLbt/oJhrtgy XQfk4Ja/ihC5b4N7oj+pbXArjp1ErDUiUsHfGVpN1prX6E4NqkVCAdTa7YKTx3RMzO CCzgc6Nc08iu4BX33NSVnfADwm12OkUR6BbVR+6uZ8my99h87K6pqcNq+QStsFkVkN iCdESVJWG2oGoQYr39+vyW7w17U6oydSDnJB0BBp/X2jXoJuRUEHiLoq2YXBgfo4D4 6JfNQmixj3sbRZEA4YVVRpfW+tL8FsdMvhdHGegCCfGhFpUx79OXEwzB9cAw0XEGkW VQwoEDItqb3tQ== Date: Thu, 23 Nov 2023 17:24:48 +0100 From: Christian Brauner To: Mark Brown Cc: Szabolcs Nagy , "Rick P. Edgecombe" , Deepak Gupta , "H.J. Lu" , Florian Weimer , Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , x86@kernel.org, "H. Peter Anvin" , Peter Zijlstra , Juri Lelli , Vincent Guittot , Dietmar Eggemann , Steven Rostedt , Ben Segall , Mel Gorman , Daniel Bristot de Oliveira , Valentin Schneider , Shuah Khan , linux-kernel@vger.kernel.org, Catalin Marinas , Will Deacon , Kees Cook , jannh@google.com, linux-kselftest@vger.kernel.org, linux-api@vger.kernel.org, David Hildenbrand , nd@arm.com Subject: Re: [PATCH RFT v3 0/5] fork: Support shadow stacks in clone3() Message-ID: <20231123-edelherzig-feiern-b53339f5a639@brauner> References: <20231120-clone3-shadow-stack-v3-0-a7b8ed3e2acc@kernel.org> <20231121-urlaub-motivieren-c9d7ee1a6058@brauner> <20231123-geflattert-mausklick-63d8ebcacffb@brauner> Precedence: bulk X-Mailing-List: linux-api@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: On Thu, Nov 23, 2023 at 11:37:54AM +0000, Mark Brown wrote: > On Thu, Nov 23, 2023 at 11:10:24AM +0100, Christian Brauner wrote: > > On Tue, Nov 21, 2023 at 04:09:40PM +0000, Mark Brown wrote: > > > On Tue, Nov 21, 2023 at 12:21:37PM +0000, Szabolcs Nagy wrote: > > > > The 11/21/2023 11:17, Christian Brauner wrote: > > > I'm still not enthusiastic that we only have one implementation for this > > in the kernel. What's the harm in waiting until the arm patches are > > merged? This shouldn't result in chicken and egg: if the implementations > > are sufficiently similar then we can do an appropriate clone3() > > extension. > > The main thing would be that it would mean that people doing userspace > enablement based on the merged x86 support can't use the stack size > control. It's not the end of the world if that has to wait a bit, it's > a bit of a detail thing, but it would make life easier, I guess the > userspace people can let us know if it's starting to be a real hassle > and we can reevaulate if that happens. > > It's also currently a dependency for the arm64 code so it'd be good to > at least get ageement that assuming nothing comes up in testing the > patches can go in along with the arm64 series, removing the dependency Oh yeah, I'm not fuzzed whose tree this goes through. By all means, take it with the arm64 series. > and then adding it as an incremental thing would be a hassle. It's > likely that the arm64 series will be held out of tree for a while to as > more complete userspace support is built up and validated so things > might be sitting for a while - we don't have hardware right now so we > can be cautious with the testing. Ok, that sounds good to me.