From: | "Heikki Linnakangas" <heikki(at)enterprisedb(dot)com> |
---|---|
To: | "Simon Riggs" <simon(at)2ndquadrant(dot)com> |
Cc: | "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>, <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Why are we waiting? |
Date: | 2008-02-05 14:14:32 |
Message-ID: | 47A86F48.1010609@enterprisedb.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Simon Riggs wrote:
> On Mon, 2008-02-04 at 17:06 -0500, Tom Lane wrote:
>
>> Basically I'd rather try to attack the problem with dtrace ...
>
> OK. I'll switch to Solaris. Or do you something I don't about dtrace on
> linux?
One idea would be to add new arguments to LWLockAcquire as you suggest,
but instead of modifying all call sites, decorate it with a macro that
passes __FILE__ and __LINE__ as the extra arguments. The good thing
about that is that it's a relatively small patch and you can easily
switch it on/off with a #ifdef. And there's no need to push for
inclusion of that into CVS, because it would be an easy patch to
maintain yourself.
--
Heikki Linnakangas
EnterpriseDB http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Magnus Hagander | 2008-02-05 14:17:28 | Re: path with spaces in config.pl |
Previous Message | Gregory Stark | 2008-02-05 13:45:04 | Re: [COMMITTERS] pgsql: configure tag'd 8.3.0 and built witih autoconf 2.59 |