diff options
| author | Jason Wang <jasowang@redhat.com> | 2013-01-28 01:05:18 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2013-01-29 15:43:03 -0500 |
| commit | 2b8b328b61c799957a456a5a8dab8cc7dea68575 (patch) | |
| tree | 7f782227156eb9aca1a0f3c9cd097e778bc8b7af /kernel/module_signing.c | |
| parent | 692a998b908ae4c612d95d1f5f5adae03eca2b79 (diff) | |
vhost_net: handle polling errors when setting backend
Currently, the polling errors were ignored, which can lead following issues:
- vhost remove itself unconditionally from waitqueue when stopping the poll,
this may crash the kernel since the previous attempt of starting may fail to
add itself to the waitqueue
- userspace may think the backend were successfully set even when the polling
failed.
Solve this by:
- check poll->wqh before trying to remove from waitqueue
- report polling errors in vhost_poll_start(), tx_poll_start(), the return value
will be checked and returned when userspace want to set the backend
After this fix, there still could be a polling failure after backend is set, it
will addressed by the next patch.
Signed-off-by: Jason Wang <jasowang@redhat.com>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'kernel/module_signing.c')
0 files changed, 0 insertions, 0 deletions