[Libvir] PATCH: Process QEMU monitor at startup
Mark McLoughlin
markmc at redhat.com
Fri Mar 2 15:41:21 UTC 2007
On Fri, 2007-03-02 at 15:34 +0000, Daniel P. Berrange wrote:
> On Fri, Mar 02, 2007 at 03:06:56PM +0000, Mark McLoughlin wrote:
> >
> > index() is a bit odd, why not strstr() ?
>
> Well we're only looking for a single character match - index() is for
> single characters, strstr() is for strings.
Sorry, strchr() (which is in the C standard itself, whereas index() is
marked legacy in POSIX ... apparently)
> > > + if (connect(fd, (struct sockaddr*)&addr, sizeof(addr)) == 0)
> >
> > Um, why not bind() (with SO_REUSEADDR)?
>
> No particular reason - either will work just fine.
I've never noticed it done with connect() anywhere else, I guess
because you'd have the same obvious race ...
I dunno, bind() will fail quickly when we probe a port, whereas
connect() will succeed more slowly and cause another context switch etc.
Not a big deal, but ...
Cheers,
Mark.
More information about the libvir-list
mailing list