From owner-cvs-src@FreeBSD.ORG Mon Mar 22 12:06:18 2004 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 8CC3E16A4D0 for ; Mon, 22 Mar 2004 12:06:18 -0800 (PST) Received: from root.org (root.org [67.118.192.226]) by mx1.FreeBSD.org (Postfix) with SMTP id 4019D43D4C for ; Mon, 22 Mar 2004 12:06:18 -0800 (PST) (envelope-from nate@root.org) Received: (qmail 34195 invoked by uid 1000); 22 Mar 2004 20:06:19 -0000 Date: Mon, 22 Mar 2004 12:06:19 -0800 (PST) From: Nate Lawson To: Bill Paul In-Reply-To: <20040322191542.CD8DE16A4CF@hub.freebsd.org> Message-ID: <20040322115832.X34157@root.org> References: <20040322191542.CD8DE16A4CF@hub.freebsd.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/amd64/isa atpic.c src/sys/contrib/dev/oltr if_oltr.c src/sys/dev/aac aac_pci.c src/sys/dev/acpic X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 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: Mon, 22 Mar 2004 20:06:18 -0000 On Mon, 22 Mar 2004, Bill Paul wrote: > > The reason why we made this > > change now is that API compatibility needs to be fixed before 5.3R. I > > MFCd the function itself to 4-STABLE, but it won't go into 5.2 or 5.2.1 > > due to the fact that only security changes are allowed there. > > I think you're missing the point. There are people with 5.2.1 CD kits -- > not just downloaded ISO images but actual physical CDs -- that don't have > this API. You can't go out and change the contents of these CDs. I > want to make it as painless as possible for people to just grab the > existing NDISulator code and drop it into a 5.2.1 system, because the > less trouble it is for them, the less trouble they make for me. Yes, I know it doesn't work for CDs. It also doesn't work for someone running 4.9R (not 4-stable) if you produced a version for 4.x. > Ok, just a minute. Do you want to: > > a) _Supplement_ bus_alloc_resource() with bus_alloc_resource_any() Yes. > Please be clear. Calling bus_alloc_resource_any() a "convenience function" > implies that it's an addition to the API, which does not preclude me > from using the older API routines if I really, really want to. You can keep using whatever API you want. I'd prefer you move to the convenience function to keep in sync with the rest of the kernel. The goal is to keep things consistent when people cut/paste to build a new driver. > Please to be explaining why this isn't a macro: > > #define bus_alloc_resource_any(dev, type, rid, flags) \ > bus_alloc_resource(dev, type, rid, 0ul, ~0ul, 1, flags) We discussed using a macro or inline and imp@ preferred inline. Again, it's a convenience function to codify the correct default values. It would be nice if you added it back for 5.3R. I just bumped FreeBSD version (502109) so you can add an #if for compat with 5.2.1 and lower. Or do nothing at all, whatever you prefer. -Nate