Commit 013ae2dd authored by Scott Feldman's avatar Scott Feldman Committed by Jeff Garzik

[PATCH] e100: stepping over err return code

* Spotted by Jay Vosburgh [fubar@us.ibm.com].  err return code was getting
stepped on in the case where we need to report low or no cb resources,
which in turn messed up the netif_stop_queue logic in xmit_frame.

Signed-off-by: scott.feldman@intel.com
parent 981a1ad3
......@@ -827,8 +827,8 @@ static inline int e100_exec_cb(struct nic *nic, struct sk_buff *skb,
cb->prev->command &= cpu_to_le16(~cb_s);
while(nic->cb_to_send != nic->cb_to_use) {
if(unlikely((err = e100_exec_cmd(nic, nic->cuc_cmd,
nic->cb_to_send->dma_addr)))) {
if(unlikely(e100_exec_cmd(nic, nic->cuc_cmd,
nic->cb_to_send->dma_addr))) {
/* Ok, here's where things get sticky. It's
* possible that we can't schedule the command
* because the controller is too busy, so
......
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