Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Tue, 18 Jun 2002 09:30:47 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Tue, 18 Jun 2002 09:30:47 -0400 Received: from employees.nextframe.net ([212.169.100.200]:12023 "EHLO sexything.nextframe.net") by vger.kernel.org with ESMTP id ; Tue, 18 Jun 2002 09:30:46 -0400 Date: Tue, 18 Jun 2002 15:39:24 +0200 From: Morten Helgesen To: linux-kernel@vger.kernel.org Subject: Re: [RFC] Getting rid of check_region() Message-ID: <20020618153924.H129@sexything> Reply-To: morten.helgesen@nextframe.net References: <20020618150055.E129@sexything> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20020618150055.E129@sexything> User-Agent: Mutt/1.3.22.1i X-Editor: VIM - Vi IMproved 6.0 X-Keyboard: PFU Happy Hacking Keyboard X-Operating-System: Slackware Linux (of course) Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2460 Lines: 67 Ok - I`ve been informed (thanks Thomas:) that William Stinson has been working on removing check_region() for a while now. I was not aware of this - I have noticed, that in recent changelogs, he has been credited for work done on code in relation to resource handling, but I did not see any check_region() stuff in there. At a closer look, I see a couple of check_region() entries in there. I`ll contact William and see if he could need assistance, if not I`ll grab another kernel/janitor project :) cheers for now, On Tue, Jun 18, 2002 at 03:00:55PM +0200, Morten Helgesen wrote: > Hey guys. > > It`s time for some real janitor work :) > > I`ve taken a clean 2.5.22 tree, and started removing > instances of check_region(). It has been on the kernel janitors` > TODO list for a long time. > > It struck me that it might be a good idea to see if anyone had any comments > or even objections before spending X hours killing it off :-) > > I see that there are lots of places where we can just remove > check_region() and instead just check the return value from > request_region(), but I`m also sure we have corner cases where > driver authors are doing 'nifty' things with check_region(), and > these drivers might need a bit more surgery. I will contact > the driver authors and ask for their comments if I can not figure out > how to work around using check_region() in the specific driver. > > When done, I`ll submit separate patches for separate directories. > > Does this sound like a sane approach to you guys ? > > == Morten > > -- > > "Livet er ikke for nybegynnere" - sitat fra en klok person. > > mvh > Morten Helgesen > UNIX System Administrator & C Developer > Nextframe AS > admin@nextframe.net / 93445641 > http://www.nextframe.net > - > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ -- "Livet er ikke for nybegynnere" - sitat fra en klok person. mvh Morten Helgesen UNIX System Administrator & C Developer Nextframe AS admin@nextframe.net / 93445641 http://www.nextframe.net - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/