Add a test drivers for linker tables. v7: address ./scripts/checkpatch.pl --codespell issues 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 <mcgrof@xxxxxxxxxx> --- 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 643bce5a5b02..dbbcbb16951d 100644 --- a/lib/Kconfig.debug +++ b/lib/Kconfig.debug @@ -1839,6 +1839,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 f5e72a728285..899b2f363ef1 100644 --- a/lib/Makefile +++ b/lib/Makefile @@ -56,6 +56,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 <linux/tables.h> +#include <linux/printk.h> + +#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 <linux/tables.h> +#include <linux/printk.h> + +#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 <linux/tables.h> +#include <linux/printk.h> + +#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 <linux/tables.h> +#include <linux/printk.h> + +#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 <linux/tables.h> +#include <linux/printk.h> + +#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..77029c346294 --- /dev/null +++ b/lib/test_linktables/test-linktables.c @@ -0,0 +1,801 @@ +/* + * Linker table test driver + * + * Copyright (C) 2016 Luis R. Rodriguez <mcgrof@xxxxxxxxxx> + * + * 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 <linux/tables.h> +#include <linux/printk.h> +#include <linux/module.h> +#include <linux/miscdevice.h> +#include <linux/vmalloc.h> +#include <linux/slab.h> + +#include "test-linktables.h" + +#define NUM_TESTS __LINKTABLE_TESTS + +struct linktable_test_device *__test_dev; +static bool init_completed; + +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) +{ + long new; + int ret; + + ret = kstrtol(buf, 10, &new); + return ret; + if (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) + 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 <mcgrof@xxxxxxxxxx>"); +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 <linux/tables.h> + +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); -- 2.11.0 -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html