Implement nacl_irt_thread and nacl_irt_tls for non-sfi mode.

Implement nacl_irt_thread and nacl_irt_tls for non-sfi mode.

This CL implements nacl_irt_thread and nacl_irt_tls.

BUG=https://code.google.com/p/nativeclient/issues/detail?id=3734
TEST=Tried to call a newly added function from plugin via AT_SYSINFO.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=245123

[hidehiko, 2014-01-15 09:50:53]

Thank you for your review in advance,
- hidehiko

[Mark Seaborn, 2014-01-15 19:06:39]

LGTM

https://codereview.chromium.org/133683011/diff/1/components/nacl/loader/nonsf...
File components/nacl/loader/nonsfi/irt_thread.cc (right):

https://codereview.chromium.org/133683011/diff/1/components/nacl/loader/nonsf...
components/nacl/loader/nonsfi/irt_thread.cc:52: return NULL;
Can you call abort() instead?  The user's function is not supposed to return.

https://codereview.chromium.org/133683011/diff/1/components/nacl/loader/nonsf...
components/nacl/loader/nonsfi/irt_thread.cc:97: return ENOSYS;
Since 'nice' is just a hint, it would be better to return 0, so that this
doesn't break any programs that call this.

[hidehiko, 2014-01-16 03:17:43]

Thank you for your review. Submitting...

https://codereview.chromium.org/133683011/diff/1/components/nacl/loader/nonsf...
File components/nacl/loader/nonsfi/irt_thread.cc (right):

https://codereview.chromium.org/133683011/diff/1/components/nacl/loader/nonsf...
components/nacl/loader/nonsfi/irt_thread.cc:52: return NULL;
On 2014/01/15 19:06:39, Mark Seaborn wrote:
> Can you call abort() instead?  The user's function is not supposed to return.

Done.

https://codereview.chromium.org/133683011/diff/1/components/nacl/loader/nonsf...
components/nacl/loader/nonsfi/irt_thread.cc:97: return ENOSYS;
On 2014/01/15 19:06:39, Mark Seaborn wrote:
> Since 'nice' is just a hint, it would be better to return 0, so that this
> doesn't break any programs that call this.

Done.

[commit-bot: I haz the power, 2014-01-16 03:19:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/hidehiko@chromium.org/133683011/100001

[commit-bot: I haz the power, 2014-01-16 04:59:00]

Change committed as 245123

[Mark Seaborn, 2014-01-16 19:04:39]

https://codereview.chromium.org/133683011/diff/100001/components/nacl/loader/...
File components/nacl/loader/nonsfi/irt_thread.cc (right):

https://codereview.chromium.org/133683011/diff/100001/components/nacl/loader/...
components/nacl/loader/nonsfi/irt_thread.cc:99: // do anything.
Nit: "doing" (I don't mind whether you fix this, but you can TBR a change for it
if you want)