From owner-freebsd-net@FreeBSD.ORG Tue Dec 10 00:22:46 2013 Return-Path: Delivered-To: freebsd-net@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 8FB4745E for ; Tue, 10 Dec 2013 00:22:46 +0000 (UTC) Received: from nm25-vm2.bullet.mail.ne1.yahoo.com (nm25-vm2.bullet.mail.ne1.yahoo.com [98.138.91.213]) by mx1.freebsd.org (Postfix) with SMTP id 504B71F52 for ; Tue, 10 Dec 2013 00:22:46 +0000 (UTC) Received: from [98.138.100.102] by nm25.bullet.mail.ne1.yahoo.com with NNFMP; 10 Dec 2013 00:20:47 -0000 Received: from [98.138.87.9] by tm101.bullet.mail.ne1.yahoo.com with NNFMP; 10 Dec 2013 00:20:47 -0000 Received: from [127.0.0.1] by omp1009.mail.ne1.yahoo.com with NNFMP; 10 Dec 2013 00:20:47 -0000 X-Yahoo-Newman-Property: ymail-3 X-Yahoo-Newman-Id: 508702.89738.bm@omp1009.mail.ne1.yahoo.com Received: (qmail 57724 invoked by uid 60001); 10 Dec 2013 00:20:47 -0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=yahoo.com; s=s1024; t=1386634847; bh=h30GFy6UwWp9wm0iXXCv05J2PCYv0v7T0ADXrI7aCRo=; h=X-YMail-OSG:Received:X-Rocket-MIMEInfo:X-Mailer:Message-ID:Date:From:Reply-To:Subject:To:MIME-Version:Content-Type; b=TUVxdFbrH9Z4n06NY2cDXR8r3hnC/hy6MWb01pK+vWzx24QgjS3aIbeRTY91StybyKjdNphT5DNmIwq6toaqj68jnI2jEceqS25G4PmVT3VQ+lJIuJOFK6CdNFKNVVoEnr1LsopWAYmRu0pHYJVJblP2iMubkAvtzoCa69fDSSQ= DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=s1024; d=yahoo.com; h=X-YMail-OSG:Received:X-Rocket-MIMEInfo:X-Mailer:Message-ID:Date:From:Reply-To:Subject:To:MIME-Version:Content-Type; b=2YpZUO68MJQ8RVteI5Kw27to3Gq2cMFhXCVnXnuUIp/Ye7BCQDP+i7Ovverco/kQznuqFwYTJHeFYUTFhzjs0Q1TyuSB+90stwkDOCynT0KPIavRmJeW8pk67Qw4mPo9u4UvHqXddOpR9Pd00j3K6TEkyv/vJGDJlBJlORO7xYk=; X-YMail-OSG: 9y4eTh8VM1mqioBnV1agtEds9IUtr.HgrSqDzqc2siuow.7 995p2AX1id.6F69Ykl0p8cKkakTd1JouXqkEq384KcPWvAH8i4E8VJEM72Wx 4w5DAnJiPMBYEhMZ.sTKdH5wbNP3UHxY._XfvNbTJh2zHPM5ul.CDgP0OnBI B3yVE1F3Yq0izDKOdPLMjUP1c2kc76SMEf2jnrSSHSOJn_dLOVLcINVELEw. EtteazwPM1N7Ax9CruxjmKg0ASJ0AJic57po._ZhgGs_srTUeVJgBHfk8xsN UTb6..v3IONWTzfIMdos626ebtrZFx8d22SyQovkqEKIOI6FPRM45HK5VTbd C64rvgSd7mQ_KTaGlK8.PtLgRSC1dTJjAo8_0MWPgJcibKQ5yqjUJV8CIZ2n 02kMOp0Ssoz0Gg3QhwT6FFMO4Z9IRmKWio5QPwHL4rJtgZwwT2eDCgSSWSI6 sTVgoRHk4N27DepiLegvCGxlxzUvZ3uLttjC3do2XswulX.y8mM9jzr7CozX 0H7JzOXAiJglhzj_gU24wt5qazEw.RawKsS2xb9WuXJxc3eZn7Q8Top6TOEc dMjMBDI2FK7rL1gLgZ_joWAqo Received: from [98.203.118.124] by web121606.mail.ne1.yahoo.com via HTTP; Mon, 09 Dec 2013 16:20:47 PST X-Rocket-MIMEInfo: 002.001, V2h5IGlzIGl0IHRoYXQgZXZlcnkgdGltZSBJIGdldCBhIG5ldyBNQiBpdCBoYXMgeWV0LWFub3RoZXIgaW50ZWwgY29udHJvbGxlciBvbiBpdD8gSG93IGFyZSB5b3Ugc3VwcG9zZWQgdG8gd3JpdGUgZ29vZCBkcml2ZXJzIHRoYXQgc3VwcG9ydCA5MDAwIGRpZmZlcmVudCBjb250cm9sbGVycz8gQXMgbXVjaCBhcyBJIGFwcHJlY2lhdGUgdGhlIHByb2dyZXNzaW9uIG9mIENQVSB0ZWNobm9sb2d5LCB0aGV5CndvdWxkIHNlcnZlIHRoZSBjb21tdW5pdHkgYmV0dGVyIGJ5IG1ha2luZyB1cCB0aGVpciBkYW1uIG0BMAEBAQE- X-Mailer: YahooMailWebService/0.8.169.609 Message-ID: <1386634847.38473.YahooMailNeo@web121606.mail.ne1.yahoo.com> Date: Mon, 9 Dec 2013 16:20:47 -0800 (PST) From: Barney Cordoba Subject: Intel Controllers [Rant] To: "freebsd-net@freebsd.org" MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.17 X-BeenThere: freebsd-net@freebsd.org X-Mailman-Version: 2.1.17 Precedence: list Reply-To: Barney Cordoba List-Id: Networking and TCP/IP with FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 10 Dec 2013 00:22:46 -0000 Why is it that every time I get a new MB it has yet-another intel controlle= r on it? How are you supposed to write good drivers that support 9000 diffe= rent controllers? As much as I appreciate the progression of CPU technology= , they=0Awould serve the community better by making up their damn minds and= just build 1 or 2 controller for each=A0=0Atarget market.=0A=0ABC From owner-freebsd-net@FreeBSD.ORG Tue Dec 10 08:40:08 2013 Return-Path: Delivered-To: freebsd-net@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 59F72713; Tue, 10 Dec 2013 08:40:08 +0000 (UTC) Received: from mail-vc0-x22e.google.com (mail-vc0-x22e.google.com [IPv6:2607:f8b0:400c:c03::22e]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id BF4561164; Tue, 10 Dec 2013 08:40:07 +0000 (UTC) Received: by mail-vc0-f174.google.com with SMTP id id10so4252989vcb.5 for ; Tue, 10 Dec 2013 00:40:06 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=Z/FczHGtKCWaoI5B7EYwKdKtwdc0+TS4QOXDKMhhkcQ=; b=qAGrlVpkpybMYZAOQca+yiWW3hpy1akF7JQ8/zYdY0r5FpJIahYzUGLXKQT4fz2uLM N4oMrdEB6P/11vScroVYwxTY/2ykju33oc6GPpuC1X/d3RT/feeKxXj7OPBjmn0vEn27 agCTO/DDbwOFA7OTefj2s5LrUZ/FzsuiCHsETfdvw7UEhski4zctuHGjFkXvuQ83pmsp 88oCGfHBzS9Pn02UzGKGwR90apYVWb5DL6KRKSlIpUFoqj38QtFw6X4SIuooA+udvHeW wRViYyUx41rtrJ3592/UN5CMy1USBjDXalG+Db2LvQ3x5Rnf+NiIwr6wVu8W6BKLsF2t m4Pw== MIME-Version: 1.0 X-Received: by 10.52.160.130 with SMTP id xk2mr201382vdb.24.1386664806799; Tue, 10 Dec 2013 00:40:06 -0800 (PST) Received: by 10.220.155.147 with HTTP; Tue, 10 Dec 2013 00:40:06 -0800 (PST) In-Reply-To: References: <521B9C2A-EECC-4412-9F68-2235320EF324@lurchi.franken.de> <20131202022338.GA3500@michelle.cdnetworks.com> <20131203021658.GC2981@michelle.cdnetworks.com> <20131205223711.GB55638@funkthat.com> <3576B69E-E943-46E0-83E5-0B2194A44ED0@lurchi.franken.de> <20131206202012.GG55638@funkthat.com> <609C63CD-9332-4EAE-AACE-5B911416DF80@lurchi.franken.de> <9E163DC1-D647-4E19-BE23-44E5DFE2F284@lurchi.franken.de> Date: Tue, 10 Dec 2013 00:40:06 -0800 Message-ID: Subject: Re: A small fix for if_em.c, if_igb.c, if_ixgbe.c From: Jack Vogel To: Adrian Chadd Content-Type: text/plain; charset=ISO-8859-1 X-Content-Filtered-By: Mailman/MimeDel 2.1.17 Cc: Yong-Hyeon Pyun , John Baldwin , "freebsd-net@freebsd.org list" , John-Mark Gurney , Jack F Vogel , Michael Tuexen X-BeenThere: freebsd-net@freebsd.org X-Mailman-Version: 2.1.17 Precedence: list List-Id: Networking and TCP/IP with FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 10 Dec 2013 08:40:08 -0000 Hey Adrian, Didn't want you to think I was ignoring you, just kind of in the middle of some deadline issues and was not able to give this a lot of cycles. I've been reading the email seeing if some kind of consensus was formed, so far I'm not strongly convinced what's right. Does someone feel there is an urgent need to get this changed? Cheers, Jack On Mon, Dec 9, 2013 at 11:37 AM, Adrian Chadd wrote: > Jack / John - thoughts? > > > -a > > On 8 December 2013 19:44, Adrian Chadd wrote: > > .. I'm saying that we should have if_transmit( )return an error only > > on the current packet, and 0 if it was queued. > > > > We don't have a mechanism to say that a queued packet actually made it > > onto the wire. Whether we should is a different discussion. > > > > Ie, after thinking about this some more, I'd like to: > > > > * change if_transmit in these drivers to do exactly what you suggest - > > it should return an error only if the given packet couldn't be queued. > > It shouldn't matter whether it is later transmitted or not - we don't > > have a feedback mechanism for that. > > * figure out a very specific definition of what xxx_mq_start_locked() > > should return - my gut feeling is an error if it couldn't queue a > > frame, and 0 if it dispatched a frame to the hardware - and then make > > the code match this definition. > > > > > > > > -adrian >