Apply error and success logic consistently to the function netisr_queue() and

its users.

netisr_queue() now returns (0) on success and ERRNO on failure.  At the
moment ENXIO (netisr queue not functional) and ENOBUFS (netisr queue full)
are supported.

Previously it would return (1) on success but the return value of IF_HANDOFF()
was interpreted wrongly and (0) was actually returned on success.  Due to this
schednetisr() was never called to kick the scheduling of the isr.  However this
was masked by other normal packets coming through netisr_dispatch() causing the
dequeueing of waiting packets.

PR:		kern/70988
Found by:	MOROHOSHI Akihiko <moro@remus.dti.ne.jp>
MFC after:	3 days

1 files changed, 5 insertions, 3 deletions

diff --git a/sys/net/netisr.c b/sys/net/netisr.c
index ca2a6cb..8f573d9 100644
--- a/sys/net/netisr.c
+++ b/sys/net/netisr.c
@@ -204,6 +204,8 @@ netisr_dispatch(int num, struct mbuf *m)
  * Same as above, but always queue.
  * This is either used in places where we are not confident that
  * direct dispatch is possible, or where queueing is required.
+ * It returns (0) on success and ERRNO on failure.  On failure the
+ * mbuf has been free'd.
  */
 int
 netisr_queue(int num, struct mbuf *m)
@@ -216,13 +218,13 @@ netisr_queue(int num, struct mbuf *m)
 	if (ni->ni_queue == NULL) {
 		isrstat.isrs_drop++;
 		m_freem(m);
-		return (1);
+		return (ENXIO);
 	}
 	isrstat.isrs_queued++;
 	if (!IF_HANDOFF(ni->ni_queue, m, NULL))
-		return (0);
+		return (ENOBUFS);	/* IF_HANDOFF has free'd the mbuf */
 	schednetisr(num);
-	return (1);
+	return (0);
 }
 
 static void