From: Lukas Wunner <lukas@wunner.de>
To: "Jozef Matejcik (Nokia)" <jozef.matejcik@nokia.com>
Cc: "linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
"Peter Sys (Nokia)" <peter.sys@nokia.com>
Subject: Re: PCI driver - hotplug of multiple devices
Date: Wed, 22 Jul 2026 10:36:00 +0200 [thread overview]
Message-ID: <amCA8AHMNorjWDli@wunner.de> (raw)
In-Reply-To: <AS4PR07MB85083609845930FF12F5C79593C12@AS4PR07MB8508.eurprd07.prod.outlook.com>
On Wed, Jul 22, 2026 at 07:12:24AM +0000, Jozef Matejcik (Nokia) wrote:
> The driver is Broadcom's linux-kernel-bde.
This seems to be an out-of-tree driver, i.e. not something that is
part of the mainline kernel. The upstream community can only provide
very limited support for such drivers. The vendor needs to upstream
the driver into the mainline kernel, otherwise there's only so much
we can do.
> The issue:
> - kernel module is inserted
> - we de-assert reset of these PCI devices (previously in reset, i.e.
> effectively powered down)
> - sometimes, very infrequently, the box freezes for 1-3 minutes and
> the HW triggers reboot
We'd need logs to say anything about it.
> When we removed the module before de-asserting the resets and then
> inserted it later, the issue did not occur. So we suspect that the
> kernel driver is accessing the device during probe too early and
> this is causing HW freeze.
Or maybe the reset puts the device in a state which is unexpected by
the driver.
> Are PCI drivers expected to handle hotplug events of any number of
> their devices simultaneously?
Yes. E.g. a global data structure in the driver needs to be properly
locked if accessed concurrently by the probe function for separate
devices.
> Can the issue I described above be considered a bug in vendor's
> implementation?
Sounds like it. Usually out-of-tree drivers haven't gone through
the rigorous review process by the community and quality may thus
be subpar. The recommendation is to ask vendors whether driver
support is in mainline already when selecting components for your
products.
Thanks,
Lukas
next prev parent reply other threads:[~2026-07-22 8:36 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-22 7:12 PCI driver - hotplug of multiple devices Jozef Matejcik (Nokia)
2026-07-22 8:36 ` Lukas Wunner [this message]
2026-07-22 9:58 ` Jozef Matejcik (Nokia)
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=amCA8AHMNorjWDli@wunner.de \
--to=lukas@wunner.de \
--cc=jozef.matejcik@nokia.com \
--cc=linux-pci@vger.kernel.org \
--cc=peter.sys@nokia.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.