| Message ID | 20241016225830.3178108-1-jm@ti.com |
|---|---|
| State | RFC |
| Delegated to: | Ryan Eatmon |
| Headers | show |
| Series | [meta-ti,kirkstone/scarthgap,RFC] ti-test: Add hwspinlocktest | expand |
On 10/16/2024 5:58 PM, Judith Mendez wrote: > Add hwspinlock recipe to build out of tree module hwspinlock test. > > Signed-off-by: Judith Mendez <jm@ti.com> > --- > .../recipes-core/packagegroups/ti-test.bb | 1 + > .../hwspinlocktest/files/LICENSE | 29 ++ > .../hwspinlocktest/files/Makefile | 19 ++ > .../files/omap_hwspinlock_test.c | 251 ++++++++++++++++++ > .../hwspinlocktest/hwspinlocktest_0.1.bb | 18 ++ > 5 files changed, 318 insertions(+) > create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/LICENSE > create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/Makefile > create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/omap_hwspinlock_test.c > create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb > > diff --git a/meta-arago-test/recipes-core/packagegroups/ti-test.bb b/meta-arago-test/recipes-core/packagegroups/ti-test.bb > index 695bed5c..91ebec44 100644 > --- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb > +++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb > @@ -17,6 +17,7 @@ TI_TEST_BASE = "\ > evtest \ > fio \ > git \ > + hwspinlocktest \ > hdparm \ > i2c-tools \ > iozone3 \ One piece of feedback. If this kind of test is only applicable to some platforms, then we should only be adding it for those platforms and not for all platforms. TI_TEST_TI_TOOLS:append:ti33x = " \ omapconf \ pru-icss \ switch-config \ hwspinlocktest \ " etc... Based on the source code, this looks something like the list: > +static const struct of_device_id omap_hwspinlock_test_of_match[] = { > + { .compatible = "ti,omap-hwspinlock-test", }, > + { .compatible = "ti,omap4-hwspinlock-test", }, > + { .compatible = "ti,omap5-hwspinlock-test", }, > + { .compatible = "ti,dra7-hwspinlock-test", }, > + { .compatible = "ti,am33xx-hwspinlock-test", }, > + { .compatible = "ti,am43xx-hwspinlock-test", }, > + { .compatible = "ti,am654-hwspinlock-test", }, > + { /* end */ }, > +}; *snip* > +static const struct hwspinlock_data soc_data[] = { > + { "ti,omap4", 32, }, > + { "ti,omap5", 32, }, > + { "ti,dra7", 256, }, > + { "ti,am33xx", 128, }, > + { "ti,am43", 128, }, > + { "ti,am654", 256, }, > + { "ti,j721e", 256, }, > + { "ti,j7200", 256, }, > + { "ti,am642", 256, }, > + { "ti,j721s2", 256, }, > + { /* sentinel */ }, > diff --git a/meta-arago-test/recipes-kernel/hwspinlocktest/files/LICENSE b/meta-arago-test/recipes-kernel/hwspinlocktest/files/LICENSE > new file mode 100644 > index 00000000..f5351a25 > --- /dev/null > +++ b/meta-arago-test/recipes-kernel/hwspinlocktest/files/LICENSE > @@ -0,0 +1,29 @@ > +BSD 3-Clause License > + > +Copyright (c) 2019, Suman Anna > +All rights reserved. > + > +Redistribution and use in source and binary forms, with or without > +modification, are permitted provided that the following conditions are met: > + > +1. Redistributions of source code must retain the above copyright notice, this > + list of conditions and the following disclaimer. > + > +2. Redistributions in binary form must reproduce the above copyright notice, > + this list of conditions and the following disclaimer in the documentation > + and/or other materials provided with the distribution. > + > +3. Neither the name of the copyright holder nor the names of its > + contributors may be used to endorse or promote products derived from > + this software without specific prior written permission. > + > +THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" > +AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE > +IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE > +DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE > +FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL > +DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR > +SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER > +CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, > +OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE > +OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. > diff --git a/meta-arago-test/recipes-kernel/hwspinlocktest/files/Makefile b/meta-arago-test/recipes-kernel/hwspinlocktest/files/Makefile > new file mode 100644 > index 00000000..d8fe76dc > --- /dev/null > +++ b/meta-arago-test/recipes-kernel/hwspinlocktest/files/Makefile > @@ -0,0 +1,19 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-3-Clause) > +# > +# TI OMAP HwSpinlock Unit Test > +# > + > +obj-m := omap_hwspinlock_test.o > + > +SRC := $(shell pwd) > + > +all: > + $(MAKE) -C $(KERNEL_SRC) M=$(SRC) > + > +modules_install: > + $(MAKE) -C $(KERNEL_SRC) M=$(SRC) modules_install > + > +clean: > + rm -f *.o *~ core .depend .*.cmd *.ko *.mod.c > + rm -f Module.markers Module.symvers modules.order > + rm -rf .tmp_versions Modules.symvers > diff --git a/meta-arago-test/recipes-kernel/hwspinlocktest/files/omap_hwspinlock_test.c b/meta-arago-test/recipes-kernel/hwspinlocktest/files/omap_hwspinlock_test.c > new file mode 100644 > index 00000000..b1801389 > --- /dev/null > +++ b/meta-arago-test/recipes-kernel/hwspinlocktest/files/omap_hwspinlock_test.c > @@ -0,0 +1,251 @@ > +// SPDX-License-Identifier: (GPL-2.0 OR BSD-3-Clause) > +/* > + * OMAP hardware spinlock test driver > + * > + * Copyright (C) 2014-2021 Texas Instruments Incorporated - https://www.ti.com > + * Suman Anna <s-anna@ti.com> > + */ > + > +#include <linux/kernel.h> > +#include <linux/module.h> > +#include <linux/init.h> > +#include <linux/err.h> > +#include <linux/sched.h> > +#include <linux/platform_device.h> > +#include <linux/of.h> > +#include <linux/of_device.h> > + > +#include <linux/hwspinlock.h> > + > +/* load-time options */ > +static int count = 2; > +module_param(count, int, 0444); > + > +struct hwspinlock_data { > + const char *compatible; > + const unsigned int max_locks; > +}; > + > +static int hwspin_lock_test(struct hwspinlock *hwlock) > +{ > + int i; > + int ret; > + > + pr_err("\nTesting lock %d\n", hwspin_lock_get_id(hwlock)); > + for (i = 0; i < count; i++) { > + ret = hwspin_trylock(hwlock); > + if (ret) { > + pr_err("%s: Initial lock failed\n", __func__); > + return -EFAULT; > + } > + pr_err("trylock #1 status[%d] = %d\n", i, ret); > + > + /* Verify lock actually works - re-acquiring it should fail */ > + ret = hwspin_trylock(hwlock); > + pr_err("trylock #2 status[%d] = %d\n", i, ret); > + if (!ret) { > + /* Keep locks balanced even in failure cases */ > + hwspin_unlock(hwlock); > + hwspin_unlock(hwlock); > + pr_err("%s: Recursive lock succeeded unexpectedly\n", > + __func__); > + return -EFAULT; > + } > + > + /* Verify unlock by re-acquiring the lock after releasing it */ > + hwspin_unlock(hwlock); > + ret = hwspin_trylock(hwlock); > + pr_err("trylock after unlock status[%d] = %d\n", i, ret); > + if (ret) { > + pr_err("%s: Unlock failed\n", __func__); > + return -EINVAL; > + } > + > + hwspin_unlock(hwlock); > + } > + > + return 0; > +} > + > +static int hwspin_lock_test_all_locks(unsigned int max_locks) > +{ > + int i; > + int ret = 0, ret1 = 0; > + struct hwspinlock *hwlock = NULL; > + > + pr_err("Testing %d locks\n", max_locks); > + for (i = 0; i < max_locks; i++) { > + hwlock = hwspin_lock_request_specific(i); > + if (!hwlock) { > + pr_err("request lock %d failed\n", i); > + ret = -EIO; > + continue; > + } > + > + ret1 = hwspin_lock_test(hwlock); > + if (ret1) { > + pr_err("hwspinlock tests failed on lock %d\n", i); > + ret = ret1; > + goto free_lock; > + } > + > +free_lock: > + ret1 = hwspin_lock_free(hwlock); > + if (ret1) { > + pr_err("hwspin_lock_free failed on lock %d\n", i); > + ret = ret1; > + } > + } > + > + return ret; > +} > + > +static const struct of_device_id omap_hwspinlock_test_of_match[] = { > + { .compatible = "ti,omap-hwspinlock-test", }, > + { .compatible = "ti,omap4-hwspinlock-test", }, > + { .compatible = "ti,omap5-hwspinlock-test", }, > + { .compatible = "ti,dra7-hwspinlock-test", }, > + { .compatible = "ti,am33xx-hwspinlock-test", }, > + { .compatible = "ti,am43xx-hwspinlock-test", }, > + { .compatible = "ti,am654-hwspinlock-test", }, > + { /* end */ }, > +}; > + > +static int hwspin_lock_test_all_phandle_locks(unsigned int max_locks) > +{ > + struct device_node *np = NULL; > + struct hwspinlock *hwlock = NULL; > + int ret = 0, ret1 = 0; > + unsigned int i; > + int num_locks; > + int hwlock_id; > + > + np = of_find_matching_node_and_match(NULL, > + omap_hwspinlock_test_of_match, > + NULL); > + if (!np) { > + pr_err("\nNo test node provided\n"); > + return 0; > + } > + > + num_locks = of_count_phandle_with_args(np, "hwlocks", "#hwlock-cells"); > + pr_err("Number of phandles = %d max_locks = %d\n", > + num_locks, max_locks); > + > + for (i = 0; i < num_locks; i++) { > + hwlock_id = of_hwspin_lock_get_id(np, i); > + if (hwlock_id < 0) { > + pr_err("unable to get hwlock_id : %d\n", hwlock_id); > + ret = -EINVAL; > + continue; > + }; > + > + hwlock = hwspin_lock_request_specific(hwlock_id); > + if (!hwlock) { > + pr_err("unable to get hwlock\n"); > + ret = -EINVAL; > + continue; > + } > + > + ret1 = hwspin_lock_test(hwlock); > + if (ret1) { > + pr_err("hwspinlock test failed on DT lock %d, ret = %d\n", > + hwspin_lock_get_id(hwlock), ret1); > + ret = ret1; > + } > + > + ret1 = hwspin_lock_free(hwlock); > + if (ret1) { > + pr_err("hwspin_lock_free failed on lock %d\n", > + hwspin_lock_get_id(hwlock)); > + ret = ret1; > + } > + } > + > + return ret; > +} > + > +static > +unsigned int omap_hwspinlock_get_locks(const struct hwspinlock_data *data) > +{ > + unsigned int locks = 0; > + > + while (data->compatible) { > + if (of_machine_is_compatible(data->compatible)) { > + locks = data->max_locks; > + break; > + } > + data++; > + } > + > + return locks; > +} > + > +static const struct of_device_id omap_hwspinlock_of_match[] = { > + { .compatible = "ti,omap4-hwspinlock", }, > + { .compatible = "ti,am654-hwspinlock", }, > + { .compatible = "ti,am64-hwspinlock", }, > + { /* end */ }, > +}; > + > +static const struct hwspinlock_data soc_data[] = { > + { "ti,omap4", 32, }, > + { "ti,omap5", 32, }, > + { "ti,dra7", 256, }, > + { "ti,am33xx", 128, }, > + { "ti,am43", 128, }, > + { "ti,am654", 256, }, > + { "ti,j721e", 256, }, > + { "ti,j7200", 256, }, > + { "ti,am642", 256, }, > + { "ti,j721s2", 256, }, > + { /* sentinel */ }, > +}; > + > +static int omap_hwspinlock_test_init(void) > +{ > + struct device_node *np; > + unsigned int max_locks; > + int ret; > + > + pr_err("\n** HwSpinLock Unit Test Module initiated **\n"); > + > + max_locks = omap_hwspinlock_get_locks(soc_data); > + if (!max_locks) { > + pr_err("\nNot a compatible platform\n"); > + return -ENODEV; > + } > + > + np = of_find_matching_node_and_match(NULL, omap_hwspinlock_of_match, > + NULL); > + if (!np || !of_device_is_available(np)) { > + pr_err("\nNo HwSpinlock node provided/enabled\n"); > + return -ENODEV; > + } > + > + pr_err("\n***** Begin - Test All Locks ****\n"); > + ret = hwspin_lock_test_all_locks(max_locks); > + if (ret) > + pr_err("hwspin_lock_test_all_locks failed, ret = %d\n", ret); > + pr_err("\n***** End - Test All Locks ****\n"); > + > + pr_err("\n***** Begin - Test All pHandle Locks ****\n"); > + ret = hwspin_lock_test_all_phandle_locks(max_locks); > + if (ret) > + pr_err("hwspin_lock_test_all_locks failed, ret = %d\n", ret); > + pr_err("\n***** End - Test All pHandle Locks ****\n"); > + > + return 0; > +} > + > +static void omap_hwspinlock_test_exit(void) > +{ > + pr_err("\n** HwSpinLock Unit Test Module finished **\n"); > +} > + > +module_init(omap_hwspinlock_test_init); > +module_exit(omap_hwspinlock_test_exit); > + > +MODULE_LICENSE("Dual BSD/GPL"); > +MODULE_DESCRIPTION("Hardware spinlock Test driver for TI SoCs"); > +MODULE_AUTHOR("Suman Anna <s-anna@ti.com>"); > diff --git a/meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb b/meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb > new file mode 100644 > index 00000000..2edd93f6 > --- /dev/null > +++ b/meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb > @@ -0,0 +1,18 @@ > +SUMMARY = "Build hwspinlock test as an external Linux kernel module" > +DESCRIPTION = "${SUMMARY}" > +LICENSE = "GPL-2.0-only | BSD-3-Clause" > +LIC_FILES_CHKSUM = "file://LICENSE;md5=bfa02c83df161e37647ee23a2c7eacd4" And like above, we can limit the applicable scope of the recipe even being supported by adding this: COMPATIBLE_MACHINE = "omapl138|ti33x|ti43x|omap-a15" I'll let you take a stab at figuring out the correct list for both here and above. > + > +inherit module > + > +SRC_URI = "file://Makefile \ > + file://omap_hwspinlock_test.c \ > + file://LICENSE \ > + " > + > +S = "${WORKDIR}" > + > +# The inherit of module.bbclass will automatically name module packages with > +# "kernel-module-" prefix as required by the oe-core build environment. > + > +RPROVIDES:${PN} += "kernel-module-hwspinlocktest"
Hi Judith, s/[meta-arago][meta-ti]/ [meta-arago] in subject. On 21/10/24 03:49, Ryan Eatmon via lists.yoctoproject.org wrote: > On 10/16/2024 5: 58 PM, Judith Mendez wrote: > Add hwspinlock recipe > to build out of tree module hwspinlock test. > > Signed-off-by: Judith > Mendez <jm@ ti. com> > --- > > .. . /recipes-core/packagegroups/ti-test. bb | 1 + > > .. . /hwspinlocktest/files/LICENSE > ZjQcmQRYFpfptBannerStart > This message was sent from outside of Texas Instruments. > Do not click links or open attachments unless you recognize the source > of this email and know the content is safe. > Report Suspicious > <https://us-phishalarm-ewt.proofpoint.com/EWT/v1/G3vK!uRdgXXcF3C07wMXEfNND_wyXhpB0lFf6zT_57GliBcGIaEQPPgCRKf-GH3PQoL5FAbNeNV6XMTUJwlRTG2jOhnNXDcw$> > > ZjQcmQRYFpfptBannerEnd > On 10/16/2024 5:58 PM, Judith Mendez wrote: > > Add hwspinlock recipe to build out of tree module hwspinlock test. > > > > Signed-off-by: Judith Mendez <jm@ti.com> > > --- > > .../recipes-core/packagegroups/ti-test.bb | 1 + > > .../hwspinlocktest/files/LICENSE | 29 ++ > > .../hwspinlocktest/files/Makefile | 19 ++ > > .../files/omap_hwspinlock_test.c | 251 ++++++++++++++++++ > > .../hwspinlocktest/hwspinlocktest_0.1.bb | 18 ++ > > 5 files changed, 318 insertions(+) > > create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/LICENSE > > create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/Makefile > > create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/omap_hwspinlock_test.c > > create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb > > Any particular reasons to maintain the files in meta-arago layer instead of leveraging the repository [1]? [1]: https://github.com/sumananna/omap-hwspinlock-test --- Chirag > > diff --git a/meta-arago-test/recipes-core/packagegroups/ti-test.bb b/meta-arago-test/recipes-core/packagegroups/ti-test.bb > > index 695bed5c..91ebec44 100644 > > --- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb > > +++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb > > @@ -17,6 +17,7 @@ TI_TEST_BASE = "\ > > evtest \ > > fio \ > > git \ > > + hwspinlocktest \ > > hdparm \ > > i2c-tools \ > > iozone3 \
Hi Chirag, >> On 10/16/2024 5:58 PM, Judith Mendez wrote: >> > Add hwspinlock recipe to build out of tree module hwspinlock test. >> > >> > Signed-off-by: Judith Mendez <jm@ti.com> >> > --- >> > .../recipes-core/packagegroups/ti-test.bb | 1 + >> > .../hwspinlocktest/files/LICENSE | 29 ++ >> > .../hwspinlocktest/files/Makefile | 19 ++ >> > .../files/omap_hwspinlock_test.c | 251 ++++++++++++++++++ >> > .../hwspinlocktest/hwspinlocktest_0.1.bb | 18 ++ >> > 5 files changed, 318 insertions(+) >> > create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/LICENSE >> > create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/Makefile >> > create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/omap_hwspinlock_test.c >> > create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb >> > > > > Any particular reasons to maintain the files in meta-arago layer instead > of leveraging the repository [1]? > > [1]: https://github.com/sumananna/omap-hwspinlock-test > <https://github.com/sumananna/omap-hwspinlock-test> ATM this test is in a personal github repo. Apparently we do not want to pull from personal repos and there is a JIRA req to move the repo to ti.com, but the target date is for 11.0. Since we want to add the test asap, the idea is to add to yocto and update the recipe later when the repo has been moved. ~ Judith
Hi Ryan, > On 10/16/2024 5:58 PM, Judith Mendez wrote: >> Add hwspinlock recipe to build out of tree module hwspinlock test. >> >> Signed-off-by: Judith Mendez <jm@ti.com> >> --- >> .../recipes-core/packagegroups/ti-test.bb | 1 + >> .../hwspinlocktest/files/LICENSE | 29 ++ >> .../hwspinlocktest/files/Makefile | 19 ++ >> .../files/omap_hwspinlock_test.c | 251 ++++++++++++++++++ >> .../hwspinlocktest/hwspinlocktest_0.1.bb | 18 ++ >> 5 files changed, 318 insertions(+) >> create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/LICENSE >> create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/Makefile >> create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/omap_hwspinlock_test.c >> create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb >> >> diff --git a/meta-arago-test/recipes-core/packagegroups/ti-test.bb b/meta-arago-test/recipes-core/packagegroups/ti-test.bb >> index 695bed5c..91ebec44 100644 >> --- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb >> +++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb >> @@ -17,6 +17,7 @@ TI_TEST_BASE = "\ >> evtest \ >> fio \ >> git \ >> + hwspinlocktest \ >> hdparm \ >> i2c-tools \ >> iozone3 \ > > One piece of feedback. If this kind of test is only applicable to some > platforms, then we should only be adding it for those platforms and not > for all platforms. I believe this test should apply for all platforms, am335x, am437x, am65x, and Sitara platforms as well. I am not 100% sure about Jacinto though. ~ Judith
diff --git a/meta-arago-test/recipes-core/packagegroups/ti-test.bb b/meta-arago-test/recipes-core/packagegroups/ti-test.bb index 695bed5c..91ebec44 100644 --- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb +++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb @@ -17,6 +17,7 @@ TI_TEST_BASE = "\ evtest \ fio \ git \ + hwspinlocktest \ hdparm \ i2c-tools \ iozone3 \ diff --git a/meta-arago-test/recipes-kernel/hwspinlocktest/files/LICENSE b/meta-arago-test/recipes-kernel/hwspinlocktest/files/LICENSE new file mode 100644 index 00000000..f5351a25 --- /dev/null +++ b/meta-arago-test/recipes-kernel/hwspinlocktest/files/LICENSE @@ -0,0 +1,29 @@ +BSD 3-Clause License + +Copyright (c) 2019, Suman Anna +All rights reserved. + +Redistribution and use in source and binary forms, with or without +modification, are permitted provided that the following conditions are met: + +1. Redistributions of source code must retain the above copyright notice, this + list of conditions and the following disclaimer. + +2. Redistributions in binary form must reproduce the above copyright notice, + this list of conditions and the following disclaimer in the documentation + and/or other materials provided with the distribution. + +3. Neither the name of the copyright holder nor the names of its + contributors may be used to endorse or promote products derived from + this software without specific prior written permission. + +THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" +AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE +IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE +DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE +FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL +DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR +SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER +CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, +OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE +OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. diff --git a/meta-arago-test/recipes-kernel/hwspinlocktest/files/Makefile b/meta-arago-test/recipes-kernel/hwspinlocktest/files/Makefile new file mode 100644 index 00000000..d8fe76dc --- /dev/null +++ b/meta-arago-test/recipes-kernel/hwspinlocktest/files/Makefile @@ -0,0 +1,19 @@ +# SPDX-License-Identifier: (GPL-2.0 OR BSD-3-Clause) +# +# TI OMAP HwSpinlock Unit Test +# + +obj-m := omap_hwspinlock_test.o + +SRC := $(shell pwd) + +all: + $(MAKE) -C $(KERNEL_SRC) M=$(SRC) + +modules_install: + $(MAKE) -C $(KERNEL_SRC) M=$(SRC) modules_install + +clean: + rm -f *.o *~ core .depend .*.cmd *.ko *.mod.c + rm -f Module.markers Module.symvers modules.order + rm -rf .tmp_versions Modules.symvers diff --git a/meta-arago-test/recipes-kernel/hwspinlocktest/files/omap_hwspinlock_test.c b/meta-arago-test/recipes-kernel/hwspinlocktest/files/omap_hwspinlock_test.c new file mode 100644 index 00000000..b1801389 --- /dev/null +++ b/meta-arago-test/recipes-kernel/hwspinlocktest/files/omap_hwspinlock_test.c @@ -0,0 +1,251 @@ +// SPDX-License-Identifier: (GPL-2.0 OR BSD-3-Clause) +/* + * OMAP hardware spinlock test driver + * + * Copyright (C) 2014-2021 Texas Instruments Incorporated - https://www.ti.com + * Suman Anna <s-anna@ti.com> + */ + +#include <linux/kernel.h> +#include <linux/module.h> +#include <linux/init.h> +#include <linux/err.h> +#include <linux/sched.h> +#include <linux/platform_device.h> +#include <linux/of.h> +#include <linux/of_device.h> + +#include <linux/hwspinlock.h> + +/* load-time options */ +static int count = 2; +module_param(count, int, 0444); + +struct hwspinlock_data { + const char *compatible; + const unsigned int max_locks; +}; + +static int hwspin_lock_test(struct hwspinlock *hwlock) +{ + int i; + int ret; + + pr_err("\nTesting lock %d\n", hwspin_lock_get_id(hwlock)); + for (i = 0; i < count; i++) { + ret = hwspin_trylock(hwlock); + if (ret) { + pr_err("%s: Initial lock failed\n", __func__); + return -EFAULT; + } + pr_err("trylock #1 status[%d] = %d\n", i, ret); + + /* Verify lock actually works - re-acquiring it should fail */ + ret = hwspin_trylock(hwlock); + pr_err("trylock #2 status[%d] = %d\n", i, ret); + if (!ret) { + /* Keep locks balanced even in failure cases */ + hwspin_unlock(hwlock); + hwspin_unlock(hwlock); + pr_err("%s: Recursive lock succeeded unexpectedly\n", + __func__); + return -EFAULT; + } + + /* Verify unlock by re-acquiring the lock after releasing it */ + hwspin_unlock(hwlock); + ret = hwspin_trylock(hwlock); + pr_err("trylock after unlock status[%d] = %d\n", i, ret); + if (ret) { + pr_err("%s: Unlock failed\n", __func__); + return -EINVAL; + } + + hwspin_unlock(hwlock); + } + + return 0; +} + +static int hwspin_lock_test_all_locks(unsigned int max_locks) +{ + int i; + int ret = 0, ret1 = 0; + struct hwspinlock *hwlock = NULL; + + pr_err("Testing %d locks\n", max_locks); + for (i = 0; i < max_locks; i++) { + hwlock = hwspin_lock_request_specific(i); + if (!hwlock) { + pr_err("request lock %d failed\n", i); + ret = -EIO; + continue; + } + + ret1 = hwspin_lock_test(hwlock); + if (ret1) { + pr_err("hwspinlock tests failed on lock %d\n", i); + ret = ret1; + goto free_lock; + } + +free_lock: + ret1 = hwspin_lock_free(hwlock); + if (ret1) { + pr_err("hwspin_lock_free failed on lock %d\n", i); + ret = ret1; + } + } + + return ret; +} + +static const struct of_device_id omap_hwspinlock_test_of_match[] = { + { .compatible = "ti,omap-hwspinlock-test", }, + { .compatible = "ti,omap4-hwspinlock-test", }, + { .compatible = "ti,omap5-hwspinlock-test", }, + { .compatible = "ti,dra7-hwspinlock-test", }, + { .compatible = "ti,am33xx-hwspinlock-test", }, + { .compatible = "ti,am43xx-hwspinlock-test", }, + { .compatible = "ti,am654-hwspinlock-test", }, + { /* end */ }, +}; + +static int hwspin_lock_test_all_phandle_locks(unsigned int max_locks) +{ + struct device_node *np = NULL; + struct hwspinlock *hwlock = NULL; + int ret = 0, ret1 = 0; + unsigned int i; + int num_locks; + int hwlock_id; + + np = of_find_matching_node_and_match(NULL, + omap_hwspinlock_test_of_match, + NULL); + if (!np) { + pr_err("\nNo test node provided\n"); + return 0; + } + + num_locks = of_count_phandle_with_args(np, "hwlocks", "#hwlock-cells"); + pr_err("Number of phandles = %d max_locks = %d\n", + num_locks, max_locks); + + for (i = 0; i < num_locks; i++) { + hwlock_id = of_hwspin_lock_get_id(np, i); + if (hwlock_id < 0) { + pr_err("unable to get hwlock_id : %d\n", hwlock_id); + ret = -EINVAL; + continue; + }; + + hwlock = hwspin_lock_request_specific(hwlock_id); + if (!hwlock) { + pr_err("unable to get hwlock\n"); + ret = -EINVAL; + continue; + } + + ret1 = hwspin_lock_test(hwlock); + if (ret1) { + pr_err("hwspinlock test failed on DT lock %d, ret = %d\n", + hwspin_lock_get_id(hwlock), ret1); + ret = ret1; + } + + ret1 = hwspin_lock_free(hwlock); + if (ret1) { + pr_err("hwspin_lock_free failed on lock %d\n", + hwspin_lock_get_id(hwlock)); + ret = ret1; + } + } + + return ret; +} + +static +unsigned int omap_hwspinlock_get_locks(const struct hwspinlock_data *data) +{ + unsigned int locks = 0; + + while (data->compatible) { + if (of_machine_is_compatible(data->compatible)) { + locks = data->max_locks; + break; + } + data++; + } + + return locks; +} + +static const struct of_device_id omap_hwspinlock_of_match[] = { + { .compatible = "ti,omap4-hwspinlock", }, + { .compatible = "ti,am654-hwspinlock", }, + { .compatible = "ti,am64-hwspinlock", }, + { /* end */ }, +}; + +static const struct hwspinlock_data soc_data[] = { + { "ti,omap4", 32, }, + { "ti,omap5", 32, }, + { "ti,dra7", 256, }, + { "ti,am33xx", 128, }, + { "ti,am43", 128, }, + { "ti,am654", 256, }, + { "ti,j721e", 256, }, + { "ti,j7200", 256, }, + { "ti,am642", 256, }, + { "ti,j721s2", 256, }, + { /* sentinel */ }, +}; + +static int omap_hwspinlock_test_init(void) +{ + struct device_node *np; + unsigned int max_locks; + int ret; + + pr_err("\n** HwSpinLock Unit Test Module initiated **\n"); + + max_locks = omap_hwspinlock_get_locks(soc_data); + if (!max_locks) { + pr_err("\nNot a compatible platform\n"); + return -ENODEV; + } + + np = of_find_matching_node_and_match(NULL, omap_hwspinlock_of_match, + NULL); + if (!np || !of_device_is_available(np)) { + pr_err("\nNo HwSpinlock node provided/enabled\n"); + return -ENODEV; + } + + pr_err("\n***** Begin - Test All Locks ****\n"); + ret = hwspin_lock_test_all_locks(max_locks); + if (ret) + pr_err("hwspin_lock_test_all_locks failed, ret = %d\n", ret); + pr_err("\n***** End - Test All Locks ****\n"); + + pr_err("\n***** Begin - Test All pHandle Locks ****\n"); + ret = hwspin_lock_test_all_phandle_locks(max_locks); + if (ret) + pr_err("hwspin_lock_test_all_locks failed, ret = %d\n", ret); + pr_err("\n***** End - Test All pHandle Locks ****\n"); + + return 0; +} + +static void omap_hwspinlock_test_exit(void) +{ + pr_err("\n** HwSpinLock Unit Test Module finished **\n"); +} + +module_init(omap_hwspinlock_test_init); +module_exit(omap_hwspinlock_test_exit); + +MODULE_LICENSE("Dual BSD/GPL"); +MODULE_DESCRIPTION("Hardware spinlock Test driver for TI SoCs"); +MODULE_AUTHOR("Suman Anna <s-anna@ti.com>"); diff --git a/meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb b/meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb new file mode 100644 index 00000000..2edd93f6 --- /dev/null +++ b/meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb @@ -0,0 +1,18 @@ +SUMMARY = "Build hwspinlock test as an external Linux kernel module" +DESCRIPTION = "${SUMMARY}" +LICENSE = "GPL-2.0-only | BSD-3-Clause" +LIC_FILES_CHKSUM = "file://LICENSE;md5=bfa02c83df161e37647ee23a2c7eacd4" + +inherit module + +SRC_URI = "file://Makefile \ + file://omap_hwspinlock_test.c \ + file://LICENSE \ + " + +S = "${WORKDIR}" + +# The inherit of module.bbclass will automatically name module packages with +# "kernel-module-" prefix as required by the oe-core build environment. + +RPROVIDES:${PN} += "kernel-module-hwspinlocktest"
Add hwspinlock recipe to build out of tree module hwspinlock test. Signed-off-by: Judith Mendez <jm@ti.com> --- .../recipes-core/packagegroups/ti-test.bb | 1 + .../hwspinlocktest/files/LICENSE | 29 ++ .../hwspinlocktest/files/Makefile | 19 ++ .../files/omap_hwspinlock_test.c | 251 ++++++++++++++++++ .../hwspinlocktest/hwspinlocktest_0.1.bb | 18 ++ 5 files changed, 318 insertions(+) create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/LICENSE create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/Makefile create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/files/omap_hwspinlock_test.c create mode 100644 meta-arago-test/recipes-kernel/hwspinlocktest/hwspinlocktest_0.1.bb