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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0C4FDC433EF for ; Tue, 5 Oct 2021 19:45:34 +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 C1E1C610A5 for ; Tue, 5 Oct 2021 19:45:33 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org C1E1C610A5 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org 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:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=ugG02gOBAiU9OsJfdxASCAwa+x6Bm0g8bs6Uf7HuiB8=; b=QfUP4l33B0BWia PAX+xmX7niNwe1YH3ACv4QIc74AvUE4q3nXpMzwxaS6S6fSgwWXuQrvAGL5inBHIoP0+/NBf3LmAb fRJVEcDF2drPU+0rKUYFnxnGu81S910vJg9jjJpJb5IsRNLxugj8rCPejI8zcf30cURdQcQhtu9uR LO5kXNdMsCJgyK0I5r+BLFIpeoAnSRUYV4stUfUgkGxlstyIQuqd/2pVP2Rcv8Y4bYO4oIPWOr0Ly oomoegw3AfhXVcvsiHJBiYwLlbAogXDXAeiutOIxYcDBdoNw6YLgn3Wlost0hdbLKMCdgxZ1BE3wL xuGjVbXrfL76LoovjexQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mXqLJ-00BgDI-90; Tue, 05 Oct 2021 19:43:13 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mXqLB-00BgAw-3y; Tue, 05 Oct 2021 19:43:06 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id 063D4610A5; Tue, 5 Oct 2021 19:43:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1633462984; bh=93hVsy5BdDKlxUJFddXFi3vlWoO0RQGiRnw/E66GdM8=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=KYhYHG3q0/jMzU2P/lSej5mQZqlS1lOlJajSBxtUbkFvwRSihhtAyHaj7cEsOqc9h Oz5+t5YI0rMT5PVj4UvfF2f4JWP6x+0L/krFZ6sKIR7SL6EnRyD2iLNcmjqhXI+9bg zCw7M5f0xZ0HIPdq7rJwjqPsG13E94u0OOXE5W6F/EWgfWpM8YHY/8nIq/LcQO9Ny7 KmmqUAOTgC6gxtRCIqNlmzs/fnfrQQ1+Z0mldWwHkb1l/mR5GYFSuHB7hj1eEgxwlv YXQkoH85br8e3dHfM9WzC6KRQ0JcUUFKNRZKRb+N0dYdrecgGUZzC3MR4ha1tYxlnc pVdBhqZAsvzUg== Received: by pali.im (Postfix) id 85139812; Tue, 5 Oct 2021 21:43:01 +0200 (CEST) Date: Tue, 5 Oct 2021 21:43:01 +0200 From: Pali =?utf-8?B?Um9ow6Fy?= To: Jeremy Linton Cc: Bjorn Helgaas , linux-pci@vger.kernel.org, lorenzo.pieralisi@arm.com, nsaenz@kernel.org, bhelgaas@google.com, rjw@rjwysocki.net, lenb@kernel.org, robh@kernel.org, kw@linux.com, f.fainelli@gmail.com, bcm-kernel-feedback-list@broadcom.com, linux-acpi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rpi-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 2/4] PCI: brcmstb: Add ACPI config space quirk Message-ID: <20211005194301.enb5jddzdgczcolx@pali> References: <20211005153209.GA1083986@bhelgaas> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20180716 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211005_124305_224406_D4A7A8CE X-CRM114-Status: GOOD ( 25.57 ) 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 Hello! On Tuesday 05 October 2021 10:57:18 Jeremy Linton wrote: > Hi, > > On 10/5/21 10:32 AM, Bjorn Helgaas wrote: > > On Thu, Aug 26, 2021 at 02:15:55AM -0500, Jeremy Linton wrote: > > > Additionally, some basic bus/device filtering exist to avoid sending > > > config transactions to invalid devices on the RP's primary or > > > secondary bus. A basic link check is also made to assure that > > > something is operational on the secondary side before probing the > > > remainder of the config space. If either of these constraints are > > > violated and a config operation is lost in the ether because an EP > > > doesn't respond an unrecoverable SERROR is raised. > > > > It's not "lost"; I assume the root port raises an error because it > > can't send a transaction over a link that is down. > > The problem is AFAIK because the root port doesn't do that. Interesting! Does it mean that PCIe Root Complex / Host Bridge (which I guess contains also logic for Root Port) does not signal transaction failure for config requests? Or it is just your opinion? Because I'm dealing with similar issues and I'm trying to find a way how to detect if some PCIe IP signal transaction error via AXI SLVERR response OR it just does not send any response back. So if you know some way how to check which one it is, I would like to know it too. > > > > Is "SERROR" an ARM64 thing? My guess is the root port would raise an > > Unsupported Request error or similar, and the root complex turns that > > into a system-specific SERROR? Yes, SError is arm64 specific. It is asynchronous CPU interrupt and syndrome code then contains what happened. > AFAIK, what is happening here the CPU core has an outstanding R/W request > for which it never receives a response from the root port. So basically its > an interconnect protocol violation that the CPU is complaining about rather > than something PCIe specific. Could you describe (ideally in commit message) which SError is triggered? Normally if kernel receive SError interrupt it also puts into dmesg or oops message also syndrome code which describe what kind of error / event occurred. It could help also to other understand what is happening there. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel