Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Fri, 21 Jan 2022 10:57:07 -0600
From:   Bjorn Helgaas <helgaas@kernel.org>
To:     Krzysztof =?utf-8?Q?Ha=C5=82asa?= <khalasa@piap.pl>
Cc:     Pali =?iso-8859-1?Q?Roh=E1r?= <pali@kernel.org>,
        Bjorn Helgaas <bhelgaas@google.com>, linux-pci@vger.kernel.org,
        linux-kernel@vger.kernel.org,
        Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>,
        Gregory Clement <gregory.clement@bootlin.com>,
        Andrew Lunn <andrew@lunn.ch>,
        Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= <kw@linux.com>
Subject: Re: PCI: Race condition in pci_create_sysfs_dev_files (can't boot)
Message-ID: <20220121165707.GA1129091@bhelgaas>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <m3bl05zhjt.fsf@t19.piap.pl>
Precedence: bulk

On Fri, Jan 21, 2022 at 08:14:46AM +0100, Krzysztof Hałasa wrote:
> Hi Bjorn, Krzysztof,
> 
> Bjorn Helgaas <helgaas@kernel.org> writes:
> 
> > On Fri, Jul 30, 2021 at 10:18:44AM +0200, Krzysztof Hałasa wrote:
> >> I'm encountering a problem booting an i.MX6-based device (Gateworks
> >> Ventana SBC). This is apparently a known issue:
> >> https://lkml.org/lkml/2020/7/16/388
> 
> > Hi Krzysztof, is this still an issue?
> 
> Well... I'm still using i.MX6 with 5.14 and Krzysiek Wilczyński's "PCI:
> Race condition in pci_create_sysfs_dev_files (can't boot)" patch (which
> fixes the problem). It seems parts of this patch are now in mainline,
> but most of it is still missing. So I guess the problem isn't fixed.
> 
> Perhaps because the DEC Alpha part of the patch is not yet ready?
> Krzysztof?
> 
> > e1d3f3268b0e ("PCI/sysfs: Convert "config" to static attribute") and
> > similar patches appeared in v5.13,
> 
> Right, but they hadn't fixed the problem. 5.13 was released in June, and
> I have started using the patch in August (apparently with pre-5.14
> first, then with final 5.14).

Thanks.  e1d3f3268b0e and related patches converted individual files
("config", "rom", "vpd", etc) to static attributes, but since the
problem you're seeing is with a directory, it's likely different.

I opened this bugzilla report to try to keep this from getting lost:
https://bugzilla.kernel.org/show_bug.cgi?id=215515

Bjorn