On Wed, Aug 20, 2014 at 2:11 AM, Jeff Davis <pg...@j-davis.com> wrote: > I attached a patch that uses two uint32 fields so that it doesn't > increase the size of MemoryContextData, and it tracks memory usage for > all contexts. I was unable to detect any performance regression vs. > master, but on my machine the results are noisy.
Still doesn't look good here. On the same PPC64 machine I've been using for testing: master: 2014-08-22 16:26:42 UTC [13153] LOG: internal sort ended, 1723933 KB used: CPU 2.19s/11.41u sec elapsed 16.40 sec 2014-08-22 16:28:18 UTC [13173] LOG: internal sort ended, 1723933 KB used: CPU 2.56s/11.37u sec elapsed 16.64 sec 2014-08-22 16:29:37 UTC [13194] LOG: internal sort ended, 1723933 KB used: CPU 2.57s/11.48u sec elapsed 16.75 sec memory-accounting-v4: 2014-08-22 16:27:35 UTC [13163] LOG: internal sort ended, 1723933 KB used: CPU 2.72s/11.91u sec elapsed 17.43 sec 2014-08-22 16:29:10 UTC [13185] LOG: internal sort ended, 1723933 KB used: CPU 2.67s/12.03u sec elapsed 17.40 sec 2014-08-22 16:30:01 UTC [13203] LOG: internal sort ended, 1723933 KB used: CPU 2.65s/11.97u sec elapsed 17.45 sec That's a 4.7% regression on the median results, worse than before. > It would be easier to resolve the performance concern if I could > reliably get the results Robert is getting. I think I was able to > reproduce the regression with the old patch, but the results were still > noisy. If you send me an SSH key by private email I can set you an account up on this machine, if that's helpful. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
