-----Original Message----- From: Christoph Hellwig [mailto:hch@lst.de] Sent: Sunday, November 15, 2020 5:54 AM To: Song Bao Hua (Barry Song) song.bao.hua@hisilicon.com Cc: iommu@lists.linux-foundation.org; hch@lst.de; robin.murphy@arm.com; m.szyprowski@samsung.com; Linuxarm linuxarm@huawei.com; linux-kselftest@vger.kernel.org; xuwei (O) xuwei5@huawei.com; Joerg Roedel joro@8bytes.org; Will Deacon will@kernel.org; Shuah Khan shuah@kernel.org Subject: Re: [PATCH v3 1/2] dma-mapping: add benchmark support for streaming DMA APIs
Lots of > 80 char lines. Please fix up the style.
Checkpatch has changed 80 to 100. That's probably why my local checkpatch didn't report any warning: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?i...
I am happy to change them to be less than 80 if you like.
I think this needs to set a dma mask as behavior for unlimited dma mask vs the default 32-bit one can be very different.
I actually prefer users bind real devices with real dma_mask to test rather than force to change the dma_mask in this benchmark.
Some device might have 32bit dma_mask while some others might have unlimited. But both of them can bind to this driver or unbind from it after the test is done. So users just need to bind those different real devices with different real dma_mask to dma_benchmark.
This can reflect the real performance of the real device better, I think.
I also think you need to be able to pass the direction or have different tests for directions. bidirectional is not exactly heavily used and pays more cache management penality.
For this, I'd like to increase a direction option in the test app and pass the option to the benchmark driver.
Thanks Barry