From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f172.google.com (mail-pl1-f172.google.com [209.85.214.172]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4944D4F3EDA for ; Mon, 28 Sep 2026 17:19:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790615995; cv=none; b=E99E6BAJFz1BWPsXsT8Mm8p3NvcAhvA+5KNHrdA/Yz6zHGe4/7NUqYiDO394S73MsQxA5hQOIh2yBGmVAG0Xn22zGRqxKbbjZeo7jHmTDc9XnmKUDeZ10WUGPl4hK9SO3yIDBWoMxOCDMJ+HQMPyMKI45nDzZzS2phs06e8CUmQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790615995; c=relaxed/simple; bh=9bVXZ77Pzbh2OYLwrCoAhJ9SNTHguTxHVRc+s2gfuIE=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=WAzY7RCKt3j7R79H0Jw3pbWaTUyL99YY1Ifsw9TEYsvLQAypto9HI3rnOpcO5th5mFILrNz9Px5CWgLMIgj8uJZWonkHMPWk0HDxf2fs0MQhGaZ98al8byCyx69gGD8S3Y3zJhHapBucPT1345RuntoCH0UgRuL+iswH+h4vVks= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=hBDq4Hjw; arc=none smtp.client-ip=209.85.214.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="hBDq4Hjw" Received: by mail-pl1-f172.google.com with SMTP id d9443c01a7336-2db33db4de9so2985ad.0 for ; Mon, 28 Sep 2026 10:19:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20251104; t=1790615991; x=1791220791; darn=vger.kernel.org; h=in-reply-to:content-disposition:content-type:mime-version :references:message-id:subject:cc:to:from:date:from:to:cc:subject :date:message-id:reply-to:content-type; bh=DWEY4l/bIKaUQ0LkmUfe65oGxH2cGGoTK+zPHQ0IPQM=; b=hBDq4HjwxP1eGmrlPhZ3odGGR3Px9kfWs0X0sj4G+s4xv1n4iSLw4UzrLWKbX1MWcu +l95p5GNeDwJNkb2CTxKGkGgTMY6OS2lg0xqK5yGrWP7ems0dbFxwmVGGlskcXr/0bGo nQZlJxfySz/Ki4xh9+ZlFUm13+Mts5WCKAMP/IdmCXjQF/oVLQzvimqKRhHGzSyfBsq2 iFIysEZ3S7yHOoWxk4jYYDSWAT65orvKrzsBBw45YXUDCxxPUkjTwsvgJ+73g7Cyu0Jg u6Hc5fIUQyylVjn0X6koEM7ca8J5IqxlETirS0AvoIEiGjxztMj5zs3yG5vQxSNcIp21 Mq2g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1790615991; x=1791220791; h=in-reply-to:content-disposition:content-type:mime-version :references:message-id:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=DWEY4l/bIKaUQ0LkmUfe65oGxH2cGGoTK+zPHQ0IPQM=; b=evMtB3gzZbjA6m9fiWSSmKh9+ZSU80mvIpNnxGaA4eNHz/ebzR1/Zv8fgnIwP4Laag FP6vEmXdASWb3lE1l8oUPSsbJtW2pq68pVrbez5l6w11N6RNyJTAcHUR07FSXQrYIpdf KWxCYOqSD0/js61x268XQtkDx8CeUa8i5funVfElsD/6wW+9yajGD2Vq6ATkJbYTRSmw SIz5N6GAM5/HJhNHtNmQzWB1pRtrRIH8UNBD2CNtgkT+y/n6LYCIWzhaYT2qheAeyLhe 9ugYfDs8eup0yjH+8vts/qUDrlDy1/06Fz7EyNVzvMuE7Vgm+UzOnBuDVOsCdRl5ct/A gI6g== X-Forwarded-Encrypted: i=1; AKwUvBx+nSpTufc5emklWe68Xjg0dTzZpNo9en4zFYyHFjc4oP01RFJ6AX/dolDZTNkP8WZsTd6MpwKt9dfXtSk=@vger.kernel.org X-Gm-Message-State: AFq9FYKBVLUDhb3Y4l3wPJdgtetg0Gccp1jdWMIBAtNzgvC+Uxlj/df2 R7LKQVyObYF0qcvftcM1PjBQpasQkFuj5r9xlFuxsQK8GR08vACJ7i0Ph8q7VDtZbQ== X-Gm-Gg: AYBFou3j24gyrXGGQ65idUxOeNuLm2hsW9p/QMN67IsdczdMpSah/KX7Z00TVkWWJEQ 0ir2TdG7Q1A7LzZ+B0N2ywKJXaddBRgRNWiJSK11hJFZ0KVFeBNVSwhnuvr8Vu3TuClwtQ2W8U8 boqzWKv/DhmM8TSO01hxFF8yyoEIInDLD/Fh3NdNdg2MhXQ/KH7W84wMLcxArItnkHkru/xf0ox YtctGPKYYg62eJXOjjneSVT4qXq9cHyWrOjUU1ot3BLGx2/48L5sOkQKxO7YPZN50P9j0thAfEG 6LcJBtPaXEdb5SyTawE0xEboM17CvDRlxBtYQoIDKFYeK48RPo+hzWWBW5MeqXMOUTmODNDdEWz 8D2DicYelCpqZYSaoODe/h8lxJbiheK/PzoREww2wBJfM8HLIUEGes0dHXmQyawu/qlJ2DASbRc qSJsOKZUJGfJo+Aav6Cykeb9c74Md+4N93qAW8u1KLDLXxkLlsHvh1BTmFjSIzWWdMu7xelTqg8 x8VtMuLuEcc0h5gtvLEz4cyaAGS20SjQ40w X-Received: by 2002:a17:902:f54b:b0:2c6:99be:9dd with SMTP id d9443c01a7336-2dfa6248695mr7310435ad.12.1790615990264; Mon, 28 Sep 2026 10:19:50 -0700 (PDT) Received: from google.com (105.211.142.34.bc.googleusercontent.com. [34.142.211.105]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-3a49858c123sm538411a91.3.2026.09.28.10.19.44 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 28 Sep 2026 10:19:49 -0700 (PDT) Date: Mon, 28 Sep 2026 17:19:41 +0000 From: Pranjal Shrivastava To: David Matlack Cc: kexec@lists.infradead.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-pci@vger.kernel.org, Adithya Jayachandran , Alexander Graf , Alex Williamson , Bjorn Helgaas , Chris Li , David Rientjes , Jacob Pan , Jason Gunthorpe , Jonathan Corbet , Josh Hilke , Leon Romanovsky , Lukas Wunner , Mike Rapoport , Parav Pandit , Pasha Tatashin , Pratyush Yadav , Randy Dunlap , Saeed Mahameed , Samiullah Khawaja , Shuah Khan , Vipin Sharma , William Tu , Yi Liu Subject: Re: [PATCH v9 09/13] PCI: liveupdate: Adopt ACS controls in incoming preserved devices Message-ID: References: <20260918200640.887030-1-dmatlack@google.com> <20260918200640.887030-10-dmatlack@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20260918200640.887030-10-dmatlack@google.com> On Fri, Sep 18, 2026 at 08:06:35PM +0000, David Matlack wrote: > Adopt Access Control Services (ACS) controls on all incoming preserved > devices (endpoints and upstream bridges) during a Live Update. > > Inheriting ACS flags avoids changing routing rules while memory > transactions are in flight from preserved devices. This is also strictly > necessary to ensure that IOMMU group assignments do not change during or > after Live Update. > > The adopted controls are recorded by the pci_save_state() call in > pci_bus_add_device(), which runs before drivers bind, so they are > automatically reapplied by pci_restore_state() if the device is reset. > If that save buffer could not be allocated there is nowhere to record > the adopted controls, and they would be silently lost by the first > reset. Program ACS from scratch in that case, which is a better outcome > than leaving ACS disabled. > > To simplify ACS inheritance, reject preserving any devices that require > quirks to enable ACS as those quirks would also have to take Live Update > into account. > > Signed-off-by: David Matlack Reviewed-by: Pranjal Shrivastava Thanks, Praan