diff options
| author |   David Vrabel <david.vrabel@citrix.com> | 2014-10-22 14:08:53 +0100 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2014-10-25 14:15:20 -0400 |
| commit | bc96f648df1bbc2729abbb84513cf4f64273a1f1 (patch) | |
| tree | ad8044147004ba38641bc727bf3b0f866c2e05ac /drivers/net/xen-netback/xenbus.c | |
| parent | ptp: restore the makefile for building the test program. (diff) | |
| download | linux-dev-bc96f648df1bbc2729abbb84513cf4f64273a1f1.tar.xz linux-dev-bc96f648df1bbc2729abbb84513cf4f64273a1f1.zip | |
xen-netback: make feature-rx-notify mandatory
Frontends that do not provide feature-rx-notify may stall because
netback depends on the notification from frontend to wake the guest Rx
thread (even if can_queue is false).
This could be fixed but feature-rx-notify was introduced in 2006 and I
am not aware of any frontends that do not implement this.
Signed-off-by: David Vrabel <david.vrabel@citrix.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/xen-netback/xenbus.c')
| -rw-r--r-- | drivers/net/xen-netback/xenbus.c | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/drivers/net/xen-netback/xenbus.c b/drivers/net/xen-netback/xenbus.c index 8079c31ac5e6..9060857c9022 100644 --- a/drivers/net/xen-netback/xenbus.c +++ b/drivers/net/xen-netback/xenbus.c @@ -873,15 +873,10 @@ static int read_xenbus_vif_flags(struct backend_info *be) if (!rx_copy) return -EOPNOTSUPP; - if (vif->dev->tx_queue_len != 0) { - if (xenbus_scanf(XBT_NIL, dev->otherend, - "feature-rx-notify", "%d", &val) < 0) - val = 0; - if (val) - vif->can_queue = 1; - else - /* Must be non-zero for pfifo_fast to work. */ - vif->dev->tx_queue_len = 1; + if (xenbus_scanf(XBT_NIL, dev->otherend, + "feature-rx-notify", "%d", &val) < 0 || val == 0) { + xenbus_dev_fatal(dev, -EINVAL, "feature-rx-notify is mandatory"); + return -EINVAL; } if (xenbus_scanf(XBT_NIL, dev->otherend, "feature-sg", |