i#1803 flaky signest test: delay the second signal.

Commit log for first patchset:
---------------
i#1803 flaky signest test: delay the second signal.

Updates test signest to delay sending the second signal utill the
first signal is delivered.
---------------

[bruening, 2016-12-01 04:38:05]

Please make it clear in the commit message *why* this is being done: that it's a
workaround for a DR bug.

Spelling error in commit message: "utill"

Similar requests on code comments.

Shouldn't need another look (but please check spelling, etc.) so LGTM w/
comments

https://codereview.appspot.com/317970043/diff/1/suite/tests/linux/signest.c
File suite/tests/linux/signest.c (right):

https://codereview.appspot.com/317970043/diff/1/suite/tests/linux/signest.c#n...
suite/tests/linux/signest.c:112: /* i#1803: DR may drop the signal if more than
one signals are delivered
grammar: "signal is"
(I think "signal are" is considered acceptable but sounds wrong to me; certainly
it cannot be "signals"; the "together" sounds wrong too -- maybe best "more than
one are delivered together")

https://codereview.appspot.com/317970043/diff/1/suite/tests/linux/signest.c#n...
suite/tests/linux/signest.c:112: /* i#1803: DR may drop the signal if more than
one signals are delivered
"incorrectly drop" to emphasize, b/c it's RT

https://codereview.appspot.com/317970043/diff/1/suite/tests/linux/signest.c#n...
suite/tests/linux/signest.c:113: * together. We delay sending the second signal
until the first signal is
But this is a bug in DR, right?  So this is a temporary workaround to make the
test pass, right?  So this comment should have a XXX and make it clear that this
is temporary.

[zhaoqin, 2016-12-01 15:46:30]

Commit log for latest patchset:
---------------
i#1803 flaky signest test: delay the second signal as a temporary workaround

DR may incorrectly drop the signal if more than one are delivered together.
We update the signest test to delay sending the second signal until
the first one is delivered as a temporary workaround.

Review-URL: https://codereview.appspot.com/317970043
---------------

[zhaoqin, 2016-12-01 16:26:42]

Committed as
https://github.com/DynamoRIO/dynamorio/commit/ac059a51bdc21a6ff138f098e92b334...

Final commit log: 
---------------
i#1803 flaky signest test: delay the second signal as a temporary workaround

DR may incorrectly drop the signal if more than one are delivered together.
We update the signest test to delay sending the second signal until
the first one is delivered as a temporary workaround.

Review-URL: https://codereview.appspot.com/317970043
---------------