Enable preemption in d8.

- Preserve bootstrapper state across thread switches (fixes issue 143).
- Make sure stack guards are properly setup even when preemption is active.
- Fix missing v8::Locker and v8::Unlocker use in d8.cc.
- Threads forked in d8 do get their own context setup.


Committed: http://code.google.com/p/v8/source/detail?r=1161

[iposva, 2009-01-26 05:25:27]

I will file the bug for the TODO after this change is reviewed.

-Ivan

[Mads Ager (chromium), 2009-01-26 07:42:51]

LGTM

http://codereview.chromium.org/18581/diff/1/4
File src/bootstrapper.cc (right):

http://codereview.chromium.org/18581/diff/1/4#newcode268
Line 268: // Support for thread preemption.
Do you need these declarations?  The same declarations are in bootstrapper.h.

[iposva, 2009-01-26 17:32:13]

Reply to comment and small update to top.cc

-Ivan

http://codereview.chromium.org/18581/diff/1/4
File src/bootstrapper.cc (right):

http://codereview.chromium.org/18581/diff/1/4#newcode268
Line 268: // Support for thread preemption.
On 2009/01/26 07:42:52, Mads Ager wrote:
> Do you need these declarations?  The same declarations are in bootstrapper.h.

Yes, these declarations are needed because Bootstrapper has to delegate to the
Genesis class to do the archival and retrieval of per thread state.

[Mads Ager (chromium), 2009-01-26 17:50:42]

LGTM!