netdev-linux: Return correct error codes on receive.
authorJesse Gross <jesse@nicira.com>
Fri, 2 Oct 2009 17:31:20 +0000 (10:31 -0700)
committerJesse Gross <jesse@nicira.com>
Fri, 2 Oct 2009 17:36:41 +0000 (10:36 -0700)
netdev_linux_receive was returning positive error codes while the
interface specifies that it should be returning negative errors.
This difference causes a huge increase in (non-existant) packet
processing with the userspace datapath.

lib/netdev-linux.c

index 50a0d2d..c71bdd4 100644 (file)
@@ -370,7 +370,7 @@ netdev_linux_recv(struct netdev *netdev_, void *data, size_t size)
 
     if (netdev->tap_fd < 0) {
         /* Device was opened with NETDEV_ETH_TYPE_NONE. */
-        return EAGAIN;
+        return -EAGAIN;
     }
 
     for (;;) {
@@ -382,7 +382,7 @@ netdev_linux_recv(struct netdev *netdev_, void *data, size_t size)
                 VLOG_WARN_RL(&rl, "error receiving Ethernet packet on %s: %s",
                              strerror(errno), netdev_get_name(netdev_));
             }
-            return errno;
+            return -errno;
         }
     }
 }