diff options
| author | Mike Pall <mike> | 2013-11-05 19:36:59 +0100 |
|---|---|---|
| committer | Mike Pall <mike> | 2013-11-05 19:36:59 +0100 |
| commit | 7b4d5306c7441a373516ee54bd942218f35348c2 (patch) | |
| tree | b2963d0af64c5282189f74244fa2d550338c4380 | |
| parent | 8944e27762712679e50962598ed1be8dc1fbe7be (diff) | |
| download | luajit-7b4d5306c7441a373516ee54bd942218f35348c2.tar.gz luajit-7b4d5306c7441a373516ee54bd942218f35348c2.tar.bz2 luajit-7b4d5306c7441a373516ee54bd942218f35348c2.zip | |
Fix GC steps threshold handling when called by JIT-compiled code.
Thanks to Cheng, Long.
| -rw-r--r-- | src/lj_gc.c | 6 | ||||
| -rw-r--r-- | src/lj_trace.c | 2 |
2 files changed, 5 insertions, 3 deletions
diff --git a/src/lj_gc.c b/src/lj_gc.c index 5c665786..7012dbec 100644 --- a/src/lj_gc.c +++ b/src/lj_gc.c | |||
| @@ -689,12 +689,14 @@ int LJ_FASTCALL lj_gc_step(lua_State *L) | |||
| 689 | } while ((int32_t)lim > 0); | 689 | } while ((int32_t)lim > 0); |
| 690 | if (g->gc.debt < GCSTEPSIZE) { | 690 | if (g->gc.debt < GCSTEPSIZE) { |
| 691 | g->gc.threshold = g->gc.total + GCSTEPSIZE; | 691 | g->gc.threshold = g->gc.total + GCSTEPSIZE; |
| 692 | g->vmstate = ostate; | ||
| 693 | return 1; | ||
| 692 | } else { | 694 | } else { |
| 693 | g->gc.debt -= GCSTEPSIZE; | 695 | g->gc.debt -= GCSTEPSIZE; |
| 694 | g->gc.threshold = g->gc.total; | 696 | g->gc.threshold = g->gc.total; |
| 697 | g->vmstate = ostate; | ||
| 698 | return 0; | ||
| 695 | } | 699 | } |
| 696 | g->vmstate = ostate; | ||
| 697 | return 0; | ||
| 698 | } | 700 | } |
| 699 | 701 | ||
| 700 | /* Ditto, but fix the stack top first. */ | 702 | /* Ditto, but fix the stack top first. */ |
diff --git a/src/lj_trace.c b/src/lj_trace.c index c70fc247..add9da6d 100644 --- a/src/lj_trace.c +++ b/src/lj_trace.c | |||
| @@ -778,7 +778,7 @@ int LJ_FASTCALL lj_trace_exit(jit_State *J, void *exptr) | |||
| 778 | setcframe_pc(cf, pc); | 778 | setcframe_pc(cf, pc); |
| 779 | if (G(L)->gc.state == GCSatomic || G(L)->gc.state == GCSfinalize) { | 779 | if (G(L)->gc.state == GCSatomic || G(L)->gc.state == GCSfinalize) { |
| 780 | if (!(G(L)->hookmask & HOOK_GC)) | 780 | if (!(G(L)->hookmask & HOOK_GC)) |
| 781 | lj_gc_step(L); /* Exited because of GC: drive GC forward. */ | 781 | lj_gc_check(L); /* Exited because of GC: drive GC forward. */ |
| 782 | } else { | 782 | } else { |
| 783 | trace_hotside(J, pc); | 783 | trace_hotside(J, pc); |
| 784 | } | 784 | } |
