Re: Add GUC to tune glibc's malloc implementation. - Mailing list pgsql-hackers

From Ronan Dunklau
Subject Re: Add GUC to tune glibc's malloc implementation.
Date
Msg-id 3686246.MHq7AAxBmi@aivenlaptop
Whole thread Raw
In response to Re: Add GUC to tune glibc's malloc implementation.  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Add GUC to tune glibc's malloc implementation.
List pgsql-hackers
Le jeudi 22 juin 2023, 15:49:36 CEST Tom Lane a écrit :
> This seems like a pretty awful idea, mainly because there's no way
> to have such a GUC mean anything on non-glibc platforms, which is
> going to cause confusion or worse.

I named the GUC glibc_malloc_max_trim_threshold, I hope this is enough to
clear up the confusion. We already have at least event_source, which is
windows specific even if it's not clear from the name.

>
> Aren't these same settings controllable via environment variables?
> I could see adding some docs suggesting that you set thus-and-such
> values in the postmaster's startup script.  Admittedly, the confusion
> argument is perhaps still raisable; but we have a similar docs section
> discussing controlling Linux OOM behavior, and I've not heard much
> complaints about that.

Yes they are, but controlling them via an environment variable for the whole
cluster defeats the point: different backends have different workloads, and
being able to make sure for example the OLAP user is memory-greedy while the
OLTP one is as conservative as possible is a worthwile goal.  Or even a
specific backend may want to raise it's work_mem and adapt glibc behaviour
accordingly, then get back to being conservative with memory until the next
such transaction.

Regards,

--
Ronan Dunklau





pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Add GUC to tune glibc's malloc implementation.
Next
From: Melanie Plageman
Date:
Subject: Re: bgwriter doesn't flush WAL stats