From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 7BB7BC6FA82 for ; Tue, 20 Sep 2022 18:07:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:MIME-Version:In-Reply-To: Content-Type:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=elt4EDAM2W6uenHPGGaT8UUQTvBHGnkaO37Ju0rmF3o=; b=L+S4cqAHuIfu8Gfuc+SSl+hn+W YXYioAJovGhu3gBsHllSuzKEqY534GbjFMOiijh0uKPlLOfnuxdzCNFVYmZKPgDdQoWW30kudsWHI LwLITxP+J2lZIQ6w/yUe2gv2bCAF8Qp79hHVRcVeC2HzamGWJm+BH9amD9GnltskV2DaoFX39ILSW kjLSRAgOcCwGmdRaETu9y1ar7tirslgLoAmdW1HXsVwp2fkbnIyx7jfXeN28IPt6pe4KgTg6WnZqr xNvFQwjxFyEwVc0nM61ggWksNwo1fh6y9WtoUe+99RqIzMWil3y424s2+sJvdlp8MftzSV4pOAfpY qoO2Oi6g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oaheI-005d97-L2; Tue, 20 Sep 2022 18:07:10 +0000 Received: from mail-co1nam11on2050.outbound.protection.outlook.com ([40.107.220.50] helo=NAM11-CO1-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oaheF-005d5d-Cq; Tue, 20 Sep 2022 18:07:08 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=V3xg4nOjr1j3GZyLuOvtEC3qH/75AilVAESlnO0LSxairds25QAAq+NUmGbckH5QEfJW0i1X2cK2c/q9UpZqRL5h3Ws+e9ITc0j9mMH+wnMYUjU7PdhLd2t87aJnDtcRE/YTmQIIMEoUxnuV52KzMzVer6JxKHdcUBXWfM+gqJx6EjX9jkLEfIULQk5ETn7JCZ9XPozg4Wq17cLp/EUm+xalHtUEo9yEWLgIbGuFDd51D4ZYsrKhXx3UxsKkwoDCbAjG5oc8gljaXcluP6MK0ZL4HII147ensp1T4wc+YKAyCiaBey/a80ZLgC+ULerahgYFA569Dx/fpUEOIJC9mg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=elt4EDAM2W6uenHPGGaT8UUQTvBHGnkaO37Ju0rmF3o=; b=S9PTxU9sHGAJf4NJrdC5USiWxSy6++Jsdy8S/VzDFAWgttIOZk0bg0xo3J54f9wb0PrOi++Za5iy51cyyRCZiyD8et3Soe80BC5SdfVfqkLjTHSLQk+LUDq5oDjpJQl+WE59cEp5FtXYuE9EpyOer1LNz8eY3+79yEqJDpflFXzP+RxMkl8ZlnrqhgZwJKmzEv2Cvitc1Gh+H1L9z1JqQBzDm1G0+0cOWQubzfIkDuq4J2dSqrV9JSuKdbyg0K1tmwCqFRP5l7MXRSAptM4+JV695tI3RiEMWwBpNzxUpUeN21A9mS3gezZoDtt06RB6SYYfrkUBD7twbV07Lvk5PA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nvidia.com; dmarc=pass action=none header.from=nvidia.com; dkim=pass header.d=nvidia.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=elt4EDAM2W6uenHPGGaT8UUQTvBHGnkaO37Ju0rmF3o=; b=pZLHOgSg4IOrBtWVNPYJZ6igClmPmPteQ2iC/oHPo0gV9vE1BiQa3MFWJ29LoQORCmLv+RsJLIcELmcHpuFGzEg2jz5lhR+YAWivyb2lvxefDKlXCx3FFyPTfKBH/zUxeCwmlK+q+qbWWFYIwlqtZoPEFi1aBP/UDovkAtcXBFu+dU+pGPKqfJETBhDASSJRZwKLVwqCM6m7yNMsVY9MZxX+MQRjrGYOpntAXQCfpfToyg2vriMSbLGn/gq1TZHBujhnzH5qVtGq72xSwl3zEYeO4r4dI4jFuQ34ckzOjO+rQun0cXHG27wp8Uub7/7ziHitYZUsDEIYoehLBKbm5g== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nvidia.com; Received: from MN2PR12MB4192.namprd12.prod.outlook.com (2603:10b6:208:1d5::15) by CY5PR12MB6600.namprd12.prod.outlook.com (2603:10b6:930:40::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5632.21; Tue, 20 Sep 2022 18:06:54 +0000 Received: from MN2PR12MB4192.namprd12.prod.outlook.com ([fe80::462:7fe:f04f:d0d5]) by MN2PR12MB4192.namprd12.prod.outlook.com ([fe80::462:7fe:f04f:d0d5%7]) with mapi id 15.20.5632.021; Tue, 20 Sep 2022 18:06:54 +0000 Date: Tue, 20 Sep 2022 15:06:52 -0300 From: Jason Gunthorpe To: "Tian, Kevin" Cc: Nicolin Chen , "joro@8bytes.org" , "suravee.suthikulpanit@amd.com" , "will@kernel.org" , "robin.murphy@arm.com" , "robdclark@gmail.com" , "dwmw2@infradead.org" , "baolu.lu@linux.intel.com" , "agross@kernel.org" , "bjorn.andersson@linaro.org" , "matthias.bgg@gmail.com" , "orsonzhai@gmail.com" , "baolin.wang@linux.alibaba.com" , "zhang.lyra@gmail.com" , "jean-philippe@linaro.org" , "sricharan@codeaurora.org" , "konrad.dybcio@somainline.org" , "yong.wu@mediatek.com" , "thierry.reding@gmail.com" , "vdumpa@nvidia.com" , "jonathanh@nvidia.com" , "tglx@linutronix.de" , "shameerali.kolothum.thodi@huawei.com" , "christophe.jaillet@wanadoo.fr" , "thunder.leizhen@huawei.com" , "quic_saipraka@quicinc.com" , "jon@solid-run.com" , "yangyingliang@huawei.com" , "iommu@lists.linux.dev" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-arm-msm@vger.kernel.org" , "linux-mediatek@lists.infradead.org" , "linux-tegra@vger.kernel.org" , "virtualization@lists.linux-foundation.org" Subject: Re: [PATCH v3 5/6] iommu: Use EINVAL for incompatible device/domain in ->attach_dev Message-ID: References: Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-ClientProxiedBy: MN2PR10CA0035.namprd10.prod.outlook.com (2603:10b6:208:120::48) To MN2PR12MB4192.namprd12.prod.outlook.com (2603:10b6:208:1d5::15) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: MN2PR12MB4192:EE_|CY5PR12MB6600:EE_ X-MS-Office365-Filtering-Correlation-Id: b07cf9da-53c4-4b8f-a5d9-08da9b32e62c X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: Da2d5DU2c98vVzCSaQnNFONwdVBTSlnfSrG4xZm8Z/rjG8QLhozOaPxoQmSLYGmYBp7arnQHF7IwkTS8FZeMJJr12pS9fG9lxoOK9h0b9F9EJd84hXidJbW1mpP+XTnT/7KjFnyKfJL6DDRR55od4BfuXGeLI2cHF2iZBFKnEPrXyrcuVStWLfIBJa9GlDfRC1y9DZKvO3g1ea/XVxDEpMUkrfU/HL8sr0iQ6kT3pVk4t1cG8aD1le7iUQwz0JGoXJ8jiXM2JxmRC/EcQV5PZHqKry6W+FUU/0MS5iMqS13aWMdtkr46MU6miIVfjmr043HDmUN+K5elsJ5aezdrxoPJFb2FX6zXEzcDbhtSH/61MxiRDc7i7pT2ew66vVFrz7Xl/wjwcddbpLc6oz/rX1PMWWDldgMES9TaDTey4cOW+dkqvktaNvWbwl4FKMb+OUMl3Br1bvEgsLREHvGxm1ffOkSZvnOarPuAZWPl4mKlceIf8pHSDZv/TB5T8MwUWDsKfeynAfe35vSJ/qooqfX/KXiRjvxgFRs2CaF/DH2wEbHr9QIgq2PygaK6oO1KL6Z9zgtqBZzHWYVS7/t05HZgM6Hk4UvFKkLf4rV2bIA0JwhiGnnpS01dhTFigrQfjOqH1TbBalu6KyS3+9pJO54FFtNz8y/AeH3FXfgo87Mm8YiHfMJMkvTMw4qzRIiAxMQfSvL8uOf5hrH8+li33M+K6kdfFeSd3l1QdvdljveyJfcea/TOwRf/sQuBDJIH X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:MN2PR12MB4192.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230022)(4636009)(346002)(366004)(376002)(39860400002)(396003)(136003)(451199015)(316002)(83380400001)(7416002)(7406005)(4326008)(8676002)(36756003)(66476007)(66556008)(66946007)(86362001)(2906002)(38100700002)(478600001)(4744005)(6486002)(6512007)(26005)(2616005)(6916009)(54906003)(8936002)(5660300002)(186003)(41300700001)(6506007)(67856001);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?ygOk7bZVq7ZtvK1iFCVbat++xxH4vb9mJ8B2YdtIno9oGFAdhgmwZX9xZhC1?= =?us-ascii?Q?a7pTYzx50iOUeZ5Dz+9SW3uNulgseV6RfKOj/Y8fSYKNJMLjaNCjQers9wk+?= =?us-ascii?Q?VF5KxSl2YecOQlfuWSi/SAc9sC9Onq1jgRhQdRn5YfJMsd0veR7KKBZN5w85?= =?us-ascii?Q?UFY/G0g0kzi4fK39WQz8dQZbNyLzcg5chnKkX3n9mtRqB8aRdBpaHmLu2Vq/?= =?us-ascii?Q?h0b1XCuDC+AxZ4KrkMzffpXxjax5xAmrh6230nDpivQPMnGZ+7YjAOEQSmbV?= =?us-ascii?Q?4X40bFxqQL2gYLIPvHAqSAXezYfaMM++6x+FqH1AZn8AbR6yppfux20M7tro?= =?us-ascii?Q?LhKkvx99RwVBmZ+COUqAvH3uVrO/lmJl/2A16YXBMCiohkBVEkjVnO87CbEy?= =?us-ascii?Q?YCtej6ocauh350qsnOq4dQ3dkRYc0t3SW/wXiaGr1sEXzzOn9t0oIb1HwnvL?= =?us-ascii?Q?6e25jTgT7yL+BJvYx4lJsoVu6ZtAvydOS6YycmThEXH4fcbkZ9WjneQ0DdP0?= =?us-ascii?Q?YFIZZBU9FSkOeUyJrqxfs5lfifDwjFJDNpE9iX1dxU0thad9wANS/ZcGKKl5?= =?us-ascii?Q?lb4JG2pAuZRNGYxkQLLvmjoKGkw0bx49Hcoblsr0ruW6wBeBqaEPGXfQMrJH?= =?us-ascii?Q?D0Rwj1Np3YJb8LMhxrWjPXQar/gRKhJ6U9HAT/GQ49HN2S+RxjBg5PP+hQ6a?= =?us-ascii?Q?WBy9+8e3WuG7eq+ss4dql37DbbeDAg1GASb8SMYeGOJp4jPofM/wI4+diITY?= =?us-ascii?Q?9bKDKIIIqz6eho0jW/ANwMr9CdZeHochiL0Lc4RVmOfZWjMwDIgONbkPgTYJ?= =?us-ascii?Q?GuusLhgpgpn5CFOOqIn9xuHtTKSm4Gwhrqwy8j91BWSfPtXbwUhGi8Sw0DZ3?= =?us-ascii?Q?X9E95j/jpsDnbkzokUmVPu+q+LoQb7udD30PSdqRGHdisj4vjqW896yycgBD?= =?us-ascii?Q?gqq3f81XvWO2rT5Gfw/2A7L3phst+TRl2a6cFCsU2lEMtNuccpmVAM/nLL3p?= =?us-ascii?Q?DZ72oTkmmM4b/ezOnW9qHPJ1+a+XaX2nP6cvOsNiZtSCaHmqoMPYDq37iCWn?= =?us-ascii?Q?9PZ43JC1nAxel06CmQU7yHfCoYBXUs+NPe0xk0SpnJvKrccOek1yecLgSsIn?= =?us-ascii?Q?pkN1/Pkq6JtTgoy9e3tYTWJ9K45Ui+LeOaqLOQAnm0Viq3+jWJYlMRkeEp4E?= =?us-ascii?Q?TyHh7ujnz03vdfSkDRQdBPhN3OvcpM2+fqE7HtY554eQdQwI+uPwCXUxZ69x?= =?us-ascii?Q?hOrfZSj/LicaNjxK7j4dfZmtgAG6CA0bmlkjg6HVwJ1q5pPA6GP+7oaORfLg?= =?us-ascii?Q?jtrYpNFKPPodd47Uri9/5Dw5fP8dZ/U5GA/vh+PLBuurmGKwWmyYXGz8SBXi?= =?us-ascii?Q?cxiCXh284y441Gf6kU91mbWkxRCCA94Pe2o4SyM1rEYfC/jKBFqWF+s843oa?= =?us-ascii?Q?UQPAmRmsi54o7ti8BfJbFQWKTQacIhFwlI4I12w1UIud29voi+BR54byO6vO?= =?us-ascii?Q?rfJzdWO+xK3RIdqAX9YTZFyEodvDCwhMLng5aCFVxROI7iD+NDcagfBUdNet?= =?us-ascii?Q?MJaazTC8gaUiPuuKTxcJZwTyMqOPZuqyOtRW72mC?= X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: b07cf9da-53c4-4b8f-a5d9-08da9b32e62c X-MS-Exchange-CrossTenant-AuthSource: MN2PR12MB4192.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Sep 2022 18:06:54.1839 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 2oHNc7FqNXiMjYyfVm+9Wygld14aa2nKrPdFzQs7LuCTpfz9xnkTZuafeodNQMdw X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY5PR12MB6600 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220920_110707_457037_B7ABC107 X-CRM114-Status: GOOD ( 11.41 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org On Tue, Sep 20, 2022 at 06:38:18AM +0000, Tian, Kevin wrote: > Above lacks of a conversion in intel-iommu: > > intel_iommu_attach_device() > if (domain->type == IOMMU_DOMAIN_UNMANAGED && > device_is_rmrr_locked(dev)) { > dev_warn(dev, "Device is ineligible for IOMMU domain attach due to platform RMRR requirement. Contact your platform vendor.\n"); > return -EPERM; > } > > since it's based on the domain type, picking a different domain > may work in theory though it won't apply to vfio which always > creates unmanaged type. IMHO this test shouldn't even be here, that is why it is so strange.. VFIO should be checking if somehow the device doesn't support unmanaged domains at all. We already have several drivers that can't support full featured unamanged domains that vfio needs, this is just another case of that. But it isn't urgent to fix, I would just ignore this branch for this series. Jason