From owner-freebsd-net@FreeBSD.ORG Sun Jan 16 17:55:14 2011 Return-Path: Delivered-To: freebsd-net@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 3A38C106564A for ; Sun, 16 Jan 2011 17:55:14 +0000 (UTC) (envelope-from bzeeb-lists@lists.zabbadoz.net) Received: from mail.cksoft.de (mail.cksoft.de [IPv6:2001:4068:10::3]) by mx1.freebsd.org (Postfix) with ESMTP id B50418FC16 for ; Sun, 16 Jan 2011 17:55:13 +0000 (UTC) Received: from localhost (amavis.fra.cksoft.de [192.168.74.71]) by mail.cksoft.de (Postfix) with ESMTP id AD2B741C735; Sun, 16 Jan 2011 18:55:12 +0100 (CET) X-Virus-Scanned: amavisd-new at cksoft.de Received: from mail.cksoft.de ([192.168.74.103]) by localhost (amavis.fra.cksoft.de [192.168.74.71]) (amavisd-new, port 10024) with ESMTP id F6KmAzMWPBdJ; Sun, 16 Jan 2011 18:55:12 +0100 (CET) Received: by mail.cksoft.de (Postfix, from userid 66) id 14AAD41C710; Sun, 16 Jan 2011 18:55:12 +0100 (CET) Received: from maildrop.int.zabbadoz.net (maildrop.int.zabbadoz.net [10.111.66.10]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mail.int.zabbadoz.net (Postfix) with ESMTP id DEB174448F3; Sun, 16 Jan 2011 17:55:06 +0000 (UTC) Date: Sun, 16 Jan 2011 17:55:06 +0000 (UTC) From: "Bjoern A. Zeeb" X-X-Sender: bz@maildrop.int.zabbadoz.net To: Marius Strobl In-Reply-To: <20110116144247.GA65399@alchemy.franken.de> Message-ID: <20110116174153.Y14966@maildrop.int.zabbadoz.net> References: <36074996.20110112192009@serebryakov.spb.ru> <20110112213208.GD12920@michelle.cdnetworks.com> <20110112225907.GA44318@alchemy.franken.de> <20110113173925.GA49356@alchemy.franken.de> <20110113212713.GC17502@michelle.cdnetworks.com> <20110114012412.GK97101@alchemy.franken.de> <178613186.20110114134109@serebryakov.spb.ru> <20110116144247.GA65399@alchemy.franken.de> X-OpenPGP-Key: 0x14003F198FEFA3E77207EE8D2B58B8F83CCF1842 MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed Cc: Pyun YongHyeon , freebsd-net@freebsd.org, Lev Serebryakov Subject: Re: [patch] re(4) problems on networks with disabled autonegotiation "solver" (WAS: Juniper e3k with ports limitied to...) -- REQUEST FOR REVIEW X-BeenThere: freebsd-net@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Networking and TCP/IP with FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 16 Jan 2011 17:55:14 -0000 On Sun, 16 Jan 2011, Marius Strobl wrote: Hi, > On Fri, Jan 14, 2011 at 01:41:09PM +0300, Lev Serebryakov wrote: >> Hello, Marius. >> You wrote 14 ?????? 2011 ?., 4:24:12: >> >>> found by ignoring the bits set in the "don't care mask". I've >>> updated the patch at the above URL accordingly and based on my >>> testing it now should actually work as expected. Sorry for the >>> glitch. >> Yes, it works for me. > > Thanks for testing! I've committed the patch to HEAD and will try > to get it into 7.4 and 8.2. > >> Only one note: I think, it is good idea to document this flag in >> re(4). >> > > Technically re.4 obviously is the wrong location for documenting it > as it's generally the business of the PHY drivers to handle the media. > Moreover, rgephy(4) driven PHYs are also commonly used in combination > with at least axe(4), nfe(4), nve(4) and sge(4) MACs so we would have > to document this special media option in all of the corresponding man > pages, which would introduce undesirable redundancy. So instead I've > added a rgephy.4 man page and documented it there. > Actually, this is a very good example for why we generally should not > document media types and options in the man pages of MAC drivers that > take advantage of mii(4) but instead add a common ifmedia.4 and just > point there and if as in this case it makes sense to man pages of > specific PHY drivers. First of all, thanks a lot for the work! Can you email me the entire set of revisions off-list so that I could have a look, especially to see if we can still sneak things into the release after the RC2s (or send them to re@ fore pre-review and I'll read them there;) Second, I start to like the idea of <$phy>.4 in addition to <$mac>.4 man pages and .Xrs given our discussion lately on the extra alias options. The only drawback I see is that the ifconfig.8 -> <$mac>.4 -> <$phy>.4 man page chain gets longer and ifconfig generally doesn't reference any drivers but miibus.4 does. Would it make a sense to add a list of man pages of the `phys' implementing miibus in addition to the list of drivers making use of miibus, rather than just having adding the .Xr at the end? The list might grow in the future now that it was started? /bz -- Bjoern A. Zeeb You have to have visions! Going to jail sucks -- All my daemons like it! http://www.freebsd.org/doc/en_US.ISO8859-1/books/handbook/jails.html