Obsolete comments about semaphores in proc.c

Started by Thomas Munroalmost 7 years ago3 messageshackers
Jump to latest
#1Thomas Munro
thomas.munro@gmail.com

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

Attachments:

0001-Remove-obsolete-comments-about-sempahores-from-proc..patchapplication/octet-stream; name=0001-Remove-obsolete-comments-about-sempahores-from-proc..patchDownload+6-9
#2Daniel Gustafsson
daniel@yesql.se
In reply to: Thomas Munro (#1)
Re: Obsolete comments about semaphores in proc.c

On 14 Jun 2019, at 01:00, Thomas Munro <thomas.munro@gmail.com> 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

#3Michael Paquier
michael@paquier.xyz
In reply to: Daniel Gustafsson (#2)
Re: Obsolete comments about semaphores in proc.c

On Mon, Jun 17, 2019 at 01:28:42PM +0200, Daniel Gustafsson wrote:

On 14 Jun 2019, at 01:00, Thomas Munro <thomas.munro@gmail.com> 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