Re: [RFC Patch V1] ioatdma: Ignore IOAT devices under hotplug-capable PCI host bridge

2015-06-28 Thread Jiang, Dave
> On Jun 28, 2015, at 7:14 PM, Jiang Liu wrote: > > Hi Dave, >Gentle Ping:) Any suggestion about this patch? > Thanks! > Gerry I'm fine with it if Dan has no objections. >> On 2015/6/2 14:36, Jiang Liu wrote: >> The dmaengine core assumes that async DMA devices will only be removed >>

Re: [RFC Patch V1] ioatdma: Ignore IOAT devices under hotplug-capable PCI host bridge

2015-06-28 Thread Jiang Liu
Hi Dave, Gentle Ping:) Any suggestion about this patch? Thanks! Gerry On 2015/6/2 14:36, Jiang Liu wrote: > The dmaengine core assumes that async DMA devices will only be removed > when they not used anymore, or it assumes dma_async_device_unregister() > will only be called by dma driver

Re: [RFC Patch V1] ioatdma: Ignore IOAT devices under hotplug-capable PCI host bridge

2015-06-28 Thread Jiang Liu
Hi Dave, Gentle Ping:) Any suggestion about this patch? Thanks! Gerry On 2015/6/2 14:36, Jiang Liu wrote: The dmaengine core assumes that async DMA devices will only be removed when they not used anymore, or it assumes dma_async_device_unregister() will only be called by dma driver exit

Re: [RFC Patch V1] ioatdma: Ignore IOAT devices under hotplug-capable PCI host bridge

2015-06-28 Thread Jiang, Dave
On Jun 28, 2015, at 7:14 PM, Jiang Liu jiang@linux.intel.com wrote: Hi Dave, Gentle Ping:) Any suggestion about this patch? Thanks! Gerry I'm fine with it if Dan has no objections. On 2015/6/2 14:36, Jiang Liu wrote: The dmaengine core assumes that async DMA devices will only

Re: [RFC Patch V1] ioatdma: Ignore IOAT devices under hotplug-capable PCI host bridge

2015-06-08 Thread Dan Williams
On Mon, Jun 8, 2015 at 8:48 AM, Vinod Koul wrote: > On Mon, Jun 08, 2015 at 07:44:43PM +0800, Jiang Liu wrote: >> On 2015/6/8 18:42, Vinod Koul wrote: >> > On Tue, Jun 02, 2015 at 02:37:31PM +0800, Jiang Liu wrote: >> >> Ccing Rafael, it's ACPI hotplug related. >> >> >> >> On 2015/6/2 14:36,

Re: [RFC Patch V1] ioatdma: Ignore IOAT devices under hotplug-capable PCI host bridge

2015-06-08 Thread Jiang Liu
On 2015/6/8 23:48, Vinod Koul wrote: > On Mon, Jun 08, 2015 at 07:44:43PM +0800, Jiang Liu wrote: >> On 2015/6/8 18:42, Vinod Koul wrote: >>> On Tue, Jun 02, 2015 at 02:37:31PM +0800, Jiang Liu wrote: Ccing Rafael, it's ACPI hotplug related. On 2015/6/2 14:36, Jiang Liu wrote: >

Re: [RFC Patch V1] ioatdma: Ignore IOAT devices under hotplug-capable PCI host bridge

2015-06-08 Thread Vinod Koul
On Mon, Jun 08, 2015 at 07:44:43PM +0800, Jiang Liu wrote: > On 2015/6/8 18:42, Vinod Koul wrote: > > On Tue, Jun 02, 2015 at 02:37:31PM +0800, Jiang Liu wrote: > >> Ccing Rafael, it's ACPI hotplug related. > >> > >> On 2015/6/2 14:36, Jiang Liu wrote: > >>> The dmaengine core assumes that async

Re: [RFC Patch V1] ioatdma: Ignore IOAT devices under hotplug-capable PCI host bridge

2015-06-08 Thread Jiang Liu
On 2015/6/8 18:42, Vinod Koul wrote: > On Tue, Jun 02, 2015 at 02:37:31PM +0800, Jiang Liu wrote: >> Ccing Rafael, it's ACPI hotplug related. >> >> On 2015/6/2 14:36, Jiang Liu wrote: >>> The dmaengine core assumes that async DMA devices will only be removed >>> when they not used anymore, or it

Re: [RFC Patch V1] ioatdma: Ignore IOAT devices under hotplug-capable PCI host bridge

2015-06-08 Thread Vinod Koul
On Tue, Jun 02, 2015 at 02:37:31PM +0800, Jiang Liu wrote: > Ccing Rafael, it's ACPI hotplug related. > > On 2015/6/2 14:36, Jiang Liu wrote: > > The dmaengine core assumes that async DMA devices will only be removed > > when they not used anymore, or it assumes dma_async_device_unregister() > >

Re: [RFC Patch V1] ioatdma: Ignore IOAT devices under hotplug-capable PCI host bridge

2015-06-08 Thread Vinod Koul
On Tue, Jun 02, 2015 at 02:37:31PM +0800, Jiang Liu wrote: Ccing Rafael, it's ACPI hotplug related. On 2015/6/2 14:36, Jiang Liu wrote: The dmaengine core assumes that async DMA devices will only be removed when they not used anymore, or it assumes dma_async_device_unregister() will only

Re: [RFC Patch V1] ioatdma: Ignore IOAT devices under hotplug-capable PCI host bridge

2015-06-08 Thread Jiang Liu
On 2015/6/8 18:42, Vinod Koul wrote: On Tue, Jun 02, 2015 at 02:37:31PM +0800, Jiang Liu wrote: Ccing Rafael, it's ACPI hotplug related. On 2015/6/2 14:36, Jiang Liu wrote: The dmaengine core assumes that async DMA devices will only be removed when they not used anymore, or it assumes

Re: [RFC Patch V1] ioatdma: Ignore IOAT devices under hotplug-capable PCI host bridge

2015-06-08 Thread Vinod Koul
On Mon, Jun 08, 2015 at 07:44:43PM +0800, Jiang Liu wrote: On 2015/6/8 18:42, Vinod Koul wrote: On Tue, Jun 02, 2015 at 02:37:31PM +0800, Jiang Liu wrote: Ccing Rafael, it's ACPI hotplug related. On 2015/6/2 14:36, Jiang Liu wrote: The dmaengine core assumes that async DMA devices will

Re: [RFC Patch V1] ioatdma: Ignore IOAT devices under hotplug-capable PCI host bridge

2015-06-08 Thread Dan Williams
On Mon, Jun 8, 2015 at 8:48 AM, Vinod Koul vinod.k...@intel.com wrote: On Mon, Jun 08, 2015 at 07:44:43PM +0800, Jiang Liu wrote: On 2015/6/8 18:42, Vinod Koul wrote: On Tue, Jun 02, 2015 at 02:37:31PM +0800, Jiang Liu wrote: Ccing Rafael, it's ACPI hotplug related. On 2015/6/2 14:36,

Re: [RFC Patch V1] ioatdma: Ignore IOAT devices under hotplug-capable PCI host bridge

2015-06-08 Thread Jiang Liu
On 2015/6/8 23:48, Vinod Koul wrote: On Mon, Jun 08, 2015 at 07:44:43PM +0800, Jiang Liu wrote: On 2015/6/8 18:42, Vinod Koul wrote: On Tue, Jun 02, 2015 at 02:37:31PM +0800, Jiang Liu wrote: Ccing Rafael, it's ACPI hotplug related. On 2015/6/2 14:36, Jiang Liu wrote: The dmaengine core

Re: [RFC Patch V1] ioatdma: Ignore IOAT devices under hotplug-capable PCI host bridge

2015-06-02 Thread Jiang Liu
Ccing Rafael, it's ACPI hotplug related. On 2015/6/2 14:36, Jiang Liu wrote: > The dmaengine core assumes that async DMA devices will only be removed > when they not used anymore, or it assumes dma_async_device_unregister() > will only be called by dma driver exit routines. But this assumption is

[RFC Patch V1] ioatdma: Ignore IOAT devices under hotplug-capable PCI host bridge

2015-06-02 Thread Jiang Liu
The dmaengine core assumes that async DMA devices will only be removed when they not used anymore, or it assumes dma_async_device_unregister() will only be called by dma driver exit routines. But this assumption is not true for the IOAT driver, which calls dma_async_device_unregister() from

[RFC Patch V1] ioatdma: Ignore IOAT devices under hotplug-capable PCI host bridge

2015-06-02 Thread Jiang Liu
The dmaengine core assumes that async DMA devices will only be removed when they not used anymore, or it assumes dma_async_device_unregister() will only be called by dma driver exit routines. But this assumption is not true for the IOAT driver, which calls dma_async_device_unregister() from

Re: [RFC Patch V1] ioatdma: Ignore IOAT devices under hotplug-capable PCI host bridge

2015-06-02 Thread Jiang Liu
Ccing Rafael, it's ACPI hotplug related. On 2015/6/2 14:36, Jiang Liu wrote: The dmaengine core assumes that async DMA devices will only be removed when they not used anymore, or it assumes dma_async_device_unregister() will only be called by dma driver exit routines. But this assumption is