* [PATCH] stress-ng: upgrade to 0.09.09 and fix musl build
@ 2018-01-10 4:14 Tim Orling
2018-01-10 4:16 ` Tim Orling
0 siblings, 1 reply; 3+ messages in thread
From: Tim Orling @ 2018-01-10 4:14 UTC (permalink / raw)
To: openembedded-devel
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=y, Size: 9350 bytes --]
* Upstream tarballs are now xz compressed
* Patch 0001 modified:
- stress-pthread and stress-context change not needed
* Add patch to fix stress-fcntl for musl
* Add patch to fix syntax errors in stress-fcntl
- Accepted upstream
* Add UPSTREAM_CHECK_URI and UPSTREAM_CHECK_REGEX
Signed-off-by: Tim Orling <timothy.t.orling@linux.intel.com>
---
.../0001-Several-changes-to-fix-musl-build.patch | 31 +-------------
.../0002-stress-fcntl-fix-build-for-musl.patch | 27 +++++++++++++
.../0003-stress-fcntl.c-fix-syntax-errors.patch | 47 ++++++++++++++++++++++
.../recipes-test/stress-ng/stress-ng_0.08.17.bb | 20 ---------
.../recipes-test/stress-ng/stress-ng_0.09.09.bb | 26 ++++++++++++
5 files changed, 102 insertions(+), 49 deletions(-)
create mode 100644 meta-oe/recipes-test/stress-ng/stress-ng/0002-stress-fcntl-fix-build-for-musl.patch
create mode 100644 meta-oe/recipes-test/stress-ng/stress-ng/0003-stress-fcntl.c-fix-syntax-errors.patch
delete mode 100644 meta-oe/recipes-test/stress-ng/stress-ng_0.08.17.bb
create mode 100644 meta-oe/recipes-test/stress-ng/stress-ng_0.09.09.bb
diff --git a/meta-oe/recipes-test/stress-ng/stress-ng/0001-Several-changes-to-fix-musl-build.patch b/meta-oe/recipes-test/stress-ng/stress-ng/0001-Several-changes-to-fix-musl-build.patch
index 3ed7efc7e..f90e704f0 100644
--- a/meta-oe/recipes-test/stress-ng/stress-ng/0001-Several-changes-to-fix-musl-build.patch
+++ b/meta-oe/recipes-test/stress-ng/stress-ng/0001-Several-changes-to-fix-musl-build.patch
@@ -11,7 +11,6 @@ swapcontext, musl provide the definition but not the implementation due
to that functions are pre-POSIX and set to be deprecated.
stress-{resources, pty}.c: Doesn't include termio.h and remove stress
operations that uses struct termio, musl doesn't provide that struct.
-stress-pthread.c: Change pthread_yield to sched_yield to be more compatible.
stress-malloc.c: Check for definition of M_MMAP_THRESHOLD musl doesn't
ptovide that constant.
stress-madvise.c: Add static poision_count integer, definition of
@@ -19,13 +18,13 @@ stress-madvise.c: Add static poision_count integer, definition of
cache.c: Define GLOB_ONLYDIR not available on MUSL.
Signed-off-by: Aníbal Limón <anibal.limon@linaro.org>
+Signed-off-by: Tim Orling <timothy.t.orling@linux.intel.com>
+
Upstream-status: Pending
---
cache.c | 4 ++++
- stress-context.c | 2 +-
stress-madvise.c | 1 +
stress-malloc.c | 2 +-
- stress-pthread.c | 2 +-
stress-pty.c | 18 ------------------
stress-resources.c | 1 -
stress-stackmmap.c | 2 +-
@@ -46,19 +45,6 @@ index e01fad0..27870ee 100644
#if defined(__linux__)
#define SYS_CPU_PREFIX "/sys/devices/system/cpu"
#define GLOB_PATTERN SYS_CPU_PREFIX "/cpu[0-9]*"
-diff --git a/stress-context.c b/stress-context.c
-index c5f50ed..67209e5 100644
---- a/stress-context.c
-+++ b/stress-context.c
-@@ -24,7 +24,7 @@
- */
- #include "stress-ng.h"
-
--#if !defined(__OpenBSD__)
-+#if !defined(__OpenBSD__) && False
-
- #include <ucontext.h>
-
diff --git a/stress-madvise.c b/stress-madvise.c
index 43ea454..2e1f8ee 100644
--- a/stress-madvise.c
@@ -84,19 +70,6 @@ index 8dbe5cc..f505f32 100644
if (get_setting("malloc-threshold", &malloc_threshold))
(void)mallopt(M_MMAP_THRESHOLD, (int)malloc_threshold);
#endif
-diff --git a/stress-pthread.c b/stress-pthread.c
-index 323a1d4..8269f4c 100644
---- a/stress-pthread.c
-+++ b/stress-pthread.c
-@@ -139,7 +139,7 @@ static void *stress_pthread_func(void *parg)
- break;
- }
- #if !defined(__NetBSD__) && !defined(__sun__)
-- (void)pthread_yield();
-+ (void)sched_yield();
- #endif
- }
- ret = pthread_mutex_unlock(&mutex);
diff --git a/stress-pty.c b/stress-pty.c
index 8c3edf8..518f118 100644
--- a/stress-pty.c
diff --git a/meta-oe/recipes-test/stress-ng/stress-ng/0002-stress-fcntl-fix-build-for-musl.patch b/meta-oe/recipes-test/stress-ng/stress-ng/0002-stress-fcntl-fix-build-for-musl.patch
new file mode 100644
index 000000000..826340169
--- /dev/null
+++ b/meta-oe/recipes-test/stress-ng/stress-ng/0002-stress-fcntl-fix-build-for-musl.patch
@@ -0,0 +1,27 @@
+From 7be4680930e2867cefd5c8fe27a4ae81a4ae2628 Mon Sep 17 00:00:00 2001
+From: Tim Orling <timothy.t.orling@linux.intel.com>
+Date: Tue, 9 Jan 2018 18:38:45 -0800
+Subject: [PATCH] stress-fcntl: fix build for musl
+
+Signed-off-by: Tim Orling <timothy.t.orling@linux.intel.com>
+
+Upstream-status: Pending
+---
+ stress-fcntl.c | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/stress-fcntl.c b/stress-fcntl.c
+index b7eade2..9fa5c38 100644
+--- a/stress-fcntl.c
++++ b/stress-fcntl.c
+@@ -413,6 +413,7 @@ ofd_lock_abort: { /* Nowt */ }
+
+ #if defined(F_GET_FILE_RW_HINT) && defined(F_SET_FILE_RW_HINT)
+ {
++ int ret;
+ size_t i;
+ unsigned long hint;
+ static const unsigned long hints[] = {
+--
+2.14.3
+
diff --git a/meta-oe/recipes-test/stress-ng/stress-ng/0003-stress-fcntl.c-fix-syntax-errors.patch b/meta-oe/recipes-test/stress-ng/stress-ng/0003-stress-fcntl.c-fix-syntax-errors.patch
new file mode 100644
index 000000000..4b6a9b391
--- /dev/null
+++ b/meta-oe/recipes-test/stress-ng/stress-ng/0003-stress-fcntl.c-fix-syntax-errors.patch
@@ -0,0 +1,47 @@
+From 8537dcafb12be4d986b571f1e1d9f3ceecdc8e63 Mon Sep 17 00:00:00 2001
+From: Tim Orling <timothy.t.orling@linux.intel.com>
+Date: Sun, 7 Jan 2018 21:29:12 -0800
+Subject: [PATCH] stress-fcntl.c: fix syntax errors
+
+Signed-off-by: Tim Orling <timothy.t.orling@linux.intel.com>
+
+Upstream-status: Accepted
+(commit 61d93d6857256cd49df6a6302caf6e7bb69b0f46)
+---
+ stress-fcntl.c | 6 +++---
+ 1 file changed, 3 insertions(+), 3 deletions(-)
+
+diff --git a/stress-fcntl.c b/stress-fcntl.c
+index 7c15d28..b7eade2 100644
+--- a/stress-fcntl.c
++++ b/stress-fcntl.c
+@@ -173,7 +173,7 @@ static int do_fcntl(const args_t *args, const int fd)
+ owner.pid = getpgrp();
+ ret = fcntl(fd, F_SETOWN_EX, &owner);
+ check_return(args, ret, "F_SETOWN_EX, F_OWNER_PGRP");
+-#else if defined(HAVE_GETPGRP) && defined(F_OWNER_GID)
++#elif defined(HAVE_GETPGRP) && defined(F_OWNER_GID)
+ owner.type = F_OWNER_GID;
+ owner.pid = getpgrp();
+ ret = fcntl(fd, F_SETOWN_EX, &owner);
+@@ -202,7 +202,7 @@ static int do_fcntl(const args_t *args, const int fd)
+ owner.type = F_OWNER_PGRP;
+ ret = fcntl(fd, F_GETOWN_EX, &owner);
+ check_return(args, ret, "F_GETOWN_EX, F_OWNER_PGRP");
+-#ele if defined(F_OWNER_GID)
++#elif defined(F_OWNER_GID)
+ owner.type = F_OWNER_GID;
+ ret = fcntl(fd, F_GETOWN_EX, &owner);
+ check_return(args, ret, "F_GETOWN_EX, F_OWNER_GID");
+@@ -436,7 +436,7 @@ ofd_lock_abort: { /* Nowt */ }
+ #endif
+ };
+
+- ret = fcntl(fd, F_GET_FILE_RW_HINT, &hint)
++ ret = fcntl(fd, F_GET_FILE_RW_HINT, &hint);
+ if (ret == 0) {
+ for (i = 0; i < SIZEOF_ARRAY(hints); i++) {
+ hint = hints[i];
+--
+2.14.3
+
diff --git a/meta-oe/recipes-test/stress-ng/stress-ng_0.08.17.bb b/meta-oe/recipes-test/stress-ng/stress-ng_0.08.17.bb
deleted file mode 100644
index dc5b2e8eb..000000000
--- a/meta-oe/recipes-test/stress-ng/stress-ng_0.08.17.bb
+++ /dev/null
@@ -1,20 +0,0 @@
-SUMMARY = "A tool to load and stress a computer system"
-HOMEPAGE = "http://kernel.ubuntu.com/~cking/stress-ng/"
-LICENSE = "GPLv2"
-LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
-
-DEPENDS = "zlib libaio"
-
-SRC_URI = "http://kernel.ubuntu.com/~cking/tarballs/${BPN}/${BP}.tar.gz \
- "
-SRC_URI_append_libc-musl = "file://0001-Several-changes-to-fix-musl-build.patch"
-
-SRC_URI[md5sum] = "e0f6497a8c06f5d652bc2ad88d449c12"
-SRC_URI[sha256sum] = "37cc73e42f5bdb0e0571ba88f6a69b8f05ee28e51afcafc2231c1058b1a5dd18"
-
-CFLAGS += "-Wall -Wextra -DVERSION='"$(VERSION)"'"
-
-do_install_append() {
- install -d ${D}${bindir}
- install -m 755 ${S}/stress-ng ${D}${bindir}/stress-ng
-}
diff --git a/meta-oe/recipes-test/stress-ng/stress-ng_0.09.09.bb b/meta-oe/recipes-test/stress-ng/stress-ng_0.09.09.bb
new file mode 100644
index 000000000..24f55ada2
--- /dev/null
+++ b/meta-oe/recipes-test/stress-ng/stress-ng_0.09.09.bb
@@ -0,0 +1,26 @@
+SUMMARY = "A tool to load and stress a computer system"
+HOMEPAGE = "http://kernel.ubuntu.com/~cking/stress-ng/"
+LICENSE = "GPLv2"
+LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
+
+DEPENDS = "zlib libaio"
+
+SRC_URI = "http://kernel.ubuntu.com/~cking/tarballs/${BPN}/${BP}.tar.xz \
+ "
+SRC_URI_append_libc-musl = " \
+ file://0001-Several-changes-to-fix-musl-build.patch \
+ file://0002-stress-fcntl-fix-build-for-musl.patch \
+ file://0003-stress-fcntl.c-fix-syntax-errors.patch"
+
+SRC_URI[md5sum] = "6e4df71095e780654ccc0a713b8423fe"
+SRC_URI[sha256sum] = "e993fddb1d990a0cdb23b2af6cf6c417a1c09e155ada7c558ad80eae8a5feed3"
+
+UPSTREAM_CHECK_URI ?= "http://kernel.ubuntu.com/~cking/tarballs/${BPN}/"
+UPSTREAM_CHECK_REGEX ?= "(?P<pver>\d+(\.\d+)+)\.tar"
+
+CFLAGS += "-Wall -Wextra -DVERSION='"$(VERSION)"'"
+
+do_install_append() {
+ install -d ${D}${bindir}
+ install -m 755 ${S}/stress-ng ${D}${bindir}/stress-ng
+}
--
2.14.3
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] stress-ng: upgrade to 0.09.09 and fix musl build
2018-01-10 4:14 [PATCH] stress-ng: upgrade to 0.09.09 and fix musl build Tim Orling
@ 2018-01-10 4:16 ` Tim Orling
2018-01-10 5:11 ` Denys Dmytriyenko
0 siblings, 1 reply; 3+ messages in thread
From: Tim Orling @ 2018-01-10 4:16 UTC (permalink / raw)
To: openembeded-devel
Meh. wrong subject line resending. Please ignore the noise.
> On Jan 9, 2018, at 8:14 PM, Tim Orling <timothy.t.orling@linux.intel.com> wrote:
>
> * Upstream tarballs are now xz compressed
> * Patch 0001 modified:
> - stress-pthread and stress-context change not needed
> * Add patch to fix stress-fcntl for musl
> * Add patch to fix syntax errors in stress-fcntl
> - Accepted upstream
> * Add UPSTREAM_CHECK_URI and UPSTREAM_CHECK_REGEX
>
> Signed-off-by: Tim Orling <timothy.t.orling@linux.intel.com>
> ---
> .../0001-Several-changes-to-fix-musl-build.patch | 31 +-------------
> .../0002-stress-fcntl-fix-build-for-musl.patch | 27 +++++++++++++
> .../0003-stress-fcntl.c-fix-syntax-errors.patch | 47 ++++++++++++++++++++++
> .../recipes-test/stress-ng/stress-ng_0.08.17.bb | 20 ---------
> .../recipes-test/stress-ng/stress-ng_0.09.09.bb | 26 ++++++++++++
> 5 files changed, 102 insertions(+), 49 deletions(-)
> create mode 100644 meta-oe/recipes-test/stress-ng/stress-ng/0002-stress-fcntl-fix-build-for-musl.patch
> create mode 100644 meta-oe/recipes-test/stress-ng/stress-ng/0003-stress-fcntl.c-fix-syntax-errors.patch
> delete mode 100644 meta-oe/recipes-test/stress-ng/stress-ng_0.08.17.bb
> create mode 100644 meta-oe/recipes-test/stress-ng/stress-ng_0.09.09.bb
>
> diff --git a/meta-oe/recipes-test/stress-ng/stress-ng/0001-Several-changes-to-fix-musl-build.patch b/meta-oe/recipes-test/stress-ng/stress-ng/0001-Several-changes-to-fix-musl-build.patch
> index 3ed7efc7e..f90e704f0 100644
> --- a/meta-oe/recipes-test/stress-ng/stress-ng/0001-Several-changes-to-fix-musl-build.patch
> +++ b/meta-oe/recipes-test/stress-ng/stress-ng/0001-Several-changes-to-fix-musl-build.patch
> @@ -11,7 +11,6 @@ swapcontext, musl provide the definition but not the implementation due
> to that functions are pre-POSIX and set to be deprecated.
> stress-{resources, pty}.c: Doesn't include termio.h and remove stress
> operations that uses struct termio, musl doesn't provide that struct.
> -stress-pthread.c: Change pthread_yield to sched_yield to be more compatible.
> stress-malloc.c: Check for definition of M_MMAP_THRESHOLD musl doesn't
> ptovide that constant.
> stress-madvise.c: Add static poision_count integer, definition of
> @@ -19,13 +18,13 @@ stress-madvise.c: Add static poision_count integer, definition of
> cache.c: Define GLOB_ONLYDIR not available on MUSL.
>
> Signed-off-by: Aníbal Limón <anibal.limon@linaro.org>
> +Signed-off-by: Tim Orling <timothy.t.orling@linux.intel.com>
> +
> Upstream-status: Pending
> ---
> cache.c | 4 ++++
> - stress-context.c | 2 +-
> stress-madvise.c | 1 +
> stress-malloc.c | 2 +-
> - stress-pthread.c | 2 +-
> stress-pty.c | 18 ------------------
> stress-resources.c | 1 -
> stress-stackmmap.c | 2 +-
> @@ -46,19 +45,6 @@ index e01fad0..27870ee 100644
> #if defined(__linux__)
> #define SYS_CPU_PREFIX "/sys/devices/system/cpu"
> #define GLOB_PATTERN SYS_CPU_PREFIX "/cpu[0-9]*"
> -diff --git a/stress-context.c b/stress-context.c
> -index c5f50ed..67209e5 100644
> ---- a/stress-context.c
> -+++ b/stress-context.c
> -@@ -24,7 +24,7 @@
> - */
> - #include "stress-ng.h"
> -
> --#if !defined(__OpenBSD__)
> -+#if !defined(__OpenBSD__) && False
> -
> - #include <ucontext.h>
> -
> diff --git a/stress-madvise.c b/stress-madvise.c
> index 43ea454..2e1f8ee 100644
> --- a/stress-madvise.c
> @@ -84,19 +70,6 @@ index 8dbe5cc..f505f32 100644
> if (get_setting("malloc-threshold", &malloc_threshold))
> (void)mallopt(M_MMAP_THRESHOLD, (int)malloc_threshold);
> #endif
> -diff --git a/stress-pthread.c b/stress-pthread.c
> -index 323a1d4..8269f4c 100644
> ---- a/stress-pthread.c
> -+++ b/stress-pthread.c
> -@@ -139,7 +139,7 @@ static void *stress_pthread_func(void *parg)
> - break;
> - }
> - #if !defined(__NetBSD__) && !defined(__sun__)
> -- (void)pthread_yield();
> -+ (void)sched_yield();
> - #endif
> - }
> - ret = pthread_mutex_unlock(&mutex);
> diff --git a/stress-pty.c b/stress-pty.c
> index 8c3edf8..518f118 100644
> --- a/stress-pty.c
> diff --git a/meta-oe/recipes-test/stress-ng/stress-ng/0002-stress-fcntl-fix-build-for-musl.patch b/meta-oe/recipes-test/stress-ng/stress-ng/0002-stress-fcntl-fix-build-for-musl.patch
> new file mode 100644
> index 000000000..826340169
> --- /dev/null
> +++ b/meta-oe/recipes-test/stress-ng/stress-ng/0002-stress-fcntl-fix-build-for-musl.patch
> @@ -0,0 +1,27 @@
> +From 7be4680930e2867cefd5c8fe27a4ae81a4ae2628 Mon Sep 17 00:00:00 2001
> +From: Tim Orling <timothy.t.orling@linux.intel.com>
> +Date: Tue, 9 Jan 2018 18:38:45 -0800
> +Subject: [PATCH] stress-fcntl: fix build for musl
> +
> +Signed-off-by: Tim Orling <timothy.t.orling@linux.intel.com>
> +
> +Upstream-status: Pending
> +---
> + stress-fcntl.c | 1 +
> + 1 file changed, 1 insertion(+)
> +
> +diff --git a/stress-fcntl.c b/stress-fcntl.c
> +index b7eade2..9fa5c38 100644
> +--- a/stress-fcntl.c
> ++++ b/stress-fcntl.c
> +@@ -413,6 +413,7 @@ ofd_lock_abort: { /* Nowt */ }
> +
> + #if defined(F_GET_FILE_RW_HINT) && defined(F_SET_FILE_RW_HINT)
> + {
> ++ int ret;
> + size_t i;
> + unsigned long hint;
> + static const unsigned long hints[] = {
> +--
> +2.14.3
> +
> diff --git a/meta-oe/recipes-test/stress-ng/stress-ng/0003-stress-fcntl.c-fix-syntax-errors.patch b/meta-oe/recipes-test/stress-ng/stress-ng/0003-stress-fcntl.c-fix-syntax-errors.patch
> new file mode 100644
> index 000000000..4b6a9b391
> --- /dev/null
> +++ b/meta-oe/recipes-test/stress-ng/stress-ng/0003-stress-fcntl.c-fix-syntax-errors.patch
> @@ -0,0 +1,47 @@
> +From 8537dcafb12be4d986b571f1e1d9f3ceecdc8e63 Mon Sep 17 00:00:00 2001
> +From: Tim Orling <timothy.t.orling@linux.intel.com>
> +Date: Sun, 7 Jan 2018 21:29:12 -0800
> +Subject: [PATCH] stress-fcntl.c: fix syntax errors
> +
> +Signed-off-by: Tim Orling <timothy.t.orling@linux.intel.com>
> +
> +Upstream-status: Accepted
> +(commit 61d93d6857256cd49df6a6302caf6e7bb69b0f46)
> +---
> + stress-fcntl.c | 6 +++---
> + 1 file changed, 3 insertions(+), 3 deletions(-)
> +
> +diff --git a/stress-fcntl.c b/stress-fcntl.c
> +index 7c15d28..b7eade2 100644
> +--- a/stress-fcntl.c
> ++++ b/stress-fcntl.c
> +@@ -173,7 +173,7 @@ static int do_fcntl(const args_t *args, const int fd)
> + owner.pid = getpgrp();
> + ret = fcntl(fd, F_SETOWN_EX, &owner);
> + check_return(args, ret, "F_SETOWN_EX, F_OWNER_PGRP");
> +-#else if defined(HAVE_GETPGRP) && defined(F_OWNER_GID)
> ++#elif defined(HAVE_GETPGRP) && defined(F_OWNER_GID)
> + owner.type = F_OWNER_GID;
> + owner.pid = getpgrp();
> + ret = fcntl(fd, F_SETOWN_EX, &owner);
> +@@ -202,7 +202,7 @@ static int do_fcntl(const args_t *args, const int fd)
> + owner.type = F_OWNER_PGRP;
> + ret = fcntl(fd, F_GETOWN_EX, &owner);
> + check_return(args, ret, "F_GETOWN_EX, F_OWNER_PGRP");
> +-#ele if defined(F_OWNER_GID)
> ++#elif defined(F_OWNER_GID)
> + owner.type = F_OWNER_GID;
> + ret = fcntl(fd, F_GETOWN_EX, &owner);
> + check_return(args, ret, "F_GETOWN_EX, F_OWNER_GID");
> +@@ -436,7 +436,7 @@ ofd_lock_abort: { /* Nowt */ }
> + #endif
> + };
> +
> +- ret = fcntl(fd, F_GET_FILE_RW_HINT, &hint)
> ++ ret = fcntl(fd, F_GET_FILE_RW_HINT, &hint);
> + if (ret == 0) {
> + for (i = 0; i < SIZEOF_ARRAY(hints); i++) {
> + hint = hints[i];
> +--
> +2.14.3
> +
> diff --git a/meta-oe/recipes-test/stress-ng/stress-ng_0.08.17.bb b/meta-oe/recipes-test/stress-ng/stress-ng_0.08.17.bb
> deleted file mode 100644
> index dc5b2e8eb..000000000
> --- a/meta-oe/recipes-test/stress-ng/stress-ng_0.08.17.bb
> +++ /dev/null
> @@ -1,20 +0,0 @@
> -SUMMARY = "A tool to load and stress a computer system"
> -HOMEPAGE = "http://kernel.ubuntu.com/~cking/stress-ng/"
> -LICENSE = "GPLv2"
> -LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
> -
> -DEPENDS = "zlib libaio"
> -
> -SRC_URI = "http://kernel.ubuntu.com/~cking/tarballs/${BPN}/${BP}.tar.gz \
> - "
> -SRC_URI_append_libc-musl = "file://0001-Several-changes-to-fix-musl-build.patch"
> -
> -SRC_URI[md5sum] = "e0f6497a8c06f5d652bc2ad88d449c12"
> -SRC_URI[sha256sum] = "37cc73e42f5bdb0e0571ba88f6a69b8f05ee28e51afcafc2231c1058b1a5dd18"
> -
> -CFLAGS += "-Wall -Wextra -DVERSION='"$(VERSION)"'"
> -
> -do_install_append() {
> - install -d ${D}${bindir}
> - install -m 755 ${S}/stress-ng ${D}${bindir}/stress-ng
> -}
> diff --git a/meta-oe/recipes-test/stress-ng/stress-ng_0.09.09.bb b/meta-oe/recipes-test/stress-ng/stress-ng_0.09.09.bb
> new file mode 100644
> index 000000000..24f55ada2
> --- /dev/null
> +++ b/meta-oe/recipes-test/stress-ng/stress-ng_0.09.09.bb
> @@ -0,0 +1,26 @@
> +SUMMARY = "A tool to load and stress a computer system"
> +HOMEPAGE = "http://kernel.ubuntu.com/~cking/stress-ng/"
> +LICENSE = "GPLv2"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
> +
> +DEPENDS = "zlib libaio"
> +
> +SRC_URI = "http://kernel.ubuntu.com/~cking/tarballs/${BPN}/${BP}.tar.xz \
> + "
> +SRC_URI_append_libc-musl = " \
> + file://0001-Several-changes-to-fix-musl-build.patch \
> + file://0002-stress-fcntl-fix-build-for-musl.patch \
> + file://0003-stress-fcntl.c-fix-syntax-errors.patch"
> +
> +SRC_URI[md5sum] = "6e4df71095e780654ccc0a713b8423fe"
> +SRC_URI[sha256sum] = "e993fddb1d990a0cdb23b2af6cf6c417a1c09e155ada7c558ad80eae8a5feed3"
> +
> +UPSTREAM_CHECK_URI ?= "http://kernel.ubuntu.com/~cking/tarballs/${BPN}/"
> +UPSTREAM_CHECK_REGEX ?= "(?P<pver>\d+(\.\d+)+)\.tar"
> +
> +CFLAGS += "-Wall -Wextra -DVERSION='"$(VERSION)"'"
> +
> +do_install_append() {
> + install -d ${D}${bindir}
> + install -m 755 ${S}/stress-ng ${D}${bindir}/stress-ng
> +}
> --
> 2.14.3
>
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] stress-ng: upgrade to 0.09.09 and fix musl build
2018-01-10 4:16 ` Tim Orling
@ 2018-01-10 5:11 ` Denys Dmytriyenko
0 siblings, 0 replies; 3+ messages in thread
From: Denys Dmytriyenko @ 2018-01-10 5:11 UTC (permalink / raw)
To: Tim Orling; +Cc: openembeded-devel
Heh, just had to do the same upgrade locally to fix some issues here. Saw your
commit in upstream too. :) Good job!
On Tue, Jan 09, 2018 at 08:16:15PM -0800, Tim Orling wrote:
> Meh. wrong subject line resending. Please ignore the noise.
>
> > On Jan 9, 2018, at 8:14 PM, Tim Orling <timothy.t.orling@linux.intel.com> wrote:
> >
> > * Upstream tarballs are now xz compressed
> > * Patch 0001 modified:
> > - stress-pthread and stress-context change not needed
> > * Add patch to fix stress-fcntl for musl
> > * Add patch to fix syntax errors in stress-fcntl
> > - Accepted upstream
> > * Add UPSTREAM_CHECK_URI and UPSTREAM_CHECK_REGEX
> >
> > Signed-off-by: Tim Orling <timothy.t.orling@linux.intel.com>
> > ---
> > .../0001-Several-changes-to-fix-musl-build.patch | 31 +-------------
> > .../0002-stress-fcntl-fix-build-for-musl.patch | 27 +++++++++++++
> > .../0003-stress-fcntl.c-fix-syntax-errors.patch | 47 ++++++++++++++++++++++
> > .../recipes-test/stress-ng/stress-ng_0.08.17.bb | 20 ---------
> > .../recipes-test/stress-ng/stress-ng_0.09.09.bb | 26 ++++++++++++
> > 5 files changed, 102 insertions(+), 49 deletions(-)
> > create mode 100644 meta-oe/recipes-test/stress-ng/stress-ng/0002-stress-fcntl-fix-build-for-musl.patch
> > create mode 100644 meta-oe/recipes-test/stress-ng/stress-ng/0003-stress-fcntl.c-fix-syntax-errors.patch
> > delete mode 100644 meta-oe/recipes-test/stress-ng/stress-ng_0.08.17.bb
> > create mode 100644 meta-oe/recipes-test/stress-ng/stress-ng_0.09.09.bb
> >
> > diff --git a/meta-oe/recipes-test/stress-ng/stress-ng/0001-Several-changes-to-fix-musl-build.patch b/meta-oe/recipes-test/stress-ng/stress-ng/0001-Several-changes-to-fix-musl-build.patch
> > index 3ed7efc7e..f90e704f0 100644
> > --- a/meta-oe/recipes-test/stress-ng/stress-ng/0001-Several-changes-to-fix-musl-build.patch
> > +++ b/meta-oe/recipes-test/stress-ng/stress-ng/0001-Several-changes-to-fix-musl-build.patch
> > @@ -11,7 +11,6 @@ swapcontext, musl provide the definition but not the implementation due
> > to that functions are pre-POSIX and set to be deprecated.
> > stress-{resources, pty}.c: Doesn't include termio.h and remove stress
> > operations that uses struct termio, musl doesn't provide that struct.
> > -stress-pthread.c: Change pthread_yield to sched_yield to be more compatible.
> > stress-malloc.c: Check for definition of M_MMAP_THRESHOLD musl doesn't
> > ptovide that constant.
> > stress-madvise.c: Add static poision_count integer, definition of
> > @@ -19,13 +18,13 @@ stress-madvise.c: Add static poision_count integer, definition of
> > cache.c: Define GLOB_ONLYDIR not available on MUSL.
> >
> > Signed-off-by: Aníbal Limón <anibal.limon@linaro.org>
> > +Signed-off-by: Tim Orling <timothy.t.orling@linux.intel.com>
> > +
> > Upstream-status: Pending
> > ---
> > cache.c | 4 ++++
> > - stress-context.c | 2 +-
> > stress-madvise.c | 1 +
> > stress-malloc.c | 2 +-
> > - stress-pthread.c | 2 +-
> > stress-pty.c | 18 ------------------
> > stress-resources.c | 1 -
> > stress-stackmmap.c | 2 +-
> > @@ -46,19 +45,6 @@ index e01fad0..27870ee 100644
> > #if defined(__linux__)
> > #define SYS_CPU_PREFIX "/sys/devices/system/cpu"
> > #define GLOB_PATTERN SYS_CPU_PREFIX "/cpu[0-9]*"
> > -diff --git a/stress-context.c b/stress-context.c
> > -index c5f50ed..67209e5 100644
> > ---- a/stress-context.c
> > -+++ b/stress-context.c
> > -@@ -24,7 +24,7 @@
> > - */
> > - #include "stress-ng.h"
> > -
> > --#if !defined(__OpenBSD__)
> > -+#if !defined(__OpenBSD__) && False
> > -
> > - #include <ucontext.h>
> > -
> > diff --git a/stress-madvise.c b/stress-madvise.c
> > index 43ea454..2e1f8ee 100644
> > --- a/stress-madvise.c
> > @@ -84,19 +70,6 @@ index 8dbe5cc..f505f32 100644
> > if (get_setting("malloc-threshold", &malloc_threshold))
> > (void)mallopt(M_MMAP_THRESHOLD, (int)malloc_threshold);
> > #endif
> > -diff --git a/stress-pthread.c b/stress-pthread.c
> > -index 323a1d4..8269f4c 100644
> > ---- a/stress-pthread.c
> > -+++ b/stress-pthread.c
> > -@@ -139,7 +139,7 @@ static void *stress_pthread_func(void *parg)
> > - break;
> > - }
> > - #if !defined(__NetBSD__) && !defined(__sun__)
> > -- (void)pthread_yield();
> > -+ (void)sched_yield();
> > - #endif
> > - }
> > - ret = pthread_mutex_unlock(&mutex);
> > diff --git a/stress-pty.c b/stress-pty.c
> > index 8c3edf8..518f118 100644
> > --- a/stress-pty.c
> > diff --git a/meta-oe/recipes-test/stress-ng/stress-ng/0002-stress-fcntl-fix-build-for-musl.patch b/meta-oe/recipes-test/stress-ng/stress-ng/0002-stress-fcntl-fix-build-for-musl.patch
> > new file mode 100644
> > index 000000000..826340169
> > --- /dev/null
> > +++ b/meta-oe/recipes-test/stress-ng/stress-ng/0002-stress-fcntl-fix-build-for-musl.patch
> > @@ -0,0 +1,27 @@
> > +From 7be4680930e2867cefd5c8fe27a4ae81a4ae2628 Mon Sep 17 00:00:00 2001
> > +From: Tim Orling <timothy.t.orling@linux.intel.com>
> > +Date: Tue, 9 Jan 2018 18:38:45 -0800
> > +Subject: [PATCH] stress-fcntl: fix build for musl
> > +
> > +Signed-off-by: Tim Orling <timothy.t.orling@linux.intel.com>
> > +
> > +Upstream-status: Pending
> > +---
> > + stress-fcntl.c | 1 +
> > + 1 file changed, 1 insertion(+)
> > +
> > +diff --git a/stress-fcntl.c b/stress-fcntl.c
> > +index b7eade2..9fa5c38 100644
> > +--- a/stress-fcntl.c
> > ++++ b/stress-fcntl.c
> > +@@ -413,6 +413,7 @@ ofd_lock_abort: { /* Nowt */ }
> > +
> > + #if defined(F_GET_FILE_RW_HINT) && defined(F_SET_FILE_RW_HINT)
> > + {
> > ++ int ret;
> > + size_t i;
> > + unsigned long hint;
> > + static const unsigned long hints[] = {
> > +--
> > +2.14.3
> > +
> > diff --git a/meta-oe/recipes-test/stress-ng/stress-ng/0003-stress-fcntl.c-fix-syntax-errors.patch b/meta-oe/recipes-test/stress-ng/stress-ng/0003-stress-fcntl.c-fix-syntax-errors.patch
> > new file mode 100644
> > index 000000000..4b6a9b391
> > --- /dev/null
> > +++ b/meta-oe/recipes-test/stress-ng/stress-ng/0003-stress-fcntl.c-fix-syntax-errors.patch
> > @@ -0,0 +1,47 @@
> > +From 8537dcafb12be4d986b571f1e1d9f3ceecdc8e63 Mon Sep 17 00:00:00 2001
> > +From: Tim Orling <timothy.t.orling@linux.intel.com>
> > +Date: Sun, 7 Jan 2018 21:29:12 -0800
> > +Subject: [PATCH] stress-fcntl.c: fix syntax errors
> > +
> > +Signed-off-by: Tim Orling <timothy.t.orling@linux.intel.com>
> > +
> > +Upstream-status: Accepted
> > +(commit 61d93d6857256cd49df6a6302caf6e7bb69b0f46)
> > +---
> > + stress-fcntl.c | 6 +++---
> > + 1 file changed, 3 insertions(+), 3 deletions(-)
> > +
> > +diff --git a/stress-fcntl.c b/stress-fcntl.c
> > +index 7c15d28..b7eade2 100644
> > +--- a/stress-fcntl.c
> > ++++ b/stress-fcntl.c
> > +@@ -173,7 +173,7 @@ static int do_fcntl(const args_t *args, const int fd)
> > + owner.pid = getpgrp();
> > + ret = fcntl(fd, F_SETOWN_EX, &owner);
> > + check_return(args, ret, "F_SETOWN_EX, F_OWNER_PGRP");
> > +-#else if defined(HAVE_GETPGRP) && defined(F_OWNER_GID)
> > ++#elif defined(HAVE_GETPGRP) && defined(F_OWNER_GID)
> > + owner.type = F_OWNER_GID;
> > + owner.pid = getpgrp();
> > + ret = fcntl(fd, F_SETOWN_EX, &owner);
> > +@@ -202,7 +202,7 @@ static int do_fcntl(const args_t *args, const int fd)
> > + owner.type = F_OWNER_PGRP;
> > + ret = fcntl(fd, F_GETOWN_EX, &owner);
> > + check_return(args, ret, "F_GETOWN_EX, F_OWNER_PGRP");
> > +-#ele if defined(F_OWNER_GID)
> > ++#elif defined(F_OWNER_GID)
> > + owner.type = F_OWNER_GID;
> > + ret = fcntl(fd, F_GETOWN_EX, &owner);
> > + check_return(args, ret, "F_GETOWN_EX, F_OWNER_GID");
> > +@@ -436,7 +436,7 @@ ofd_lock_abort: { /* Nowt */ }
> > + #endif
> > + };
> > +
> > +- ret = fcntl(fd, F_GET_FILE_RW_HINT, &hint)
> > ++ ret = fcntl(fd, F_GET_FILE_RW_HINT, &hint);
> > + if (ret == 0) {
> > + for (i = 0; i < SIZEOF_ARRAY(hints); i++) {
> > + hint = hints[i];
> > +--
> > +2.14.3
> > +
> > diff --git a/meta-oe/recipes-test/stress-ng/stress-ng_0.08.17.bb b/meta-oe/recipes-test/stress-ng/stress-ng_0.08.17.bb
> > deleted file mode 100644
> > index dc5b2e8eb..000000000
> > --- a/meta-oe/recipes-test/stress-ng/stress-ng_0.08.17.bb
> > +++ /dev/null
> > @@ -1,20 +0,0 @@
> > -SUMMARY = "A tool to load and stress a computer system"
> > -HOMEPAGE = "http://kernel.ubuntu.com/~cking/stress-ng/"
> > -LICENSE = "GPLv2"
> > -LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
> > -
> > -DEPENDS = "zlib libaio"
> > -
> > -SRC_URI = "http://kernel.ubuntu.com/~cking/tarballs/${BPN}/${BP}.tar.gz \
> > - "
> > -SRC_URI_append_libc-musl = "file://0001-Several-changes-to-fix-musl-build.patch"
> > -
> > -SRC_URI[md5sum] = "e0f6497a8c06f5d652bc2ad88d449c12"
> > -SRC_URI[sha256sum] = "37cc73e42f5bdb0e0571ba88f6a69b8f05ee28e51afcafc2231c1058b1a5dd18"
> > -
> > -CFLAGS += "-Wall -Wextra -DVERSION='"$(VERSION)"'"
> > -
> > -do_install_append() {
> > - install -d ${D}${bindir}
> > - install -m 755 ${S}/stress-ng ${D}${bindir}/stress-ng
> > -}
> > diff --git a/meta-oe/recipes-test/stress-ng/stress-ng_0.09.09.bb b/meta-oe/recipes-test/stress-ng/stress-ng_0.09.09.bb
> > new file mode 100644
> > index 000000000..24f55ada2
> > --- /dev/null
> > +++ b/meta-oe/recipes-test/stress-ng/stress-ng_0.09.09.bb
> > @@ -0,0 +1,26 @@
> > +SUMMARY = "A tool to load and stress a computer system"
> > +HOMEPAGE = "http://kernel.ubuntu.com/~cking/stress-ng/"
> > +LICENSE = "GPLv2"
> > +LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
> > +
> > +DEPENDS = "zlib libaio"
> > +
> > +SRC_URI = "http://kernel.ubuntu.com/~cking/tarballs/${BPN}/${BP}.tar.xz \
> > + "
> > +SRC_URI_append_libc-musl = " \
> > + file://0001-Several-changes-to-fix-musl-build.patch \
> > + file://0002-stress-fcntl-fix-build-for-musl.patch \
> > + file://0003-stress-fcntl.c-fix-syntax-errors.patch"
> > +
> > +SRC_URI[md5sum] = "6e4df71095e780654ccc0a713b8423fe"
> > +SRC_URI[sha256sum] = "e993fddb1d990a0cdb23b2af6cf6c417a1c09e155ada7c558ad80eae8a5feed3"
> > +
> > +UPSTREAM_CHECK_URI ?= "http://kernel.ubuntu.com/~cking/tarballs/${BPN}/"
> > +UPSTREAM_CHECK_REGEX ?= "(?P<pver>\d+(\.\d+)+)\.tar"
> > +
> > +CFLAGS += "-Wall -Wextra -DVERSION='"$(VERSION)"'"
> > +
> > +do_install_append() {
> > + install -d ${D}${bindir}
> > + install -m 755 ${S}/stress-ng ${D}${bindir}/stress-ng
> > +}
> > --
> > 2.14.3
> >
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-01-10 5:18 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-01-10 4:14 [PATCH] stress-ng: upgrade to 0.09.09 and fix musl build Tim Orling
2018-01-10 4:16 ` Tim Orling
2018-01-10 5:11 ` Denys Dmytriyenko
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox