ChangeLogTag: Fri Mar 24 01:58:02 UTC 2006 J.T. Conklin <jtc@acorntoolworks.com>

3 files changed, 27 insertions, 1 deletions

diff --git a/TAO/ChangeLog b/TAO/ChangeLog
index 81383bdfc61..53e7418bfa1 100644
--- a/TAO/ChangeLog
+++ b/TAO/ChangeLog
@@ -1,3 +1,12 @@
+Fri Mar 24 01:58:02 UTC 2006  J.T. Conklin  <jtc@acorntoolworks.com>
+
+        * orbsvcs/Notify_Service/Notify_Service.cpp:
+        * orbsvcs/tests/Notify/lib/Driver.cpp:
+        
+          Removed logging of ORB worker thread start up, as that fails on
+          systems without thread priorities.  See bugzilla issue 2477 for
+          details.
+
 Wed Mar 22 16:18:00 UTC 2006  Simon Massey  <sma@prismtech.com>
 
         * tests/Bug_1495_Regression/run_test.pl:
diff --git a/TAO/orbsvcs/Notify_Service/Notify_Service.cpp b/TAO/orbsvcs/Notify_Service/Notify_Service.cpp
index e9e7fde3c1a..e7d95c4dd99 100644
--- a/TAO/orbsvcs/Notify_Service/Notify_Service.cpp
+++ b/TAO/orbsvcs/Notify_Service/Notify_Service.cpp
@@ -411,6 +411,13 @@ Worker::orb (CORBA::ORB_ptr orb)
 int
 Worker::svc (void)
 {
+#if 0
+  // ACE_Thread::getprio() fails on systems that do not support thread
+  // priorities.  While we could just treat the failure as benign, I'm
+  // just disabling it altogether.  It doesn't provide much value, and
+  // makes service startup needlessly more verbose.  See bugzilla 2477
+  // for details.
+  
   ACE_hthread_t current;
   ACE_Thread::self (current);
 
@@ -422,6 +429,7 @@ Worker::svc (void)
     }
 
   ACE_DEBUG ((LM_DEBUG, "Activated Worker Thread to run the ORB @ priority:%d \n", priority));
+#endif
 
   ACE_DECLARE_NEW_CORBA_ENV;
   ACE_TRY
diff --git a/TAO/orbsvcs/tests/Notify/lib/Driver.cpp b/TAO/orbsvcs/tests/Notify/lib/Driver.cpp
index 6c4c0036b4f..8985aa6f772 100644
--- a/TAO/orbsvcs/tests/Notify/lib/Driver.cpp
+++ b/TAO/orbsvcs/tests/Notify/lib/Driver.cpp
@@ -76,6 +76,7 @@ TAO_Notify_Tests_Worker::command_builder (TAO_Notify_Tests_Command_Builder* cmd_
 int
 TAO_Notify_Tests_Worker::svc (void)
 {
+#if 0
   ACE_hthread_t current;
   ACE_Thread::self (current);
 
@@ -87,6 +88,7 @@ TAO_Notify_Tests_Worker::svc (void)
     }
 
   ACE_DEBUG ((LM_ERROR, "Activated Worker Thread for commands @ priority:%d \n", priority));
+#endif
 
   ACE_DECLARE_NEW_CORBA_ENV;
 
@@ -129,6 +131,13 @@ TAO_Notify_Tests_ORB_Run_Worker::run_period (ACE_Time_Value run_period)
 int
 TAO_Notify_Tests_ORB_Run_Worker::svc (void)
 {
+#if 0
+  // ACE_Thread::getprio() fails on systems that do not support thread
+  // priorities.  While we could just treat the failure as benign, I'm
+  // just disabling it altogether.  It doesn't provide much value, and
+  // makes service startup needlessly more verbose.  See bugzilla 2477
+  // for details.
+  
   ACE_hthread_t current;
   ACE_Thread::self (current);
 
@@ -139,8 +148,8 @@ TAO_Notify_Tests_ORB_Run_Worker::svc (void)
       return -1;
     }
 
-
   ACE_DEBUG ((LM_ERROR, "Activated ORB Run Worker Thread to run the ORB @ priority:%d \n", priority));
+#endif
 
   ACE_DECLARE_NEW_CORBA_ENV;
   ACE_TRY