From owner-cvs-src-old@FreeBSD.ORG Fri Jun 17 20:07:13 2011 Return-Path: Delivered-To: cvs-src-old@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id CC37D106564A for ; Fri, 17 Jun 2011 20:07:13 +0000 (UTC) (envelope-from jhb@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id BB4A08FC08 for ; Fri, 17 Jun 2011 20:07:13 +0000 (UTC) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.4/8.14.4) with ESMTP id p5HK7Dv8083693 for ; Fri, 17 Jun 2011 20:07:13 GMT (envelope-from jhb@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.4/8.14.4/Submit) id p5HK7Dou083692 for cvs-src-old@freebsd.org; Fri, 17 Jun 2011 20:07:13 GMT (envelope-from jhb@repoman.freebsd.org) Message-Id: <201106172007.p5HK7Dou083692@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to jhb@repoman.freebsd.org using -f From: John Baldwin Date: Fri, 17 Jun 2011 20:06:52 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: HEAD Subject: cvs commit: src/sys/dev/e1000 if_igb.c if_igb.h X-BeenThere: cvs-src-old@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: **OBSOLETE** CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 17 Jun 2011 20:07:13 -0000 jhb 2011-06-17 20:06:52 UTC FreeBSD src repository Modified files: sys/dev/e1000 if_igb.c if_igb.h Log: SVN rev 223198 on 2011-06-17 20:06:52Z by jhb - Use a dedicated task to handle deferred transmits from the if_transmit method instead of reusing the existing per-queue interrupt task. Reusing the per-queue interrupt task could result in both an interrupt thread and the taskqueue thread trying to handle received packets on a single queue resulting in out-of-order packet processing. - Don't define igb_start() at all on 8.0 and where if_transmit is used. Replace last remaining call to igb_start() with a loop to kick off transmit on each queue instead. - Call ether_ifdetach() earlier in igb_detach(). - Drain tasks and free taskqueues during igb_detach(). Reviewed by: jfv MFC after: 1 week Revision Changes Path 1.69 +63 -10 src/sys/dev/e1000/if_igb.c 1.22 +1 -0 src/sys/dev/e1000/if_igb.h