Re: 9.4 regression

From: Stephen Frost <sfrost(at)snowman(dot)net>
To: Andres Freund <andres(at)2ndquadrant(dot)com>
Cc: Jon Nelson <jnelson+pgsql(at)jamponi(dot)net>, Bruce Momjian <bruce(at)momjian(dot)us>, Thom Brown <thom(at)linux(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: 9.4 regression
Date: 2013-08-20 01:27:57
Message-ID: 20130820012757.GT2706@tamriel.snowman.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

* Andres Freund (andres(at)2ndquadrant(dot)com) wrote:
> I vote for adapting the patch to additionally zero out the file via
> write(). In your tests that seemed to perform at least as good as the
> old method... It also has the advantage that we can use it a littlebit
> more as a testbed for possibly using it for heap extensions one day.
> We're pretty early in the cycle, so I am not worried about this too much...

I dunno, I'm pretty disappointed that this doesn't actually improve
things. Just following this casually, it looks like it might be some
kind of locking issue in the kernel that's causing it to be slower; or
at least some code path that isn't exercise terribly much and therefore
hasn't been given the love that it should.

Definitely interested in what Ts'o says, but if we can't figure out why
it's slower *without* writing out the zeros, I'd say we punt on this
until Linux and the other OS folks improve the situation.

Thanks,

Stephen

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Stephen Frost 2013-08-20 01:30:33 Re: [PATCH] pg_sleep(interval)
Previous Message Noah Misch 2013-08-20 00:35:00 Re: Fix Windows socket error checking for MinGW