From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965700AbbKEMB5 (ORCPT ); Thu, 5 Nov 2015 07:01:57 -0500 Received: from mga11.intel.com ([192.55.52.93]:8305 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965299AbbKEMBz (ORCPT ); Thu, 5 Nov 2015 07:01:55 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.20,247,1444719600"; d="scan'208";a="594494373" Date: Thu, 5 Nov 2015 17:35:27 +0530 From: Vinod Koul To: Sinan Kaya Cc: Andy Shevchenko , Dan Williams , "dmaengine@vger.kernel.org" , timur@codeaurora.org, cov@codeaurora.org, jcm@redhat.com, "linux-kernel@vger.kernel.org" Subject: Re: [PATCH V2 2/3] dmaselftest: add memcpy selftest support functions Message-ID: <20151105120527.GX12910@localhost> 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> <20151103063009.GT21326@localhost> <20151103160800.GD12910@localhost> <563AC226.30303@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <563AC226.30303@codeaurora.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Nov 04, 2015 at 09:42:46PM -0500, Sinan Kaya wrote: > Here is what I proposed. > > - a common file that gets compiled into a module that wants to use > self-test with a public API. It can be called from driver's probe > routine. > - the test is independent of my implementation. It uses dmaengine > API and should be portable to most drivers. > - there *are* still drivers in the kernel that has selftest code > embedded inside them. I followed the design pattern from other > drivers thinking this must have been a good idea and it paid off for > me. > > As far as I understand, there is interest in doing more than this > and reusing the dmatest code for code duplication. the code that selftest uses to test will be very similar to dmatest code, both of these _should_ share this common code so that fixes get done for both! > Facts: > - Dmatest can be actually configured to run during boot. > - Nobody besides the dma driver developer uses dmatest. This leaves > holes for regressions that are really hard to debug due to > interaction with the rest of the system. > - Dmatest doesn't exist in most distribution kernels. That doesn't mean it is not useful. This line of thought is not quite right. You are trying to say dmatest in not important and selftest is. Sorry but you are wrong, both are equally important and since both try to test and use similar routines (dmaengien API) they need to share the code and not duplicate it > If we want to do something else, I need clear directions. I can > remove the self test code completely from my driver. But, I need an > equivalent functionality. Add selftest to dmatest, we need both! > > > > > That part is tricky, you need to do so thru clients, spi/audio/serial etc > > > > My selftest code actually attaches to all slave devices and issues a > memcpy command and then detaches from the slave devices. Not everyone supports memcpy! -- ~Vinod