NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit7b063ff

committed

Search for SLRU page only in its own bank

One of the two slot scans in SlruSelectLRUPage was not walking only theslots in the specific bank where the buffer could be; change it to dothat.Oversight in53c2a97.Author: Sergey Sargsyan <sergey.sargsyan.2001@gmail.com>Discussion:https://postgr.es/m/18582-5f301dd30ba91a38@postgresql.org

1 parent2793ace commit7b063ffCopy full SHA for 7b063ff

File tree

1 file changed

-1

lines changed

src/backend/access/transam
- slru.c

1 file changed

-1

lines changed

`‎src/backend/access/transam/slru.c`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -1184,7 +1184,7 @@ SlruSelectLRUPage(SlruCtl ctl, int64 pageno)`
`1184`	`1184`	`Assert(LWLockHeldByMe(SimpleLruGetBankLock(ctl,pageno)));`
`1185`	`1185`
`1186`	`1186`	`/* See if page already has a buffer assigned */`
`1187`		`-for (intslotno=0;slotno<shared->num_slots;slotno++)`
	`1187`	`+for (intslotno=bankstart;slotno<bankend;slotno++)`
`1188`	`1188`	`{`
`1189`	`1189`	`if (shared->page_status[slotno]!=SLRU_PAGE_EMPTY&&`
`1190`	`1190`	`shared->page_number[slotno]==pageno)`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit7b063ff

File tree

1 file changed

1 file changed

`‎src/backend/access/transam/slru.c`

0 commit comments