Content-Type: multipart/mixed; boundary="------------1rJD1N2wK3d2zYQf15aGiEbm"
Message-ID: <a05ca971-a8ea-422f-85cd-b5edc46c5a9a@vondra.me>
Date: Tue, 28 Oct 2025 16:14:43 +0100
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: failed NUMA pages inquiry status: Operation not permitted
To: Christoph Berg <myon@debian.org>
Cc: Jakub Wartak <jakub.wartak@enterprisedb.com>,
 pgsql-hackers@lists.postgresql.org
References: <E1u1tr8-003BbN-2E@gemulon.postgresql.org>
 <aPDZOxjrmEo_1JRG@msg.df7cb.de>
 <7bbc582b-cc70-4a6f-bbf2-b5fd9b13a867@vondra.me>
 <aPEHINwaAxsQNAIP@msg.df7cb.de> <aPEJ7yH7ZJAF_MYW@msg.df7cb.de>
 <aPENGAkcrLyB_NLC@msg.df7cb.de>
Content-Language: en-US
From: Tomas Vondra <tomas@vondra.me>
In-Reply-To: <aPENGAkcrLyB_NLC@msg.df7cb.de>
Archived-At: 
 <https://www.postgresql.org/message-id/a05ca971-a8ea-422f-85cd-b5edc46c5a9a%40vondra.me>
Precedence: bulk

This is a multi-part message in MIME format.
--------------1rJD1N2wK3d2zYQf15aGiEbm
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit

On 10/16/25 17:19, Christoph Berg wrote:
>> So maybe all that's needed is a get_mempolicy() call in
>> pg_numa_available() ?
> 
> ...
> 
> So maybe PG should implement numa_available itself like that. (Or
> accept the output difference so the regression tests are passing.)
> 

I'm not sure which of those options is better. I'm a bit worried just
accepting the alternative output would hide some failures in the future
(although it's a low risk).

So I'm leaning to adjust pg_numa_init() to also check EPERM, per the
attached patch. It still calls numa_available(), so that we don't
silently miss future libnuma changes.

Can you check this makes it work inside the docker container?


regards

-- 
Tomas Vondra
--------------1rJD1N2wK3d2zYQf15aGiEbm
Content-Type: text/x-patch; charset=UTF-8;
 name="0001-Handle-EPERM-in-pg_numa_init.patch"
Content-Disposition: attachment;
 filename="0001-Handle-EPERM-in-pg_numa_init.patch"
Content-Transfer-Encoding: base64

RnJvbSBiNTU1MGFlNmY1YmFjM2RlMTRhODZhMGY3Njc3ZGI3NTViMjdhYTczIE1vbiBTZXAg
MTcgMDA6MDA6MDAgMjAwMQpGcm9tOiBUb21hcyBWb25kcmEgPHRvbWFzQHZvbmRyYS5tZT4K
RGF0ZTogVHVlLCAyOCBPY3QgMjAyNSAxNjowMDowNyArMDEwMApTdWJqZWN0OiBbUEFUQ0hd
IEhhbmRsZSBFUEVSTSBpbiBwZ19udW1hX2luaXQKCi0tLQogc3JjL3BvcnQvcGdfbnVtYS5j
IHwgMTIgKysrKysrKysrKystCiAxIGZpbGUgY2hhbmdlZCwgMTEgaW5zZXJ0aW9ucygrKSwg
MSBkZWxldGlvbigtKQoKZGlmZiAtLWdpdCBhL3NyYy9wb3J0L3BnX251bWEuYyBiL3NyYy9w
b3J0L3BnX251bWEuYwppbmRleCAzMzY4YTQzYTMzOC4uNTQwYWRhM2Y4ZWYgMTAwNjQ0Ci0t
LSBhL3NyYy9wb3J0L3BnX251bWEuYworKysgYi9zcmMvcG9ydC9wZ19udW1hLmMKQEAgLTQ3
LDcgKzQ3LDE3IEBACiBpbnQKIHBnX251bWFfaW5pdCh2b2lkKQogewotCWludAkJCXIgPSBu
dW1hX2F2YWlsYWJsZSgpOworCWludAkJCXI7CisKKwkvKgorCSAqIFhYWCBsaWJudW1hIHZl
cnNpb25zIGJlZm9yZSAyLjAuMTkgZG9uJ3QgaGFuZGxlIEVQRVJNIGJ5IGRpc2FibGluZwor
CSAqIE5VTUEsIHdoaWNoIHRoZW4gbGVhZHMgdG8gdW5leHBlY3RlZCBmYWlsdXJlcyBsYXRl
ci4gVGhpcyBhZmZlY3RzCisJICogY29udGFpbmVycyB0aGF0IGRpc2FibGUgZ2V0X21lbXBv
bGljeSBieSBhIHNlY2NvbXAgcHJvZmlsZS4KKwkgKi8KKwlpZiAoZ2V0X21lbXBvbGljeShO
VUxMLCBOVUxMLCAwLCAwLCAwKSA8IDAgJiYgKGVycm5vID09IEVQRVJNKSkKKwkJciA9IC0x
OworCWVsc2UKKwkJciA9IG51bWFfYXZhaWxhYmxlKCk7CiAKIAlyZXR1cm4gcjsKIH0KLS0g
CjIuNTEuMAoK

--------------1rJD1N2wK3d2zYQf15aGiEbm--