Обсуждение: BufferSync() & FlushRelationBuffers() conflict
Just committed changes in bufmgr.c Regress tests passed but need more specific tests, as usually. Descr as in CVS: > Check bufHdr->cntxDirty and call StartBufferIO in BufferSync() > *before* acquiring shlock on buffer context. This way we should be > protected against conflicts with FlushRelationBuffers. > (Seems we never do excl lock and then StartBufferIO for the same > buffer, so there should be no deadlock here, - but we'd better > check this very soon). Vadim
Tom, since you appear to be able to recreate the bug, can you comment on this, as to whether we are okay now? On Wed, 21 Mar 2001, Vadim Mikheev wrote: > Just committed changes in bufmgr.c > Regress tests passed but need more specific tests, > as usually. Descr as in CVS: > > > Check bufHdr->cntxDirty and call StartBufferIO in BufferSync() > > *before* acquiring shlock on buffer context. This way we should be > > protected against conflicts with FlushRelationBuffers. > > (Seems we never do excl lock and then StartBufferIO for the same > > buffer, so there should be no deadlock here, - but we'd better > > check this very soon). > > Vadim > > > > ---------------------------(end of broadcast)--------------------------- > TIP 4: Don't 'kill -9' the postmaster > Marc G. Fournier ICQ#7615664 IRC Nick: Scrappy Systems Administrator @ hub.org primary: scrappy@hub.org secondary: scrappy@{freebsd|postgresql}.org
The Hermit Hacker <scrappy@hub.org> writes: > Tom, since you appear to be able to recreate the bug, can you comment on > this, as to whether we are okay now? Sorry for the delay --- I was down in Norfolk all day, and am just now catching up on email. I will pull Vadim's update and run the test some more. However, last night I only saw the failure once in about an hour's worth of testing, so it's not that easy to reproduce anyway... regards, tom lane
okay, baring you bein able to recreate the bug between now and, say, 13:00AST tomorrow, I'll wrap up RC1 and get her out the door ... On Wed, 21 Mar 2001, Tom Lane wrote: > The Hermit Hacker <scrappy@hub.org> writes: > > Tom, since you appear to be able to recreate the bug, can you comment on > > this, as to whether we are okay now? > > Sorry for the delay --- I was down in Norfolk all day, and am just now > catching up on email. I will pull Vadim's update and run the test some > more. However, last night I only saw the failure once in about an > hour's worth of testing, so it's not that easy to reproduce anyway... > > regards, tom lane > > ---------------------------(end of broadcast)--------------------------- > TIP 3: if posting/reading through Usenet, please send an appropriate > subscribe-nomail command to majordomo@postgresql.org so that your > message can get through to the mailing list cleanly > Marc G. Fournier ICQ#7615664 IRC Nick: Scrappy Systems Administrator @ hub.org primary: scrappy@hub.org secondary: scrappy@{freebsd|postgresql}.org
> > Tom, since you appear to be able to recreate the bug, can you comment on > > this, as to whether we are okay now? > > Sorry for the delay --- I was down in Norfolk all day, and am just now > catching up on email. I will pull Vadim's update and run the test some > more. However, last night I only saw the failure once in about an > hour's worth of testing, so it's not that easy to reproduce anyway... I saw >~ 10 failures with -B 32 in ~ 3 minutes of testing. With old code, of course -:) Vadim