From: | "Vadim Mikheev" <vmikheev(at)sectorbase(dot)com> |
---|---|
To: | "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Btree runtime recovery. Stuck spins. |
Date: | 2001-02-09 02:58:38 |
Message-ID: | 005f01c09244$341e3440$4a79583f@sectorbase.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> > Shouldn't we increase S_MAX_BUSY and use ERROR instead of FATAL?
>
> No. If you have delays exceeding a minute, or that are even a visible
> fraction of a minute, then a spinlock is NOT the correct mechanism to be
> using to wait ... because guess what, it's spinning, and consuming
> processor time to no purpose. You should be using a lock instead for
> anything that involves more than a trivial amount of delay.
"Amount of delay" depends on system load - something we can't control...
Btree uses spins to lock buffers (as all other access methods) and so
I could use only spins in new code. And though tree recovery locks buffers
for longer time than normal insert operations it's possible to get
"stuck" spins when using concurrent buffers locks *everywhere* under
heavy load (especially with WAL which requires holding buffer locks
for duration of logging).
So, probably we have to use some kind of light locks (without deadlock
detection) for buffers, in future.
Vadim
From | Date | Subject | |
---|---|---|---|
Next Message | Nathan Myers | 2001-02-09 03:38:47 | Re: Syslog and pg_options (for RPMs) |
Previous Message | Justin Clift | 2001-02-09 02:33:30 | SPI_exec - Trying to access SPI_tuptable - error of 'dereferencing pointer to incomplete type' |