All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tim Orling <timothy.t.orling@linux.intel.com>
To: openembedded-devel@lists.openembedded.org
Subject: [meta-oe][PATCH v2] stress-ng: upgrade to 0.09.09 and fix musl build
Date: Tue,  9 Jan 2018 22:23:45 -0800	[thread overview]
Message-ID: <20180110062345.43847-1-timothy.t.orling@linux.intel.com> (raw)

* 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>
---

Per Denys Demytriyenko request, v2 is with -M05 threshold

 .../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 ++++++++++++++++++++++
 .../{stress-ng_0.08.17.bb => stress-ng_0.09.09.bb} | 14 +++++--
 4 files changed, 86 insertions(+), 33 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
 rename meta-oe/recipes-test/stress-ng/{stress-ng_0.08.17.bb => stress-ng_0.09.09.bb} (49%)

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.09.09.bb
similarity index 49%
rename from meta-oe/recipes-test/stress-ng/stress-ng_0.08.17.bb
rename to meta-oe/recipes-test/stress-ng/stress-ng_0.09.09.bb
index dc5b2e8eb..24f55ada2 100644
--- a/meta-oe/recipes-test/stress-ng/stress-ng_0.08.17.bb
+++ b/meta-oe/recipes-test/stress-ng/stress-ng_0.09.09.bb
@@ -5,12 +5,18 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
 
 DEPENDS = "zlib libaio"
 
-SRC_URI = "http://kernel.ubuntu.com/~cking/tarballs/${BPN}/${BP}.tar.gz \
+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"
+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] = "e0f6497a8c06f5d652bc2ad88d449c12"
-SRC_URI[sha256sum] = "37cc73e42f5bdb0e0571ba88f6a69b8f05ee28e51afcafc2231c1058b1a5dd18"
+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)"'"
 
-- 
2.14.3



             reply	other threads:[~2018-01-10  6:23 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-01-10  6:23 Tim Orling [this message]
2018-01-10  7:23 ` [meta-oe][PATCH v2] stress-ng: upgrade to 0.09.09 and fix musl build Denys Dmytriyenko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180110062345.43847-1-timothy.t.orling@linux.intel.com \
    --to=timothy.t.orling@linux.intel.com \
    --cc=openembedded-devel@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.