From: Johannes Schindelin <Johannes.Schindelin@gmx.de>
To: Johannes Sixt <j.sixt@viscovery.net>
Cc: git@vger.kernel.org, Junio C Hamano <gitster@pobox.com>
Subject: Re: [PATCH v2] git daemon: avoid waking up too often
Date: Wed, 23 Jul 2008 11:17:00 +0200 (CEST) [thread overview]
Message-ID: <alpine.DEB.1.00.0807231112310.2830@eeepc-johanness> (raw)
In-Reply-To: <4886D503.7030106@viscovery.net>
Hi,
On Wed, 23 Jul 2008, Johannes Sixt wrote:
> Johannes Schindelin schrieb:
> > To avoid waking up unnecessarily, a pipe is set up that is only ever
> > written to by child_handler(), when a child disconnects, as suggested
> > per Junio.
> >
> > This avoids waking up the main process every second to see if a child
> > was disconnected.
>
> This makes porting this beast to Windows practically impossible because we
> cannot have a poll() implementation that waits both on a listening socket
> and a pipe. :-(
Umm. We also do not have signal handlers, do we? AFAICT you wanted to
simulate _this_ program's fork() with a start_async(), right? All you
have to do is to refactor the code with the pipe to do the update
(properly guarded by a mutex, which is called "CriticalSection" in Windows
speak, since it is impossible for Microsoft to accept common conventions).
The biggest part to get this beast running on Windows is to move it to the
start_async() method, not above-mentioned refactoring.
Ciao,
Dscho
next prev parent reply other threads:[~2008-07-23 9:17 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-07-22 21:52 [PATCH] git daemon: avoid waking up too often Johannes Schindelin
2008-07-22 22:03 ` [PATCH v2] " Johannes Schindelin
2008-07-23 6:51 ` Johannes Sixt
2008-07-23 9:17 ` Johannes Schindelin [this message]
2008-07-23 19:26 ` Avery Pennarun
2008-07-23 19:39 ` Johannes Schindelin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=alpine.DEB.1.00.0807231112310.2830@eeepc-johanness \
--to=johannes.schindelin@gmx.de \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=j.sixt@viscovery.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox