Merge "Document bionic's targetSdkVersion-specific behavior."
diff --git a/docs/status.md b/docs/status.md
index 6467143..0aaa0b3 100644
--- a/docs/status.md
+++ b/docs/status.md
@@ -1,6 +1,69 @@
Android bionic status
=====================
+# Target API level behavioral differences
+
+Most bionic bug fixes and improvements have been made without checks for
+the app's `targetSdkVersion`. As of O there were exactly two exceptions,
+but there are likely to be more in future because of Project Treble.
+
+Invalid `pthread_t` handling (targetSdkVersion >= O)
+----------------------------------------------------
+
+As part of a long-term goal to remove the global thread list,
+and in an attempt to flush out racy code, we changed how an invalid
+`pthread_t` is handled. For `pthread_detach`, `pthread_getcpuclockid`,
+`pthread_getschedparam`/`pthread_setschedparam`, `pthread_join`, and
+`pthread_kill`, instead of returning ESRCH when passed an invalid
+`pthread_t`, if you're targeting O or above, they'll abort with the
+message "attempt to use invalid pthread\_t".
+
+Note that this doesn't change behavior as much as you might think: the
+old lookup only held the global thread list lock for the duration of
+the lookup, so there was still a race between that and the dereference
+in the caller, given that callers actually need the tid to pass to some
+syscall or other, and sometimes update fields in the `pthread_internal_t`
+struct too.
+
+We can't check a thread's tid against 0 to see whether a `pthread_t`
+is still valid because a dead thread gets its thread struct unmapped
+along with its stack, so the dereference isn't safe.
+
+To fix your code, taking the affected functions one by one:
+
+ * `pthread_getcpuclockid` and `pthread_getschedparam`/`pthread_setschedparam`
+ should be fine. Unsafe calls to those seem highly unlikely.
+
+ * Unsafe `pthread_detach` callers probably want to switch to
+ `pthread_attr_setdetachstate` instead, or use
+ `pthread_detach(pthread_self());` from the new thread's start routine
+ rather than calling detach in the parent.
+
+ * `pthread_join` calls should be safe anyway, because a joinable thread
+ won't actually exit and unmap until it's joined. If you're joining an
+ unjoinable thread, the fix is to stop marking it detached. If you're
+ joining an already-joined thread, you need to rethink your design!
+
+ * Unsafe `pthread_kill` calls aren't portably fixable. (And are obviously
+ inherently non-portable as-is.) The best alternative on Android is to
+ use `pthread_gettid_np` at some point that you know the thread to be
+ alive, and then call `kill`/`tgkill` with signal 0 (which checks
+ whether a process exists rather than actually sending a
+ signal). That's still not completely safe because if you're too late
+ the tid may have been reused, but your code is inherently unsafe without
+ a redesign anyway.
+
+Interruptable `sem_wait` (targetSdkVersion >= N)
+------------------------------------------------
+
+POSIX says that `sem_wait` can be interrupted by delivery of a
+signal. This wasn't historically true in Android, and when we fixed this
+bug we found that existing code relied on the old behavior. To preserve
+compatibility, `sem_wait` can only return EINTR on Android if the app
+targets N or later.
+
+# Bionic function availability
+
libc
----
@@ -8,18 +71,19 @@
New libc functions in P:
* `__freading`/`__fwriting` (completing <stdio_ext.h>)
- * `endhostent`/endnetent`/`endprotoent`/`getnetent`/`getprotoent`/`sethostent`/`setnetent`/`setprotoent` (completing <netdb.h>)
+ * `endhostent`/`endnetent`/`endprotoent`/`getnetent`/`getprotoent`/`sethostent`/`setnetent`/`setprotoent` (completing <netdb.h>)
* `fexecve`
* `fflush_unlocked`/`fgetc_unlocked`/`fgets_unlocked`/`fputc_unlocked`/`fputs_unlocked`/`fread_unlocked`/`fwrite_unlocked`
* `getentropy`/`getrandom` (adding <sys/random.h>)
* `getlogin_r`
* `glob`/`globfree` (adding <glob.h>)
- * `hcreate`/hcreate_r`/`hdestroy`/`hdestroy_r`/`hsearch`/`hsearch_r` (completing <search.h>)
+ * `hcreate`/`hcreate_r`/`hdestroy`/`hdestroy_r`/`hsearch`/`hsearch_r` (completing <search.h>)
* `iconv`/`iconv_close`/`iconv_open` (adding <iconv.h>)
* `pthread_attr_getinheritsched`/`pthread_attr_setinheritsched`/`pthread_setschedprio`
* <spawn.h>
* `swab`
* `syncfs`
+ * %C and %S support in the printf family (previously only the wprintf family supported these).
New libc functions in O:
* `sendto` FORTIFY support