This is the mail archive of the cygwin-developers mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Can you undo change in sig_dispatch_pending() ?


On Wed, 31 Oct 2007, Christopher Faylor wrote:

> However, if you think that this check can be put back then it seems like
> the next logical step is to remove the sig_dispatch_pending from readv
> and writev.  Have you tried that?

I was going to try and get back to analyzing this performance issue again,
but I see that you committed this change:

$ cvs diff -r 1.466 -r 1.467 syscalls.cc
Index: syscalls.cc
===================================================================
RCS file: /cvs/src/src/winsup/cygwin/syscalls.cc,v
retrieving revision 1.466
retrieving revision 1.467
diff -u -p -r1.466 -r1.467
--- syscalls.cc 24 Aug 2007 14:55:56 -0000      1.466
+++ syscalls.cc 8 Nov 2007 14:36:49 -0000       1.467
@@ -743,8 +743,6 @@ readv (int fd, const struct iovec *const

   while (1)
     {
-      sig_dispatch_pending ();
-
       cygheap_fdget cfd (fd);
       if (cfd < 0)
        break;
@@ -814,7 +812,6 @@ extern "C" ssize_t
 writev (const int fd, const struct iovec *const iov, const int iovcnt)
 {
   int res = -1;
-  sig_dispatch_pending ();
   const ssize_t tot = check_iovec_for_write (iov, iovcnt);

   cygheap_fdget cfd (fd);

with this ChangeLog entry:

2007-11-08  Christopher Faylor  <me+cygwin@cgf.cx>

        * dllfixdbg: Eliminate extra objcopy step.

on main line.  Which was the oversight, the syscalls.cc change or the
missing ChangeLog entry?  Thanks.

-- 
Brian Ford
Lead Realtime Software Engineer
VITAL - Visual Simulation Systems
FlightSafety International
the best safety device in any aircraft is a well-trained crew...


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]