Re: [PATCH 2/2] ioatdma: add DMA_PRIVATE capabilities flag

2013-06-26 Thread Dan Williams
On 6/26/13 4:55 PM, "Jon Mason" wrote: >Unfortunately, the dma_find_channel model does not led itself to >optimal usage of the available channels, as it seems to give out the >same channel. Adding some randomizer (or other way to spread the >channel selection over multiple channels) would be gr

Re: [PATCH 2/2] ioatdma: add DMA_PRIVATE capabilities flag

2013-06-26 Thread Jon Mason
On Wed, Jun 19, 2013 at 01:44:08PM -0700, Dan Williams wrote: > On Wed, Jun 19, 2013 at 1:10 PM, Jon Mason wrote: > > On Wed, Jun 19, 2013 at 11:56:38AM -0700, Dan Williams wrote: > >> On Wed, Jun 19, 2013 at 10:52 AM, Jon Mason wrote: > >> >> DMA_PRIVATE here keeps all channels private, so they

Re: [PATCH 2/2] ioatdma: add DMA_PRIVATE capabilities flag

2013-06-19 Thread Dan Williams
On Wed, Jun 19, 2013 at 1:10 PM, Jon Mason wrote: > On Wed, Jun 19, 2013 at 11:56:38AM -0700, Dan Williams wrote: >> On Wed, Jun 19, 2013 at 10:52 AM, Jon Mason wrote: >> >> DMA_PRIVATE here keeps all channels private, so they couldn't be used >> >> elsewhere, for example raid offload. Do you ne

Re: [PATCH 2/2] ioatdma: add DMA_PRIVATE capabilities flag

2013-06-19 Thread Jon Mason
On Wed, Jun 19, 2013 at 11:56:38AM -0700, Dan Williams wrote: > On Wed, Jun 19, 2013 at 10:52 AM, Jon Mason wrote: > >> DMA_PRIVATE here keeps all channels private, so they couldn't be used > >> elsewhere, for example raid offload. Do you need a private allocation > >> or can you get away with a

Re: [PATCH 2/2] ioatdma: add DMA_PRIVATE capabilities flag

2013-06-19 Thread Dan Williams
On Wed, Jun 19, 2013 at 10:52 AM, Jon Mason wrote: >> DMA_PRIVATE here keeps all channels private, so they couldn't be used >> elsewhere, for example raid offload. Do you need a private allocation >> or can you get away with a dynamically assigned channel? > > I would like to have a dedicated DMA

Re: [PATCH 2/2] ioatdma: add DMA_PRIVATE capabilities flag

2013-06-19 Thread Jon Mason
On Tue, Jun 18, 2013 at 05:59:59PM -0700, Dan Williams wrote: > On Tue, Jun 18, 2013 at 5:46 PM, Jon Mason wrote: > > Set the DMA_PRIVATE dma_transaction_type in the capability mask. This > > enables the ability to get an exclusive ioatdma DMA channel for any > > devices that requests one via the

Re: [PATCH 2/2] ioatdma: add DMA_PRIVATE capabilities flag

2013-06-19 Thread Dave Jiang
On 06/18/2013 05:59 PM, Dan Williams wrote: On Tue, Jun 18, 2013 at 5:46 PM, Jon Mason wrote: Set the DMA_PRIVATE dma_transaction_type in the capability mask. This enables the ability to get an exclusive ioatdma DMA channel for any devices that requests one via the dma_request_channel function

Re: [PATCH 2/2] ioatdma: add DMA_PRIVATE capabilities flag

2013-06-18 Thread Dan Williams
On Tue, Jun 18, 2013 at 5:46 PM, Jon Mason wrote: > Set the DMA_PRIVATE dma_transaction_type in the capability mask. This > enables the ability to get an exclusive ioatdma DMA channel for any > devices that requests one via the dma_request_channel function call. > > Signed-off-by: Jon Mason > Ac