Project

General

Profile

Actions

Feature #15805

closed

Let memory sizes of the various IMEMO object types be reflected correctly

Added by methodmissing (Lourens Naudé) over 5 years ago. Updated over 5 years ago.

Status:
Closed
Assignee:
-
Target version:
-
[ruby-core:92451]

Description

References Github PR https://github.com/ruby/ruby/pull/2140

In current trunk only the imemo_tmpbuf type's auxiliary malloc heap buffer is factored into obj_memsize_of. The following IMEMO types also allocate on the malloc heap:

  • imemo_env (similar pattern as imemo_tmpbuf, an array of VALUEs)
  • imemo_ment (single struct allocated)
  • imemo_iseq (massaged the iseq_memsize API a little)
  • imemo_ast (more difficult to get right, need to walk buffer elements too)

The imemo_memsize function introduced attempts to be the entry point for object size calculation of the IMEMO types.

I think these are important to reflect actual size for heap dumps etc. too. Thoughts?

Actions #1

Updated by nobu (Nobuyoshi Nakada) over 5 years ago

  • Tracker changed from Misc to Feature
Actions #2

Updated by methodmissing (Lourens Naudé) over 5 years ago

  • Status changed from Open to Closed

Applied in changeset git|90c4bd2d2bd10b19c2b09834396553742bc7e8a4.


Let memory sizes of the various IMEMO object types be reflected correctly

[Feature #15805]

Closes: https://github.com/ruby/ruby/pull/2140

Actions

Also available in: Atom PDF

Like0
Like0Like0