From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp2.osuosl.org (smtp2.osuosl.org [140.211.166.133]) (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 507B6148FE3 for ; Fri, 10 May 2024 03:21:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=140.211.166.133 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715311308; cv=none; b=dHRzMy8++oTgWKVreMk5bFU/sYVhMW8c89qid0DwuZLA3khWAFvRpjQH4ryWXpXuIdawbDOtCYnYNJPfGYN+L+lBPRnBHxVncMbQL9wXe4ckw7xe4eVw1L4uz0k8K0c9eOhImDJ52sw17c2mwe1GCiUS8nZHYkRThd3fseKZa1A= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715311308; c=relaxed/simple; bh=fPVJrhZEYUI+SsesiqExa8OII6CVAsM+KT62Fck9Rik=; h=Message-ID:Date:MIME-Version:Cc:Subject:To:References:From: In-Reply-To:Content-Type; b=aPllz0KvHEpDjKbtz5Fc5/Wdk6PpweQYdKbXHEGt5WTaPZ/Q+WfFadW/MLW8UtydDz8VWXFHvxfns+V676NsCqoqygemkfk099SFPOKW/XDZceHt5qTyxiXH8NjBlxJnkq4A1jrUxNF0X5cYf65KpQ+mkU9iv53v/3z0H+EVG4A= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=cwYtapUI; arc=none smtp.client-ip=140.211.166.133 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="cwYtapUI" Received: from localhost (localhost [127.0.0.1]) by smtp2.osuosl.org (Postfix) with ESMTP id D1CD54245B for ; Fri, 10 May 2024 03:21:46 +0000 (UTC) X-Virus-Scanned: amavis at osuosl.org X-Spam-Flag: NO X-Spam-Score: -1.999 X-Spam-Level: Received: from smtp2.osuosl.org ([127.0.0.1]) by localhost (smtp2.osuosl.org [127.0.0.1]) (amavis, port 10024) with ESMTP id fCnmCJEGf3-7 for ; Fri, 10 May 2024 03:21:46 +0000 (UTC) Received-SPF: None (mailfrom) identity=mailfrom; client-ip=192.198.163.12; helo=mgamail.intel.com; envelope-from=baolu.lu@linux.intel.com; receiver= DMARC-Filter: OpenDMARC Filter v1.4.2 smtp2.osuosl.org DB3EA40A40 Authentication-Results: smtp2.osuosl.org; dmarc=none (p=none dis=none) header.from=linux.intel.com DKIM-Filter: OpenDKIM Filter v2.11.0 smtp2.osuosl.org DB3EA40A40 Authentication-Results: smtp2.osuosl.org; dkim=pass (2048-bit key, unprotected) header.d=intel.com header.i=@intel.com header.a=rsa-sha256 header.s=Intel header.b=cwYtapUI Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.12]) by smtp2.osuosl.org (Postfix) with ESMTPS id DB3EA40A40 for ; Fri, 10 May 2024 03:21:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1715311306; x=1746847306; h=message-id:date:mime-version:cc:subject:to:references: from:in-reply-to:content-transfer-encoding; bh=fPVJrhZEYUI+SsesiqExa8OII6CVAsM+KT62Fck9Rik=; b=cwYtapUIRgOwOvmS38gvP5rJe2sPKgCCzduGze99No588eJ5w30RJL9w 7+L/zSUzdHF5NSVFrAjLkDOJU6f1hRGkZDLiL4I03w86j8sZvBq8FjgYC 1HJISANcYn6l6LrPdxJQrHnsOCsTbKhdLVQKvW22T7wr7n1GOxojHfVWZ uOaPJt2WMoJ5HW4D7H/WGAcS4PX6VagbuqI3wL1EJm21cXbq1R+l2wp0L A7NJGrEIb2dMgnh2T6dwUQ0FK3YXRBXxkYjiY+kdwGkFVLeGZGJXa9ygG wAFcPOFe5Jrx+9458HnCqIFHBnrha2CU0yo/u2Jrkb/sFkzLeNIrCLiiI g==; X-CSE-ConnectionGUID: Nbn/Utg8RdKFnBz1ezaSHQ== X-CSE-MsgGUID: T1CXD0ZoQMCuyQI7zfKdGQ== X-IronPort-AV: E=McAfee;i="6600,9927,11068"; a="15081473" X-IronPort-AV: E=Sophos;i="6.08,149,1712646000"; d="scan'208";a="15081473" Received: from fmviesa010.fm.intel.com ([10.60.135.150]) by fmvoesa106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 09 May 2024 20:21:45 -0700 X-CSE-ConnectionGUID: eSsYCvnLScGnz0BLJXcK6w== X-CSE-MsgGUID: XpWBGYUJTx2MnxFq6z+pUg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.08,149,1712646000"; d="scan'208";a="29531328" Received: from unknown (HELO [10.239.159.127]) ([10.239.159.127]) by fmviesa010.fm.intel.com with ESMTP; 09 May 2024 20:21:42 -0700 Message-ID: <065cfc43-b164-4817-9be7-8f37585742ec@linux.intel.com> Date: Fri, 10 May 2024 11:20:01 +0800 Precedence: bulk X-Mailing-List: virtualization@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Cc: baolu.lu@linux.intel.com, Kevin Tian , Joerg Roedel , Will Deacon , Robin Murphy , Jean-Philippe Brucker , Nicolin Chen , Yi Liu , Jacob Pan , Joel Granados , iommu@lists.linux.dev, virtualization@lists.linux-foundation.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v5 6/9] iommufd: Fault-capable hwpt attach/detach/replace To: Jason Gunthorpe References: <20240430145710.68112-1-baolu.lu@linux.intel.com> <20240430145710.68112-7-baolu.lu@linux.intel.com> <20240508001840.GO4718@ziepe.ca> Content-Language: en-US From: Baolu Lu In-Reply-To: <20240508001840.GO4718@ziepe.ca> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 5/8/24 8:18 AM, Jason Gunthorpe wrote: > On Tue, Apr 30, 2024 at 10:57:07PM +0800, Lu Baolu wrote: >> diff --git a/drivers/iommu/iommufd/fault.c b/drivers/iommu/iommufd/fault.c >> index 13125c0feecb..6357229bf3b4 100644 >> --- a/drivers/iommu/iommufd/fault.c >> +++ b/drivers/iommu/iommufd/fault.c >> @@ -15,6 +15,124 @@ >> #include "../iommu-priv.h" >> #include "iommufd_private.h" >> >> +static int iommufd_fault_iopf_enable(struct iommufd_device *idev) >> +{ >> + int ret; >> + >> + if (idev->iopf_enabled) >> + return 0; >> + >> + ret = iommu_dev_enable_feature(idev->dev, IOMMU_DEV_FEAT_IOPF); >> + if (ret) >> + return ret; >> + >> + idev->iopf_enabled = true; >> + >> + return 0; >> +} > I would greatly prefer we remove this from the drivers :\ I guess it > is Ok for now > > Doesn't this need a counter? We can have many fault capable PASIDs? > That will get changed in the PASID series? Okay, let's design this more gracefully after the PASID interfaces are landed. For now, we assume that the device driver will do this. Best regards, baolu