linux-kernel - RE: [EXT] Re: [PATCHv5 1/6] PCI: mobiveil: Refactor Mobiveil PCIe Host Bridge IP driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 30 Apr 2019 00:23:47 +0000
From:   "Z.q. Hou" <zhiqiang.hou@....com>
To:     Arnd Bergmann <arnd@...db.de>
CC:     "linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "bhelgaas@...gle.com" <bhelgaas@...gle.com>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "mark.rutland@....com" <mark.rutland@....com>,
        "l.subrahmanya@...iveil.co.in" <l.subrahmanya@...iveil.co.in>,
        "shawnguo@...nel.org" <shawnguo@...nel.org>,
        Leo Li <leoyang.li@....com>,
        "lorenzo.pieralisi@....com" <lorenzo.pieralisi@....com>,
        "catalin.marinas@....com" <catalin.marinas@....com>,
        "will.deacon@....com" <will.deacon@....com>,
        "M.h. Lian" <minghuan.lian@....com>,
        Xiaowei Bao <xiaowei.bao@....com>,
        Mingkai Hu <mingkai.hu@....com>
Subject: RE: [EXT] Re: [PATCHv5 1/6] PCI: mobiveil: Refactor Mobiveil PCIe
 Host Bridge IP driver

Hi Arnd,

Thanks a lot for your comments!

> -----Original Message-----
> From: Arnd Bergmann [mailto:arnd@...db.de]
> Sent: 2019年4月12日 22:39
> To: Z.q. Hou <zhiqiang.hou@....com>
> Cc: linux-pci@...r.kernel.org; linux-arm-kernel@...ts.infradead.org;
> devicetree@...r.kernel.org; linux-kernel@...r.kernel.org;
> bhelgaas@...gle.com; robh+dt@...nel.org; mark.rutland@....com;
> l.subrahmanya@...iveil.co.in; shawnguo@...nel.org; Leo Li
> <leoyang.li@....com>; lorenzo.pieralisi@....com;
> catalin.marinas@....com; will.deacon@....com; M.h. Lian
> <minghuan.lian@....com>; Xiaowei Bao <xiaowei.bao@....com>; Mingkai
> Hu <mingkai.hu@....com>
> Subject: [EXT] Re: [PATCHv5 1/6] PCI: mobiveil: Refactor Mobiveil PCIe Host
> Bridge IP driver
> 
> WARNING: This email was created outside of NXP. DO NOT CLICK links or
> attachments unless you recognize the sender and know the content is safe.
> 
> 
> 
> On Fri, Apr 12, 2019 at 11:53 AM Z.q. Hou <zhiqiang.hou@....com> wrote:
> 
> > +int mobiveil_pcie_host_probe(struct mobiveil_pcie *pcie)
> >  {
> > -       struct mobiveil_pcie *pcie;
> >         struct pci_bus *bus;
> >         struct pci_bus *child;
> >         struct pci_host_bridge *bridge;
> > -       struct device *dev = &pdev->dev;
> > +       struct device *dev = &pcie->pdev->dev;
> >         resource_size_t iobase;
> >         int ret;
> >
> > -       /* allocate the PCIe port */
> > -       bridge = devm_pci_alloc_host_bridge(dev, sizeof(*pcie));
> > -       if (!bridge)
> > -               return -ENOMEM;
> > -
> > -       pcie = pci_host_bridge_priv(bridge);
> > -
> > -       pcie->pdev = pdev;
> > +       INIT_LIST_HEAD(&pcie->resources);
> >
> >         ret = mobiveil_pcie_parse_dt(pcie);
> >         if (ret) {
> > @@ -928,7 +560,10 @@ static int mobiveil_pcie_probe(struct
> platform_device *pdev)
> >                 return ret;
> >         }
> >
> > -       INIT_LIST_HEAD(&pcie->resources);
> > +       /* allocate the PCIe port */
> > +       bridge = devm_pci_alloc_host_bridge(dev, 0);
> > +       if (!bridge)
> > +               return -ENOMEM;
> >
> 
> > +static int mobiveil_pcie_probe(struct platform_device *pdev) {
> > +       struct mobiveil_pcie *pcie;
> > +       struct device *dev = &pdev->dev;
> > +
> > +       pcie = devm_kzalloc(dev, sizeof(*pcie), GFP_KERNEL);
> > +       if (!pcie)
> > +               return -ENOMEM;
> > +
> > +       pcie->pdev = pdev;
> > +
> > +       return mobiveil_pcie_host_probe(pcie); }
> 
> I think you need to pull the allocation of the host structure out into the main
> driver here, to keep both allocations together, otherwise the release function
> will free the mobiveil_pcie structure before freeing the pci_host_bridge if that
> is still in use.

Yes, that will be arranged in v6.

> 
> > +
> > +struct mobiveil_pcie {
> > +       struct platform_device *pdev;
> > +       struct list_head resources;
> 
> These two should not be needed here, as they are already part of the
> pci_host_bridge structure.

The resources list structure has been changed to a pointer to the bridge->windows.
It is better to keep these 2 pointers in the private structure to make it easy to use them
in the PCIe controller driver.

> 
>      Arnd

Regards,
Zhiqiang