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 6615EC6FA89 for ; Wed, 7 Sep 2022 14:28:01 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:To:Subject:MIME-Version: Date:Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=z0hb11+JCHiKABnM14cfihHd2yrXMopgQ/6fkFWLxiI=; b=UBIbMlZsOvIHeZYV6XV58sNbOa D8lU5zJWtvMAVXCZtE9zAmdWe0gug4LZfFHI5lxBrB74FMR1GxIJkV6vWj/VxBD0lUZ7jKdcVUJhW vGJAeiCfa6iuRJdmjj9prPh+T/q1wQOko4Jjme7bnmmedGnLXukEZ9YWUSamd4j2FHjorGfDkepqL R5+orPwEJ5QyIpQ8EG3FpLG2cklUlfX+Bd7O3kh7YPi9dIQ/pjXYwll7IC/YtyqYi0Y/URbeU2kC1 Hyy8fbxaXnIkzfeF34vDGR1XFyWA5Pl/N3wjVJNjQaPcBoG6TizGP7QioYhIuqc84LHmv6KZiBO6u d2qmslwA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oVw13-006rrT-0Z; Wed, 07 Sep 2022 14:26:57 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oVvxh-006pOp-7u for linux-arm-kernel@lists.infradead.org; Wed, 07 Sep 2022 14:23:30 +0000 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 CCDD91042; Wed, 7 Sep 2022 07:23:32 -0700 (PDT) Received: from [10.57.15.197] (unknown [10.57.15.197]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 3B9E63F71A; Wed, 7 Sep 2022 07:23:15 -0700 (PDT) Message-ID: <9f91f187-2767-13f9-68a2-a5458b888f00@arm.com> Date: Wed, 7 Sep 2022 15:23:09 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; rv:102.0) Gecko/20100101 Thunderbird/102.2.1 Subject: Re: [PATCH v6 1/5] iommu: Return -EMEDIUMTYPE for incompatible domain and device/group Content-Language: en-GB To: Jason Gunthorpe , Joerg Roedel References: <20220815181437.28127-1-nicolinc@nvidia.com> <20220815181437.28127-2-nicolinc@nvidia.com> From: Robin Murphy In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220907_072329_557642_2260CA10 X-CRM114-Status: GOOD ( 20.72 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: marcan@marcan.st, mjrosato@linux.ibm.com, linux-kernel@vger.kernel.org, thierry.reding@gmail.com, will@kernel.org, alyssa@rosenzweig.io, jean-philippe@linaro.org, kvm@vger.kernel.org, zhang.lyra@gmail.com, jon@solid-run.com, jonathanh@nvidia.com, iommu@lists.linux.dev, Nicolin Chen , yangyingliang@huawei.com, orsonzhai@gmail.com, gerald.schaefer@linux.ibm.com, kevin.tian@intel.com, sven@svenpeter.dev, linux-arm-msm@vger.kernel.org, christophe.jaillet@wanadoo.fr, baolin.wang@linux.alibaba.com, thunder.leizhen@huawei.com, linux-tegra@vger.kernel.org, tglx@linutronix.de, virtualization@lists.linux-foundation.org, linux-arm-kernel@lists.infradead.org, linux-s390@vger.kernel.org, cohuck@redhat.com, alex.williamson@redhat.com, shameerali.kolothum.thodi@huawei.com, robdclark@gmail.com, asahi@lists.linux.dev, suravee.suthikulpanit@amd.com, dwmw2@infradead.org, baolu.lu@linux.intel.com Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 2022-09-07 14:47, Jason Gunthorpe wrote: > On Wed, Sep 07, 2022 at 02:41:54PM +0200, Joerg Roedel wrote: >> On Mon, Aug 15, 2022 at 11:14:33AM -0700, Nicolin Chen wrote: >>> Provide a dedicated errno from the IOMMU driver during attach that the >>> reason attached failed is because of domain incompatability. EMEDIUMTYPE >>> is chosen because it is never used within the iommu subsystem today and >>> evokes a sense that the 'medium' aka the domain is incompatible. >> >> I am not a fan of re-using EMEDIUMTYPE or any other special value. What >> is needed here in EINVAL, but with a way to tell the caller which of the >> function parameters is actually invalid. > > Using errnos to indicate the nature of failure is a well established > unix practice, it is why we have hundreds of error codes and don't > just return -EINVAL for everything. > > What don't you like about it? > > Would you be happier if we wrote it like > > #define IOMMU_EINCOMPATIBLE_DEVICE xx > > Which tells "which of the function parameters is actually invalid" ? FWIW, we're now very close to being able to validate dev->iommu against where the domain came from in core code, and so short-circuit ->attach_dev entirely if they don't match. At that point -EINVAL at the driver callback level could be assumed to refer to the domain argument, while anything else could be taken as something going unexpectedly wrong when the attach may otherwise have worked. I've forgotten if we actually had a valid case anywhere for "this is my device but even if you retry with a different domain it's still never going to work", but I think we wouldn't actually need that anyway - it should be clear enough to a caller that if attaching to an existing domain fails, then allocating a fresh domain and attaching also fails, that's the point to give up. Robin. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel