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=-2.4 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,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 195AAC432C3 for ; Tue, 3 Dec 2019 22:09:37 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id BC62C2080A for ; Tue, 3 Dec 2019 22:09:36 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=broadcom.com header.i=@broadcom.com header.b="geuMiRjA" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727605AbfLCWJf (ORCPT ); Tue, 3 Dec 2019 17:09:35 -0500 Received: from mail-pf1-f196.google.com ([209.85.210.196]:33614 "EHLO mail-pf1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727539AbfLCWJf (ORCPT ); Tue, 3 Dec 2019 17:09:35 -0500 Received: by mail-pf1-f196.google.com with SMTP id y206so2540697pfb.0 for ; Tue, 03 Dec 2019 14:09:34 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=mSBIYOscptZnYWaI4AeOjEh64oDv5ru63qkreKiXvlY=; b=geuMiRjARcxU+pIzzuOeH6lx6cneFVRMrKbm5tRMvY3hwxiwCgtP/MLufBqpU4RJ7h 3UO8ZYfwl1j2qXpO4wsmCZlyRB/63Co3z+e4AVTFnaN3wUIXNoU58uL87/FjdMvz5C9t dSKy47buYSGEPK+Do9zawbCYg+AbEQpnBDPBU= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=mSBIYOscptZnYWaI4AeOjEh64oDv5ru63qkreKiXvlY=; b=nqmT1zP/V8VPJTpPHDAtj0oOY+7Nk4kjSSVjqLE6nNW5ubbnJnwFonGaL9RJEQLDit fc6CT7On2T83pAjORrJwZR2Xizv6DNY6EDhu+FnHxNNg32ByEjc2sRfEICTpcVLVHc67 pP557ITpgjHFajPXTvxFuf/GUcYT9Zu0xu/Zq33f/IqBt7uAb/r3Cyjkhw9d0FfwYdPl gkkmg0xuCoowdUWh6Snfl9FhwHMrzGLSZmhZetzO+iN+O499W9eiQnZLAJiV47nHKnwe KPElhYVciN9kdFRzNRakUHvscvNPlRrcesKYBJsOYtbme0uMb/Canvro6gpHQuNITxbO RMlw== X-Gm-Message-State: APjAAAVu+Gh+WEFsR4DJebfd7s2bz+kNVtyGpO1z3bO8ChjJ/xMKQDeU saBLEXsABG0as+ig2XzSICJvSco/zKZxkJvbDf0uYz4UQj8k4asxRLbsBjtKgRzse/nkCyU0DkX dyoWReWS/MXr3eNnU1v/V/A7806Jc90NstWm7PoGLF0Dp3BAvBpzXeMc7cPxBzxgHG2AMWWS2ax 1t X-Google-Smtp-Source: APXvYqylfTPPGwnkmI3aoo7y5yzJGuEDLuZJ1o057PLvAd+XFAn29jSN6QkIzubcCN2MEgWSyp95Dg== X-Received: by 2002:a63:6704:: with SMTP id b4mr8002284pgc.424.1575410973842; Tue, 03 Dec 2019 14:09:33 -0800 (PST) Received: from rj-aorus.ric.broadcom.com ([192.19.228.250]) by smtp.gmail.com with ESMTPSA id k12sm4471207pgm.65.2019.12.03.14.09.26 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 03 Dec 2019 14:09:33 -0800 (PST) Subject: Re: [PATCH v3 2/6] PCI: iproc: Add INTx support with better modeling To: Andy Shevchenko , Andrew Murray Cc: Srinath Mannam , Lorenzo Pieralisi , Bjorn Helgaas , Florian Fainelli , Ray Jui , Rob Herring , Mark Rutland , Arnd Bergmann , bcm-kernel-feedback-list , linux-pci@vger.kernel.org, devicetree , linux-arm Mailing List , Linux Kernel Mailing List References: <1575349026-8743-1-git-send-email-srinath.mannam@broadcom.com> <1575349026-8743-3-git-send-email-srinath.mannam@broadcom.com> <20191203155514.GE18399@e119886-lin.cambridge.arm.com> From: Ray Jui Message-ID: <40fffa66-4b06-a851-84c2-4de36d5c6777@broadcom.com> Date: Tue, 3 Dec 2019 14:09:22 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.2.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 12/3/19 11:27 AM, Andy Shevchenko wrote: > On Tue, Dec 3, 2019 at 5:55 PM Andrew Murray wrote: >> On Tue, Dec 03, 2019 at 10:27:02AM +0530, Srinath Mannam wrote: > >>> + /* go through INTx A, B, C, D until all interrupts are handled */ >>> + do { >>> + status = iproc_pcie_read_reg(pcie, IPROC_PCIE_INTX_CSR); >> >> By performing this read once and outside of the do/while loop you may improve >> performance. I wonder how probable it is to get another INTx whilst handling >> one? > > May I ask how it can be improved? > One read will be needed any way, and so does this code. > I guess the current code will cause the IPROC_PCIE_INTX_CSR register to be read TWICE, if it's ever set to start with. But then if we do it outside of the while loop, if we ever receive an interrupt while servicing one, the interrupt will still need to be serviced, and in this case, it will cause additional context switch overhead by going out and back in the interrupt context. My take is that it's probably more ideal to leave this portion of code as it is. >>> + for_each_set_bit(bit, &status, PCI_NUM_INTX) { >>> + virq = irq_find_mapping(pcie->irq_domain, bit); >>> + if (virq) >>> + generic_handle_irq(virq); >>> + else >>> + dev_err(dev, "unexpected INTx%u\n", bit); >>> + } >>> + } while ((status & SYS_RC_INTX_MASK) != 0); >