Re: [PATCHES] Reorganization of spinlock defines - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: [PATCHES] Reorganization of spinlock defines
Date
Msg-id 200309120418.h8C4Irj07692@candle.pha.pa.us
Whole thread Raw
In response to Re: [PATCHES] Reorganization of spinlock defines  ("Marc G. Fournier" <scrappy@postgresql.org>)
List pgsql-hackers
Marc G. Fournier wrote:
> > > >From what I understand, "not working properly" means slow, not broken, no?
> > > Which means ppl could submit a problem report and it could be fixed for
> > > v7.4.1 ... its not so much  'not working properly' as it is 'not optimal
> > > performance' ...
> >
> > Right, though I am not sure people will know _slow_ configuration vs.
> > PostgreSQL is slow.
> 
> No, but definitely something for those discussion performance to add
> to their checklist :)
> 
> BTW, post-compile, running system ... how do you check this?  Or can you?
> 
> For instance, having a checklist, of sorts, that ppl can go through when
> trying to investigate performance issues could include stuff like:
> 
> check swap usage (albeit obvious to alot of ppl, not to all)
> check disk usage using iostat
> check spinlocks in use using ... ??

We can add the compiler test that will throw an error if they aren't
using spinlocks, and they have to use a configure flag to enable it. 
The only issue there is that this is going to throw up perhaps many
cases we haven't gotten working, and they might dribble out for weeks or
after final, while a clean solution could break more platforms in the
short term, but could catch more in the long term.

--  Bruce Momjian                        |  http://candle.pha.pa.us pgman@candle.pha.pa.us               |  (610)
359-1001+  If your life is a hard drive,     |  13 Roberts Road +  Christ can be your backup.        |  Newtown Square,
Pennsylvania19073
 


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [PATCHES] Reorganization of spinlock defines
Next
From: Tom Lane
Date:
Subject: Heads up: beta3 planned for Monday