Commit 26c0e102 authored by David Vrabel's avatar David Vrabel Committed by David S. Miller

xen-netback: support frontends without feature-rx-notify again

Commit bc96f648 (xen-netback: make
feature-rx-notify mandatory) incorrectly assumed that there were no
frontends in use that did not support this feature.  But the frontend
driver in MiniOS does not and since this is used by (qemu) stubdoms,
these stopped working.

Netback sort of works as-is in this mode except:

- If there are no Rx requests and the internal Rx queue fills, only
  the drain timeout will wake the thread.  The default drain timeout
  of 10 s would give unacceptable pauses.

- If an Rx stall was detected and the internal Rx queue is drained,
  then the Rx thread would never wake.

Handle these two cases (when feature-rx-notify is disabled) by:

- Reducing the drain timeout to 30 ms.

- Disabling Rx stall detection.
Reported-by: default avatarJohn <jw@nuclearfallout.net>
Tested-by: default avatarJohn <jw@nuclearfallout.net>
Signed-off-by: default avatarDavid Vrabel <david.vrabel@citrix.com>
Reviewed-by: default avatarWei Liu <wei.liu2@citrix.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 0e324cf6
...@@ -230,6 +230,8 @@ struct xenvif { ...@@ -230,6 +230,8 @@ struct xenvif {
*/ */
bool disabled; bool disabled;
unsigned long status; unsigned long status;
unsigned long drain_timeout;
unsigned long stall_timeout;
/* Queues */ /* Queues */
struct xenvif_queue *queues; struct xenvif_queue *queues;
...@@ -328,7 +330,7 @@ irqreturn_t xenvif_interrupt(int irq, void *dev_id); ...@@ -328,7 +330,7 @@ irqreturn_t xenvif_interrupt(int irq, void *dev_id);
extern bool separate_tx_rx_irq; extern bool separate_tx_rx_irq;
extern unsigned int rx_drain_timeout_msecs; extern unsigned int rx_drain_timeout_msecs;
extern unsigned int rx_drain_timeout_jiffies; extern unsigned int rx_stall_timeout_msecs;
extern unsigned int xenvif_max_queues; extern unsigned int xenvif_max_queues;
#ifdef CONFIG_DEBUG_FS #ifdef CONFIG_DEBUG_FS
......
...@@ -166,7 +166,7 @@ static int xenvif_start_xmit(struct sk_buff *skb, struct net_device *dev) ...@@ -166,7 +166,7 @@ static int xenvif_start_xmit(struct sk_buff *skb, struct net_device *dev)
goto drop; goto drop;
cb = XENVIF_RX_CB(skb); cb = XENVIF_RX_CB(skb);
cb->expires = jiffies + rx_drain_timeout_jiffies; cb->expires = jiffies + vif->drain_timeout;
xenvif_rx_queue_tail(queue, skb); xenvif_rx_queue_tail(queue, skb);
xenvif_kick_thread(queue); xenvif_kick_thread(queue);
...@@ -414,6 +414,8 @@ struct xenvif *xenvif_alloc(struct device *parent, domid_t domid, ...@@ -414,6 +414,8 @@ struct xenvif *xenvif_alloc(struct device *parent, domid_t domid,
vif->ip_csum = 1; vif->ip_csum = 1;
vif->dev = dev; vif->dev = dev;
vif->disabled = false; vif->disabled = false;
vif->drain_timeout = msecs_to_jiffies(rx_drain_timeout_msecs);
vif->stall_timeout = msecs_to_jiffies(rx_stall_timeout_msecs);
/* Start out with no queues. */ /* Start out with no queues. */
vif->queues = NULL; vif->queues = NULL;
......
...@@ -60,14 +60,12 @@ module_param(separate_tx_rx_irq, bool, 0644); ...@@ -60,14 +60,12 @@ module_param(separate_tx_rx_irq, bool, 0644);
*/ */
unsigned int rx_drain_timeout_msecs = 10000; unsigned int rx_drain_timeout_msecs = 10000;
module_param(rx_drain_timeout_msecs, uint, 0444); module_param(rx_drain_timeout_msecs, uint, 0444);
unsigned int rx_drain_timeout_jiffies;
/* The length of time before the frontend is considered unresponsive /* The length of time before the frontend is considered unresponsive
* because it isn't providing Rx slots. * because it isn't providing Rx slots.
*/ */
static unsigned int rx_stall_timeout_msecs = 60000; unsigned int rx_stall_timeout_msecs = 60000;
module_param(rx_stall_timeout_msecs, uint, 0444); module_param(rx_stall_timeout_msecs, uint, 0444);
static unsigned int rx_stall_timeout_jiffies;
unsigned int xenvif_max_queues; unsigned int xenvif_max_queues;
module_param_named(max_queues, xenvif_max_queues, uint, 0644); module_param_named(max_queues, xenvif_max_queues, uint, 0644);
...@@ -2020,7 +2018,7 @@ static bool xenvif_rx_queue_stalled(struct xenvif_queue *queue) ...@@ -2020,7 +2018,7 @@ static bool xenvif_rx_queue_stalled(struct xenvif_queue *queue)
return !queue->stalled return !queue->stalled
&& prod - cons < XEN_NETBK_RX_SLOTS_MAX && prod - cons < XEN_NETBK_RX_SLOTS_MAX
&& time_after(jiffies, && time_after(jiffies,
queue->last_rx_time + rx_stall_timeout_jiffies); queue->last_rx_time + queue->vif->stall_timeout);
} }
static bool xenvif_rx_queue_ready(struct xenvif_queue *queue) static bool xenvif_rx_queue_ready(struct xenvif_queue *queue)
...@@ -2038,8 +2036,9 @@ static bool xenvif_have_rx_work(struct xenvif_queue *queue) ...@@ -2038,8 +2036,9 @@ static bool xenvif_have_rx_work(struct xenvif_queue *queue)
{ {
return (!skb_queue_empty(&queue->rx_queue) return (!skb_queue_empty(&queue->rx_queue)
&& xenvif_rx_ring_slots_available(queue, XEN_NETBK_RX_SLOTS_MAX)) && xenvif_rx_ring_slots_available(queue, XEN_NETBK_RX_SLOTS_MAX))
|| xenvif_rx_queue_stalled(queue) || (queue->vif->stall_timeout &&
|| xenvif_rx_queue_ready(queue) (xenvif_rx_queue_stalled(queue)
|| xenvif_rx_queue_ready(queue)))
|| kthread_should_stop() || kthread_should_stop()
|| queue->vif->disabled; || queue->vif->disabled;
} }
...@@ -2092,6 +2091,9 @@ int xenvif_kthread_guest_rx(void *data) ...@@ -2092,6 +2091,9 @@ int xenvif_kthread_guest_rx(void *data)
struct xenvif_queue *queue = data; struct xenvif_queue *queue = data;
struct xenvif *vif = queue->vif; struct xenvif *vif = queue->vif;
if (!vif->stall_timeout)
xenvif_queue_carrier_on(queue);
for (;;) { for (;;) {
xenvif_wait_for_rx_work(queue); xenvif_wait_for_rx_work(queue);
...@@ -2118,10 +2120,12 @@ int xenvif_kthread_guest_rx(void *data) ...@@ -2118,10 +2120,12 @@ int xenvif_kthread_guest_rx(void *data)
* while it's probably not responsive, drop the * while it's probably not responsive, drop the
* carrier so packets are dropped earlier. * carrier so packets are dropped earlier.
*/ */
if (xenvif_rx_queue_stalled(queue)) if (vif->stall_timeout) {
xenvif_queue_carrier_off(queue); if (xenvif_rx_queue_stalled(queue))
else if (xenvif_rx_queue_ready(queue)) xenvif_queue_carrier_off(queue);
xenvif_queue_carrier_on(queue); else if (xenvif_rx_queue_ready(queue))
xenvif_queue_carrier_on(queue);
}
/* Queued packets may have foreign pages from other /* Queued packets may have foreign pages from other
* domains. These cannot be queued indefinitely as * domains. These cannot be queued indefinitely as
...@@ -2192,9 +2196,6 @@ static int __init netback_init(void) ...@@ -2192,9 +2196,6 @@ static int __init netback_init(void)
if (rc) if (rc)
goto failed_init; goto failed_init;
rx_drain_timeout_jiffies = msecs_to_jiffies(rx_drain_timeout_msecs);
rx_stall_timeout_jiffies = msecs_to_jiffies(rx_stall_timeout_msecs);
#ifdef CONFIG_DEBUG_FS #ifdef CONFIG_DEBUG_FS
xen_netback_dbg_root = debugfs_create_dir("xen-netback", NULL); xen_netback_dbg_root = debugfs_create_dir("xen-netback", NULL);
if (IS_ERR_OR_NULL(xen_netback_dbg_root)) if (IS_ERR_OR_NULL(xen_netback_dbg_root))
......
...@@ -887,9 +887,15 @@ static int read_xenbus_vif_flags(struct backend_info *be) ...@@ -887,9 +887,15 @@ static int read_xenbus_vif_flags(struct backend_info *be)
return -EOPNOTSUPP; return -EOPNOTSUPP;
if (xenbus_scanf(XBT_NIL, dev->otherend, if (xenbus_scanf(XBT_NIL, dev->otherend,
"feature-rx-notify", "%d", &val) < 0 || val == 0) { "feature-rx-notify", "%d", &val) < 0)
xenbus_dev_fatal(dev, -EINVAL, "feature-rx-notify is mandatory"); val = 0;
return -EINVAL; if (!val) {
/* - Reduce drain timeout to poll more frequently for
* Rx requests.
* - Disable Rx stall detection.
*/
be->vif->drain_timeout = msecs_to_jiffies(30);
be->vif->stall_timeout = 0;
} }
if (xenbus_scanf(XBT_NIL, dev->otherend, "feature-sg", if (xenbus_scanf(XBT_NIL, dev->otherend, "feature-sg",
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment