From patchwork Mon Jan 9 14:58:32 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Luis Chamberlain X-Patchwork-Id: 9505099 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 7545360710 for ; Mon, 9 Jan 2017 15:03:05 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 668EE28339 for ; Mon, 9 Jan 2017 15:03:05 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 5A47F284EB; Mon, 9 Jan 2017 15:03:05 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.2 required=2.0 tests=BAYES_00, RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) (using TLSv1.2 with cipher AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id B67B228339 for ; Mon, 9 Jan 2017 15:03:03 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.84_2) (envelope-from ) id 1cQbQU-0008MR-VS; Mon, 09 Jan 2017 14:59:58 +0000 Received: from mail6.bemta5.messagelabs.com ([195.245.231.135]) by lists.xenproject.org with esmtp (Exim 4.84_2) (envelope-from ) id 1cQbQT-0008Ki-Cz for xen-devel@lists.xensource.com; Mon, 09 Jan 2017 14:59:57 +0000 Received: from [85.158.139.211] by server-5.bemta-5.messagelabs.com id 0C/62-04025-C65A3785; Mon, 09 Jan 2017 14:59:56 +0000 X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrEIsWRWlGSWpSXmKPExsVybKJsh2720uI Ig+crVC3uTXnP7sDosb1vF3sAYxRrZl5SfkUCa8bcK6+YCo68Y6zo2f+btYHx4xnGLkZODiGB qYwSk5dzdTFyAdnLmCS+LzgNlODgYBPQlbh5WwIkLiLwj0ni3aQGVhCHWeANt8TZVTfYQbqFB Wwklqw4D9bAIqAqMX9fDkiYV8BcYsHLt2wgtoSAvMSutousIDYnUHz+hbWsEIuTJM6tvsAOUS 8ocXLmExaQMcwC6hLr5wmBhJmBWpu3zmaGGJMhMa9nDiuE7SWx6MYlKFtN4uq5TcwTGAVnIZk 0C2HSLCSTFjAyr2LUKE4tKkst0jW00EsqykzPKMlNzMzRNTQw1ctNLS5OTE/NSUwq1kvOz93E CAxbBiDYwdi03fMQoyQHk5Ior2NUQYQQX1J+SmVGYnFGfFFpTmrxIUYZDg4lCd6KJcURQoJFq empFWmZOcAIgklLcPAoifBWg6R5iwsSc4sz0yFSpxgVpcR5Q0ESAiCJjNI8uDZY1F5ilJUS5m UEOkSIpyC1KDezBFX+FaM4B6OSMG89yBSezLwSuOmvgBYzAS2OtANbXJKIkJJqYHTc+MHSOfW JzMksc48V7bJiBj82ezQ8fqo6PUr0js4BDt5Xj9vnT/r9rK1cdtFkw/kXHITNfpmKmt23qssK jRf9tnJu5QFOuzuP3d/Ger568X1WUqhLccDvfIUI1Yq5PxZF9HnYTFqYvTF7jlpPqHnEbm2Wx Obrf/ZEW5Vx9uuc65q08qyQjhJLcUaioRZzUXEiAKAMb2/VAgAA X-Env-Sender: mcgrof@kernel.org X-Msg-Ref: server-2.tower-206.messagelabs.com!1483973992!58387962!1 X-Originating-IP: [198.145.29.136] X-SpamReason: No, hits=0.0 required=7.0 tests= X-StarScan-Received: X-StarScan-Version: 9.1.1; banners=-,-,- X-VirusChecked: Checked Received: (qmail 45739 invoked from network); 9 Jan 2017 14:59:55 -0000 Received: from mail.kernel.org (HELO mail.kernel.org) (198.145.29.136) by server-2.tower-206.messagelabs.com with DHE-RSA-AES256-GCM-SHA384 encrypted SMTP; 9 Jan 2017 14:59:55 -0000 Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 1AB61203A1; Mon, 9 Jan 2017 14:59:50 +0000 (UTC) Received: from garbanzo.do-not-panic.com (c-73-15-241-2.hsd1.ca.comcast.net [73.15.241.2]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id F228E203E6; Mon, 9 Jan 2017 14:59:44 +0000 (UTC) From: "Luis R. Rodriguez" To: hpa@zytor.com, acme@redhat.com, tglx@linutronix.de, mingo@redhat.com, jpoimboe@redhat.com, bp@alien8.de, npiggin@gmail.com, mhiramat@kernel.org, masami.hiramatsu.pt@hitachi.com, jbaron@akamai.com, heiko.carstens@de.ibm.com, ananth@linux.vnet.ibm.com, anil.s.keshavamurthy@intel.com, davem@davemloft.net, realmz6@gmail.com, dalias@libc.org, linux@arm.linux.org.uk Date: Mon, 9 Jan 2017 06:58:32 -0800 Message-Id: <20170109145833.11502-15-mcgrof@kernel.org> X-Mailer: git-send-email 2.11.0 In-Reply-To: <20170109145833.11502-1-mcgrof@kernel.org> References: <20161222023811.21246-1-mcgrof@kernel.org> <20170109145833.11502-1-mcgrof@kernel.org> MIME-Version: 1.0 X-Virus-Scanned: ClamAV using ClamSMTP Cc: gnomes@lxorguk.ukuu.org.uk, linux-ia64@vger.kernel.org, jkosina@suse.cz, benh@kernel.crashing.org, ming.lei@canonical.com, linux@rasmusvillemoes.dk, platform-driver-x86@vger.kernel.org, James.Bottomley@HansenPartnership.com, paul.gortmaker@windriver.com, paulus@samba.org, sparclinux@vger.kernel.org, wangnan0@huawei.com, mchehab@osg.samsung.com, linux-arch@vger.kernel.org, markus.heiser@darmarit.de, xen-devel@lists.xensource.com, msalter@redhat.com, linux-sh@vger.kernel.org, mpe@ellerman.id.au, joro@8bytes.org, x86@kernel.org, fontana@sharpeleven.org, dsahern@gmail.com, linux-arm-kernel@lists.infradead.org, catalin.marinas@arm.com, dvhart@infradead.org, dwmw2@infradead.org, linux@roeck-us.net, linux-xtensa@linux-xtensa.org, pali.rohar@gmail.com, keescook@chromium.org, arnd@arndb.de, jani.nikula@intel.com, will.deacon@arm.com, rusty@rustcorp.com.au, rostedt@goodmis.org, ak@linux.intel.com, namhyung@kernel.org, andriy.shevchenko@linux.intel.com, mcb30@ipxe.org, linux-kbuild@vger.kernel.org, alan@linux.intel.com, jgross@suse.com, chris@zankel.net, pebolle@tiscali.nl, tony.luck@intel.com, jcmvbkbc@gmail.com, gregkh@linuxfoundation.org, dmitry.torokhov@gmail.com, adrian.hunter@intel.com, luto@amacapital.net, "Luis R. Rodriguez" , mmarek@suse.com, david.vrabel@citrix.com, jolsa@kernel.org, andrew.cooper3@citrix.com, akpm@linux-foundation.org, torvalds@linux-foundation.org, korea.drzix@gmail.com Subject: [Xen-devel] [PATCH v6 14/14] lib: add linker tables test driver X-BeenThere: xen-devel@lists.xen.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" X-Virus-Scanned: ClamAV using ClamSMTP Add a test drivers for linker tables. v6: rename table macro as suggested by Andy Shevchenko v5: added this commit for the first time in this series. Signed-off-by: Luis R. Rodriguez --- lib/Kconfig.debug | 6 + lib/Makefile | 1 + lib/test_linktables/Makefile | 13 + lib/test_linktables/test-linktables-00.c | 27 ++ lib/test_linktables/test-linktables-01.c | 27 ++ lib/test_linktables/test-linktables-02.c | 27 ++ lib/test_linktables/test-linktables-03.c | 27 ++ lib/test_linktables/test-linktables-04.c | 27 ++ lib/test_linktables/test-linktables.c | 801 +++++++++++++++++++++++++++++++ lib/test_linktables/test-linktables.h | 48 ++ 10 files changed, 1004 insertions(+) create mode 100644 lib/test_linktables/Makefile create mode 100644 lib/test_linktables/test-linktables-00.c create mode 100644 lib/test_linktables/test-linktables-01.c create mode 100644 lib/test_linktables/test-linktables-02.c create mode 100644 lib/test_linktables/test-linktables-03.c create mode 100644 lib/test_linktables/test-linktables-04.c create mode 100644 lib/test_linktables/test-linktables.c create mode 100644 lib/test_linktables/test-linktables.h diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug index 5655bd95919a..8c99ee3c9fd7 100644 --- a/lib/Kconfig.debug +++ b/lib/Kconfig.debug @@ -1827,6 +1827,12 @@ config TEST_HASH This is intended to help people writing architecture-specific optimized versions. If unsure, say N. +config TEST_LINKTABLES + bool "Test linker table functionality" + default n + help + Enable this option to test the kernel's linker table functionality. + endmenu # runtime tests config PROVIDE_OHCI1394_DMA_INIT diff --git a/lib/Makefile b/lib/Makefile index 94ed5cc4e89c..b5c5a7a9146c 100644 --- a/lib/Makefile +++ b/lib/Makefile @@ -55,6 +55,7 @@ obj-$(CONFIG_TEST_STATIC_KEYS) += test_static_key_base.o obj-$(CONFIG_TEST_PRINTF) += test_printf.o obj-$(CONFIG_TEST_BITMAP) += test_bitmap.o obj-$(CONFIG_TEST_UUID) += test_uuid.o +obj-$(CONFIG_TEST_LINKTABLES) += test_linktables/ ifeq ($(CONFIG_DEBUG_KOBJECT),y) CFLAGS_kobject.o += -DDEBUG diff --git a/lib/test_linktables/Makefile b/lib/test_linktables/Makefile new file mode 100644 index 000000000000..a1a67d7fd57a --- /dev/null +++ b/lib/test_linktables/Makefile @@ -0,0 +1,13 @@ +__LINKTABLE_TESTS := $(shell find ./ -name test-linktables-\*.c | wc -l) +ccflags-y += -D__LINKTABLE_TESTS=$(__LINKTABLE_TESTS) + +# Keep these disorganized, if linker tables did not work then +# things would not be ordered for us automatically at link time. +# Linker tables grants us the ability to be explicit about +# order. +obj-$(CONFIG_TEST_LINKTABLES) += test-linktables-04.o +obj-$(CONFIG_TEST_LINKTABLES) += test-linktables-02.o +obj-$(CONFIG_TEST_LINKTABLES) += test-linktables-01.o +obj-$(CONFIG_TEST_LINKTABLES) += test-linktables-03.o +obj-$(CONFIG_TEST_LINKTABLES) += test-linktables-00.o +obj-$(CONFIG_TEST_LINKTABLES) += test-linktables.o diff --git a/lib/test_linktables/test-linktables-00.c b/lib/test_linktables/test-linktables-00.c new file mode 100644 index 000000000000..ea162bd69179 --- /dev/null +++ b/lib/test_linktables/test-linktables-00.c @@ -0,0 +1,27 @@ +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt + +#include +#include + +#include "test-linktables.h" + +static int __init test_linktable_00_init(int input) +{ + return 0 * input; +} + +static int __initdata test_linktable_00_init_data(int input) +{ + return 0 * input; +} + +static int test_linktable_00(int input) +{ + return 0 * input; +} + +test_linktable_init_data(00, test_linktable_00_init_data); /* .init.data */ +test_linktable_init_text(00, test_linktable_00_init); /* .init.text */ +test_linktable(00, test_linktable_00); /* .data */ +test_linktable_text(00, test_linktable_00); /* .text */ +test_linktable_rodata(00, test_linktable_00); /* .rodata */ diff --git a/lib/test_linktables/test-linktables-01.c b/lib/test_linktables/test-linktables-01.c new file mode 100644 index 000000000000..39578990fd59 --- /dev/null +++ b/lib/test_linktables/test-linktables-01.c @@ -0,0 +1,27 @@ +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt + +#include +#include + +#include "test-linktables.h" + +static int __init test_linktable_01_init(int input) +{ + return 1 * input; +} + +static int __initdata test_linktable_01_init_data(int input) +{ + return 1 * input; +} + +static int test_linktable_01(int input) +{ + return 1 * input; +} + +test_linktable_init_data(01, test_linktable_01_init_data); /* .init.data */ +test_linktable_init_text(01, test_linktable_01_init); /* .init.text */ +test_linktable(01, test_linktable_01); /* .data */ +test_linktable_text(01, test_linktable_01); /* .text */ +test_linktable_rodata(01, test_linktable_01); /* .rodata */ diff --git a/lib/test_linktables/test-linktables-02.c b/lib/test_linktables/test-linktables-02.c new file mode 100644 index 000000000000..7f4b34240ecb --- /dev/null +++ b/lib/test_linktables/test-linktables-02.c @@ -0,0 +1,27 @@ +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt + +#include +#include + +#include "test-linktables.h" + +static int __init test_linktable_02_init(int input) +{ + return 2 * input; +} + +static int __initdata test_linktable_02_init_data(int input) +{ + return 2 * input; +} + +static int test_linktable_02(int input) +{ + return 2 * input; +} + +test_linktable_init_data(02, test_linktable_02_init_data); /* .init.data */ +test_linktable_init_text(02, test_linktable_02_init); /* .init.text */ +test_linktable(02, test_linktable_02); /* .data */ +test_linktable_text(02, test_linktable_02); /* .text */ +test_linktable_rodata(02, test_linktable_02); /* .rodata */ diff --git a/lib/test_linktables/test-linktables-03.c b/lib/test_linktables/test-linktables-03.c new file mode 100644 index 000000000000..47ca77e30c24 --- /dev/null +++ b/lib/test_linktables/test-linktables-03.c @@ -0,0 +1,27 @@ +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt + +#include +#include + +#include "test-linktables.h" + +static int __init test_linktable_03_init(int input) +{ + return 3 * input; +} + +static int __initdata test_linktable_03_init_data(int input) +{ + return 3 * input; +} + +static int test_linktable_03(int input) +{ + return 3 * input; +} + +test_linktable_init_data(03, test_linktable_03_init_data); /* .init.data */ +test_linktable_init_text(03, test_linktable_03_init); /* .init.text */ +test_linktable(03, test_linktable_03); /* .data */ +test_linktable_text(03, test_linktable_03); /* .text */ +test_linktable_rodata(03, test_linktable_03); /* .rodata */ diff --git a/lib/test_linktables/test-linktables-04.c b/lib/test_linktables/test-linktables-04.c new file mode 100644 index 000000000000..4123e0e16715 --- /dev/null +++ b/lib/test_linktables/test-linktables-04.c @@ -0,0 +1,27 @@ +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt + +#include +#include + +#include "test-linktables.h" + +static int __init test_linktable_04_init(int input) +{ + return 4 * input; +} + +static int __initdata test_linktable_04_init_data(int input) +{ + return 4 * input; +} + +static int test_linktable_04(int input) +{ + return 4 * input; +} + +test_linktable_init_data(04, test_linktable_04_init_data); /* .init.data */ +test_linktable_init_text(04, test_linktable_04_init); /* .init.text */ +test_linktable(04, test_linktable_04); /* .data */ +test_linktable_text(04, test_linktable_04); /* .text */ +test_linktable_rodata(04, test_linktable_04); /* .rodata */ diff --git a/lib/test_linktables/test-linktables.c b/lib/test_linktables/test-linktables.c new file mode 100644 index 000000000000..b78fb5b554cb --- /dev/null +++ b/lib/test_linktables/test-linktables.c @@ -0,0 +1,801 @@ +/* + * Linker table test driver + * + * Copyright (C) 2016 Luis R. Rodriguez + * + * This program is free software; you can redistribute it and/or modify it + * under the terms of copyleft-next (version 0.3.1 or later) as published + * at http://copyleft-next.org/. + */ + +/* + * This driver provides an interface to trigger and test the linker table API + * through a series of configurations and a few triggers. This driver + * must be built-in as linker tables currently lack generalized modular + * support -- additional work is required on each module right now if you + * want to use linker tables, in the future we may make this much easier. + * For now just build this into your kernel. + * + * echo -n .text > /sys/devices/virtual/misc/test_linktable/config_section_name + * echo -n 3 > /sys/devices/virtual/misc/test_linktable/config_input + * echo 1 > /sys/devices/virtual/misc/test_linktable/trigger_config + * cat /sys/devices/virtual/misc/test_linktable/test_result + * 30 + * + * echo -n .data > /sys/devices/virtual/misc/test_linktable/config_section_name + * echo -n .text > /sys/devices/virtual/misc/test_linktable/config_section_name + * echo -n .rodata > /sys/devices/virtual/misc/test_linktable/config_section_name + * + * To check the configuration: + * + * cat /sys/devices/virtual/misc/test_linktable/config + * + * NUM_TESTS + * ∑ test(n, input) + * n=0 + * + * Each linker table entry on each section has a series of entries. Each entry + * has a function which just multiplies the test case number by input value. + * + * ... + * test-linktables-03.c multiplies (3 * input) + * test-linktables-04.c multiplies (4 * input) + * ... + * + * A full run on a section produces the sum of these values. So with input set + * to 3 we have: + * + * (0 * 3) + (1 * 3) + (2 * 3) + (3 * 3) + (4 * 3) + * 0 + 3 + 6 + 9 + 12 + * 30 + * + * This is nothing important, its just a basic test. We had to pick something. + * Other than the above tests, this also demos and shows proper const use on + * all sections which we need as read-only: .text, .init.text, .rodata. + * Furthermore it does a silly write test to ensure write works on .data and + * init.data. Since the read-only sections use const we obviously are forced + * by the compiler to not be able to write to these sections. + * + * One of the more important items, the order, is tested. We could develop + * a fancy simple math algorithm that depends on order for correctness but + * instead we just annotate the expected order as we run. If the order listed + * on test->expected does not match with the actual order a routine was run + * in then we fail and complain. + */ +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt + +#include +#include +#include +#include +#include +#include + +#include "test-linktables.h" + +#define NUM_TESTS __LINKTABLE_TESTS + +struct linktable_test_device *__test_dev = NULL; +static bool init_completed = false; + +static const char *test_dev_name = "test_linktable"; + +/* + * Used for the default test to start configuration with if we reset the + * test from scratch. Don't use .init.* sections given we can only test + * these on init, when init_completed is true we can't .init.* section + * code anymore. + */ +#define TEST_SECTION_START ".data" + +DEFINE_LINKTABLE(struct test_linktable, test_fns_data); +DEFINE_LINKTABLE_TEXT(struct test_linktable, test_fns_text); +DEFINE_LINKTABLE_RO(struct test_linktable, test_fns_rodata); +DEFINE_LINKTABLE_INIT(struct test_linktable, test_fns_init_text); +DEFINE_LINKTABLE_INIT_DATA(struct test_linktable, test_fns_init_data); + +struct test_config { + char *section_name; + int input; + int test_result; +}; + +/** + * linktable_test_device - test device to help test linker tables + * + * @dev_idx: unique ID for test device + * @misc_dev: we use a misc device under the hood + * @dev: pointer to misc_dev's own struct device + * @config_mutex: protects configuration of test + * @trigger_mutex: the test trigger can only be fired once at a time + */ +struct linktable_test_device { + struct test_config config; + struct miscdevice misc_dev; + struct device *dev; + struct mutex config_mutex; + struct mutex trigger_mutex; + unsigned int num_called; +}; + +/** + * enum linktable_test_case - linker table test case + * + * @TEST_LINKTABLE_INIT: tests .init.text (this is all const) + * @TEST_LINKTABLE_INIT_DATA: tests .init.data + * @TEST_LINKTABLE: tests .data + * @TEST_LINKTABLE_TEXT: tests .text (this is all const) + * @TEST_LINKTABLE_RO: tests .rodata (this is all const) + */ +enum linktable_test_case { + __TEST_LINKTABLE_INVALID = 0, + + TEST_LINKTABLE_INIT, + TEST_LINKTABLE_INIT_DATA, + TEST_LINKTABLE, + TEST_LINKTABLE_TEXT, + TEST_LINKTABLE_RO, +}; + +static struct miscdevice *dev_to_misc_dev(struct device *dev) +{ + return dev_get_drvdata(dev); +} + +static +struct linktable_test_device *misc_dev_to_test_dev(struct miscdevice *misc_dev) +{ + return container_of(misc_dev, struct linktable_test_device, misc_dev); +} + +static struct linktable_test_device *dev_to_test_dev(struct device *dev) +{ + struct miscdevice *misc_dev; + + misc_dev = dev_to_misc_dev(dev); + + return misc_dev_to_test_dev(misc_dev); +} + +static ssize_t config_show(struct device *dev, + struct device_attribute *attr, + char *buf) +{ + struct linktable_test_device *test_dev = dev_to_test_dev(dev); + struct test_config *config = &test_dev->config; + int len = 0; + + mutex_lock(&test_dev->config_mutex); + + len += snprintf(buf, PAGE_SIZE, + "Custom trigger configuration for: %s\n", + dev_name(dev)); + + if (config->section_name) + len += snprintf(buf+len, PAGE_SIZE, + "section name:\t%s\n", + config->section_name); + else + len += snprintf(buf+len, PAGE_SIZE, + "default name:\tEMTPY\n"); + + mutex_unlock(&test_dev->config_mutex); + + return len; +} +static DEVICE_ATTR_RO(config); + +static int write_test(struct test_linktable *test, int write_val) +{ + test->write_test = write_val; + if (test->write_test != write_val) { + pr_warn("Write test failed\n"); + return -EPERM; + } + return 0; +} + +static int __test_linktable(struct linktable_test_device *test_dev, + struct test_linktable *test) +{ + struct test_config *config = &test_dev->config; + int rc; + + if (test->expected != test_dev->num_called) { + pr_warn("test routine ID %d called on order %d\n", + test->expected, test_dev->num_called); + return -EINVAL; + } + + rc = write_test(test, test->expected+1); + if (rc) + return rc; + + rc = test->op(config->input); + test_dev->num_called++; + + return rc; +} + +static int __test_linktable_ro(struct linktable_test_device *test_dev, + const struct test_linktable *test) +{ + struct test_config *config = &test_dev->config; + int rc; + + if (test->expected != test_dev->num_called) { + pr_info("test routine ID %d called on order %d\n", + test->expected, test_dev->num_called); + return -EINVAL; + } + + /* + * Note, compiler would complain if we tried write_test() so + * no need to test that. + */ + + rc = test->op(config->input); + test_dev->num_called++; + + return rc; +} + +static enum linktable_test_case get_test_case(const char *section_name) +{ + if (strcmp(".init.text", section_name) == 0) + return TEST_LINKTABLE_INIT; + if (strcmp(".init.data", section_name) == 0) + return TEST_LINKTABLE_INIT_DATA; + if (strcmp(".data", section_name) == 0) + return TEST_LINKTABLE; + if (strcmp(".text", section_name) == 0) + return TEST_LINKTABLE_TEXT; + if (strcmp(".rodata", section_name) == 0) + return TEST_LINKTABLE_TEXT; + + return __TEST_LINKTABLE_INVALID; +} + +static int __run_sanity_test(struct linktable_test_device *test_dev, + int num_entries) +{ + if (!num_entries) { + pr_warn("no tests -- this is invalid\n"); + return -EINVAL; + } + + pr_debug("number of tests: %d\n", num_entries); + + if (num_entries != NUM_TESTS) { + pr_warn("expected: %d test\n", NUM_TESTS); + return -EINVAL; + } + + return 0; +} + +static int __init run_init_text_test(struct linktable_test_device *test_dev) +{ + int ret, total = 0; + const struct test_linktable *test; + unsigned int num_entries = LINKTABLE_SIZE(test_fns_init_text); + + ret = __run_sanity_test(test_dev, num_entries); + if (ret) + return ret; + + linktable_for_each(test, test_fns_init_text) + total += __test_linktable_ro(test_dev, test); + + return total; +} + +static int __init run_init_data_test(struct linktable_test_device *test_dev) +{ + int ret, total = 0; + struct test_linktable *test; + unsigned int num_entries = LINKTABLE_SIZE(test_fns_init_data); + + ret = __run_sanity_test(test_dev, num_entries); + if (ret) + return ret; + + linktable_for_each(test, test_fns_init_data) + total += __test_linktable(test_dev, test); + + return total; +} + +static int run_data_test(struct linktable_test_device *test_dev) +{ + int ret, total = 0; + struct test_linktable *test; + unsigned int num_entries = LINKTABLE_SIZE(test_fns_data); + + ret = __run_sanity_test(test_dev, num_entries); + if (ret) + return ret; + + linktable_for_each(test, test_fns_data) + total += __test_linktable(test_dev, test); + + return total; +} + +static int run_text_test(struct linktable_test_device *test_dev) +{ + int ret, total = 0; + const struct test_linktable *test; + unsigned int num_entries = LINKTABLE_SIZE(test_fns_text); + + ret = __run_sanity_test(test_dev, num_entries); + if (ret) + return ret; + + linktable_for_each(test, test_fns_text) + total += __test_linktable_ro(test_dev, test); + + return total; +} + +static int run_rodata_test(struct linktable_test_device *test_dev) +{ + int ret, total = 0; + const struct test_linktable *test; + unsigned int num_entries = LINKTABLE_SIZE(test_fns_rodata); + + ret = __run_sanity_test(test_dev, num_entries); + if (ret) + return ret; + + linktable_for_each(test, test_fns_rodata) + total += __test_linktable_ro(test_dev, test); + + return total; +} + +static int __ref __trigger_config_run(struct linktable_test_device *test_dev) +{ + enum linktable_test_case test_case; + struct test_config *config = &test_dev->config; + + test_dev->num_called = 0; + test_case = get_test_case(config->section_name); + + switch (test_case) { + case TEST_LINKTABLE_INIT: + if (!init_completed) + return run_init_text_test(test_dev); + else + return -EACCES; + case TEST_LINKTABLE_INIT_DATA: + if (!init_completed) + return run_init_data_test(test_dev); + else + return -EACCES; + case TEST_LINKTABLE: + return run_data_test(test_dev); + case TEST_LINKTABLE_TEXT: + return run_text_test(test_dev); + case TEST_LINKTABLE_RO: + return run_rodata_test(test_dev); + default: + pr_warn("Invalid test case requested: %s\n", + config->section_name); + return -EINVAL; + } +} + +static int trigger_config_run(struct linktable_test_device *test_dev) +{ + struct test_config *config = &test_dev->config; + int rc; + + mutex_lock(&test_dev->trigger_mutex); + mutex_lock(&test_dev->config_mutex); + + pr_debug("running test on '%s'\n", config->section_name); + + rc = __trigger_config_run(test_dev); + + config->test_result = rc; + pr_debug("result: %d\n", rc); + + if (rc < 0) { + rc = -EINVAL; + goto out; + } + + rc = 0; + +out: + mutex_unlock(&test_dev->config_mutex); + mutex_unlock(&test_dev->trigger_mutex); + + return rc; +} + +static ssize_t +trigger_config_store(struct device *dev, + struct device_attribute *attr, + const char *buf, size_t count) +{ + struct linktable_test_device *test_dev = dev_to_test_dev(dev); + int rc; + + rc = trigger_config_run(test_dev); + if (rc) + goto out; + + rc = count; +out: + return rc; +} +static DEVICE_ATTR_WO(trigger_config); + +/* + * XXX: move to kstrncpy() once merged. + * + * Users should use kfree_const() when freeing these. + */ +static int __kstrncpy(char **dst, const char *name, size_t count, gfp_t gfp) +{ + *dst = kstrndup(name, count, gfp); + if (!*dst) + return -ENOSPC; + return count; +} + +static int config_copy_section_name(struct test_config *config, + const char *name, + size_t count) +{ + return __kstrncpy(&config->section_name, name, count, GFP_KERNEL); +} + +static void __linktable_config_free(struct test_config *config) +{ + kfree_const(config->section_name); + config->section_name = NULL; +} + +static void linktable_config_free(struct linktable_test_device *test_dev) +{ + struct test_config *config = &test_dev->config; + + mutex_lock(&test_dev->config_mutex); + __linktable_config_free(config); + mutex_unlock(&test_dev->config_mutex); +} + +static ssize_t config_section_name_store(struct device *dev, + struct device_attribute *attr, + const char *buf, size_t count) +{ + struct linktable_test_device *test_dev = dev_to_test_dev(dev); + struct test_config *config = &test_dev->config; + int rc; + + mutex_lock(&test_dev->config_mutex); + rc = config_copy_section_name(config, buf, count); + mutex_unlock(&test_dev->config_mutex); + + return rc; +} + +/* + * As per sysfs_kf_seq_show() the buf is max PAGE_SIZE. + */ +static ssize_t config_test_show_str(struct mutex *config_mutex, + char *dst, + char *src) +{ + int len; + + mutex_lock(config_mutex); + len = snprintf(dst, PAGE_SIZE, "%s\n", src); + mutex_unlock(config_mutex); + + return len; +} + +static ssize_t config_section_name_show(struct device *dev, + struct device_attribute *attr, + char *buf) +{ + struct linktable_test_device *test_dev = dev_to_test_dev(dev); + struct test_config *config = &test_dev->config; + + return config_test_show_str(&test_dev->config_mutex, buf, + config->section_name); +} +static DEVICE_ATTR(config_section_name, 0644, config_section_name_show, + config_section_name_store); + +static int trigger_config_run_named(struct linktable_test_device *test_dev, + const char *section_name) +{ + int copied; + struct test_config *config = &test_dev->config; + + mutex_lock(&test_dev->config_mutex); + copied = config_copy_section_name(config, section_name, + strlen(section_name)); + mutex_unlock(&test_dev->config_mutex); + + if (copied <=0 || copied != strlen(section_name)) + return -EINVAL; + + return trigger_config_run(test_dev); +} + +static int __linktable_config_init(struct test_config *config) +{ + int ret; + + ret = config_copy_section_name(config, TEST_SECTION_START, + strlen(TEST_SECTION_START)); + if (ret < 0) + goto out; + + config->input = 3; + config->test_result = 0; + +out: + return ret; +} + +static ssize_t reset_store(struct device *dev, + struct device_attribute *attr, + const char *buf, size_t count) +{ + struct linktable_test_device *test_dev = dev_to_test_dev(dev); + struct test_config *config = &test_dev->config; + int ret; + + mutex_lock(&test_dev->trigger_mutex); + mutex_lock(&test_dev->config_mutex); + + __linktable_config_free(config); + + ret = __linktable_config_init(config); + if (ret < 0) { + ret = -ENOMEM; + dev_err(dev, "could not alloc settings for config trigger: %d\n", + ret); + goto out; + } + + dev_info(dev, "reset\n"); + ret = count; + +out: + mutex_unlock(&test_dev->config_mutex); + mutex_unlock(&test_dev->trigger_mutex); + + return ret; +} +static DEVICE_ATTR_WO(reset); + +static int test_dev_config_update_int(struct linktable_test_device *test_dev, + const char *buf, size_t size, + int *config) +{ + char *end; + long new = simple_strtol(buf, &end, 0); + if (end == buf || new > INT_MAX || new < INT_MIN) + return -EINVAL; + mutex_lock(&test_dev->config_mutex); + *config = new; + mutex_unlock(&test_dev->config_mutex); + /* Always return full write size even if we didn't consume all */ + return size; +} + +static ssize_t test_dev_config_show_int(struct linktable_test_device *test_dev, + char *buf, + int config) +{ + int val; + + mutex_lock(&test_dev->config_mutex); + val = config; + mutex_unlock(&test_dev->config_mutex); + + return snprintf(buf, PAGE_SIZE, "%d\n", val); +} + +static ssize_t test_result_store(struct device *dev, + struct device_attribute *attr, + const char *buf, size_t count) +{ + struct linktable_test_device *test_dev = dev_to_test_dev(dev); + struct test_config *config = &test_dev->config; + + return test_dev_config_update_int(test_dev, buf, count, + &config->test_result); +} + +static ssize_t config_input_store(struct device *dev, + struct device_attribute *attr, + const char *buf, size_t count) +{ + struct linktable_test_device *test_dev = dev_to_test_dev(dev); + struct test_config *config = &test_dev->config; + + return test_dev_config_update_int(test_dev, buf, count, + &config->input); +} + +static ssize_t config_input_show(struct device *dev, + struct device_attribute *attr, + char *buf) +{ + struct linktable_test_device *test_dev = dev_to_test_dev(dev); + struct test_config *config = &test_dev->config; + + return test_dev_config_show_int(test_dev, buf, config->input); +} +static DEVICE_ATTR(config_input, 0644, config_input_show, config_input_store); + +static ssize_t test_result_show(struct device *dev, + struct device_attribute *attr, + char *buf) +{ + struct linktable_test_device *test_dev = dev_to_test_dev(dev); + struct test_config *config = &test_dev->config; + + return test_dev_config_show_int(test_dev, buf, config->test_result); +} +static DEVICE_ATTR(test_result, 0644, test_result_show, test_result_store); + +#define LINKTABLE_DEV_ATTR(name) &dev_attr_##name.attr + +static struct attribute *test_dev_attrs[] = { + LINKTABLE_DEV_ATTR(trigger_config), + LINKTABLE_DEV_ATTR(config), + LINKTABLE_DEV_ATTR(reset), + + LINKTABLE_DEV_ATTR(config_section_name), + LINKTABLE_DEV_ATTR(config_input), + LINKTABLE_DEV_ATTR(test_result), + + NULL, +}; + +ATTRIBUTE_GROUPS(test_dev); + +static int linktable_config_init(struct linktable_test_device *test_dev) +{ + struct test_config *config = &test_dev->config; + int ret; + + mutex_lock(&test_dev->config_mutex); + ret = __linktable_config_init(config); + mutex_unlock(&test_dev->config_mutex); + + return ret; +} + +static struct linktable_test_device *alloc_test_dev_linktable(void) +{ + int rc; + struct linktable_test_device *test_dev; + struct miscdevice *misc_dev; + + test_dev = vmalloc(sizeof(struct linktable_test_device)); + if (!test_dev) { + pr_err("Cannot alloc test_dev\n"); + goto err_out; + } + + memset(test_dev, 0, sizeof(struct linktable_test_device)); + + mutex_init(&test_dev->config_mutex); + mutex_init(&test_dev->trigger_mutex); + + rc = linktable_config_init(test_dev); + if (rc < 0) { + pr_err("Cannot alloc linktable_config_init()\n"); + goto err_out_free; + } + + misc_dev = &test_dev->misc_dev; + + misc_dev->minor = MISC_DYNAMIC_MINOR; + misc_dev->name = test_dev_name; + misc_dev->groups = test_dev_groups; + + return test_dev; + +err_out_free: + kfree(test_dev); +err_out: + return NULL; +} + +static void free_test_dev_linktable(struct linktable_test_device *test_dev) +{ + test_dev->misc_dev.name = NULL; + vfree(test_dev); + test_dev = NULL; + linktable_config_free(test_dev); +} + +static struct linktable_test_device *register_test_dev_linktable(void) +{ + struct linktable_test_device *test_dev = NULL; + int rc; + + test_dev = alloc_test_dev_linktable(); + if (!test_dev) { + return NULL; + } + + rc = misc_register(&test_dev->misc_dev); + if (rc) { + pr_err("could not register misc device: %d\n", rc); + free_test_dev_linktable(test_dev); + return NULL; + } + + test_dev->dev = test_dev->misc_dev.this_device; + dev_dbg(test_dev->dev, "interface ready\n"); + + return test_dev; +} + +static int __init test_linktable_init(void) +{ + struct linktable_test_device *test_dev; + int rc; + + test_dev = register_test_dev_linktable(); + if (!test_dev) { + pr_err("Cannot add test linktable device\n"); + return -ENODEV; + } + + rc = trigger_config_run_named(test_dev, ".init.text"); + if (WARN_ON(rc)) + return rc; + rc = trigger_config_run_named(test_dev, ".init.data"); + if (WARN_ON(rc)) + return rc; + + init_completed = true; + + rc = trigger_config_run_named(test_dev, ".data"); + if (WARN_ON(rc)) + return rc; + rc = trigger_config_run_named(test_dev, ".text"); + if (WARN_ON(rc)) + return rc; + rc = trigger_config_run_named(test_dev, ".rodata"); + if (WARN_ON(rc)) + return rc; + + pr_info("linker table tests: OK!\n"); + + return 0; +} +late_initcall(test_linktable_init); + +static +void unregister_test_dev_linktable(struct linktable_test_device *test_dev) +{ + dev_info(test_dev->dev, "removing interface\n"); + misc_deregister(&test_dev->misc_dev); + free_test_dev_linktable(test_dev); +} + +static void __exit test_linktable_exit(void) +{ + struct linktable_test_device *test_dev = __test_dev; + + unregister_test_dev_linktable(test_dev); +} +module_exit(test_linktable_exit); + +MODULE_AUTHOR("Luis R. Rodriguez "); +MODULE_LICENSE("GPL"); diff --git a/lib/test_linktables/test-linktables.h b/lib/test_linktables/test-linktables.h new file mode 100644 index 000000000000..c9c9d5c87263 --- /dev/null +++ b/lib/test_linktables/test-linktables.h @@ -0,0 +1,48 @@ +#include + +struct test_linktable { + int expected; + int write_test; + int (*op)(int input_digit); +}; + +#define test_linktable_init_text(__level, __op) \ + static LINKTABLE_INIT(test_fns_init_text, __level) \ + __test_fn_init_text_##__op = { \ + .expected = __level, \ + .op = __op, \ +}; + +#define test_linktable_init_data(__level, __op) \ + static LINKTABLE_INIT_DATA(test_fns_init_data, __level) \ + __test_fn_init_data_##__op = { \ + .expected = __level, \ + .op = __op, \ +}; + +#define test_linktable(__level, __op) \ + static LINKTABLE(test_fns_data, __level) \ + __test_fn_data_##__op = { \ + .expected = __level, \ + .op = __op, \ +}; + +#define test_linktable_text(__level, __op) \ + static LINKTABLE_TEXT(test_fns_text, __level) \ + __test_fn_text_##__op = { \ + .expected = __level, \ + .op = __op, \ +}; + +#define test_linktable_rodata(__level, __op) \ + static LINKTABLE_RO(test_fns_rodata, __level) \ + __test_fn_rodata_##__op = { \ + .expected = __level, \ + .op = __op, \ +}; + +DECLARE_LINKTABLE_RO(struct test_linktable, test_fns_init_text); +DECLARE_LINKTABLE(struct test_linktable, test_fns_init_data); +DECLARE_LINKTABLE(struct test_linktable, test_fns_data); +DECLARE_LINKTABLE_RO(struct test_linktable, test_fns_text); +DECLARE_LINKTABLE_RO(struct test_linktable, test_fns_rodata);