Assume safe iteration of workers in the arbiter

With reaping handled in the main loop and kill_worker delegating
responsibility for cleanup to the reaping loop, iterate over the workers
dictionary everywhere else without concern for concurrent modification.

gunicorn/arbiter.py

@@ -512,8 +512,7 @@ def murder_workers(self):
         """
         if not self.timeout:
             return
-        workers = list(self.WORKERS.items())
-        for (pid, worker) in workers:
+        for (pid, worker) in self.WORKERS.items():
             try:
                 if time.time() - worker.tmp.last_update() <= self.timeout:
                     continue
@@ -644,8 +643,7 @@ def kill_workers(self, sig):
         Kill all workers with the signal `sig`
         :attr sig: `signal.SIG*` value
         """
-        worker_pids = list(self.WORKERS.keys())
-        for pid in worker_pids:
+        for pid in self.WORKERS:
             self.kill_worker(pid, sig)
 
     def kill_worker(self, pid, sig):