On Tue, Sep 1, 2026 at 9:21 AM David Geier <[email protected]> wrote:
> On 13.07.2026 00:44, Ayoub Kazar wrote: > > > I chose to keep an accurate running count of the memory footprint per > > backend by tracking both the sizeof(Vfd) and the exact filename string > > lengths. > > > > We add the string length to the total footprint when a file is opened, > and > > subtract it when the VFD is freed (i suppose this is not "Too much" of a > > work done, although it's done in a bit hot place); v5 of the patch is > > attached. > Better use GetMemoryChunkSpace() instead of using strlen() + 1, to get > the true allocation size. > That wouldn't work because filename is malloc'd and not palloc'd, which is what GetMemoryChunkSpace() works on IIUC. Am i correct here? Regards, Ayoub
