From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755285AbbKCQ5r (ORCPT ); Tue, 3 Nov 2015 11:57:47 -0500 Received: from smtp.codeaurora.org ([198.145.29.96]:59534 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751289AbbKCQ5o (ORCPT ); Tue, 3 Nov 2015 11:57:44 -0500 Subject: Re: [PATCH V2 2/3] dmaselftest: add memcpy selftest support functions To: Timur Tabi , Vinod Koul References: <1446444460-21600-1-git-send-email-okaya@codeaurora.org> <1446444460-21600-3-git-send-email-okaya@codeaurora.org> <20151103041528.GS21326@localhost> <5638359D.9080507@codeaurora.org> <5638C55D.50001@codeaurora.org> <20151103161013.GE12910@localhost> <5638E0A5.4060401@codeaurora.org> <5638E4EA.1040905@codeaurora.org> Cc: dmaengine@vger.kernel.org, cov@codeaurora.org, jcm@redhat.com, Dan Williams , linux-kernel@vger.kernel.org From: Sinan Kaya Message-ID: <5638E785.2070009@codeaurora.org> Date: Tue, 3 Nov 2015 11:57:41 -0500 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <5638E4EA.1040905@codeaurora.org> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 11/3/2015 11:46 AM, Timur Tabi wrote: > Sinan Kaya wrote: >> 1. Bug in ARM64 DMA subsystem. >> 2. Bug in IOMMU driver. >> 3. Bug in another newly introduced driver. The new driver would hog the >> CPU and won't allow HIDMA interrupts to execute. Therefore, the test >> times out. > > Which driver? Some other driver that is not upstream yet. > > Wouldn't these problems already be exposed by dmatest? I was asking > whether it's possible that, every now and then, your DMA internal test > could fail and then the driver would unload. I'm not talking about hard > bugs in other code that always causes the DMA driver test to fail. > The point is that dmatest is a kernel module that requires manual interaction. It does not run automatically and is generally used by the dma engine driver developer during development only. Other developers like UART, SATA, USB, IOMMU etc. They don't care about dmatest and I have seen their changes broke self-test multiple times. I see the value of self test all the time. I can make dma-self test a new kernel module. It could discover all DMA devices with MEMCPY capability and run a self test on them. We could tell the self test code deregister all DMA devices that fail the test. I can also make it kernel command line dependent and disabled by default. Those who want this functionality all the time can change their kernel command line. I hope this addresses concerns. -- Sinan Kaya Qualcomm Technologies, Inc. on behalf of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project