From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | maryedie(at)osdl(dot)org |
Cc: | pgsql-hackers(at)postgresql(dot)org, Mark Wong <markw(at)osdl(dot)org>, Josh Berkus <josh(at)agliodbs(dot)com>, Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
Subject: | Re: data on devel code perf dip |
Date: | 2005-08-12 01:02:03 |
Message-ID: | 6557.1123808523@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Mary Edie Meredith <maryedie(at)osdl(dot)org> writes:
> I have an example of runs that illustrate a performance
> problem that occurred between installing the 7/18 and 8/1
> development release codes.
I dug through the CVS logs to see what had changed, and I'm afraid there
is just one plausible-looking candidate:
2005-07-28 23:22 momjian
* src/backend/access/transam/xlog.c:
Use O_DIRECT if available when using O_SYNC for wal_sync_method.
Also, write multiple WAL buffers out in one write() operation.
ITAGAKI Takahiro
Most of the CVS activity in that time period had to with stuff like
roles and the interval datatype. It's conceivable that these things
had some marginal performance cost, but if so I'd have expected it to
show up as extra CPU effort (more time checking permissions, say).
This figure:
> samples % app name symbol name
> 164623113 70.5372 kernel-2.6.11.3 .shared_idle
says pretty clearly that your problem is all I/O wait, and there are
no other commits that might have increased our tendency to wait for I/O.
I am sure I will get some pushback if I propose reverting the O_DIRECT
patch, so could you try to get some more-specific evidence? Like pull
the CVS tree from just before and just after this patch and compare
performance?
BTW I did check that both runs are using wal_sync_method = fdatasync
and wal_buffers = 1000, so it's not a problem of those parameters having
been changed by the patch.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | William ZHANG | 2005-08-12 01:26:42 | Re: SELECT for UPDATE and outer join? |
Previous Message | Mary Edie Meredith | 2005-08-11 22:13:15 | data on devel code perf dip |
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2005-08-12 01:31:12 | Re: data on devel code perf dip |
Previous Message | Tom Lane | 2005-08-11 22:56:49 | Re: BUG #1467: fe_connect doesn't handle EINTR right |