diff options
| author | Michael S. Tsirkin <mst@redhat.com> | 2010-09-22 12:31:53 +0200 |
|---|---|---|
| committer | Michael S. Tsirkin <mst@redhat.com> | 2010-09-22 16:21:33 +0200 |
| commit | 5786aee8bf6d747ea59595601a19e78ad33d6929 (patch) | |
| tree | 63d45f00716dea2ab0813361f7af79f15ce24142 | |
| parent | vhost-net: fix range checking in mrg bufs case (diff) | |
| download | linux-5786aee8bf6d747ea59595601a19e78ad33d6929.tar.xz linux-5786aee8bf6d747ea59595601a19e78ad33d6929.zip | |
vhost: fix log ctx signalling
The log eventfd signalling got put in dead code.
We didn't notice because qemu currently does polling
instead of eventfd select.
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
| -rw-r--r-- | drivers/vhost/vhost.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/drivers/vhost/vhost.c b/drivers/vhost/vhost.c index c579dcc9200c..dd3d6f7406f8 100644 --- a/drivers/vhost/vhost.c +++ b/drivers/vhost/vhost.c @@ -858,11 +858,12 @@ int vhost_log_write(struct vhost_virtqueue *vq, struct vhost_log *log, if (r < 0) return r; len -= l; - if (!len) + if (!len) { + if (vq->log_ctx) + eventfd_signal(vq->log_ctx, 1); return 0; + } } - if (vq->log_ctx) - eventfd_signal(vq->log_ctx, 1); /* Length written exceeds what we have stored. This is a bug. */ BUG(); return 0; |