Date: Wed, 27 Jan 2016 14:45:42 +0300
From: Aleksander Alekseev <a.alekseev@postgrespro.ru>
To: Dilip Kumar <dilipbalaut@gmail.com>
Cc: "pgsql-hackers@postgresql.org" <pgsql-hackers@postgresql.org>
Subject: Re: Patch: fix lock contention for HASHHDR.mutex
Message-ID: <20160127144542.224c0741@fujitsu>
In-Reply-To: 
 <CAFiTN-srHqMXRU-EO0EnzVEtwAqyqx=SX7YXWkdRotvN0SU_dw@mail.gmail.com>
References: <20151230160137.034371e0@fujitsu>
 <20160112001642.GA802875@alvherre.pgsql>
 <20160112112005.26675392@fujitsu>
 <CAFiTN-vJ2M1tn0yrFsOUn1-anQfzpB2n1SyEA_PBu4zk4KCe_w@mail.gmail.com>
 <20160122131432.2f11dac4@fujitsu>
 <CAFiTN-srHqMXRU-EO0EnzVEtwAqyqx=SX7YXWkdRotvN0SU_dw@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Precedence: bulk
Sender: pgsql-hackers-owner@postgresql.org

> > This patch affects header files. By any chance didn't you forget to
> > run `make clean` after applying it? As we discussed above, when you
> > change .h files autotools doesn't rebuild dependent .c files:
> >
> 
> Yes, actually i always compile using "make clean;make -j20; make
> install" If you want i will run it again may be today or tomorrow and
> post the result.
> 
> 

Most likely HASHHDR.mutex is not only bottleneck in your case so my
patch doesn't help much. Unfortunately I don't have access to any
POWER8 server so I can't investigate this issue. I suggest to use a
gettimeofday trick I described in a first message of this thread. Its
time consuming but it gives a clear understanding which code is keeping
a lock.


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers