Bug #15270: [PATCH] Fix TracePoint for code loaded using ISeq.load_from_binary

• Status: Closed
• Assignee: ko1 (Koichi Sasada)
• Target version: -
• Backport: 2.3: UNKNOWN, 2.4: UNKNOWN, 2.5: REQUIRED

[alanwu (Alan Wu)]

Hello. This is a patch for Bug #14702, which I don't think is fully addressed.
The tests in the patch should fail on both trunk and ruby_2_5.

    Fix TracePoint for nested iseq loaded from binary [Bug #14702]

    When loading iseq from binary while a TracePoint is on, we need to
    recompile instructions to their "trace_" variant. Before this commit
    we only recompiled instructions in the top level iseq, which meant
    that TracePoint was malfunctioning for code inside module/class/method
    definitions.

    * compile.c: Move rb_iseq_init_trace to rb_ibf_load_iseq_complete.
      It is called on all iseqs during loading.

    * test_iseq.rb: Test that tracepoints fire within children iseq when
      using load_from_binary.

[alanwu (Alan Wu)]

• File 0001-Fix-TracePoint-for-nested-iseq-loaded-from-binary.patch 0001-Fix-TracePoint-for-nested-iseq-loaded-from-binary.patch added

[alanwu (Alan Wu)]

• File deleted (0001-Fix-TracePoint-for-nested-iseq-loaded-from-binary.patch)

[alanwu (Alan Wu)]

• Description updated (diff)

Typos

[ko1 (Koichi Sasada)]

• Status changed from Open to Closed
• Assignee set to ko1 (Koichi Sasada)

Thank you! You are completely right.

applied at r65567 (I forget to add ticket reference on commit log).

[rafaelfranca (Rafael França)]

@ko1 (Koichi Sasada) can you mark this issue to be backported to 2.5.x please?

[nagachika (Tomoyuki Chikanaga)]

• Backport changed from 2.3: UNKNOWN, 2.4: UNKNOWN, 2.5: UNKNOWN to 2.3: UNKNOWN, 2.4: UNKNOWN, 2.5: REQUIRED

[nagachika (Tomoyuki Chikanaga)]

• Backport changed from 2.3: UNKNOWN, 2.4: UNKNOWN, 2.5: REQUIRED to 2.3: UNKNOWN, 2.4: UNKNOWN, 2.5: DONE

ruby_2_5 r66225 merged revision(s) 64736,65567.

[nagachika (Tomoyuki Chikanaga)]

• Related to Bug #15717: Backport #15270 to Ruby 2.5.x added

[nagachika (Tomoyuki Chikanaga)]

• Backport changed from 2.3: UNKNOWN, 2.4: UNKNOWN, 2.5: DONE to 2.3: UNKNOWN, 2.4: UNKNOWN, 2.5: REQUIRED

I'll set Backport field for 2.5 as REQUIRED since https://bugs.ruby-lang.org/issues/15717 claims the issue was not fixed on ruby_2_5.

[nagachika (Tomoyuki Chikanaga)]

merge revision(s) 64736,65567: [Backport #15270]

    iseq.c: prefix rb_ to non-static iseq functions

    I assume we always prefix rb_ to non-static functions to avoid conflict.
    These functions are not exported and safe to be renamed.

    iseq.h: ditto
    compile.c: ditto

    Fix TracePoint for nested iseq loaded from binary [Bug#14702]

    When loading iseq from binary while a TracePoint is on, we need to
    recompile instructions to their "trace_" variant. Before this commit
    we only recompiled instructions in the top level iseq, which meant
    that TracePoint was malfunctioning for code inside module/class/method
    definitions.

    * compile.c: Move rb_iseq_init_trace to rb_ibf_load_iseq_complete.
      It is called on all iseqs during loading.

    * test_iseq.rb: Test that tracepoints fire within children iseq when
      using load_from_binary.

    This patch is from: Alan Wu <[e-mail]>

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_5@66225 b2dd03c8-39d4-4d8f-98ff-823fe69b080e