]> git.ipfire.org Git - thirdparty/qemu.git/commit
9pfs: fix off-by-one error in PDU free list
authorGreg Kurz <groug@kaod.org>
Tue, 21 Mar 2017 08:59:16 +0000 (09:59 +0100)
committerMichael Roth <mdroth@linux.vnet.ibm.com>
Tue, 21 Mar 2017 19:53:30 +0000 (14:53 -0500)
commit7f515a96ab191f4eff0983a1c2a27a7a4d54eff8
tree99c37669754ee5e1c3dfb69e4df9bfc2090ee774
parentd437262fa8edd0d9fbe038a515dda3dbf7c5bb54
9pfs: fix off-by-one error in PDU free list

The server can handle MAX_REQ - 1 PDUs at a time and the virtio-9p
device has a MAX_REQ sized virtqueue. If the client manages to fill
up the virtqueue, pdu_alloc() will fail and the request won't be
processed without any notice to the client (it actually causes the
linux 9p client to hang).

This has been there since the beginning (commit 9f10751365b2 "virtio-9p:
Add a virtio 9p device to qemu"), but it needs an agressive workload to
run in the guest to show up.

We actually allocate MAX_REQ PDUs and I see no reason not to link them
all into the free list, so let's fix the init loop.

Reported-by: Tuomas Tynkkynen <tuomas@tuxera.com>
Suggested-by: Al Viro <viro@ZenIV.linux.org.uk>
Signed-off-by: Greg Kurz <groug@kaod.org>
(cherry picked from commit 0d78289c3dca3de8e614a551a3d4a9415168ace0)

Conflicts:
hw/9pfs/9p.c

* drop context dep on 583f21f8

Signed-off-by: Greg Kurz <groug@kaod.org>
Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
hw/9pfs/9p.c