From patchwork Mon Oct 19 06:11:46 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Akinobu Mita X-Patchwork-Id: 36358 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from bombadil.infradead.org (bombadil.infradead.org [18.85.46.34]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 6E10FB7B9E for ; Mon, 19 Oct 2009 17:16:21 +1100 (EST) Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.69 #1 (Red Hat Linux)) id 1MzlTO-0000EE-Ug; Mon, 19 Oct 2009 06:12:02 +0000 Received: from mail-gx0-f219.google.com ([209.85.217.219]) by bombadil.infradead.org with esmtp (Exim 4.69 #1 (Red Hat Linux)) id 1MzlTI-0000CC-GH for linux-mtd@lists.infradead.org; Mon, 19 Oct 2009 06:12:01 +0000 Received: by gxk19 with SMTP id 19so487222gxk.5 for ; Sun, 18 Oct 2009 23:11:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:from:to:cc:subject:date :message-id:x-mailer:in-reply-to:references; bh=rBBbLAgI9UIOw2R763BtfVuahuHkDX2+jaZM6qGCDDk=; b=s8RweuRRVCPhFwh1Z68QKF4ZcFKcspGO01AbUC2co4nYjNbAI9f866LUogYYiC0mpm e2pKCk72s7FCPyIqv7yY1UVrNNpn9J+kGVRDVXe3MqeQeSLu8ANP3WeAXlA0UTK4DeCf QMzxJ921wV7kfK9BZ6gl19PqaZuxG51mtZY/Q= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:cc:subject:date:message-id:x-mailer:in-reply-to:references; b=i/ILtHyrAGTEQUkBPOlscBo4WdQDKezN2AMrPGjikgH2pRnHPQQowh/ydjhLG+neT1 d2T3eCVEy9a7N64wTNnkNtdnUKbusduXRCIZJ6O3gXhy2HLVR2MqWiRvq+BczJxvMI4f 2EP0P3AtZM2ho99nEIxYdUMCvFArvWDwaYa+s= Received: by 10.150.233.3 with SMTP id f3mr7330954ybh.148.1255932715781; Sun, 18 Oct 2009 23:11:55 -0700 (PDT) Received: from localhost ([220.110.185.192]) by mx.google.com with ESMTPS id 20sm2589831ywh.2.2009.10.18.23.11.54 (version=TLSv1/SSLv3 cipher=RC4-MD5); Sun, 18 Oct 2009 23:11:54 -0700 (PDT) From: Akinobu Mita To: linux-mtd@lists.infradead.org Subject: [PATCH 2/2] MTD: Add nand_ecc test module Date: Mon, 19 Oct 2009 15:11:46 +0900 Message-Id: <1255932706-23824-2-git-send-email-akinobu.mita@gmail.com> X-Mailer: git-send-email 1.5.4.3 In-Reply-To: <1255932706-23824-1-git-send-email-akinobu.mita@gmail.com> References: <> <1255932706-23824-1-git-send-email-akinobu.mita@gmail.com> X-CRM114-Version: 20090807-BlameThorstenAndJenny ( TRE 0.7.6 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20091019_021156_671664_CDD5405E X-CRM114-Status: GOOD ( 15.98 ) X-Spam-Score: 0.0 (/) X-Spam-Report: SpamAssassin version 3.2.5 on bombadil.infradead.org summary: Content analysis details: (0.0 points) pts rule name description ---- ---------------------- -------------------------------------------------- _SUMMARY_ Cc: Artem Bityutskiy , David Woodhouse , Akinobu Mita X-BeenThere: linux-mtd@lists.infradead.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Sender: linux-mtd-bounces@lists.infradead.org Errors-To: linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org This module tests NAND ECC functions. The test is simple. 1. Create a 256 or 512 bytes block of data filled with random bytes (data) 2. Duplicate the data block and inject single bit error (error_data) 3. Try to correct error_data 4. Compare data and error_data This test is added into new MTD test group called MTD NAND test which requires MTD NAND device support. Signed-off-by: Akinobu Mita Cc: David Woodhouse Cc: linux-mtd@lists.infradead.org Cc: Artem Bityutskiy --- drivers/mtd/Kconfig | 6 +++ drivers/mtd/tests/Makefile | 1 + drivers/mtd/tests/nand_ecc-test.c | 88 +++++++++++++++++++++++++++++++++++++ 3 files changed, 95 insertions(+), 0 deletions(-) create mode 100644 drivers/mtd/tests/nand_ecc-test.c diff --git a/drivers/mtd/Kconfig b/drivers/mtd/Kconfig index ecf90f5..abe5c7f 100644 --- a/drivers/mtd/Kconfig +++ b/drivers/mtd/Kconfig @@ -33,6 +33,12 @@ config MTD_TESTS should normally be compiled as kernel modules. The modules perform various checks and verifications when loaded. +config MTD_NAND_TESTS + tristate "MTD NAND tests support" + depends on MTD_TESTS && MTD_NAND + help + This option enables MTD tests which require NAND Device support. + config MTD_CONCAT tristate "MTD concatenating support" help diff --git a/drivers/mtd/tests/Makefile b/drivers/mtd/tests/Makefile index c1d5013..3994d5a 100644 --- a/drivers/mtd/tests/Makefile +++ b/drivers/mtd/tests/Makefile @@ -5,3 +5,4 @@ obj-$(CONFIG_MTD_TESTS) += mtd_speedtest.o obj-$(CONFIG_MTD_TESTS) += mtd_stresstest.o obj-$(CONFIG_MTD_TESTS) += mtd_subpagetest.o obj-$(CONFIG_MTD_TESTS) += mtd_torturetest.o +obj-$(CONFIG_MTD_NAND_TESTS) += nand_ecc-test.o diff --git a/drivers/mtd/tests/nand_ecc-test.c b/drivers/mtd/tests/nand_ecc-test.c new file mode 100644 index 0000000..54bbb4a --- /dev/null +++ b/drivers/mtd/tests/nand_ecc-test.c @@ -0,0 +1,88 @@ +#include +#include +#include +#include +#include +#include +#include +#include + +void inject_single_bit_error(void *data, size_t size) +{ + unsigned long offset = random32() % (size * BITS_PER_BYTE); + + __change_bit(offset, data); +} + +#if defined(CONFIG_MTD_NAND) || defined(CONFIG_MTD_NAND_MODULE) + +#include + +static unsigned char data[512]; +static unsigned char error_data[512]; + +static int nand_ecc_test(const size_t size) +{ + unsigned char code[3]; + unsigned char error_code[3]; + char testname[30]; + + BUG_ON(sizeof(data) < size); + + sprintf(testname, "nand-ecc-%zu", size); + + get_random_bytes(data, size); + + memcpy(error_data, data, size); + inject_single_bit_error(error_data, size); + + __nand_calculate_ecc(data, size, code); + __nand_calculate_ecc(error_data, size, error_code); + __nand_correct_data(error_data, code, error_code, size); + + if (!memcmp(data, error_data, size)) { + printk(KERN_INFO "nand_ecc-test: ok - %s\n", testname); + return 0; + } + + printk(KERN_ERR "nand_ecc-test: not ok - %s\n", testname); + + printk(KERN_DEBUG "hexdump of data:\n"); + print_hex_dump(KERN_DEBUG, "", DUMP_PREFIX_OFFSET, 16, 4, + data, size, false); + printk(KERN_DEBUG "hexdump of error data:\n"); + print_hex_dump(KERN_DEBUG, "", DUMP_PREFIX_OFFSET, 16, 4, + error_data, size, false); + + return -1; +} + +#else + +static int nand_ecc_test(const size_t size) +{ + return 0; +} + +#endif + +static int __init ecc_test_init(void) +{ + srandom32(jiffies); + + nand_ecc_test(256); + nand_ecc_test(512); + + return 0; +} + +static void __exit ecc_test_exit(void) +{ +} + +module_init(ecc_test_init); +module_exit(ecc_test_exit); + +MODULE_DESCRIPTION("ECC test module"); +MODULE_AUTHOR("Akinobu Mita"); +MODULE_LICENSE("GPL");