On 3/29/2017 4:36 PM, Sinan Kaya wrote: > On 3/29/2017 4:16 PM, Patel, Mayurkumar wrote: >> Actually, I have taken same patches of yours which proposed and applied it correctly. Will try it with clean >> build if that helps. >> >> But what I could understand from following patch is https://patchwork.codeaurora.org/patch/207653/ >> >> We still have a flow of pci_aspm_init() -> alloc_pcie_link_state() -> pci_function_0() >> Where I see the first Kernel NULL pointer crash while accessing pdev->subordinate in pci_function_0() function. > > OK. It looks like somebody moved function_0 call around. I was looking at 4.11-rc1. > > 3bd7db63a841e8c5297bb18ad801df67d5e38ad2 (PCI/ASPM: Always set link->downstream to avoid NULL dereference on remove) > > I see the change in 4.11-rc3 now. > > Let me see what's going on. > I posted V6 a minute ago with the bugfix. Let me know how that works for you. -- Sinan Kaya Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm Technologies, Inc. Qualcomm Technologies, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project.
