From: chrubis@suse.cz
To: Jan Stancek <jstancek@redhat.com>
Cc: ltp-list@lists.sourceforge.net
Subject: Re: [LTP] [PATCH v5 2/2] open_posix_testsuite/../mq_timedsend/12-1: fix race
Date: Thu, 15 Aug 2013 14:53:41 +0200 [thread overview]
Message-ID: <20130815125341.GA336@rei> (raw)
In-Reply-To: <cfa1ee92801ae50856d8d7b000c5bb046724475e.1376479412.git.jstancek@redhat.com>
Hi!
> Test can send SIGUSR1 signal after thread has already completed
> and report failure from main() and success from a_thread_func():
>
> conformance/interfaces/mq_timedsend/12-1: execution: UNRESOLVED: Output:
> Error: in pthread_kill
> : Success
> thread: mq_timedsend interrupted by signal and correctly set errno to EINTR
>
> Synchronize exit of a_thread_func(), so that thread can exit only
> after loop sending SIGUSR1 completed.
>
> Signed-off-by: Jan Stancek <jstancek@redhat.com>
> ---
> Changes in v5:
> - return value from pthread_exit is not used, always return NULL
Looks good to me. Acked.
--
Cyril Hrubis
chrubis@suse.cz
------------------------------------------------------------------------------
Get 100% visibility into Java/.NET code with AppDynamics Lite!
It's a free troubleshooting tool designed for production.
Get down to code-level detail for bottlenecks, with <2% overhead.
Download for free and get started troubleshooting in minutes.
http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
next prev parent reply other threads:[~2013-08-15 12:53 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-24 6:39 [LTP] [PATCH 1/2] open_posix_testsuite/../mq_timedsend/12-1: fix long lines Jan Stancek
2013-07-24 6:39 ` [LTP] [PATCH 2/2] open_posix_testsuite/../mq_timedsend/12-1: fix race Jan Stancek
2013-07-25 10:07 ` [LTP] [PATCH v2 " Jan Stancek
2013-07-31 14:30 ` [LTP] [PATCH v3 " Jan Stancek
2013-08-12 14:43 ` chrubis
2013-08-13 13:38 ` [LTP] [PATCH v4 " Jan Stancek
2013-08-13 15:13 ` chrubis
2013-08-14 11:28 ` [LTP] [PATCH v5 " Jan Stancek
2013-08-15 12:53 ` chrubis [this message]
2013-08-27 11:51 ` chrubis
2013-07-30 17:35 ` [LTP] [PATCH " chrubis
[not found] ` <111760967.10033378.1375213746603.JavaMail.root@redhat.com>
2013-07-31 11:07 ` chrubis
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=20130815125341.GA336@rei \
--to=chrubis@suse.cz \
--cc=jstancek@redhat.com \
--cc=ltp-list@lists.sourceforge.net \
/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.