helps to attach, eh?
On Mon, Jul 19, 2004 at 06:18:49PM -0400, Chris Shoemaker wrote: > On Sun, Jul 18, 2004 at 08:37:03PM -0700, Wayne Davison wrote: > > On Sun, Jul 18, 2004 at 06:20:59PM -0400, Chris Shoemaker wrote: > > > Could a simplified version of this index notification take place over > > > the existing error-pipe pair? > > > > The data is traveling in the opposite direction for what we need (and > > it's not bidirectional). > > > > Ok, how about this: Instead of index notification, run the generator > and receiver serially. > > I've attached a concept patch. I don't like creating such a > special-case for read-batch, but it seems like it might solve the > problem you found. > > OTOH, I can see some benefit to mainlining some sort of g2r index > notification just as an on-going sanity check. > > -chris > > > ..wayne.. > -- > To unsubscribe or change options: http://lists.samba.org/mailman/listinfo/rsync > Before posting, read: http://www.catb.org/~esr/faqs/smart-questions.html
Index: main.c =================================================================== RCS file: /cvsroot/rsync/main.c,v retrieving revision 1.208 diff -u -r1.208 main.c --- main.c 19 Jul 2004 17:11:41 -0000 1.208 +++ main.c 20 Jul 2004 03:04:53 -0000 @@ -463,6 +463,18 @@ io_flush(NORMAL_FLUSH); + if (read_batch) { + io_start_buffering_out(); + set_msg_fd_in(error_pipe[0]); + set_msg_fd_out(error_pipe[1]); + send_msg(MSG_DONE, "", 0); + generate_files(f_out, flist, local_name); + recv_files(f_in, flist, local_name); + io_flush(FULL_FLUSH); + report(f_in); + return 0; + } + if ((pid = do_fork()) == 0) { close(error_pipe[0]); if (f_in != f_out)
-- To unsubscribe or change options: http://lists.samba.org/mailman/listinfo/rsync Before posting, read: http://www.catb.org/~esr/faqs/smart-questions.html