Re: [PATCH v5 0/2] PCI: dwc: fix two MSI issues
On Thu, 8 Oct 2020 12:36:14 +0100 Lorenzo Pieralisi wrote: > > On Wed, Sep 30, 2020 at 09:12:05AM +0800, Jisheng Zhang wrote: > > Fix two MSI issues. One to skip PCIE_MSI_INTR0* programming if MSI is > > disabled, another to use an address in the driver data for MSI address, > > to fix the MSI page leakage during suspend/resume. > > > > Since v4: > > - fix pci-dra7xx.c > > > > Since v3: > > - add Acked-by tag > > - change patch2 commit msg to make it clear > > - map the MSI msg with dma_map_single_attrs() for some platforms > > which either has seperate addrs for dma and phy or has mem access > > limitation for the PCIe. > > > > Since v2: > > - add Acked-by tag > > - use an address in the driver data for MSI address. Thank Ard and Rob > > for pointing out this correct direction. > > - Since the MSI page has gone, the leak issue doesn't exist anymore, > > remove unnecessary patches. > > - Remove dw_pcie_free_msi rename and the last patch. They could be > > targeted to next. So will send out patches in a separate series. > > > > Since v1: > > - add proper error handling patches. > > - solve the msi page leakage by moving dw_pcie_msi_init() from each > > users to designware host > > > > > > Jisheng Zhang (2): > > PCI: dwc: Skip PCIE_MSI_INTR0* programming if MSI is disabled > > PCI: dwc: Fix MSI page leakage in suspend/resume > > > > drivers/pci/controller/dwc/pci-dra7xx.c | 18 +- > > .../pci/controller/dwc/pcie-designware-host.c | 34 +-- > > drivers/pci/controller/dwc/pcie-designware.h | 2 +- > > 3 files changed, 35 insertions(+), 19 deletions(-) > > Hi, > > can you rebase this series against my pci/dwc branch please ? > Done in v6. Thanks
Re: [PATCH v5 0/2] PCI: dwc: fix two MSI issues
On Wed, Sep 30, 2020 at 09:12:05AM +0800, Jisheng Zhang wrote: > Fix two MSI issues. One to skip PCIE_MSI_INTR0* programming if MSI is > disabled, another to use an address in the driver data for MSI address, > to fix the MSI page leakage during suspend/resume. > > Since v4: > - fix pci-dra7xx.c > > Since v3: > - add Acked-by tag > - change patch2 commit msg to make it clear > - map the MSI msg with dma_map_single_attrs() for some platforms > which either has seperate addrs for dma and phy or has mem access > limitation for the PCIe. > > Since v2: > - add Acked-by tag > - use an address in the driver data for MSI address. Thank Ard and Rob > for pointing out this correct direction. > - Since the MSI page has gone, the leak issue doesn't exist anymore, > remove unnecessary patches. > - Remove dw_pcie_free_msi rename and the last patch. They could be > targeted to next. So will send out patches in a separate series. > > Since v1: > - add proper error handling patches. > - solve the msi page leakage by moving dw_pcie_msi_init() from each > users to designware host > > > Jisheng Zhang (2): > PCI: dwc: Skip PCIE_MSI_INTR0* programming if MSI is disabled > PCI: dwc: Fix MSI page leakage in suspend/resume > > drivers/pci/controller/dwc/pci-dra7xx.c | 18 +- > .../pci/controller/dwc/pcie-designware-host.c | 34 +-- > drivers/pci/controller/dwc/pcie-designware.h | 2 +- > 3 files changed, 35 insertions(+), 19 deletions(-) Hi, can you rebase this series against my pci/dwc branch please ? Thanks, Lorenzo
[PATCH v5 0/2] PCI: dwc: fix two MSI issues
Fix two MSI issues. One to skip PCIE_MSI_INTR0* programming if MSI is disabled, another to use an address in the driver data for MSI address, to fix the MSI page leakage during suspend/resume. Since v4: - fix pci-dra7xx.c Since v3: - add Acked-by tag - change patch2 commit msg to make it clear - map the MSI msg with dma_map_single_attrs() for some platforms which either has seperate addrs for dma and phy or has mem access limitation for the PCIe. Since v2: - add Acked-by tag - use an address in the driver data for MSI address. Thank Ard and Rob for pointing out this correct direction. - Since the MSI page has gone, the leak issue doesn't exist anymore, remove unnecessary patches. - Remove dw_pcie_free_msi rename and the last patch. They could be targeted to next. So will send out patches in a separate series. Since v1: - add proper error handling patches. - solve the msi page leakage by moving dw_pcie_msi_init() from each users to designware host Jisheng Zhang (2): PCI: dwc: Skip PCIE_MSI_INTR0* programming if MSI is disabled PCI: dwc: Fix MSI page leakage in suspend/resume drivers/pci/controller/dwc/pci-dra7xx.c | 18 +- .../pci/controller/dwc/pcie-designware-host.c | 34 +-- drivers/pci/controller/dwc/pcie-designware.h | 2 +- 3 files changed, 35 insertions(+), 19 deletions(-) -- 2.28.0