diff options
| author | Andrei Vagin <avagin@google.com> | 2024-11-15 21:35:00 +0000 |
|---|---|---|
| committer | Dmitry Vyukov <dvyukov@google.com> | 2024-11-18 10:07:10 +0000 |
| commit | e0aa7963ba56a926d1a741e7f580d24ce94c4e65 (patch) | |
| tree | 6bfe42b766735a1a126001982ec38c3ed28c80ce /pkg/rpcserver | |
| parent | e7bb5d6eaa12da79db8e670ea78af25fb5a49e02 (diff) | |
executor: use any executor if the avoid mask included all of them
After 9fc8fe026baa ("executor: better handling for hanged test
processes"), yz-executor's responses may reference procids outside of
the [0;procs] range.
If procids are no longer dense on the syz-executor side, we cannot rely
on this check in pkg/rpcserver:
```
if avoid == (uint64(1)<<runner.procs)-1 {
avoid = 0
}
```
Signed-off-by: Andrei Vagin <avagin@google.com>
Diffstat (limited to 'pkg/rpcserver')
| -rw-r--r-- | pkg/rpcserver/runner.go | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/pkg/rpcserver/runner.go b/pkg/rpcserver/runner.go index 1af35b6d2..29e79bad5 100644 --- a/pkg/rpcserver/runner.go +++ b/pkg/rpcserver/runner.go @@ -321,9 +321,6 @@ func (runner *Runner) sendRequest(req *queue.Request) error { avoid |= uint64(1 << id.Proc) } } - if avoid == (uint64(1)<<runner.procs)-1 { - avoid = 0 - } msg := &flatrpc.HostMessage{ Msg: &flatrpc.HostMessages{ Type: flatrpc.HostMessagesRawExecRequest, |