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=-16.1 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable 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 75940C4338F for ; Wed, 4 Aug 2021 14:04:59 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 4328060F14 for ; Wed, 4 Aug 2021 14:04:59 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 4328060F14 Authentication-Results: mail.kernel.org; dmarc=fail (p=quarantine dis=none) header.from=ti.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date: Message-ID:From:References:CC:To:Subject:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Owner; bh=8Nj/79pfQ/R4l1YAUF4ZPucSz+A60gT9lcHUj7KofPc=; b=vfueL0MDadrJXQqoS91bps1N2V kBuRqkHoGOxBhZA0udKvGhtajXmhuEcnZeVxRrpPKogJsrYYqaKTf5txsrcMfYc967zZ7kVnCucfe zG799M7l01eNY9tV0AE+jlGXmTn1w3tS9N+kH66zU6UwY41KRpakNl63C/qNhOjOPQF3CQkmq8o7e nMa8JRgmo/fECD83LTh29Vhk7fCf8rPzv1MY/wH1EZwylv177PHnSl9Y6DKhNTtu/waZtihMaGyGD jcIFu68rcm+7aVpjWNuKi5zh/XkFPsfrXNpBGZlxMq/6bOqBUrSW2q/4YbkoVJvKPWOtuefwWYdDr w0oHWGOw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mBHU7-006H2d-TM; Wed, 04 Aug 2021 14:03:04 +0000 Received: from fllv0015.ext.ti.com ([198.47.19.141]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mBHU3-006H1X-M5 for linux-arm-kernel@lists.infradead.org; Wed, 04 Aug 2021 14:03:01 +0000 Received: from fllv0035.itg.ti.com ([10.64.41.0]) by fllv0015.ext.ti.com (8.15.2/8.15.2) with ESMTP id 174E2oMH093964; Wed, 4 Aug 2021 09:02:50 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1628085770; bh=sMUS15sZoryFM7mBrI98D75T6KWCTkls5l3Y2MZV98k=; h=Subject:To:CC:References:From:Date:In-Reply-To; b=u/njbcLgd+kGIYKB+ct73TobZqhkgR+x45v6RJLvEMnJOUhs6YtD4fWMu2AtBo8Lt FiRu/TiAZIQoJuR7ceTRHUAixAA4dYzlMHk+eSZxw2k+2p/XVzKD/rx4vJPPoWnTZX 0CePXUTAg/MhN7Did2dH4HottkAhY1mAdtNpDl8s= Received: from DLEE105.ent.ti.com (dlee105.ent.ti.com [157.170.170.35]) by fllv0035.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 174E2oOO027011 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Wed, 4 Aug 2021 09:02:50 -0500 Received: from DLEE108.ent.ti.com (157.170.170.38) by DLEE105.ent.ti.com (157.170.170.35) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2176.2; Wed, 4 Aug 2021 09:02:49 -0500 Received: from fllv0040.itg.ti.com (10.64.41.20) by DLEE108.ent.ti.com (157.170.170.38) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2176.2 via Frontend Transport; Wed, 4 Aug 2021 09:02:49 -0500 Received: from [10.250.233.161] (ileax41-snat.itg.ti.com [10.172.224.153]) by fllv0040.itg.ti.com (8.15.2/8.15.2) with ESMTP id 174E2iWO099533; Wed, 4 Aug 2021 09:02:45 -0500 Subject: Re: [PATCH v2 5/6] misc: pci_endpoint_test: Do not request or allocate IRQs in probe To: Lorenzo Pieralisi CC: Arnd Bergmann , Rob Herring , Bjorn Helgaas , Lokesh Vutla , Greg Kroah-Hartman , Tom Joseph , , , , , References: <20210803074932.19820-1-kishon@ti.com> <20210803074932.19820-6-kishon@ti.com> <20210803095839.GA11252@lpieralisi> From: Kishon Vijay Abraham I Message-ID: <02c1ddb7-539e-20a0-1bef-e10e76922a0e@ti.com> Date: Wed, 4 Aug 2021 19:32:44 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <20210803095839.GA11252@lpieralisi> Content-Language: en-US X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210804_070259_863757_D4A87304 X-CRM114-Status: GOOD ( 29.30 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Lorenzo, On 03/08/21 3:28 pm, Lorenzo Pieralisi wrote: > On Tue, Aug 03, 2021 at 01:19:31PM +0530, Kishon Vijay Abraham I wrote: >> Allocation of IRQ vectors and requesting IRQ is done as part of >> PCITEST_SET_IRQTYPE. Do not request or allocate IRQs in probe for >> AM654 and J721E so that the user space test script has better control >> of the devices for which the IRQs are configured. Since certain user >> space scripts could rely on allocation of IRQ vectors during probe, >> remove allocation of IRQs only for TI's K3 platform. >> >> Signed-off-by: Kishon Vijay Abraham I >> --- >> drivers/misc/pci_endpoint_test.c | 19 +++++++++++++------ >> 1 file changed, 13 insertions(+), 6 deletions(-) > > I don't claim to understand the inner details of the endpoint test > device but it looks like this approach should be redesigned. > > I don't believe using devices quirks is the best approach to > expose/remove a feature to userspace, this can soon become > unmaintenable. > > Maybe you can elaborate a bit more on what the real issue is please ? The actual reason for introducing this patch (affects only AM654 and J721E) is due to Errata ID #i2101 GIC: ITS Misbehavior (https://www.ti.com/lit/er/sprz455a/sprz455a.pdf). So if more than 5 devices use GIC ITS simultaneously, GIC fails to raise interrupts. Though this patch is not an actual workaround for the issue (the workaround is in GIC ITS driver provided in the errata document), it helps to keep testing PCIe RC/EP using pci-endpoint-test even when multiple pci-epf-test endpoint devices are connected (Normal test-setup having J721E-J721E back to back connection can support 21 pci-epf-test devices). So this patch lets user to individually enable interrupts for each of the devices and could disable after the interrupt test. Since pci_endpoint_test is used only for testing PCIE RC/EP communication and pci-endpoint-test has already implemented PCITEST_SET_IRQTYPE for the userspace to enable interrupt, tried to not enable the interrupts of all the devices by default in the probe (for AM654 and J721E where this errata applies). Thanks, Kishon > > Thanks, > Lorenzo > >> diff --git a/drivers/misc/pci_endpoint_test.c b/drivers/misc/pci_endpoint_test.c >> index c7ee34013485..9740f2a0e7cd 100644 >> --- a/drivers/misc/pci_endpoint_test.c >> +++ b/drivers/misc/pci_endpoint_test.c >> @@ -79,6 +79,9 @@ >> #define PCI_DEVICE_ID_RENESAS_R8A774C0 0x002d >> #define PCI_DEVICE_ID_RENESAS_R8A774E1 0x0025 >> >> +#define is_j721e_pci_dev(pdev) \ >> + ((pdev)->device == PCI_DEVICE_ID_TI_J721E) >> + >> static DEFINE_IDA(pci_endpoint_test_ida); >> >> #define to_endpoint_test(priv) container_of((priv), struct pci_endpoint_test, \ >> @@ -810,9 +813,11 @@ static int pci_endpoint_test_probe(struct pci_dev *pdev, >> >> pci_set_master(pdev); >> >> - if (!pci_endpoint_test_alloc_irq_vectors(test, irq_type)) { >> - err = -EINVAL; >> - goto err_disable_irq; >> + if (!(is_am654_pci_dev(pdev) || is_j721e_pci_dev(pdev))) { >> + if (!pci_endpoint_test_alloc_irq_vectors(test, irq_type)) { >> + err = -EINVAL; >> + goto err_disable_irq; >> + } >> } >> >> for (bar = 0; bar < PCI_STD_NUM_BARS; bar++) { >> @@ -850,9 +855,11 @@ static int pci_endpoint_test_probe(struct pci_dev *pdev, >> goto err_ida_remove; >> } >> >> - if (!pci_endpoint_test_request_irq(test)) { >> - err = -EINVAL; >> - goto err_kfree_test_name; >> + if (!(is_am654_pci_dev(pdev) || is_j721e_pci_dev(pdev))) { >> + if (!pci_endpoint_test_request_irq(test)) { >> + err = -EINVAL; >> + goto err_kfree_test_name; >> + } >> } >> >> misc_device = &test->miscdev; >> -- >> 2.17.1 >> _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel