Re: [PATCH 2/9] I/OAT

[Evgeniy Polyakov]

On Wed, Mar 29, 2006 at 02:54:04PM -0800, Chris Leech (christopher.leech@xxxxxxxxx) wrote:
> [I/OAT] Driver for the Intel(R) I/OAT DMA engine
> 
> From: Chris Leech <christopher.leech@xxxxxxxxx>
> 
> Adds a new ioatdma driver
> 
> Signed-off-by: Chris Leech <christopher.leech@xxxxxxxxx>

Let's do it again.
Could you please describe how struct ioat_dma_chan channels are freed?
For example when device is removed just after it has been added.

ioat_probe() -> enumerate_dma_channels() (failures are ok now) ->
kmalloc a lot of channels.

ioat_remove() -> dma_async_device_unregister() which does not cleanup
ioat_dma_chan channels, but only clients.
It ends up in dma_async_device_cleanup() only.

-- 
	Evgeniy Polyakov
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/