]> granicus.if.org Git - postgresql/commit
Revise bgwriter fsync-request mechanism to improve robustness when a table
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 17 Jan 2007 00:17:21 +0000 (00:17 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 17 Jan 2007 00:17:21 +0000 (00:17 +0000)
commit6d660587f631d5c23d72ed791e37f62844e65372
treebdc7bc6601687786f668ee71fcdebe2b34b984ed
parent7f58ed1a106981e020df549e842e93d2250bbe75
Revise bgwriter fsync-request mechanism to improve robustness when a table
is deleted.  A backend about to unlink a file now sends a "revoke fsync"
request to the bgwriter to make it clean out pending fsync requests.  There
is still a race condition where the bgwriter may try to fsync after the unlink
has happened, but we can resolve that by rechecking the fsync request queue
to see if a revoke request arrived meanwhile.  This eliminates the former
kluge of "just assuming" that an ENOENT failure is okay, and lets us handle
the fact that on Windows it might be EACCES too without introducing any
questionable assumptions.  After an idea of mine improved by Magnus.

The HEAD patch doesn't apply cleanly to 8.2, but I'll see about a back-port
later.  In the meantime this could do with some testing on Windows; I've been
able to force it through the code path via ENOENT, but that doesn't prove that
it actually fixes the Windows problem ...
src/backend/postmaster/bgwriter.c
src/backend/storage/smgr/md.c