Messages in this thread | | | Date | Tue, 3 Nov 2015 12:00:09 +0530 | From | Vinod Koul <> | Subject | Re: [PATCH V2 2/3] dmaselftest: add memcpy selftest support functions |
| |
On Mon, Nov 02, 2015 at 11:18:37PM -0500, Sinan Kaya wrote: > > > On 11/2/2015 11:15 PM, Vinod Koul wrote: > >On Mon, Nov 02, 2015 at 01:07:38AM -0500, Sinan Kaya wrote: > >>This patch adds supporting utility functions > >>for selftest. The intention is to share the self > >>test code between different drivers. > >> > >>Supported test cases include: > >>1. dma_map_single > >>2. streaming DMA > >>3. coherent DMA > >>4. scatter-gather DMA > > > >This seems quite similar to dmatest, any reason why you cannot use/enhance > >that? > > > Dmatest is a standalone kernel module intended for stress testing > DMA engines from userspace with N number of threads and M size > combinations etc. > > This one; on the other hand, is selftest to verify hardware is > working as expected during power up. > > Almost all DMA engine drivers come with some sort of selftest code > called from probe. I followed the same design pattern.
which ones ?
> > I think the goal is to remove the duplicate self test code in all > drivers over time.
and what prevents us from having common selftest plus dmatest code. Most of the code here to do selftest is _same_ dmaengine routine code used in dmatest
We can have common code which is used for dmatest as well as selftest. I do not want to see same code duplicated..
-- ~Vinod
| |