| Message ID | 20241211080105.11104-1-manivannan.sadhasivam@linaro.org |
|---|---|
| Headers | show |
| Series | Migrate PCI Endpoint Subsystem tests to Kselftest | expand |
Hello Mani, On Wed, Dec 11, 2024 at 01:31:01PM +0530, Manivannan Sadhasivam wrote: > Hi, > > This series carries forward the effort to add Kselftest for PCI Endpoint > Subsystem started by Aman Gupta [1] a while ago. I reworked the initial version > based on another patch that fixes the return values of IOCTLs in > pci_endpoint_test driver and did many cleanups. Since the resulting work > modified the initial version substantially, I took over the authorship. > > This series also incorporates the review comment by Shuah Khan [2] to move the > existing tests from 'tools/pci' to 'tools/testing/kselftest/pci_endpoint' before > migrating to Kselftest framework. I made sure that the tests are executable in > each commit and updated documentation accordingly. > > NOTE: Patch 1 is strictly not related to this series, but necessary to execute > Kselftests with Qualcomm Endpoint devices. So this can be merged separately. Having to write a big NOTE is usually a hint that you should just have done things differently :) If you need to respin this series, I strongly suggest that you send the Qcom fix separately. It is totally independent, and should be merged ASAP. As you know, this series conflicts with: https://lore.kernel.org/linux-pci/20241116032045.2574168-2-cassel@kernel.org/ I don't see any reason why the above patch has not been merged yet, but it would be really nice if the above could be picked up first, so this series could also add a kselftest testcase for the above. Kind regards, Niklas
On Mon, Dec 16, 2024 at 11:33:37AM +0530, Manivannan Sadhasivam wrote: > On Thu, Dec 12, 2024 at 10:25:53AM +0100, Niklas Cassel wrote: > > > > If you need to respin this series, I strongly suggest that you send the > > Qcom fix separately. It is totally independent, and should be merged ASAP. > > > > Even though it is an independent fix, it is needed to get Kselftests (also the > legacy ones) passing without failures. That's why I kept it as patch 1. > Otherwise, someone may test it and report failures. If qcom needs this fix, then surely pcitest.sh is already failing for the BAR test for BAR 1 and BAR 3, for all qcom SoCs, and must have been doing so since the introduction of the introduction of the qcom-pcie-ep driver. That should be fixed of course, but I do not see why converting the tests to kselftests is related in any way. It seems cleaner if this series just converts the tests cases to kselftests, and nothing else. EPC drivers that was passing before should still pass after this conversion. I understand that the qcom fix is important though, which is even bigger reason that it should be sent separately, so that it can go in ASAP, and not be blocked on this series landing. Because, AFAICT, you need that qcom fix to make the pcitest.sh test cases (even without converting to kselftests). Kind regards, Niklas
Hello, > This series carries forward the effort to add Kselftest for PCI Endpoint > Subsystem started by Aman Gupta [1] a while ago. I reworked the initial version > based on another patch that fixes the return values of IOCTLs in > pci_endpoint_test driver and did many cleanups. Since the resulting work > modified the initial version substantially, I took over the authorship. > > This series also incorporates the review comment by Shuah Khan [2] to move the > existing tests from 'tools/pci' to 'tools/testing/kselftest/pci_endpoint' before > migrating to Kselftest framework. I made sure that the tests are executable in > each commit and updated documentation accordingly. > > NOTE: Patch 1 is strictly not related to this series, but necessary to execute > Kselftests with Qualcomm Endpoint devices. So this can be merged separately. Applied to selftests, thank you! [01/04] PCI: qcom-ep: Mark BAR0/BAR2 as 64bit BARs and BAR1/BAR3 as RESERVED https://git.kernel.org/pci/pci/c/71ae1c3a342c [02/04] misc: pci_endpoint_test: Fix the return value of IOCTL https://git.kernel.org/pci/pci/c/7908208a2f6a [03/04] selftests: Move PCI Endpoint tests from tools/pci to Kselftests https://git.kernel.org/pci/pci/c/5c892b60e4c6 [04/04] selftests: pci_endpoint: Migrate to Kselftest framework https://git.kernel.org/pci/pci/c/62f966e676b5 Krzysztof
On Thu, Dec 19, 2024 at 09:01:12AM +0900, Krzysztof Wilczyński wrote: > Hello, > > > This series carries forward the effort to add Kselftest for PCI Endpoint > > Subsystem started by Aman Gupta [1] a while ago. I reworked the initial version > > based on another patch that fixes the return values of IOCTLs in > > pci_endpoint_test driver and did many cleanups. Since the resulting work > > modified the initial version substantially, I took over the authorship. > > > > This series also incorporates the review comment by Shuah Khan [2] to move the > > existing tests from 'tools/pci' to 'tools/testing/kselftest/pci_endpoint' before > > migrating to Kselftest framework. I made sure that the tests are executable in > > each commit and updated documentation accordingly. > > > > NOTE: Patch 1 is strictly not related to this series, but necessary to execute > > Kselftests with Qualcomm Endpoint devices. So this can be merged separately. > > Applied to selftests, thank you! > > [01/04] PCI: qcom-ep: Mark BAR0/BAR2 as 64bit BARs and BAR1/BAR3 as RESERVED > https://git.kernel.org/pci/pci/c/71ae1c3a342c > > [02/04] misc: pci_endpoint_test: Fix the return value of IOCTL > https://git.kernel.org/pci/pci/c/7908208a2f6a > > [03/04] selftests: Move PCI Endpoint tests from tools/pci to Kselftests > https://git.kernel.org/pci/pci/c/5c892b60e4c6 > > [04/04] selftests: pci_endpoint: Migrate to Kselftest framework > https://git.kernel.org/pci/pci/c/62f966e676b5 > > Krzysztof I'm a bit surprised that this series was picked up, since as you could see earlier in this same thread: https://lore.kernel.org/linux-pci/20241219000112.GE1444967@rocinante/T/#m7bb0e624a4bf88f5cc13dc3804972c4fa9a79bcd Mani suggested that my patch (which conflicts with this), should be picked up first. Is there a reason for the sudden chance of plans? Please advice on how to proceed. Kind regards, Niklas
Hello, [...] > > Applied to selftests, thank you! > > > > [01/04] PCI: qcom-ep: Mark BAR0/BAR2 as 64bit BARs and BAR1/BAR3 as RESERVED > > https://git.kernel.org/pci/pci/c/71ae1c3a342c > > > > [02/04] misc: pci_endpoint_test: Fix the return value of IOCTL > > https://git.kernel.org/pci/pci/c/7908208a2f6a > > > > [03/04] selftests: Move PCI Endpoint tests from tools/pci to Kselftests > > https://git.kernel.org/pci/pci/c/5c892b60e4c6 > > > > [04/04] selftests: pci_endpoint: Migrate to Kselftest framework > > https://git.kernel.org/pci/pci/c/62f966e676b5 > > > > Krzysztof > > I'm a bit surprised that this series was picked up, > since as you could see earlier in this same thread: > https://lore.kernel.org/linux-pci/20241219000112.GE1444967@rocinante/T/#m7bb0e624a4bf88f5cc13dc3804972c4fa9a79bcd > > Mani suggested that my patch (which conflicts with this), > should be picked up first. > > Is there a reason for the sudden chance of plans? No, no change to the plan here. There were some mixed signals between the mailing list, IRC and the Patchwork queue. But I will proceed as planned there. > Please advice on how to proceed. I will pick your patch and drop Mani's series. Mani told me on IRC that he plans to work on it a bit more. Krzysztof
On Fri, Dec 20, 2024 at 12:55:04AM +0900, Krzysztof Wilczyński wrote: [...] > > Mani suggested that my patch (which conflicts with this), > > should be picked up first. > > > > Is there a reason for the sudden chance of plans? > > No, no change to the plan here. > > There were some mixed signals between the mailing list, IRC and > the Patchwork queue. But I will proceed as planned there. > > > Please advice on how to proceed. > > I will pick your patch and drop Mani's series. Mani told me on IRC that he > plans to work on it a bit more. Ok, thanks! I really hope that the kselftest conversion will have time to land this cycle as well. Feel free to CC me on the next revision and I will make sure to send a Tested-by and/or Reviewed-by. Kind regards, Niklas