Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1jYufp-0001Zx-Th for pgsql-hackers@arkaria.postgresql.org; Wed, 13 May 2020 16:56:01 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1jYufo-0007zd-LF for pgsql-hackers@arkaria.postgresql.org; Wed, 13 May 2020 16:56:00 +0000 Received: from makus.postgresql.org ([2001:4800:3e1:1::229]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1jYufo-0007zK-Bx for pgsql-hackers@lists.postgresql.org; Wed, 13 May 2020 16:56:00 +0000 Received: from sss.pgh.pa.us ([66.207.139.130]) by makus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1jYufm-0005hQ-3u for pgsql-hackers@postgresql.org; Wed, 13 May 2020 16:55:59 +0000 Received: from sss1.sss.pgh.pa.us (localhost [127.0.0.1]) by sss.pgh.pa.us (8.14.4/8.14.4) with ESMTP id 04DGtogD007839; Wed, 13 May 2020 12:55:50 -0400 From: Tom Lane To: Fujii Masao cc: Alvaro Herrera , Tomas Vondra , Masahiko Sawada , "tsunakawa.takay@fujitsu.com" , "pgsql-hackers@postgresql.org" Subject: Re: SLRU statistics In-reply-to: References: <20200513161406.GA437@alvherre.pgsql> Comments: In-reply-to Fujii Masao message dated "Thu, 14 May 2020 01:41:57 +0900" MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <7837.1589388950.1@sss.pgh.pa.us> Date: Wed, 13 May 2020 12:55:50 -0400 Message-ID: <7838.1589388950@sss.pgh.pa.us> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Precedence: bulk Fujii Masao writes: > But since the counter that postmaster incremented is propagated to > child processes via fork, it should be zeroed at postmaster or the > beginning of child process? Otherwise that counter always starts > with non-zero in child process. Yes, if the postmaster is incrementing these counts then we would have to reset them at the start of each child process. I share Alvaro's feeling that that's bad and we don't want to do it. >> (IIRC only the Async module is doing that.) Hm, maybe we can fix that. regards, tom lane