Commit4d4ca24

committed

spinlock emulation: Fix bug when more than INT_MAX spinlocks are initialized.

Once the counter goes negative we ended up with spinlocks that erroredout on first use (due to check in tas_sema).Author: Andres FreundReviewed-By: Robert HaasDiscussion:https://postgr.es/m/20200606023103.avzrctgv7476xj7i@alap3.anarazel.deBackpatch: 9.5-

1 parentfd49d53 commit4d4ca24Copy full SHA for 4d4ca24

File tree

-1

lines changed

-1

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -106,7 +106,7 @@ SpinlockSemaInit(void)`
`106`	`106`	`void`
`107`	`107`	`s_init_lock_sema(volatileslock_t*lock,boolnested)`
`108`	`108`	`{`
`109`		`-staticintcounter=0;`
	`109`	`+staticuint32counter=0;`
`110`	`110`
`111`	`111`	`*lock= ((++counter) %NUM_SPINLOCK_SEMAPHORES)+1;`
`112`	`112`	`}`

Comments

(0)