Obsolete comments about semaphores in proc.c

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

Obsolete comments about semaphores in proc.c

Thomas Munro-5
Hi,

Commit 6753333f switched from a semaphore-based waiting to latch-based
waiting for ProcSleep()/ProcWakeup(), but left behind some stray
references to semaphores.  PSA.

--
Thomas Munro
https://enterprisedb.com

0001-Remove-obsolete-comments-about-sempahores-from-proc..patch (3K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: Obsolete comments about semaphores in proc.c

Daniel Gustafsson
> On 14 Jun 2019, at 01:00, Thomas Munro <[hidden email]> wrote:

> Commit 6753333f switched from a semaphore-based waiting to latch-based
> waiting for ProcSleep()/ProcWakeup(), but left behind some stray
> references to semaphores.  PSA.

LGTM

cheers ./daniel


Reply | Threaded
Open this post in threaded view
|

Re: Obsolete comments about semaphores in proc.c

Michael Paquier-2
On Mon, Jun 17, 2019 at 01:28:42PM +0200, Daniel Gustafsson wrote:
>> On 14 Jun 2019, at 01:00, Thomas Munro <[hidden email]> wrote:
>
>> Commit 6753333f switched from a semaphore-based waiting to latch-based
>> waiting for ProcSleep()/ProcWakeup(), but left behind some stray
>> references to semaphores.  PSA.
>
> LGTM

Fine seen from here as well.  I am not spotting other areas, FWIW.
--
Michael

signature.asc (849 bytes) Download Attachment