From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f180.google.com (mail-pl1-f180.google.com [209.85.214.180]) (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 0BEBF3CB2D2 for ; Thu, 30 Jul 2026 23:20:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.180 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785453619; cv=none; b=qCjWjz1D9x3NlQOVjxk93ihVqApb0ai12/o2bOngb+JzNCpLadEyQyZEVKYnhOE2MjxgwXqRvtIe30VYKlfmihJzVnzV0LfvhwfmkdmLiaCcqZiTRrY8pfSuj9OaLG1XwzIeRWqSLUdwIo/9Coc6qKcfgizNRDz0d3frB/LIDbs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785453619; c=relaxed/simple; bh=D1HG6b77iifw3DUMszzZ7R8qVj01yGi3PV2UPXRdquM=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=PThJue8vHYyGtnjANlbQ5cNFjhjTVtlbaBk9KW+51C/5JRRL8Ba9jkugCTTj8wq/N9+ZGSSiPJnEtvziZLT0aHw3NBLYsPcZCX0/NQSzTOolr1TpGOFvvndNEDXjy6tRfuUp0o0n6DM368y8I0vgbYe00Q0duVXkq4iH3Fh9Q40= 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=EJmG/Hjr; arc=none smtp.client-ip=209.85.214.180 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="EJmG/Hjr" Received: by mail-pl1-f180.google.com with SMTP id d9443c01a7336-2ccdf36f63dso57155ad.0 for ; Thu, 30 Jul 2026 16:20:17 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20251104; t=1785453617; x=1786058417; 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=i8nq0/Hmm93fF2JW2THyDFZojbW5xmKXoinFZToOHmQ=; b=EJmG/Hjr+YdbZnwRfiRY1eYlZTOV2eeL3RN70Cfe37RhydlKsY4jDKXOu5lN2oG6IU bQ4k8Q6L9PNwpwpMlbXVd0lOkfgBCvkxJWGUy79dQDNRks8IqS2hnBPVpESe4wqDsU9E ovBfcHDSazjA/FoAfbUialeirbvZmMyKt/LE9TxLXX5FcrawobnODpL7zKyB3fOujiwf yCX5+KuwiDvjW0F2ZliHFDvPh0j8Zd4UZBFtw/wHztJzjMcDI59mvAZNTyWLcNBUq8R7 dSF8qCRCKU4kvHQaew/7BUzKZ6bL6i5ZtMRahhx+M795j0pHgn4q330wbzJaJ8ewFNut R56Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1785453617; x=1786058417; 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=i8nq0/Hmm93fF2JW2THyDFZojbW5xmKXoinFZToOHmQ=; b=Tsb5GeSupIEh+xomirkVF9jlg1LJUwwgIVzWY9Ho50zj624KGdRd7vTy2V64gxIfIu WZp712WOrTKLER0p/AgTCjkpiDuynmEO+eEYr/QQa4HHTa5dMN4rFlTl4Ov5WzMD66r2 LNqgyvKJ1moGNE8huXSvPvofD0dLkAqnOBJJN+zfpOIHj+9q5BeStp/sCHa92jPvmDE5 w+TEry7wWol8vGYFcxHIlvCTuU2mkfyFetqi3noU0ECK91mJK/RPhoQ2r1oIN3ChUCzp A+cIOWQcE+PGp8Am9gAMGG7+fCGxnuIzidfTbG53Kx5ewBgAgbGU5xEviWDZPvrPx7mI Mq2A== X-Forwarded-Encrypted: i=1; AHgh+Rpv1dA8SJlzuWpH/y6Z2uNX9KklzGq8y8P90A8nRLzdmgjjGvxISeOVg4EBWCagnwamne5i4HUIdRs=@vger.kernel.org X-Gm-Message-State: AOJu0YyvlwsI8HaKuZ878mGu22X0YVsTF9iV2YPh8LPpF4kISnu66ozM p/5yOumvWve54p0hGRaaLbsZbFlO1mPzrWKuY+BAs0yEeG19WEF1Jo5vQUTFGBAyww== X-Gm-Gg: AR+sD13BW1IpPpqZnp+Oy0M1k/gQFuESHtKBdaFezQRzCW2aY2WmC7wp/XDBqeAtNmJ XMTRCGLeutlrlmaPsQqxHVstwI8IjLH+VdwrR9FRvgFBkK+f8HiT5MFf5gD7eWiDiwPuju5pqnu 1/YkZ4boBt4DuvFpJr+TsMcascA6oTeqF7Jk4RU0gUOIoI3KJyo+PUToCjvG7Bm9VHOXBk6UhsE 7rQOK7B2hYt/uYARizIIxgMDemKCWt1Ko8T62nY7vHBwv+c7ijNmJ0OQRbUoPmhOnuaRQLxSAiI 5BSCGIdiVlB5HSaJuJ0Y3DrWoIzioWPKck+tCxEceZH/n3ZUttYyElk3lfQciWvEvXTghjKdzwj TsmRskz4K3ggKXu2PInKmeaTMh0EPY61ihSBmo63DdDojPSZIIgqGi4k91RgH1ogAghk0tRTR40 z6ncmApYAeBo8VABZt9Q/vmol83j7bPNrG5mc4ViqMT3ZMgFFcSSXpYZpr3Ir93DZAWs7WUQqKs XYHb2PCVAGzqLC9WXfBVVY= X-Received: by 2002:a17:903:2a88:b0:2c9:d89f:fd98 with SMTP id d9443c01a7336-2d043c66fc3mr2363495ad.1.1785453616883; Thu, 30 Jul 2026 16:20:16 -0700 (PDT) Received: from google.com (21.168.124.34.bc.googleusercontent.com. [34.124.168.21]) by smtp.gmail.com with ESMTPSA id a92af1059eb24-13e7265be48sm24706277c88.8.2026.07.30.16.20.12 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 30 Jul 2026 16:20:16 -0700 (PDT) Date: Thu, 30 Jul 2026 23:20:08 +0000 From: Pranjal Shrivastava To: Matt Evans Cc: Alex Williamson , Leon Romanovsky , Jason Gunthorpe , Alex Mastro , Christian =?iso-8859-1?Q?K=F6nig?= , Bjorn Helgaas , Logan Gunthorpe , Kevin Tian , Longfang Liu , Mahmoud Adam , David Matlack , =?iso-8859-1?Q?Bj=F6rn_T=F6pel?= , Sumit Semwal , Ankit Agrawal , Alistair Popple , Vivek Kasireddy , linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, dri-devel@lists.freedesktop.org, linaro-mm-sig@lists.linaro.org, kvm@vger.kernel.org, linux-pci@vger.kernel.org Subject: Re: [PATCH v5 7/9] vfio/pci: Clean up BAR zap and revocation Message-ID: References: <20260715174737.15287-1-matt@ozlabs.org> <20260715174737.15287-8-matt@ozlabs.org> Precedence: bulk X-Mailing-List: linux-pci@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: <20260715174737.15287-8-matt@ozlabs.org> On Wed, Jul 15, 2026 at 06:47:30PM +0100, Matt Evans wrote: > Previously, vfio_pci_zap_bars() (and the wrapper > vfio_pci_zap_and_down_write_memory_lock()) calls were paired with > calls to vfio_pci_dma_buf_move(). > > This commit replaces them with a unified new function, > vfio_pci_revoke_bars() containing both the vfio_pci_dma_buf_move() and > the unmap_mapping_range(), making it harder for callers to omit one. > It adds a wrapper, vfio_pci_lock_revoke_bars(), which takes the write > memory_lock before zapping, and adds a new vfio_pci_unrevoke_bars() > for the re-enable path. > > As of "vfio/pci: Convert BAR mmap() to use a DMABUF", the zap via > unmap_mapping_range() is no longer performed for vfio-pci since the > DMABUFs used for BAR mappings already zap PTEs when the > vfio_pci_dma_buf_move() occurs. > > However, it must be assumed that VFIO drivers which override the .mmap > op could create mappings _not_ backed by DMABUFs. So, the zap is > still performed on revoke if .mmap is overridden, using a new > zap_bars_on_revoke flag. A driver can explicitly opt out; the flag is > cleared by the hisi_acc_vfio_pci driver, since its .mmap just wraps > vfio_pci_core_mmap() and so still uses DMABUFs. > > Signed-off-by: Matt Evans Reviewed-by: Pranjal Shrivastava Thanks Praan