On Tuesday, February 04, 2014 1:01 AM, Bjørn Erik Nilsen wrote: > On Mon, 2014-02-03 at 10:55 +0100, Jingoo Han wrote: > > On Monday, February 03, 2014 6:29 PM, Jingoo Han wrote: > > > On Thursday, January 30, 2014 1:29 AM, Bjørn Erik Nilsen wrote: > > > > On Fri, 2014-01-24 at 07:48 +0100, Pratyush Anand wrote: > > > > > > > > > > Yes, I browsed pcie sysfs rescan and reset code and it does not go > > > > > into that path. Sorry for the wrong pointers. I should have looked > > > > > into code earlier :( > > > > > > > > > > Coming to the issue, I am still not convinced that msi_init part should > > > > > go to msi_setup_irq. May be something is missing. > > > > > @Mohit, Jingoo: How does remove, reset and rescan behave with your > > > > > platform. > > > > > > > > Any comments? > > > > > > Sorry for late response. > > > I tested 'remove, reset and rescan' on Exynos platform > > > as you did. > > > > > > (remove endpoints) > > > # echo 1 > /sys/bus/pci/devices/0000:01:00.0/remove > > > # echo 1 > /sys/bus/pci/devices/0000:01:00.1/remove > > > > > > (reset bridge) > > > # echo 1 > /sys/bus/pci/devices/0000:00:00.0/reset > > > > > > (rescan bus) > > > # echo 1 > /sys/bus/pci/rescan > > > > > > Without MSI (legacy INTx mode), it works properly. However, > > > with MSI, it makes the problem. > > > > Oh, there was my mistake. There is no problem on Exynos platform. > > I tested two different LAN cards. With MSI, 'remove, reset and > > rescan' works properly on Exynos platform using the current > > pci/next tree. Sorry for the confusion. > > Thank you. > > Right. > > So if I understand you correctly it works fine with pci/next, however > you were able to reproduce with an earlier kernel version? That is > indeed very good news. I will try to pull in the latest and greatest > from pci/next to verify this. Hi Bjørn Erik Nilsen, I tested an earlier kernel v3.13. As you know, this version have Some problems about designware PCIe MSI handling. I tested the latest v3.14-rc1 kernel (vanilla kernel). It works properly. Currently, v3.14-rc1 kernel was merged to the current pci/next kernel. So, you can test 'mainline kernel v3.14-rc1' or 'the latest pci/next kernel'. (the commit 7f2d86 "Merge branch 'pci/numa-cleanup' into next") If you have some problems, please let us know. :-) Thank you. Best regards, Jingoo Han -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
