Re: [PATCH 02/11] powerpc/powernv/ioda: Protect PE list

2019-11-19 Thread Frederic Barrat
Le 19/11/2019 à 14:22, Oliver O'Halloran a écrit : On Tue, Nov 19, 2019 at 11:57 PM Frederic Barrat wrote: Do the other accessors of ioda.pe_list also need mutex protection? pnv_ioda_setup_bus_PE() pnv_pci_dma_bus_setup() pnv_pci_init_ioda_phb() pnv_pci_ioda_setup_PEs() I think we could

Re: [PATCH 02/11] powerpc/powernv/ioda: Protect PE list

2019-11-19 Thread Oliver O'Halloran
On Tue, Nov 19, 2019 at 11:57 PM Frederic Barrat wrote: > > > Do the other accessors of ioda.pe_list also need mutex protection? > > pnv_ioda_setup_bus_PE() > > pnv_pci_dma_bus_setup() > > pnv_pci_init_ioda_phb() > > pnv_pci_ioda_setup_PEs() > > > I think we could also use it there, it wouldn't hu

Re: [PATCH 02/11] powerpc/powernv/ioda: Protect PE list

2019-11-19 Thread Frederic Barrat
Le 10/09/2019 à 02:34, Alastair D'Silva a écrit : On Mon, 2019-09-09 at 17:45 +0200, Frederic Barrat wrote: Protect the PHB's list of PE. Probably not needed as long as it was populated during PHB creation, but it feels right and will become required once we can add/remove opencapi devices on

Re: [PATCH 02/11] powerpc/powernv/ioda: Protect PE list

2019-11-18 Thread Andrew Donnellan
On 10/9/19 1:45 am, Frederic Barrat wrote: Protect the PHB's list of PE. Probably not needed as long as it was populated during PHB creation, but it feels right and will become required once we can add/remove opencapi devices on hotplug. Signed-off-by: Frederic Barrat Reviewed-by: Andrew Donn

Re: [PATCH 02/11] powerpc/powernv/ioda: Protect PE list

2019-09-09 Thread Alastair D'Silva
On Mon, 2019-09-09 at 17:45 +0200, Frederic Barrat wrote: > Protect the PHB's list of PE. Probably not needed as long as it was > populated during PHB creation, but it feels right and will become > required once we can add/remove opencapi devices on hotplug. > > Signed-off-by: Frederic Barrat > -