public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
From: Petr Vorel <pvorel@suse.cz>
To: Li Wang <liwang@redhat.com>
Cc: ltp@lists.linux.it, automated-testing@lists.yoctoproject.org
Subject: Re: [LTP] [RFC][PATCH 2/2] runtest: Move ipc into syscalls
Date: Mon, 7 Apr 2025 14:19:05 +0200	[thread overview]
Message-ID: <20250407121905.GB87982@pevik> (raw)
In-Reply-To: <CAEemH2dsdhgKNHTJY8apYEoe1a2RWrZFvMefd8rML1GwP8Lxkw@mail.gmail.com>

Hi

[ Cc automated-testing, Tim and Anders: FYI disruptive change,
content of runtest/ipc is moved into runtest/syscalls ]

Kind regards,
Petr

> On Thu, Apr 3, 2025 at 6:19 PM Petr Vorel <pvorel@suse.cz> wrote:

> > It's not worth to keep separate runtest file for 8 very quick tests.
> > syscalls-ipc would be obvious runtest to move, but it contains tests for
> > message queues, semaphores etc. Other IPC tests are in syscalls runtest,
> > therefore move tests there.

> > Remove test description (can be visible from help and the test output,
> > it was also below the test not above).

> > Signed-off-by: Petr Vorel <pvorel@suse.cz>

> ---
> >  runtest/ipc      | 28 ----------------------------
> >  runtest/syscalls |  9 +++++++++
> >  2 files changed, 9 insertions(+), 28 deletions(-)
> >  delete mode 100644 runtest/ipc

> > diff --git a/runtest/ipc b/runtest/ipc
> > deleted file mode 100644
> > index f5cdea1974..0000000000
> > --- a/runtest/ipc
> > +++ /dev/null
> > @@ -1,28 +0,0 @@



> > -#DESCRIPTION:Interprocess communication stress
> > -# These tests use tests/pipeio to put pipes (named or unnamed) through a
> > workout



> > -#
> > -pipeio_1 pipeio -T pipeio_1 -c 5 -s 4090 -i 100 -b -f x80
> > -# spawns 5 children to write 100 chunks of 4090 bytes to a named pipe
> > -# using blocking I/O
> > -pipeio_2 pipeio -T pipeio_2 -c 5 -s 4090 -i 100 -f x80
> > -# spawns 5 children to write 100 chunks of 4090 bytes to a named pipe
> > -# using non-blocking I/O
> > -pipeio_3 pipeio -T pipeio_3 -c 5 -s 4090 -i 100 -u -b -f x80
> > -# spawns 5 children to write 100 chunks of 4090 bytes to an unnamed pipe
> > -# using blocking I/O
> > -pipeio_4 pipeio -T pipeio_4 -c 5 -s 4090 -i 100 -u -f x80
> > -# spawns 5 children to write 100 chunks of 4090 bytes to an unnamed pipe
> > -# using non-blocking I/O
> > -pipeio_5 pipeio -T pipeio_5 -c 5 -s 5000 -i 10 -b -f x80
> > -# spawns 5 children to write 10 chunks of 5000 bytes to a named pipe
> > -# using blocking I/O
> > -pipeio_6 pipeio -T pipeio_6 -c 5 -s 5000 -i 10 -b -u -f x80
> > -# spawns 5 children to write 10 chunks of 5000 bytes to an unnamed pipe
> > -# using blocking I/O
> > -pipeio_7 pipeio -T pipeio_7 -c 5 -s 5000 -i 10 -f x80
> > -# spawns 5 children to write 10 chunks of 5000 bytes to a named pipe
> > -# using non-blocking I/O
> > -pipeio_8 pipeio -T pipeio_8 -c 5 -s 5000 -i 10 -u -f x80
> > -# spawns 5 children to write 10 chunks of 5000 bytes to an unnamed pipe
> > -# using non-blocking I/O
> > -
> > diff --git a/runtest/syscalls b/runtest/syscalls
> > index 05b3e0d376..b911aba7b6 100644
> > --- a/runtest/syscalls
> > +++ b/runtest/syscalls
> > @@ -1047,6 +1047,15 @@ pipe2_01 pipe2_01
> >  pipe2_02 pipe2_02
> >  pipe2_04 pipe2_04


> Maybe we can add one-line comments here to explain that these are
> Interprocess communication stress test?

> Since syscalls is a large file (containing so many tests), it makes sense
> to comment on something there to explain the subtest blocks.


> +pipeio_1 pipeio -T pipeio_1 -c 5 -s 4090 -i 100 -b -f x80
> > +pipeio_2 pipeio -T pipeio_2 -c 5 -s 4090 -i 100 -f x80
> > +pipeio_3 pipeio -T pipeio_3 -c 5 -s 4090 -i 100 -u -b -f x80
> > +pipeio_4 pipeio -T pipeio_4 -c 5 -s 4090 -i 100 -u -f x80
> > +pipeio_5 pipeio -T pipeio_5 -c 5 -s 5000 -i 10 -b -f x80
> > +pipeio_6 pipeio -T pipeio_6 -c 5 -s 5000 -i 10 -b -u -f x80
> > +pipeio_7 pipeio -T pipeio_7 -c 5 -s 5000 -i 10 -f x80
> > +pipeio_8 pipeio -T pipeio_8 -c 5 -s 5000 -i 10 -u -f x80



> Reviewed-by: Li Wang <liwang@redhat.com>

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

  parent reply	other threads:[~2025-04-07 12:19 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-04-03 10:19 [LTP] [RFC][PATCH 0/2] Merge runtest files Petr Vorel
2025-04-03 10:19 ` [LTP] [RFC][PATCH 1/2] runtest/ipc: Enable named pipes on non-blocking IO Petr Vorel
2025-04-06  2:55   ` Li Wang via ltp
2025-04-08 18:37   ` Cyril Hrubis
2025-04-03 10:19 ` [LTP] [RFC][PATCH 2/2] runtest: Move ipc into syscalls Petr Vorel
2025-04-06  3:01   ` Li Wang via ltp
2025-04-07 12:15     ` Petr Vorel
2025-04-07 12:19     ` Petr Vorel [this message]
2025-04-17  9:22   ` Cyril Hrubis
2025-04-17 11:14     ` Petr Vorel
2025-04-03 11:33 ` [LTP] [RFC][PATCH 0/2] Merge runtest files Andrea Cervesato via ltp
2025-04-03 15:47   ` Cyril Hrubis
2025-04-06  2:52     ` Li Wang via ltp
2025-04-06  2:43 ` Li Wang via ltp

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=20250407121905.GB87982@pevik \
    --to=pvorel@suse.cz \
    --cc=automated-testing@lists.yoctoproject.org \
    --cc=liwang@redhat.com \
    --cc=ltp@lists.linux.it \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox