From: tenderlove@... Date: 2018-01-17T20:12:57+00:00 Subject: [ruby-core:84909] [Ruby trunk Feature#14370] Directly mark instruction operands and avoid mark_ary usage on rb_iseq_constant_body Issue #14370 has been reported by tenderlovemaking (Aaron Patterson). ---------------------------------------- Feature #14370: Directly mark instruction operands and avoid mark_ary usage on rb_iseq_constant_body https://bugs.ruby-lang.org/issues/14370 * Author: tenderlovemaking (Aaron Patterson) * Status: Open * Priority: Normal * Assignee: ko1 (Koichi Sasada) * Target version: ---------------------------------------- Hi, I've attached a patch that changes rb_iseq_mark to directly mark instruction operands rather than adding them to a mark array. I observed a ~3% memory reduction by directly marking operands, and I didn't observe any difference in GC time. To test memory usage, I used a basic Rails application, logged all malloc / free calls to a file, then wrote a script that would sum the live memory at each sample (each sample being a call to malloc). I graphed these totals so that I could see the memory usage as malloc calls were made: ![memory usage graph](https://user-images.githubusercontent.com/3124/35020270-1b0ded20-fae0-11e7-9cbd-1d028a6c9484.png) The red line is trunk, the blue line is trunk + the patch I've attached. Since the X axis is sample number (not time), the blue line is not as long as the red line because the blue line calls `malloc` fewer times. The Y axis in the graph is the total number of "live" bytes that have been allocated (all allocations minus their corresponding frees). You can see from the graph that memory savings start adding up as more code gets loaded. I was concerned that this patch might impact GC time, but `make gcbench-rdoc` didn't seem to show any significant difference in GC time between trunk and this patch. If it turns out there is a performance impact, I think I could improve the time while still keeping memory usage low by generating a bitmap during iseq compilation. There is a bit more information where I've been working, but I think I've summarized everything here. https://github.com/github/ruby/pull/39 ---Files-------------------------------- iseq_mark.diff (6.28 KB) -- https://bugs.ruby-lang.org/ Unsubscribe: