Ah, sorry for the late response (and for any confusion), but the only
thing I tested was Qingqing's rewritten semaphore implementation.  I
didn't test the proposed bug fixes to the existing semaphore
implementation.  

We've never been able reproduce (or even trigger) the original "sem_ctl
failed" error in a testing environment so it would be hard to say if the
changes to win32/sema.c have an impact on it or not.  On the other hand,
win32_sema.c seems to solve the pgbench lockups reported earlier by Jim
N. and it successfully completes a reasonably brutal stress test with
real world data and real world queries (which at least is a good
indication that it basically works).

Pete

>>> Bruce Momjian <pgman@candle.pha.pa.us> 05/11/06 4:08 am >>>
Tom Lane wrote:
> Bruce Momjian <pgman@candle.pha.pa.us> writes:
> >> Well we did have a bug report by Peter Brant, and a test by him
with the
> >> patch that fixes it, so it seems it should be applied.  The URLs
I
> >> posted had that information.


---------------------------(end of broadcast)---------------------------
TIP 3: Have you checked our extensive FAQ?

               http://www.postgresql.org/docs/faq

Reply via email to