On Mon, May 23, 2022 at 09:18:28PM +0300, Dmitry Baryshkov wrote:
> I have replied with my Tested-by to the patch at [2], which has landed
> in the linux-next as the commit 20f1bfb8dd62 ("PCI: qcom:
> Add support for handling MSIs from 8 endpoints"). However lately I
> noticed that during the tests I still had 'pcie_pme=nomsi', so the
> device was not forced to use higher MSI vectors.
>
> After removing this option I noticed that hight MSI vectors are not
> delivered on tested platforms. After additional research I stumbled upon
> a patch in msm-4.14 ([1]), which describes that each group of MSI
> vectors is mapped to the separate interrupt. Implement corresponding
> mapping.
>
> The first patch in the series is a revert of [2] (landed in pci-next).
> Either both patches should be applied or both should be dropped.
>
> Patchseries dependecies: [3] (for the schema change).
>
> Changes since v11 (suggested by Johan):
> - Added back reporting errors for the "msi0" interrupt,
> - Stopped overriding num_vectors field if it is less than the amount of
> MSI vectors deduced from interrupt list,
> - Added a warning (and an override) if the host specifies more MSI
> vectors than available,
> - Moved has_split_msi_irq variable to the patch where it is used.
You forgot to CC me this version. Please remember to keep reviewers on
CC.
Johan
