* [LTP] [PATCH] [RFC] syscalls/ipc: Rename some msgctl tests
@ 2018-06-06 12:37 Cyril Hrubis
2018-06-07 5:44 ` Li Wang
0 siblings, 1 reply; 5+ messages in thread
From: Cyril Hrubis @ 2018-06-06 12:37 UTC (permalink / raw)
To: ltp
The test from msgctl08 to msgctl11 are actually msg stress tests, these
do not actually call msgctl() during the test but rather than that they
fork large number of processes and send message around. Renaming them to
msgstress seems to be more fitting.
Signed-off-by: Cyril Hrubis <chrubis@suse.cz>
---
runtest/ltplite | 4 +--
runtest/stress.part3 | 4 +--
runtest/syscalls | 8 +++---
runtest/syscalls-ipc | 8 +++---
testcases/kernel/syscalls/ipc/msgctl/.gitignore | 4 ---
testcases/kernel/syscalls/ipc/msgstress/.gitignore | 4 +++
testcases/kernel/syscalls/ipc/msgstress/Makefile | 29 ++++++++++++++++++++++
.../{msgctl/msgctl08.c => msgstress/msgstress01.c} | 0
.../{msgctl/msgctl09.c => msgstress/msgstress02.c} | 0
.../{msgctl/msgctl10.c => msgstress/msgstress03.c} | 0
.../{msgctl/msgctl11.c => msgstress/msgstress04.c} | 0
11 files changed, 45 insertions(+), 16 deletions(-)
create mode 100644 testcases/kernel/syscalls/ipc/msgstress/.gitignore
create mode 100644 testcases/kernel/syscalls/ipc/msgstress/Makefile
rename testcases/kernel/syscalls/ipc/{msgctl/msgctl08.c => msgstress/msgstress01.c} (100%)
rename testcases/kernel/syscalls/ipc/{msgctl/msgctl09.c => msgstress/msgstress02.c} (100%)
rename testcases/kernel/syscalls/ipc/{msgctl/msgctl10.c => msgstress/msgstress03.c} (100%)
rename testcases/kernel/syscalls/ipc/{msgctl/msgctl11.c => msgstress/msgstress04.c} (100%)
diff --git a/runtest/ltplite b/runtest/ltplite
index 5f16dec00..3c6f51270 100644
--- a/runtest/ltplite
+++ b/runtest/ltplite
@@ -487,8 +487,8 @@ msgctl04 msgctl04
msgctl05 msgctl05
msgctl06 msgctl06
msgctl07 msgctl07
-msgctl08 msgctl08
-msgctl09 msgctl09
+msgstress01 msgstress01
+msgstress02 msgstress02
msgget01 msgget01
msgget02 msgget02
diff --git a/runtest/stress.part3 b/runtest/stress.part3
index 0420827e8..e0a8869e8 100644
--- a/runtest/stress.part3
+++ b/runtest/stress.part3
@@ -403,8 +403,8 @@ msgctl04 msgctl04
msgctl05 msgctl05
msgctl06 msgctl06
msgctl07 msgctl07
-msgctl08 msgctl08
-msgctl09 msgctl09
+msgstress01 msgstress01
+msgstress02 msgstress02
msgget01 msgget01
msgget02 msgget02
diff --git a/runtest/syscalls b/runtest/syscalls
index 738d2242c..95d470cde 100644
--- a/runtest/syscalls
+++ b/runtest/syscalls
@@ -693,10 +693,10 @@ msgctl04 msgctl04
msgctl05 msgctl05
msgctl06 msgctl06
msgctl07 msgctl07
-msgctl08 msgctl08
-msgctl09 msgctl09
-msgctl10 msgctl10
-msgctl11 msgctl11
+msgstress01 msgstress01
+msgstress02 msgstress02
+msgstress03 msgstress03
+msgstress04 msgstress04
msgctl12 msgctl12
msgctl13 msgctl13
diff --git a/runtest/syscalls-ipc b/runtest/syscalls-ipc
index de32c6ba9..c734e239d 100644
--- a/runtest/syscalls-ipc
+++ b/runtest/syscalls-ipc
@@ -5,10 +5,10 @@ msgctl04 msgctl04
msgctl05 msgctl05
msgctl06 msgctl06
msgctl07 msgctl07
-msgctl08 msgctl08
-msgctl09 msgctl09
-msgctl10 msgctl10
-msgctl11 msgctl11
+msgstress01 msgstress01
+msgstress02 msgstress02
+msgstress03 msgstress03
+msgstress04 msgstress04
msgctl12 msgctl12
msgctl13 msgctl13
diff --git a/testcases/kernel/syscalls/ipc/msgctl/.gitignore b/testcases/kernel/syscalls/ipc/msgctl/.gitignore
index 3d3a8bf52..2edde5de4 100644
--- a/testcases/kernel/syscalls/ipc/msgctl/.gitignore
+++ b/testcases/kernel/syscalls/ipc/msgctl/.gitignore
@@ -5,9 +5,5 @@
/msgctl05
/msgctl06
/msgctl07
-/msgctl08
-/msgctl09
-/msgctl10
-/msgctl11
/msgctl12
/msgctl13
diff --git a/testcases/kernel/syscalls/ipc/msgstress/.gitignore b/testcases/kernel/syscalls/ipc/msgstress/.gitignore
new file mode 100644
index 000000000..a8f675399
--- /dev/null
+++ b/testcases/kernel/syscalls/ipc/msgstress/.gitignore
@@ -0,0 +1,4 @@
+/msgstress01
+/msgstress02
+/msgstress03
+/msgstress04
diff --git a/testcases/kernel/syscalls/ipc/msgstress/Makefile b/testcases/kernel/syscalls/ipc/msgstress/Makefile
new file mode 100644
index 000000000..4472eeb1c
--- /dev/null
+++ b/testcases/kernel/syscalls/ipc/msgstress/Makefile
@@ -0,0 +1,29 @@
+#
+# Copyright (c) International Business Machines Corp., 2001
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
+# the GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
+#
+
+top_srcdir ?= ../../../../..
+
+include $(top_srcdir)/include/mk/testcases.mk
+
+LIBMSGCTL := $(LIBDIR)/libmsgctl.a
+LDLIBS += -lmsgctl
+
+MAKE_DEPS := $(LIBMSGCTL)
+
+include $(abs_srcdir)/../Makefile.inc
+include $(top_srcdir)/include/mk/generic_leaf_target.mk
diff --git a/testcases/kernel/syscalls/ipc/msgctl/msgctl08.c b/testcases/kernel/syscalls/ipc/msgstress/msgstress01.c
similarity index 100%
rename from testcases/kernel/syscalls/ipc/msgctl/msgctl08.c
rename to testcases/kernel/syscalls/ipc/msgstress/msgstress01.c
diff --git a/testcases/kernel/syscalls/ipc/msgctl/msgctl09.c b/testcases/kernel/syscalls/ipc/msgstress/msgstress02.c
similarity index 100%
rename from testcases/kernel/syscalls/ipc/msgctl/msgctl09.c
rename to testcases/kernel/syscalls/ipc/msgstress/msgstress02.c
diff --git a/testcases/kernel/syscalls/ipc/msgctl/msgctl10.c b/testcases/kernel/syscalls/ipc/msgstress/msgstress03.c
similarity index 100%
rename from testcases/kernel/syscalls/ipc/msgctl/msgctl10.c
rename to testcases/kernel/syscalls/ipc/msgstress/msgstress03.c
diff --git a/testcases/kernel/syscalls/ipc/msgctl/msgctl11.c b/testcases/kernel/syscalls/ipc/msgstress/msgstress04.c
similarity index 100%
rename from testcases/kernel/syscalls/ipc/msgctl/msgctl11.c
rename to testcases/kernel/syscalls/ipc/msgstress/msgstress04.c
--
2.13.6
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [LTP] [PATCH] [RFC] syscalls/ipc: Rename some msgctl tests
2018-06-06 12:37 [LTP] [PATCH] [RFC] syscalls/ipc: Rename some msgctl tests Cyril Hrubis
@ 2018-06-07 5:44 ` Li Wang
2018-06-07 8:45 ` Cyril Hrubis
2018-06-12 14:53 ` Cyril Hrubis
0 siblings, 2 replies; 5+ messages in thread
From: Li Wang @ 2018-06-07 5:44 UTC (permalink / raw)
To: ltp
On Wed, Jun 6, 2018 at 8:37 PM, Cyril Hrubis <chrubis@suse.cz> wrote:
> The test from msgctl08 to msgctl11 are actually msg stress tests, these
> do not actually call msgctl() during the test but rather than that they
> fork large number of processes and send message around. Renaming them to
> msgstress seems to be more fitting.
>
Agreed. These tests fork too many times, and they often trigger OOM at
runtime. So I think this patch make sense.
​
Another thought of mine is that to remove them from the ltplite and
syscalls runtest file. what about just putting them in stress.part3?
--
Regards,
Li Wang
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.linux.it/pipermail/ltp/attachments/20180607/a6cefd8d/attachment.html>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [LTP] [PATCH] [RFC] syscalls/ipc: Rename some msgctl tests
2018-06-07 5:44 ` Li Wang
@ 2018-06-07 8:45 ` Cyril Hrubis
2018-06-07 9:01 ` Li Wang
2018-06-12 14:53 ` Cyril Hrubis
1 sibling, 1 reply; 5+ messages in thread
From: Cyril Hrubis @ 2018-06-07 8:45 UTC (permalink / raw)
To: ltp
Hi!
> Agreed. These tests fork too many times, and they often trigger OOM at
> runtime. So I think this patch make sense.
> ???
> Another thought of mine is that to remove them from the ltplite and
> syscalls runtest file. what about just putting them in stress.part3?
These shouldn't be part of ltplite for sure, I will remove them in a
follow up patch. I would like to keep them in the syscalls runtest file
though and make sure that they scale better accordingly to system
RAM/CPUs later on.
--
Cyril Hrubis
chrubis@suse.cz
^ permalink raw reply [flat|nested] 5+ messages in thread
* [LTP] [PATCH] [RFC] syscalls/ipc: Rename some msgctl tests
2018-06-07 8:45 ` Cyril Hrubis
@ 2018-06-07 9:01 ` Li Wang
0 siblings, 0 replies; 5+ messages in thread
From: Li Wang @ 2018-06-07 9:01 UTC (permalink / raw)
To: ltp
On Thu, Jun 7, 2018 at 4:45 PM, Cyril Hrubis <chrubis@suse.cz> wrote:
> Hi!
> > Agreed. These tests fork too many times, and they often trigger OOM at
> > runtime. So I think this patch make sense.
> > ???
> > Another thought of mine is that to remove them from the ltplite and
> > syscalls runtest file. what about just putting them in stress.part3?
>
> These shouldn't be part of ltplite for sure, I will remove them in a
> follow up patch. I would like to keep them in the syscalls runtest file
> though and make sure that they scale better accordingly to system
> RAM/CPUs later on.
>
Great. It sounds good to adjust their fork numbers according to the system
config.
> --
> Cyril Hrubis
> chrubis@suse.cz
>
--
Regards,
Li Wang
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.linux.it/pipermail/ltp/attachments/20180607/faff90da/attachment.html>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [LTP] [PATCH] [RFC] syscalls/ipc: Rename some msgctl tests
2018-06-07 5:44 ` Li Wang
2018-06-07 8:45 ` Cyril Hrubis
@ 2018-06-12 14:53 ` Cyril Hrubis
1 sibling, 0 replies; 5+ messages in thread
From: Cyril Hrubis @ 2018-06-12 14:53 UTC (permalink / raw)
To: ltp
Hi!
Pushed the patch with your ackq, as well as removal from the ltplite
runtest file, thanks.
--
Cyril Hrubis
chrubis@suse.cz
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-06-12 14:53 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-06 12:37 [LTP] [PATCH] [RFC] syscalls/ipc: Rename some msgctl tests Cyril Hrubis
2018-06-07 5:44 ` Li Wang
2018-06-07 8:45 ` Cyril Hrubis
2018-06-07 9:01 ` Li Wang
2018-06-12 14:53 ` Cyril Hrubis
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.