linux-kernel - Re: [PATCH v3 0/4] PCI: Keystone: Enable loadable module support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <3sjuplupmdoxqhyz2i2p4he5vw7krqokixoy6ddoiox6p536n6@xzfcyhwjx3hv>
Date: Mon, 22 Sep 2025 14:02:43 +0530
From: Manivannan Sadhasivam <mani@...nel.org>
To: lpieralisi@...nel.org, kwilczynski@...nel.org, robh@...nel.org, 
	bhelgaas@...gle.com, jingoohan1@...il.com, christian.bruel@...s.st.com, 
	quic_wenbyao@...cinc.com, inochiama@...il.com, mayank.rana@....qualcomm.com, 
	thippeswamy.havalige@....com, shradha.t@...sung.com, cassel@...nel.org, kishon@...nel.org, 
	sergio.paracuellos@...il.com, 18255117159@....com, rongqianfeng@...o.com, jirislaby@...nel.org, 
	Siddharth Vadapalli <s-vadapalli@...com>
Cc: linux-pci@...r.kernel.org, linux-kernel@...r.kernel.org, 
	linux-arm-kernel@...ts.infradead.org, srk@...com
Subject: Re: [PATCH v3 0/4] PCI: Keystone: Enable loadable module support

On Mon, Sep 22, 2025 at 01:56:08PM +0530, Manivannan Sadhasivam wrote:
> 
> On Mon, 22 Sep 2025 12:42:12 +0530, Siddharth Vadapalli wrote:
> > This series enables support for the 'pci-keystone.c' driver to be built
> > as a loadable module. The motivation for the series is that PCIe is not
> > a necessity for booting Linux due to which the 'pci-keystone.c' driver
> > does not need to be built-in.
> > 
> > Series is based on commit
> > dc72930fe22e Merge branch 'pci/misc'
> > of pci/next.
> > 
> > [...]
> 
> Applied, thanks!
> 
> [1/4] PCI: Export pci_get_host_bridge_device() for use by pci-keystone
>       commit: c514ba0fa8938ae09370beecb77257868c1568a7
> [2/4] PCI: dwc: Export dw_pcie_allocate_domains() and dw_pcie_ep_raise_msix_irq()
>       commit: db9ff606a5535aee94bf41682f03aba500ff3ad6
> [3/4] PCI: keystone: Exit ks_pcie_probe() for invalid mode
>       commit: 76d23c87a3e06af003ae3a08053279d06141c716
> [4/4] PCI: keystone: Add support to build as a loadable module
>       commit: e82d56b5f3844189f2b2240b1c3eaeeafc8f1fd2
> 

I just noticed the build dependency mentioned in the cover letter after applying
the series. This is problematic since there is no guarantee that the dependent
commit will reach mainline first. So if this series gets applied by Linus first,
then building this driver as module will break the build. We should not have the
build error at any cost.

So I'm dropping this series now. Please repost once the fix is in mainline
(which will be next cycle).

- Mani

-- 
மணிவண்ணன் சதாசிவம்