From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 74C633B95E3; Fri, 25 Sep 2026 17:50:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790358657; cv=none; b=Ig8K6F1hmLpJjn7UftqXdpeL5ebBlXera26oCrEIUzI2FASPhrXUStTOpBjRWfjPqdK1Ye4ChkIuXVtzwuZw7UDkntZ4iMgYWNXRMLOLScOCla5ygup6kMNwfLdp1Ttia9y0t3lWJW5aLHE0eRqfpbePvYDqEDITNZfyv7ivM9A= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790358657; c=relaxed/simple; bh=U/3AEvhYN3bQ45ip9yZFKLNi4yGIFwSVeEPN9JGNLek=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=NaLPo/ylyLEphHRwYNMk8TnkiVFp63mt4ZVf/ddrWAbd+omhDhspODSYHSv/YCjJx/U7l9kho3NsYhKjccuCbMHG1DogmxZYSmrB//+MRHPDgh2onFcUS2ZURfT6WY16Yo7/u5z2OCZQfxmekJhMY2cVM71iNEF4+cEVbrlkwqs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=dFZk2SzP; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="dFZk2SzP" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 571AE169C; Fri, 25 Sep 2026 10:50:51 -0700 (PDT) Received: from [10.57.10.17] (unknown [10.57.10.17]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 9350F3F632; Fri, 25 Sep 2026 10:50:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1790358654; bh=U/3AEvhYN3bQ45ip9yZFKLNi4yGIFwSVeEPN9JGNLek=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=dFZk2SzPVDQgWnnh4XxPw2GQ/FPJzzTgl+TPEKAtdhxC1UETQXiQp8awi2r/92qSe EAU5BkUHqy0hNZw48kmRVAINYUnWkycHwuINFqNeWr8H/Alr/UhB1hnmesJNlaANuu XtkgONeVIEa4++XxO6ir+TGmuhrvZCO2vBo48tfk= Message-ID: <12e83d88-de4c-4230-aac2-1796123320ad@arm.com> Date: Fri, 25 Sep 2026 18:50:50 +0100 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v19 5/7] firmware: arm_rmm: Activate the RMM Content-Language: en-GB To: Catalin Marinas Cc: kvm@vger.kernel.org, kvmarm@lists.linux.dev, maz@kernel.org, will@kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, steven.price@arm.com, aneesh.kumar@kernel.org, oupton@kernel.org, gshan@redhat.com, joey.gouly@arm.com, tabba@google.com, yuzenghui@huawei.com, linux-coco@lists.linux.dev, gankulkarni@os.amperecomputing.com, sdonthineni@nvidia.com, alpergun@google.com, fj0570is@fujitsu.com, WeiLin.Chang@arm.com, lpieralisi@kernel.org, enju.kohei@fujitsu.com, sudeep.holla@arm.com, jonathan.cameron@oss.qualcomm.com References: <20260924135201.850038-1-suzuki.poulose@arm.com> <20260924135201.850038-6-suzuki.poulose@arm.com> From: Suzuki K Poulose In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Hi Catalin On 25/09/2026 17:42, Catalin Marinas wrote: > On Fri, Sep 25, 2026 at 04:02:24PM +0100, Suzuki K Poulose wrote: >> On 25/09/2026 13:17, Catalin Marinas wrote: >>> On Thu, Sep 24, 2026 at 02:51:59PM +0100, Suzuki K Poulose wrote: >>>> From: Steven Price >>>> >>>> Activate the RMM after the basic configuration. This is a memory >>>> transferring stateful operation. >>>> ... >>>> + >>>> + ret = rmi_sro_memxfer_cmd(sro, GFP_KERNEL, SMC_RMI_RMM_ACTIVATE); >>>> + if (ret) { >>>> + pr_err("RMM activate failed (%d)\n", ret); >>>> + ret = ret < 0 ? ret : -ENXIO; >>>> + } >>>> + >>>> + return ret; >>> >>> It was raised earlier this year [1] but I'm not sure it concluded. How >>> do we handle kexec and kdump? I think RMI_RMM_DEACTIVATE only succeeds >>> if nothing is delegated, so it would need all realms torn down first. If >>> that's not feasible, we could at least block (non-crash) kexec like pKVM >>> does. >> >> You are right, we can't DEACTIVATE until all granules have been >> "undelegated" back. Not just the Realms, but also the GPTs/Tracking >> Metadata etc would need to be reclaimed (when we get to support >> dynamic GPT/Tracking metadata). For now, we should block the kexec. > > Looking more into this (and the memory hotplug story), I find it strange > that simply having RME and a valid RMM imposes all these restrictions > even if we never run or intend to run a realm. How common will > RME-capable systems with RMM firmware be that are not used for CoCo? Or > do we expect only CoCo systems to have capable/configured firmware (RME > may be present in silicon anyway)? There is another angle to this : RMM may act as a TSM (as in the Trusted Security Manager in PCI TDISP) context and provide setting up IDE connection between the RootPort/ EndPoint. So, the trigger point for the RMM activation would become the "First Delegate" request. Running an RMM just for the "TSM" functionality is not ideal. In the absence of RMM, Linux can act as the baremetal TSM. But when the RMM is present, we must use the RMM as the TSM, especially if the Device will be assigned to a Realm. FEAT_RME capable systems don't need to enable RMM unless they want to run CoCo guests. > > Ideally we'd defer the RMM configuration and activation (and the > tracking/GPT checks) until we first attempt to start a realm, keeping > only the RMI_VERSION/FEATURES probing at boot. Not sure how feasible > this is (memory is more fragmented by then for any contiguous donation). > If we manage it, kexec and memory hotplug just work on hosts that never > start a realm. > > The next best thing for kexec is to tear down the realms, undelegate > the granules and deactivate the RMM before invoking kexec (with kdump > that's harder as we likely no longer have a controlled shutdown). This is quite complicated, when we get the dynamic metadata support, especially with the self-describing L1GPT and Tracking metadata. But not impossible. > > Similarly with memory hotplug, allow it if we haven't started any realms > and refuse realm creation afterwards if untracked memory was onlined. > Also, if the memory is onlined to ZONE_MOVABLE, neither guest_memfd nor > the kernel allocations we delegate come from there, so we could allow > it even with realms running. I will double check this. > >>> Kdump gets even more interesting if it starts accessing delegated pages >>> and getting GPF. >>> >>> [1] https://lore.kernel.org/r/CABpDEukEO4Y_fg8fv5Nr1_Pw_qOK=2UmioXk=WyPEzoEprPcGA@mail.gmail.com >> >> Kdump may be a bit more easier, as the kdump kernel is supposed to use >> the "reserved" region and vmcore access could handle the GPF and >> provide "0"s to the reader ? May be this is one case where the >> host needs to be able to handle GPFs. > > Yes, I think so. I now realised that metadata delegation to the RMM > doesn't come from guest_memfd, so that's another case not helped by the > old series to unmap guest_memfd from the linear map. I'll reply to the > GPF thread. Ok, lets discuss on that thread. Cheers Suzuki