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 X-Spam-Level: X-Spam-Status: No, score=-5.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5DC65C433E0 for ; Fri, 29 Jan 2021 19:02:52 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id EAC2364E0B for ; Fri, 29 Jan 2021 19:02:51 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org EAC2364E0B Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Type: Content-Transfer-Encoding:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=3+5Y6N+BWXKAOF9vjPbzYjrXHZj0qN7PR3CPwch+VXI=; b=hfFyBtJTHyPDIKNG9q/OuZT1d 5wifONQslLURMVKStoF2K35iOdNYUW9MDt01Q7E2xoqHKBu6aJ58NEeNVgKXz61Y4l9iOb8I8Ae8f F0C+MOg4OkEGI5LSz36oUCbAhWHRpHNRs7rLB08K7mI4lcSjfD6Elc+ooqN5hcdOfuzksV1XRTMFN s/+E0QxpF01JANomTmaubGIB5PuKqIyiuJyeJYXLcIeeHU91uzaH3Pp2KevYeWoHgpfI9efKbq01+ ks4mpakpZSuq5js3jMpdDY69uBf0iNcVkElVTIWqnndB0o9Nd5LBK63b7jA6p4afPpBISy8Y+XxcT DCP7FDy0w==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1l5XBs-0000hs-2Z; Fri, 29 Jan 2021 17:04:12 +0000 Received: from foss.arm.com ([217.140.110.172]) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1l5XBo-0000h3-8G for linux-arm-kernel@lists.infradead.org; Fri, 29 Jan 2021 17:04:09 +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 EF3D013A1; Fri, 29 Jan 2021 09:04:02 -0800 (PST) Received: from [10.57.35.163] (unknown [10.57.35.163]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 63C843F71B; Fri, 29 Jan 2021 09:04:01 -0800 (PST) Subject: Re: [PATCH v3 2/3] perf/smmuv3: Add a MODULE_SOFTDEP() to indicate dependency on SMMU To: John Garry , Zhen Lei , Will Deacon , Mark Rutland , Joerg Roedel , linux-arm-kernel , iommu , linux-kernel References: <20210127113258.1421-1-thunder.leizhen@huawei.com> <20210127113258.1421-3-thunder.leizhen@huawei.com> <67e0859b-2633-3516-527f-57557e210fa1@arm.com> <4b116f38-84da-9ef9-350c-8900944b0433@huawei.com> From: Robin Murphy Message-ID: Date: Fri, 29 Jan 2021 17:03:59 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; rv:78.0) Gecko/20100101 Thunderbird/78.7.0 MIME-Version: 1.0 In-Reply-To: <4b116f38-84da-9ef9-350c-8900944b0433@huawei.com> Content-Language: en-GB X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210129_120408_378110_09187FB0 X-CRM114-Status: GOOD ( 15.04 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Jean-Philippe Brucker 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 2021-01-29 15:34, John Garry wrote: > On 29/01/2021 15:12, Robin Murphy wrote: >> On 2021-01-27 11:32, Zhen Lei wrote: >>> The MODULE_SOFTDEP() gives user space a hint of the loading sequence. >>> And >>> when command "modprobe arm_smmuv3_pmu" is executed, the >>> arm_smmu_v3.ko is >>> automatically loaded in advance. >> >> Why do we need this? If probe order doesn't matter when both drivers >> are built-in, why should module load order? >> >> TBH I'm not sure why we even have a Kconfig dependency on ARM_SMMU_V3, >> given that the drivers operate completely independently :/ > > Can that Kconfig dependency just be removed? I think that it was added > under the idea that there is no point in having the SMMUv3 PMU driver > without the SMMUv3 driver. A PMCG *might* be usable for simply counting transactions to measure device activity regardless of its associated SMMU being enabled. Either way, it's not really Kconfig's job to decide what makes sense (beyond the top-level "can this driver *ever* be used on this platform" visibility choices). Imagine if we gave every PCI/USB/etc. device driver an explicit dependency on at least one host controller driver being enabled... Robin. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel