Qingqing Zhou <zhouqq(at)cs(dot)toronto(dot)edu> writes:
> There is no LWLock protecting the spinlock in UnpinBuffer(),
I don't believe this is necessary, and if it is necessary the patch
is wrong anyway. Because the code uses _NoHoldoff, there won't be
any check of InterruptPending in that segment of code. Furthermore,
any callers who don't have their own interrupt holdoff in force are
probably broken --- it's unlikely that the state of bufmgr is globally
correct if the caller of UnpinBuffer loses control.
Exactly which path of control are you worried about?
regards, tom lane
In response to
pgsql-patches by date
|Next:||From: Qingqing Zhou||Date: 2005-12-29 00:04:28|
|Subject: Re: Fix spinlock usage in UnpinBuffer() |
|Previous:||From: Qingqing Zhou||Date: 2005-12-28 19:09:28|
|Subject: Fix spinlock usage in UnpinBuffer()|