Add profiling to remoteActionBuildingSemaphore.acquire()

Without this there can be large gaps in the profile when using remote execution that are hard to reason about.

Closes bazelbuild#20474.

PiperOrigin-RevId: 590475989
Change-Id: Ic6e042c36f85e8098a468c73c62bd45cc367423e

src/main/java/com/google/devtools/build/lib/remote/RemoteExecutionService.java

@@ -517,7 +517,10 @@ private ToolSignature getToolSignature(Spawn spawn, SpawnExecutionContext contex
   /** Creates a new {@link RemoteAction} instance from spawn. */
   public RemoteAction buildRemoteAction(Spawn spawn, SpawnExecutionContext context)
       throws IOException, ExecException, ForbiddenActionInputException, InterruptedException {
-    remoteActionBuildingSemaphore.acquire();
+    try (SilentCloseable c =
+        Profiler.instance().profile(ProfilerTask.REMOTE_SETUP, "acquiring semaphore")) {
+      remoteActionBuildingSemaphore.acquire();
+    }
     try {
       ToolSignature toolSignature = getToolSignature(spawn, context);
       final MerkleTree merkleTree = buildInputMerkleTree(spawn, context, toolSignature);
@@ -1460,7 +1463,10 @@ public void uploadInputsIfNotPresent(RemoteAction action, boolean force)
     // concurrency. This prevents memory exhaustion. We assume that
     // ensureInputsPresent() provides enough parallelism to saturate the
     // network connection.
-    remoteActionBuildingSemaphore.acquire();
+    try (SilentCloseable c =
+        Profiler.instance().profile(ProfilerTask.UPLOAD_TIME, "acquiring semaphore")) {
+      remoteActionBuildingSemaphore.acquire();
+    }
     try {
       MerkleTree merkleTree = action.getMerkleTree();
       if (merkleTree == null) {