From: Pedro Alves <palves@redhat.com>
Date: Thu, 20 Mar 2014 14:09:53 +0000 (+0000)
Subject: Further cleanup of signal-while-stepping-over-bp-other-thread.c.
X-Git-Tag: gdb-7.8-release~769
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=40acf43aadb4d5348cff0dd554ae97de4dd775af;p=external%2Fbinutils.git

Further cleanup of signal-while-stepping-over-bp-other-thread.c.

This test now uses pthread_kill instead of the host's kill command, so
no longer need to block signals, or store the the inferior's PID.

gdb/testsuite/
2014-03-20  Pedro Alves  <palves@redhat.com>

	* gdb.threads/signal-while-stepping-over-bp-other-thread.c (pid):
	Delete.
	(block_signals, unblock_signals): Delete.
	(child_function_2, main): Remove references to deleted variable
	and functions.
---

diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index bc6d6df..31e0e5d 100644
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,5 +1,13 @@
 2014-03-20  Pedro Alves  <palves@redhat.com>
 
+	* gdb.threads/signal-while-stepping-over-bp-other-thread.c (pid):
+	Delete.
+	(block_signals, unblock_signals): Delete.
+	(child_function_2, main): Remove references to deleted variable
+	and functions.
+
+2014-03-20  Pedro Alves  <palves@redhat.com>
+
 	* gdb.threads/signal-while-stepping-over-bp-other-thread.c (main):
 	Use pthread_kill to signal thread 2.
 	* gdb.threads/signal-while-stepping-over-bp-other-thread.exp:
diff --git a/gdb/testsuite/gdb.threads/signal-while-stepping-over-bp-other-thread.c b/gdb/testsuite/gdb.threads/signal-while-stepping-over-bp-other-thread.c
index 8839a6f..0f3d17a 100644
--- a/gdb/testsuite/gdb.threads/signal-while-stepping-over-bp-other-thread.c
+++ b/gdb/testsuite/gdb.threads/signal-while-stepping-over-bp-other-thread.c
@@ -22,7 +22,6 @@
 
 unsigned int args[2];
 
-pid_t pid;
 pthread_barrier_t barrier;
 pthread_t child_thread_2, child_thread_3;
 
@@ -38,24 +37,6 @@ callme (void)
 {
 }
 
-void
-block_signals (void)
-{
-  sigset_t mask;
-
-  sigfillset (&mask);
-  sigprocmask (SIG_BLOCK, &mask, NULL);
-}
-
-void
-unblock_signals (void)
-{
-  sigset_t mask;
-
-  sigfillset (&mask);
-  sigprocmask (SIG_UNBLOCK, &mask, NULL);
-}
-
 void *
 child_function_3 (void *arg)
 {
@@ -79,8 +60,6 @@ child_function_2 (void *arg)
   int my_number =  (long) arg;
   volatile int *myp = (int *) &args[my_number];
 
-  unblock_signals ();
-
   pthread_barrier_wait (&barrier);
 
   while (*myp > 0)
@@ -106,10 +85,6 @@ main ()
   int res;
   long i;
 
-  /* Block signals in all threads but one, so that we're sure which
-     thread gets the signal we send from the command line.  */
-  block_signals ();
-
   signal (SIGUSR1, handler);
 
   /* Call these early so that PLTs for these are resolved soon,
@@ -120,10 +95,6 @@ main ()
 
   pthread_barrier_init (&barrier, NULL, 2);
 
-  /* The test uses this global to know where to send the signal
-     to.  */
-  pid = getpid ();
-
   i = 0;
   args[i] = 1;
   res = pthread_create (&child_thread_2,