Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754048Ab1DTHle (ORCPT ); Wed, 20 Apr 2011 03:41:34 -0400 Received: from mail-qw0-f46.google.com ([209.85.216.46]:35902 "EHLO mail-qw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753652Ab1DTHld convert rfc822-to-8bit (ORCPT ); Wed, 20 Apr 2011 03:41:33 -0400 MIME-Version: 1.0 In-Reply-To: <20110417154622.GC17935@S2100-06.ap.freescale.net> References: <1302116833-24540-1-git-send-email-per.forlin@linaro.org> <1302116833-24540-4-git-send-email-per.forlin@linaro.org> <20110417154622.GC17935@S2100-06.ap.freescale.net> Date: Wed, 20 Apr 2011 09:41:32 +0200 Message-ID: Subject: Re: [PATCH v2 03/12] mmc: mmc_test: add test for none blocking transfers From: Per Forlin To: Shawn Guo Cc: linux-mmc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linaro-dev@lists.linaro.org, Chris Ball Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1460 Lines: 38 On 17 April 2011 17:46, Shawn Guo wrote: > On Wed, Apr 06, 2011 at 09:07:04PM +0200, Per Forlin wrote: > [...] >> +static int mmc_test_rw_multiple(struct mmc_test_card *test, >> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? struct mmc_test_multiple_rw *tdata, >> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? unsigned int reqsize, unsigned int size) >> +{ >> + ? ? unsigned int dev_addr; >> + ? ? struct mmc_test_area *t = &test->area; >> + ? ? int ret = 0; >> + ? ? int max_reqsize = max(t->mem->size_min_cmn * >> + ? ? ? ? ? ? ? ? ? ? ? ? ? min(t->max_segs, t->mem->cnt), t->max_tfr); >> + > The 'max(..., t->max_tfr)' probably should be 'min(..., t->max_tfr)'. > Otherwise, I see mmc_test failure on my mxs-mmc setup. > > mmc0: Test case 37. Write performance with blocking req 4k to 4MB... > mmc0: Transfer of 64 x 2048 sectors (64 x 1024 KiB) took 5.412563314 seconds (12 > 398 kB/s, 12108 KiB/s, 11.82 IOPS) > mmc0: Failed to map sg list > [mmc_test_rw_multiple] error > mmc0: Result: ERROR (-22) Thanks for letting me know. I think I should simplify it and use t->max_tfr only. I tried to optimize for one of my case when max_tfr is only 1MiB but mmc request size is 32MiB. > > -- > Regards, > Shawn Regards, Per -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/