Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Fix an issue when empty the usap pool."

Browse Source
This commit is contained in:
Treehugger Robot
2021-11-29 07:46:36 +00:00
committed by Gerrit Code Review
parent 286d24fd0d 1bc2b9c612
commit faf1b1cf0b
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
core/java/com/android/internal/os/Zygote.java
View File

@@ -724,9 +724,6 @@ public final class Zygote {
DataOutputStream usapOutputStream = null; DataOutputStream usapOutputStream = null;
ZygoteArguments args = null; ZygoteArguments args = null;
// Block SIGTERM so we won't be killed if the Zygote flushes the USAP pool.
blockSigTerm();
LocalSocket sessionSocket = null; LocalSocket sessionSocket = null;
if (argBuffer == null) { if (argBuffer == null) {
// Read arguments from usapPoolSocket instead. // Read arguments from usapPoolSocket instead.
@@ -742,6 +739,10 @@ public final class Zygote {
ZygoteCommandBuffer tmpArgBuffer = null; ZygoteCommandBuffer tmpArgBuffer = null;
try { try {
sessionSocket = usapPoolSocket.accept(); sessionSocket = usapPoolSocket.accept();
// Block SIGTERM so we won't be killed if the Zygote flushes the USAP pool.
// This is safe from a race condition because the pool is only flushed after
// the SystemServer changes its internal state to stop using the USAP pool.
blockSigTerm();
usapOutputStream = usapOutputStream =
new DataOutputStream(sessionSocket.getOutputStream()); new DataOutputStream(sessionSocket.getOutputStream());
@@ -759,9 +760,10 @@ public final class Zygote {
unblockSigTerm(); unblockSigTerm();
IoUtils.closeQuietly(sessionSocket); IoUtils.closeQuietly(sessionSocket);
IoUtils.closeQuietly(tmpArgBuffer); IoUtils.closeQuietly(tmpArgBuffer);
blockSigTerm();
} }
} else { } else {
// Block SIGTERM so we won't be killed if the Zygote flushes the USAP pool.
blockSigTerm();
try { try {
args = ZygoteArguments.getInstance(argBuffer); args = ZygoteArguments.getInstance(argBuffer);
} catch (Exception ex) { } catch (Exception ex) {