From owner-cvs-src@FreeBSD.ORG Sun Nov 13 21:23:45 2005 Return-Path: X-Original-To: cvs-src@FreeBSD.org Delivered-To: cvs-src@FreeBSD.org Received: by hub.freebsd.org (Postfix, from userid 618) id 330A316A420; Sun, 13 Nov 2005 21:23:45 +0000 (GMT) In-Reply-To: <200511111604.jABG4xw3011845@repoman.freebsd.org> from Ruslan Ermilov at "Nov 11, 2005 04:04:59 pm" To: ru@FreeBSD.org (Ruslan Ermilov) Date: Sun, 13 Nov 2005 21:23:45 +0000 (GMT) X-Mailer: ELM [version 2.4ME+ PL54 (25)] MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Message-Id: <20051113212345.330A316A420@hub.freebsd.org> From: wpaul@FreeBSD.ORG (Bill Paul) Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/net if.c if_arcsubr.c if_arp.h if_bridge.c if_ef.c if_ethersubr.c if_fddisubr.c if_fwsubr.c if_iso88 X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 13 Nov 2005 21:23:45 -0000 [...] > sys/compat/ndis subr_ndis.c [...] > - Store pointer to the link-level address right in "struct ifnet" > rather than in ifindex_table[]; all (except one) accesses are > through ifp anyway. IF_LLADDR() works faster, and all (except > one) ifaddr_byindex() users were converted to use ifp->if_addr. > > - Stop storing a (pointer to) Ethernet address in "struct arpcom", > and drop the IFP2ENADDR() macro; all users have been converted > to use IF_LLADDR() instead. For the umpity umpth time: I'm trying to keep it so that the same NDIS code will compile on 7.x, 6.x and 5.x. Having one set of code means I don't need to go nuts doing merges from one branch to the next: I can just pull the files over and be done with it. You did not consider this when modifying subr_ndis.c, and now you have broken it for the 6.x and 5.x branches. More importantly, you broke it in a very subtle way that only shows up as a kernel panic a runtime. I don't mind if people make necessary updates to the NDIS code, but I do mind it very much when people mess up the interoperability between branches that I've tried very hard to maintain. In the future, you will not make any changes to any of this code unless you can guarantee it will remain backwards compatible. You don't have to do the MFCs yourself, but you do have to fix it so that the code can be dropped directly into an earlier branch and still build and run. If you don't think you can do this, tell me about it first and I'll do it. -Bill -- ============================================================================= -Bill Paul (510) 749-2329 | Senior Engineer, Master of Unix-Fu wpaul@windriver.com | Wind River Systems ============================================================================= you're just BEGGING to face the moose =============================================================================