[PATCH v2 1/2] PCI: Make pci_msi_setup_pci_dev() non-static for use by arch code
Guilherme G. Piccoli
gpiccoli at linux.vnet.ibm.com
Fri Aug 21 05:10:46 AEST 2015
On 08/19/2015 10:02 PM, Michael Ellerman wrote:
> In future you should send a reply like the above to my mail, and then
> separately send the new patch series. My preference is that the new series is
> not a reply to anything, though some other maintainers may disagree on that
OK, sure. I can send new patch series as new messages instead of replies
to the same thread.
> The other question, which I neglected to ask yesterday, is what is the symptom
> of the bug? ie. does the system fail to boot or otherwise crash etc.?
This is briefly explained on cover-letter, but I can elaborate a bit
more: I was testing driver issues on kernel 2.6.32 (RHEL 6.6), and when
I tried the mainline kernel, the driver wasn't able to enable MSI-X
capabilities. Interestingly, on kernel 4.1 this behavior doesn't happen
and the driver can use MSI-X interrupts.
So, I figured that something was wrong and found the problem described
on the patches. I tried the proposed solution (calling manually the
function that is not reachable anymore) and it works.
Regarding the bnx2x driver, below are two dmesg outputs:
1) With kernel 4.2-rc7
bnx2x 0000:01:00.0: no msix capability found
2) With kernel 4.1
bnx2x 0000:01:00.0: msix capability found
bnx2x 0000:01:00.0 eth2: using MSI-X IRQs: sp 24 fp 26 ... fp 33
> This is changes *in* v2, or since v1.
My bad, sorry.
> Or anywhere after the first '---', which means the version commentary is
> discarded in the final commit.
I used scissors, but there's no problem in stop using it in this list.
Thanks for the suggestion.
More information about the Linuxppc-dev