From patchwork Thu Aug 8 16:06:07 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Jos=C3=A9_Luis_Salvador_Rufo?= X-Patchwork-Id: 1970618 Return-Path: X-Original-To: incoming-buildroot@patchwork.ozlabs.org Delivered-To: patchwork-incoming-buildroot@legolas.ozlabs.org Authentication-Results: legolas.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=buildroot.org (client-ip=2605:bc80:3010::136; helo=smtp3.osuosl.org; envelope-from=buildroot-bounces@buildroot.org; receiver=patchwork.ozlabs.org) Received: from smtp3.osuosl.org (smtp3.osuosl.org [IPv6:2605:bc80:3010::136]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (secp384r1) server-digest SHA384) (No client certificate requested) by legolas.ozlabs.org (Postfix) with ESMTPS id 4WfsRq0HfYz1yfC for ; Fri, 9 Aug 2024 02:09:10 +1000 (AEST) Received: from localhost (localhost [127.0.0.1]) by smtp3.osuosl.org (Postfix) with ESMTP id 3D5A660A81; Thu, 8 Aug 2024 16:09:09 +0000 (UTC) X-Virus-Scanned: amavis at osuosl.org Received: from smtp3.osuosl.org ([127.0.0.1]) by localhost (smtp3.osuosl.org [127.0.0.1]) (amavis, port 10024) with ESMTP id JHj0qEveIVW3; Thu, 8 Aug 2024 16:09:07 +0000 (UTC) X-Comment: SPF check N/A for local connections - client-ip=140.211.166.34; helo=ash.osuosl.org; envelope-from=buildroot-bounces@buildroot.org; receiver= DKIM-Filter: OpenDKIM Filter v2.11.0 smtp3.osuosl.org 094CF60AFD Received: from ash.osuosl.org (ash.osuosl.org [140.211.166.34]) by smtp3.osuosl.org (Postfix) with ESMTP id 094CF60AFD; Thu, 8 Aug 2024 16:09:06 +0000 (UTC) X-Original-To: buildroot@lists.busybox.net Delivered-To: buildroot@osuosl.org Received: from smtp4.osuosl.org (smtp4.osuosl.org [140.211.166.137]) by ash.osuosl.org (Postfix) with ESMTP id 5DFA31BF28E for ; Thu, 8 Aug 2024 16:09:04 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp4.osuosl.org (Postfix) with ESMTP id 4A0E6409AE for ; Thu, 8 Aug 2024 16:09:04 +0000 (UTC) X-Virus-Scanned: amavis at osuosl.org Received: from smtp4.osuosl.org ([127.0.0.1]) by localhost (smtp4.osuosl.org [127.0.0.1]) (amavis, port 10024) with ESMTP id LkYRf4uyzaZo for ; Thu, 8 Aug 2024 16:09:02 +0000 (UTC) Received-SPF: Pass (mailfrom) identity=mailfrom; client-ip=2a00:1450:4864:20::335; helo=mail-wm1-x335.google.com; envelope-from=salvador.joseluis@gmail.com; receiver= DMARC-Filter: OpenDMARC Filter v1.4.2 smtp4.osuosl.org 4F8FD409A6 DKIM-Filter: OpenDKIM Filter v2.11.0 smtp4.osuosl.org 4F8FD409A6 Received: from mail-wm1-x335.google.com (mail-wm1-x335.google.com [IPv6:2a00:1450:4864:20::335]) by smtp4.osuosl.org (Postfix) with ESMTPS id 4F8FD409A6 for ; Thu, 8 Aug 2024 16:09:02 +0000 (UTC) Received: by mail-wm1-x335.google.com with SMTP id 5b1f17b1804b1-4281c164408so8281685e9.1 for ; Thu, 08 Aug 2024 09:09:02 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1723133340; x=1723738140; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=i0mgoSOthxgGnUHG1fblqzaiQr92LC+0qpPRCXrMn9U=; b=NVXxq2nXnGug2sPuVX34HfV3l94Yzi1QaNc9kRsGV/2FMz8Cc1+df8nvO6OeWry6cS m61qm06ZzzD0IFynCm7g9iW0m0qyNy66zURWsFBY42/+fhdHpBlvQzkYzDPZNdsDGAMV Ka2i+x1dhkjU66qBigFqXLZNVfaFqDxP4mN77f3V9vMIZhISwqqv+aH5NuLPnOKkJxrm o8H96JWPTsTn43h+T/WnN6dFmATSWWmLPNMmIWy7gPHMy8z++7dqq/r3OWugqCxOz4JP MpzRHWV7Vg17zI+LzgR9TI70YVYMn2DbT0bLQLdFip42Q3YfXZQ8gFFOTHkEEi4zXPuG qAmQ== X-Gm-Message-State: AOJu0YwJWwBQSZkcKm/i/BvFfCViI2p1PNWjecmqtILbmf0WQ+PqfufG I05Iq4WRkfJOTU1l8YJIjN7mT/6thK/3jDNJ861n5Ll65YV21XM+bXV1O0sk X-Google-Smtp-Source: AGHT+IH0hE0ZDvtuenc4OOT7Jm7AKkXlRf5a+tMJRd5RZOGlWCTEHFYwobP5XcXVvD7Usv0ZdaBQig== X-Received: by 2002:a05:600c:458a:b0:428:f41:d467 with SMTP id 5b1f17b1804b1-4290aeb3a84mr17526955e9.10.1723133339639; Thu, 08 Aug 2024 09:08:59 -0700 (PDT) Received: from cuervo.intranet.jlsalvador.online ([78.30.23.54]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-429059cbea0sm81835855e9.43.2024.08.08.09.08.58 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 08 Aug 2024 09:08:59 -0700 (PDT) From: =?utf-8?q?Jos=C3=A9_Luis_Salvador_Rufo?= To: buildroot@buildroot.org Date: Thu, 8 Aug 2024 18:06:07 +0200 Message-ID: <20240808160606.60188-3-salvador.joseluis@gmail.com> X-Mailer: git-send-email 2.46.0 In-Reply-To: <20240807120619.66304f2e@windsurf> References: <20240807120619.66304f2e@windsurf> MIME-Version: 1.0 X-Mailman-Original-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1723133340; x=1723738140; darn=buildroot.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=i0mgoSOthxgGnUHG1fblqzaiQr92LC+0qpPRCXrMn9U=; b=IJrY0Mjg1SHo3JL9pd+Yo8+YVcwdJSKgT8r24cmhggIVh5HVKKfqgTf/cE1ZFkfAlK 1qaQoDe47tuhl1bPoj8Ia11NlxK+sHrJU/fRkNXIZCgEdQ/od0hckpyTNHrCL60r5kPK qIZGbUWM4RrN6ajGUHTV3LbLAraSp4WOZiYJHz+jZDD8Wu0OWmQqR5Mt3ccpV456OQKY azRTOaIgZGwA1pD4wC6e1r5UEvrf62/uSoHqMELwTpJlEUOcpXlEkynhapFGAjpwveCy sjeiqGOvJTApnhxdX9XgJV1s/SU9MTKEecyCmFjLSwKYa9ERpukg+qdAL5dA0ZDQgrmk LBRg== X-Mailman-Original-Authentication-Results: smtp4.osuosl.org; dmarc=pass (p=none dis=none) header.from=gmail.com X-Mailman-Original-Authentication-Results: smtp4.osuosl.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256 header.s=20230601 header.b=IJrY0Mjg Subject: [Buildroot] [PATCH v3 1/2] package/zfs: replace wip patch by upstream patch X-BeenThere: buildroot@buildroot.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Discussion and development of buildroot List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Alexander Mukhin , =?utf-8?q?Jos=C3=A9_Lui?= =?utf-8?q?s_Salvador_Rufo?= , Thomas Petazzoni Errors-To: buildroot-bounces@buildroot.org Sender: "buildroot" Signed-off-by: José Luis Salvador Rufo --- Changes v2 -> v3: - Added Signed-off-by (suggested by Thomas Petazzoni) Changes v1 -> v2: - Removed ZFS_AUTORECONF (suggested by Baruch Siach) ...ll-fix-for-SEEK_DATA-SEEK_HOLE-tests.patch | 70 ++++++++ ...s-ignore-if-SEEK_DATA-is-not-defined.patch | 162 ------------------ package/zfs/zfs.mk | 3 - 3 files changed, 70 insertions(+), 165 deletions(-) create mode 100644 package/zfs/0001-ZTS-small-fix-for-SEEK_DATA-SEEK_HOLE-tests.patch delete mode 100644 package/zfs/0001-tests-cp_files-ignore-if-SEEK_DATA-is-not-defined.patch diff --git a/package/zfs/0001-ZTS-small-fix-for-SEEK_DATA-SEEK_HOLE-tests.patch b/package/zfs/0001-ZTS-small-fix-for-SEEK_DATA-SEEK_HOLE-tests.patch new file mode 100644 index 0000000000..e47079d85e --- /dev/null +++ b/package/zfs/0001-ZTS-small-fix-for-SEEK_DATA-SEEK_HOLE-tests.patch @@ -0,0 +1,70 @@ +From 2ccefd4aff98cf355c7d13b3f92bb4d390dfa522 Mon Sep 17 00:00:00 2001 +From: Tino Reichardt +Date: Sun, 4 Aug 2024 11:58:13 +0200 +Subject: [PATCH] ZTS: small fix for SEEK_DATA/SEEK_HOLE tests + +Some libc's like uClibc lag the proper definition of SEEK_DATA +and SEEK_HOLE. Since we have only two files in ZTS which use +these definitons, let's define them by hand: + +``` +#ifndef SEEK_DATA +#define SEEK_DATA 3 +#endif +#ifndef SEEK_HOLE +#define SEEK_HOLE 4 +#endif +``` + +There should be no failures, because: +- FreeBSD has support for SEEK_DATA/SEEK_HOLE since FreeBSD 8 +- Linux has it since Linux 3.1 +- the libc will submit the parameters unchanged to the kernel + +Signed-off-by: Tino Reichardt +Signed-off-by: José Luis Salvador Rufo +Upstream: https://github.com/openzfs/zfs/pull/16413 +--- + tests/zfs-tests/cmd/mmap_seek.c | 10 ++++++++++ + tests/zfs-tests/tests/functional/cp_files/seekflood.c | 7 +++++++ + 2 files changed, 17 insertions(+) + +diff --git a/tests/zfs-tests/cmd/mmap_seek.c b/tests/zfs-tests/cmd/mmap_seek.c +index 7be92d109565..2d250554a13f 100644 +--- a/tests/zfs-tests/cmd/mmap_seek.c ++++ b/tests/zfs-tests/cmd/mmap_seek.c +@@ -35,6 +35,16 @@ + #include + #endif + ++/* some older uClibc's lack the defines, so we'll manually define them */ ++#ifdef __UCLIBC__ ++#ifndef SEEK_DATA ++#define SEEK_DATA 3 ++#endif ++#ifndef SEEK_HOLE ++#define SEEK_HOLE 4 ++#endif ++#endif ++ + static void + seek_data(int fd, off_t offset, off_t expected) + { +diff --git a/tests/zfs-tests/tests/functional/cp_files/seekflood.c b/tests/zfs-tests/tests/functional/cp_files/seekflood.c +index 02c2c8e6eca5..f832db85970d 100644 +--- a/tests/zfs-tests/tests/functional/cp_files/seekflood.c ++++ b/tests/zfs-tests/tests/functional/cp_files/seekflood.c +@@ -36,6 +36,13 @@ + #include + #include + ++/* some older uClibc's lack the defines, so we'll manually define them */ ++#ifdef __UCLIBC__ ++#ifndef SEEK_DATA ++#define SEEK_DATA 3 ++#endif ++#endif ++ + #define DATASIZE (4096) + char data[DATASIZE]; + diff --git a/package/zfs/0001-tests-cp_files-ignore-if-SEEK_DATA-is-not-defined.patch b/package/zfs/0001-tests-cp_files-ignore-if-SEEK_DATA-is-not-defined.patch deleted file mode 100644 index f47c615d10..0000000000 --- a/package/zfs/0001-tests-cp_files-ignore-if-SEEK_DATA-is-not-defined.patch +++ /dev/null @@ -1,162 +0,0 @@ -From 93e7f8889072047276da11fe6a525d3f0ea16205 Mon Sep 17 00:00:00 2001 -From: =?UTF-8?q?Jos=C3=A9=20Luis=20Salvador=20Rufo?= - -Date: Sat, 11 May 2024 22:40:12 +0200 -Subject: [PATCH] tests/cp_files: ignore if SEEK_DATA is not defined -MIME-Version: 1.0 -Content-Type: text/plain; charset=UTF-8 -Content-Transfer-Encoding: 8bit - -Not all C libraries support SEEK_DATA (e.g., uClibc). Skip the test case -cp_files if SEEK_DATA is not defined. - -Signed-off-by: José Luis Salvador Rufo -Upstream: https://github.com/openzfs/zfs/pull/16169 ---- - config/user-unistd.m4 | 24 ++++++++++++++++++++++++ - config/user.m4 | 1 + - config/zfs-build.m4 | 1 + - tests/test-runner/bin/zts-report.py.in | 1 + - tests/zfs-tests/Makefile.am | 2 ++ - tests/zfs-tests/cmd/Makefile.am | 6 +++++- - tests/zfs-tests/tests/Makefile.am | 14 +++++++++----- - 7 files changed, 43 insertions(+), 6 deletions(-) - create mode 100644 config/user-unistd.m4 - -diff --git a/config/user-unistd.m4 b/config/user-unistd.m4 -new file mode 100644 -index 000000000..302bc0bde ---- /dev/null -+++ b/config/user-unistd.m4 -@@ -0,0 +1,24 @@ -+dnl # -+dnl # Check for SEEK_DATA - only used for cp_files/seekflood test case. -+dnl # -+AC_DEFUN([ZFS_AC_CONFIG_USER_UNISTD_SEEK_DATA], [ -+ AC_MSG_CHECKING(whether host toolchain supports SEEK_DATA) -+ -+ AC_COMPILE_IFELSE([AC_LANG_SOURCE([[ -+ #ifndef _GNU_SOURCE -+ #define _GNU_SOURCE -+ #endif -+ #include -+ #if defined(SEEK_DATA) -+ int ok; -+ #else -+ error fail -+ #endif -+ ]])], [ -+ user_unistd_seek_data=yes -+ AC_MSG_RESULT([yes]) -+ ], [ -+ user_unistd_seek_data=no -+ AC_MSG_RESULT([no]) -+ ]) -+]) -diff --git a/config/user.m4 b/config/user.m4 -index 6ec27a5b2..2326a44be 100644 ---- a/config/user.m4 -+++ b/config/user.m4 -@@ -23,6 +23,7 @@ AC_DEFUN([ZFS_AC_CONFIG_USER], [ - ZFS_AC_CONFIG_USER_LIBAIO - ZFS_AC_CONFIG_USER_LIBATOMIC - ZFS_AC_CONFIG_USER_LIBFETCH -+ ZFS_AC_CONFIG_USER_UNISTD_SEEK_DATA - ZFS_AC_CONFIG_USER_AIO_H - ZFS_AC_CONFIG_USER_CLOCK_GETTIME - ZFS_AC_CONFIG_USER_PAM -diff --git a/config/zfs-build.m4 b/config/zfs-build.m4 -index bb5a85d81..578692676 100644 ---- a/config/zfs-build.m4 -+++ b/config/zfs-build.m4 -@@ -294,6 +294,7 @@ AC_DEFUN([ZFS_AC_CONFIG], [ - [test "x$qatsrc" != x ]) - AM_CONDITIONAL([WANT_DEVNAME2DEVID], [test "x$user_libudev" = xyes ]) - AM_CONDITIONAL([WANT_MMAP_LIBAIO], [test "x$user_libaio" = xyes ]) -+ AM_CONDITIONAL([WANT_UNISTD_SEEK_DATA], [test "x$user_unistd_seek_data" = xyes ]) - AM_CONDITIONAL([PAM_ZFS_ENABLED], [test "x$enable_pam" = xyes]) - ]) - -diff --git a/tests/test-runner/bin/zts-report.py.in b/tests/test-runner/bin/zts-report.py.in -index ecc50f487..3a00345ed 100755 ---- a/tests/test-runner/bin/zts-report.py.in -+++ b/tests/test-runner/bin/zts-report.py.in -@@ -162,6 +162,7 @@ known = { - ['FAIL', rewind_reason], - 'cli_user/misc/zfs_share_001_neg': ['SKIP', na_reason], - 'cli_user/misc/zfs_unshare_001_neg': ['SKIP', na_reason], -+ 'cp_files/cp_stress': ['SKIP', 16169], - 'pool_checkpoint/checkpoint_discard_busy': ['SKIP', 12053], - 'privilege/setup': ['SKIP', na_reason], - 'refreserv/refreserv_004_pos': ['FAIL', known_reason], -diff --git a/tests/zfs-tests/Makefile.am b/tests/zfs-tests/Makefile.am -index 3dd1a6452..8ae790057 100644 ---- a/tests/zfs-tests/Makefile.am -+++ b/tests/zfs-tests/Makefile.am -@@ -13,8 +13,10 @@ scripts_zfs_tests_functional_hkdf_PROGRAMS = %D%/tests/functional/hkdf/hkdf_test - %C%_tests_functional_hkdf_hkdf_test_LDADD = \ - libzpool.la - -+if WANT_UNISTD_SEEK_DATA - scripts_zfs_tests_functional_cp_filesdir = $(datadir)/$(PACKAGE)/zfs-tests/tests/functional/cp_files - scripts_zfs_tests_functional_cp_files_PROGRAMS = %D%/tests/functional/cp_files/seekflood -+endif - - if BUILD_LINUX - scripts_zfs_tests_functional_tmpfiledir = $(datadir)/$(PACKAGE)/zfs-tests/tests/functional/tmpfile -diff --git a/tests/zfs-tests/cmd/Makefile.am b/tests/zfs-tests/cmd/Makefile.am -index 23848a82f..69bba3039 100644 ---- a/tests/zfs-tests/cmd/Makefile.am -+++ b/tests/zfs-tests/cmd/Makefile.am -@@ -5,7 +5,6 @@ scripts_zfs_tests_bin_PROGRAMS = %D%/chg_usr_exec - scripts_zfs_tests_bin_PROGRAMS += %D%/clonefile - scripts_zfs_tests_bin_PROGRAMS += %D%/clone_mmap_cached - scripts_zfs_tests_bin_PROGRAMS += %D%/clone_mmap_write --scripts_zfs_tests_bin_PROGRAMS += %D%/cp_files - scripts_zfs_tests_bin_PROGRAMS += %D%/ctime - scripts_zfs_tests_bin_PROGRAMS += %D%/dir_rd_update - scripts_zfs_tests_bin_PROGRAMS += %D%/dosmode_readonly_write -@@ -16,6 +15,11 @@ scripts_zfs_tests_bin_PROGRAMS += %D%/truncate_test - scripts_zfs_tests_bin_PROGRAMS += %D%/zfs_diff-socket - - -+if WANT_UNISTD_SEEK_DATA -+scripts_zfs_tests_bin_PROGRAMS += %D%/cp_files -+endif -+ -+ - scripts_zfs_tests_bin_PROGRAMS += %D%/badsend - %C%_badsend_LDADD = \ - libzfs_core.la \ -diff --git a/tests/zfs-tests/tests/Makefile.am b/tests/zfs-tests/tests/Makefile.am -index cc66d762f..1e9f06a7e 100644 ---- a/tests/zfs-tests/tests/Makefile.am -+++ b/tests/zfs-tests/tests/Makefile.am -@@ -1394,11 +1394,6 @@ nobase_dist_datadir_zfs_tests_tests_SCRIPTS += \ - functional/compression/l2arc_encrypted.ksh \ - functional/compression/l2arc_encrypted_no_compressed_arc.ksh \ - functional/compression/setup.ksh \ -- functional/cp_files/cleanup.ksh \ -- functional/cp_files/cp_files_001_pos.ksh \ -- functional/cp_files/cp_files_002_pos.ksh \ -- functional/cp_files/cp_stress.ksh \ -- functional/cp_files/setup.ksh \ - functional/crtime/cleanup.ksh \ - functional/crtime/crtime_001_pos.ksh \ - functional/crtime/setup.ksh \ -@@ -2108,3 +2103,12 @@ nobase_dist_datadir_zfs_tests_tests_SCRIPTS += \ - functional/idmap_mount/idmap_mount_003.ksh \ - functional/idmap_mount/idmap_mount_004.ksh \ - functional/idmap_mount/idmap_mount_005.ksh -+ -+if WANT_UNISTD_SEEK_DATA -+nobase_dist_datadir_zfs_tests_tests_SCRIPTS += \ -+ functional/cp_files/cleanup.ksh \ -+ functional/cp_files/cp_files_001_pos.ksh \ -+ functional/cp_files/cp_files_002_pos.ksh \ -+ functional/cp_files/cp_stress.ksh \ -+ functional/cp_files/setup.ksh -+endif --- -2.45.2 - diff --git a/package/zfs/zfs.mk b/package/zfs/zfs.mk index e0fa392670..e51ade55a4 100644 --- a/package/zfs/zfs.mk +++ b/package/zfs/zfs.mk @@ -12,9 +12,6 @@ ZFS_LICENSE_FILES = LICENSE COPYRIGHT ZFS_CPE_ID_VENDOR = openzfs ZFS_CPE_ID_PRODUCT = openzfs -# 0001-tests-cp_files-ignore-if-SEEK_DATA-is-not-defined.patch -ZFS_AUTORECONF = YES - ZFS_DEPENDENCIES = libaio openssl udev util-linux zlib libcurl linux # sysvinit installs only a commented-out modules-load.d/ config file