* exceptions.cc (set_signal_mask): Remove useless debugging output.

* fhandler.cc (fhandler_base::write): Ditto.
(fhandler_base_overlapped::close): Cancel any ongoing I/O before closing.
* syscalls.cc (write): Default to always reporting all writes in strace output
via syscall_printf.
* wait.cc (wait4): Fix debugging output.  Use standard syscall leaver output.

diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog
index fed985c..aa9d379 100644 (file)
--- a/winsup/cygwin/ChangeLog
+++ b/winsup/cygwin/ChangeLog
@@ -1,3 +1,15 @@
+2011-12-17  Christopher Faylor  <me.cygwin2011@cgf.cx>
+
+       * exceptions.cc (set_signal_mask): Remove useless debugging output.
+
+       * fhandler.cc (fhandler_base::write): Ditto.
+       (fhandler_base_overlapped::close): Cancel any ongoing I/O before
+       closing.
+       * syscalls.cc (write): Default to always reporting all writes in strace
+       output via syscall_printf.
+       * wait.cc (wait4): Fix debugging output.  Use standard syscall leaver
+       output.
+

 2011-12-16  Christopher Faylor  <me.cygwin2011@cgf.cx>
 
        Implement fhandler reference counting.
 2011-12-16  Christopher Faylor  <me.cygwin2011@cgf.cx>
 
        Implement fhandler reference counting.

diff --git a/winsup/cygwin/exceptions.cc b/winsup/cygwin/exceptions.cc
index 3fcd91c..a15c9c3 100644 (file)
--- a/winsup/cygwin/exceptions.cc
+++ b/winsup/cygwin/exceptions.cc
@@ -1144,8 +1144,6 @@ set_signal_mask (sigset_t newmask, sigset_t& oldmask)
   oldmask = newmask;
   if (mask_bits)
     sig_dispatch_pending (true);
   oldmask = newmask;
   if (mask_bits)
     sig_dispatch_pending (true);

-  else
-    sigproc_printf ("not calling sig_dispatch_pending");

   mask_sync.release ();
 }
 
   mask_sync.release ();
 }
 

diff --git a/winsup/cygwin/fhandler.cc b/winsup/cygwin/fhandler.cc
index 04499e6..6624bc7 100644 (file)
--- a/winsup/cygwin/fhandler.cc
+++ b/winsup/cygwin/fhandler.cc
@@ -839,16 +839,13 @@ fhandler_base::write (const void *ptr, size_t len)
          NTSTATUS status;
          status = NtFsControlFile (get_output_handle (), NULL, NULL, NULL,
                                    &io, FSCTL_SET_SPARSE, NULL, 0, NULL, 0);
          NTSTATUS status;
          status = NtFsControlFile (get_output_handle (), NULL, NULL, NULL,
                                    &io, FSCTL_SET_SPARSE, NULL, 0, NULL, 0);

-         syscall_printf ("%p = NtFsControlFile(%S, FSCTL_SET_SPARSE)",
-                         status, pc.get_nt_native_path ());
+         debug_printf ("%p = NtFsControlFile(%S, FSCTL_SET_SPARSE)",
+                       status, pc.get_nt_native_path ());

        }
     }
 
   if (wbinary ())
        }
     }
 
   if (wbinary ())

-    {
-      debug_printf ("binary write");
-      res = raw_write (ptr, len);
-    }
+    res = raw_write (ptr, len);

   else
     {
       debug_printf ("text write");
   else
     {
       debug_printf ("text write");


@@ -1222,6 +1219,10 @@ fhandler_base_overlapped::close ()
     }
   else
     {
     }
   else
     {

+      /* Cancelling seems to be necessary for cases where a reader is
+        still executing either in another thread or when a signal handler
+        performs a close.  */
+      CancelIo (get_io_handle ());

       destroy_overlapped ();
       res = fhandler_base::close ();
     }
       destroy_overlapped ();
       res = fhandler_base::close ();
     }

diff --git a/winsup/cygwin/syscalls.cc b/winsup/cygwin/syscalls.cc
index f77c451..cdf3c85 100644 (file)
--- a/winsup/cygwin/syscalls.cc
+++ b/winsup/cygwin/syscalls.cc
@@ -1170,10 +1170,7 @@ write (int fd, const void *ptr, size_t len)
   res = cfd->write (ptr, len);
 
 done:
   res = cfd->write (ptr, len);
 
 done:

-  if (fd == 1 || fd == 2)
-    paranoid_printf ("%R = write(%d, %p, %d)", res, fd, ptr, len);
-  else
-    syscall_printf ("%R = write(%d, %p, %d)", res, fd, ptr, len);
+  syscall_printf ("%R = write(%d, %p, %d)", res, fd, ptr, len);

 
   MALLOC_CHECK;
   return res;
 
   MALLOC_CHECK;
   return res;

diff --git a/winsup/cygwin/wait.cc b/winsup/cygwin/wait.cc
index 0d965e1..91a7623 100644 (file)
--- a/winsup/cygwin/wait.cc
+++ b/winsup/cygwin/wait.cc
@@ -81,7 +81,7 @@ wait4 (int intpid, int *status, int options, struct rusage *r)
 
       res = cancelable_wait (waitfor);
 
 
       res = cancelable_wait (waitfor);
 

-      sigproc_printf ("%d = WaitForSingleObject (...)", res);
+      sigproc_printf ("%d = cancelable_wait (...)", res);

 
       if (w->ev == NULL)
        {
 
       if (w->ev == NULL)
        {


@@ -109,10 +109,7 @@ wait4 (int intpid, int *status, int options, struct rusage *r)
       break;
     }
 
       break;
     }
 

-  sigproc_printf ("intpid %d, status %p, w->status %d, options %d, res %d",
-                 intpid, status, w->status, options, res);
+  syscall_printf ("%R = wait4(%d, %p, %d, %p)", res, intpid, w->status, options, r);

   w->status = -1;
   w->status = -1;

-  if (res < 0)
-    sigproc_printf ("*** errno %d", get_errno ());

   return res;
 }
   return res;
 }