| From: | Andres Freund <andres(at)2ndquadrant(dot)com> |
|---|---|
| To: | Peter Geoghegan <pg(at)heroku(dot)com> |
| Cc: | pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: Wait free LW_SHARED acquisition - v0.2 |
| Date: | 2014-01-31 09:54:19 |
| Message-ID: | 20140131095419.GB13199@alap3.anarazel.de |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
On 2014-01-28 21:27:29 -0800, Peter Geoghegan wrote:
> On Fri, Nov 15, 2013 at 11:47 AM, Andres Freund <andres(at)2ndquadrant(dot)com> wrote:
> > 1) I've added an abstracted atomic ops implementation. Needs a fair
> > amount of work, also submitted as a separate CF entry. (Patch 1 & 2)
>
> Commit 220b34331f77effdb46798ddd7cca0cffc1b2858 caused bitrot when
> applying 0002-Very-basic-atomic-ops-implementation.patch. Please
> rebase.
I've pushed a rebased version of the patchset to
http://git.postgresql.org/gitweb/?p=users/andresfreund/postgres.git
branch rwlock contention.
220b34331f77effdb46798ddd7cca0cffc1b2858 actually was the small problem,
ea9df812d8502fff74e7bc37d61bdc7d66d77a7f was the major PITA.
I plan to split the atomics patch into smaller chunks before
reposting. Imo the "Convert the PGPROC->lwWaitLink list into a dlist
instead of open coding it." is worth being applied independently from
the rest of the series, it simplies code and it fixes a bug...
Greetings,
Andres Freund
--
Andres Freund http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Sawada Masahiko | 2014-01-31 09:59:05 | Re: [PATCH] pg_basebackup: progress report max once per second |
| Previous Message | Andres Freund | 2014-01-31 09:40:23 | Re: Patch: show xid and xmin in pg_stat_activity and pg_stat_replication |