Re: [PATCH v4] Add recipe for linux kselftest

["vijaikumar....@gmail.com" <vijaikumar.kanagarajan@gmail.com>]

[-- Attachment #1.1: Type: text/plain, Size: 7410 bytes --]



On Thursday, November 5, 2020 at 2:14:50 AM UTC+5:30 Henning Schild wrote:

> Did not follow the thread and coming late to the table. Did anyone 
> consider pushing this upstream into the kernel? 
>

No. Atleast, I did not try it.
 

>
> For the kernel itself we eventually failed. But because it turned out 
> to be too complicated to find a generic debianization that fits all the 
> needs. For a tool it might be a different story.   


> Henning 
>
> Am Mon, 2 Nov 2020 12:55:11 +0530 
> schrieb Vijai Kumar K <Vijaikumar_...@mentor.com>: 
>
> > Kselftest is a set of unit and regression tests under the 
> > tools/testing/selftests/ directory of the Linux kernel. These are used 
> > for testing individual code paths in the kernel. 
> > 
> > Kselftest is not debianized and is not available as part of the 
> > official Debian repository. 
> > 
> > This patch provides a recipe to build kselftest from the linux source 
> > of your choice. 
> > 
> > Sometimes developers/testers tend to use the latest version of 
> > selftests from Linux upstream stable releases for testing and hence 
> > the need for a separate recipe instead of being part of the linux 
> > recipe of ISAR. 
> > 
> > Some kselftest test cases build are known to break in certain linux 
> > versions, this has to be addressed upstream. Right now kselftest build 
> > succeeds even if atleast one of the testcase case build succeeds. 
> > 
> > You can change that by setting KSELFTEST_FORCE_TARGETS=1. 
> > 
> > TARGETS, TARGETS_SKIP, FORCE_TARGETS can be set using the appropriate 
> > KSELFTEST_* recipe variable. 
> > 
> > Signed-off-by: Vijai Kumar K <Vijaikumar_...@mentor.com> 
> > --- 
> > Changes since v3: 
> > - Remove libnuma-dev. Since it is not available in stretch. 
> > - remove entry in isar-image-debug. Added to CI image using 
> > IMAGE_INSTALL. 
> > 
> > Changes since v2: 
> > - Use deb_debianize instead of individual calls for changelog, compat 
> > & control. 
> > - Add package to isar-image-debug 
> > 
> > Changes since v1: 
> > - Addressed review comments in v1 
> > - Used debianize APIs for control, compat and changelog. 
> > - Moved Depends & S definitions to include file. 
> > - Added cross compilation bits in rules. 
> > - Change clang, llvm to clang:native & llvm:native 
> > 
> > .../kselftest/kselftest_5.6.13.bb | 11 ++++ 
> > meta/recipes-kernel/kselftest/files/rules | 41 +++++++++++++ 
> > meta/recipes-kernel/kselftest/kselftest.inc | 58 
> > +++++++++++++++++++ scripts/ci_build.sh | 
> > 1 + 4 files changed, 111 insertions(+) 
> > create mode 100644 
> > meta-isar/recipes-kernel/kselftest/kselftest_5.6.13.bb create mode 
> > 100755 meta/recipes-kernel/kselftest/files/rules create mode 100644 
> > meta/recipes-kernel/kselftest/kselftest.inc 
> > 
> > diff --git a/meta-isar/recipes-kernel/kselftest/kselftest_5.6.13.bb 
> > b/meta-isar/recipes-kernel/kselftest/kselftest_5.6.13.bb new file 
> > mode 100644 index 0000000..0c9818a 
> > --- /dev/null 
> > +++ b/meta-isar/recipes-kernel/kselftest/kselftest_5.6.13.bb 
> > @@ -0,0 +1,11 @@ 
> > +# Kselftest package 
> > +# 
> > +# This software is a part of ISAR. 
> > +# Copyright (c) Mentor Graphics, a Siemens business, 2020 
> > +# 
> > +# SPDX-License-Identifier: MIT 
> > + 
> > +require recipes-kernel/kselftest/kselftest.inc 
> > + 
> > +SRC_URI += 
> > "https://cdn.kernel.org/pub/linux/kernel/v5.x/linux-${PV}.tar.xz" 
> > +SRC_URI[sha256sum] = 
> > "f125d79c8f6974213638787adcad6b575bbd35a05851802fd83f622ec18ff987" 
> > diff --git a/meta/recipes-kernel/kselftest/files/rules 
> > b/meta/recipes-kernel/kselftest/files/rules new file mode 100755 
> > index 0000000..776d341 --- /dev/null +++ 
> > b/meta/recipes-kernel/kselftest/files/rules @@ -0,0 +1,41 @@ 
> > +#!/usr/bin/make -f 
> > + 
> > +# Debian rules for Kselftest build 
> > +# 
> > +# This software is a part of ISAR. 
> > +# Copyright (c) Mentor, A Siemens Business 
> > +# 
> > +# SPDX-License-Identifier: MIT 
> > + 
> > +deb_top_dir:=$(CURDIR)/debian 
> > +kselftest_install_dir 
> > :=$(deb_top_dir)/kselftest/usr/libexec/kselftest +export 
> > CROSS_COMPILE=$(DEB_HOST_GNU_TYPE)- + 
> > +ifeq ($(DEB_HOST_GNU_CPU), arm) 
> > +export ARCH=arm 
> > +endif 
> > +ifeq ($(DEB_HOST_GNU_CPU), aarch64) 
> > +export ARCH=arm64 
> > +endif 
> > +ifeq ($(DEB_HOST_GNU_CPU), riscv64) 
> > +export ARCH=riscv 
> > +endif 
> > +ifneq (,$(findstring 86,$(DEB_HOST_GNU_CPU))) 
> > +export ARCH=x86 
> > +endif 
> > + 
> > +PARALLEL_BUILD=-j $(shell echo $$(($$(nproc) * 2))) 
> > + 
> > +override_dh_auto_build: 
> > + $(MAKE) defconfig 
> > + $(MAKE) $(PARALLEL_BUILD) -C tools/testing/selftests/ 
> > ${KSELFTEST_ARGS} + 
> > +override_dh_auto_install: 
> > + mkdir -p ${kselftest_install_dir} 
> > + cd tools/testing/selftests/ && 
> > KSFT_INSTALL_PATH=${kselftest_install_dir} $(MAKE) $(PARALLEL_BUILD) 
> > install ${KSELFTEST_ARGS} + +override_dh_auto_clean: 
> > + rm -rf ${deb_top_dir}/kselftest/ 
> > + 
> > +%: 
> > + CFLAGS= LDFLAGS= dh $@ --parallel 
> > diff --git a/meta/recipes-kernel/kselftest/kselftest.inc 
> > b/meta/recipes-kernel/kselftest/kselftest.inc new file mode 100644 
> > index 0000000..2a76028 
> > --- /dev/null 
> > +++ b/meta/recipes-kernel/kselftest/kselftest.inc 
> > @@ -0,0 +1,58 @@ 
> > +# Kselftest package for Linux 
> > +# 
> > +# This software is a part of ISAR. 
> > +# Copyright (c) Mentor Graphics, a Siemens Business, 2020 
> > +# 
> > +# SPDX-License-Identifier: MIT 
> > + 
> > +inherit dpkg 
> > + 
> > +FILESEXTRAPATHS_prepend := "${THISDIR}/files:" 
> > + 
> > +# Some test-cases like memfd compilation failed in cross-compilation. 
> > +# Force native compilation for now to have consistent output across 
> > +# ISAR native/cross compilation setups. 
> > +ISAR_CROSS_COMPILE = "0" 
> > + 
> > +DESCRIPTION ?= "Kernel selftests from Linux kernel ${PV}" 
> > + 
> > +DEBIAN_BUILD_DEPENDS ?= " \ 
> > + rsync, \ 
> > + flex, \ 
> > + bison, \ 
> > + fuse, \ 
> > + libelf-dev:native, \ 
> > + libcap-ng-dev:native, \ 
> > + libpopt-dev:native, \ 
> > + libcap-dev:native, \ 
> > + libmount-dev:native, \ 
> > + libfuse-dev:native, \ 
> > + libmnl-dev:native, \ 
> > + pkg-config, \ 
> > + clang:native, \ 
> > + llvm:native, \ 
> > +" 
> > + 
> > +SRC_URI += "file://rules" 
> > +S = "${WORKDIR}/linux-${PV}" 
> > + 
> > +KSELFTEST_TARGETS ?= "" 
> > +KSELFTEST_SKIP_TARGETS ?= "" 
> > +KSELFTEST_FORCE_TARGETS ?= "0" 
> > + 
> > +do_prepare_build[cleandirs] += "${S}/debian" 
> > +do_prepare_build() { 
> > + deb_debianize 
> > +} 
> > + 
> > +dpkg_runbuild_prepend() { 
> > + if [ -n "${KSELFTEST_TARGETS}" ];then 
> > + export KSELFTEST_ARGS="TARGETS=\"${KSELFTEST_TARGETS}\"" 
> > + fi 
> > + if [ ${KSELFTEST_FORCE_TARGETS} -eq 1 ];then 
> > + export KSELFTEST_ARGS="${KSELFTEST_ARGS} FORCE_TARGETS=1" 
> > + fi 
> > + if [ -n "${KSELFTEST_SKIP_TARGETS}" ];then 
> > + export KSELFTEST_ARGS="${KSELFTEST_ARGS} 
> > SKIP_TARGETS=\"${KSELFTEST_SKIP_TARGETS}\"" 
> > + fi 
> > +} 
> > diff --git a/scripts/ci_build.sh b/scripts/ci_build.sh 
> > index c0a032e..2d06543 100755 
> > --- a/scripts/ci_build.sh 
> > +++ b/scripts/ci_build.sh 
> > @@ -143,6 +143,7 @@ cat >>conf/local.conf <<EOF 
> > ISAR_ENABLE_COMPAT_ARCH_amd64 = "1" 
> > ISAR_ENABLE_COMPAT_ARCH_arm64 = "1" 
> > ISAR_ENABLE_COMPAT_ARCH_debian-stretch_amd64 = "0" 
> > +IMAGE_INSTALL += "kselftest" 
> > EOF 
> > 
> > if [ -n "$CROSS_BUILD" ]; then 
>
>

[-- Attachment #1.2: Type: text/html, Size: 10597 bytes --]