From: Morten Helgesen <morten.helgesen@nextframe.net>
To: linux-kernel@vger.kernel.org
Subject: Re: [RFC] Getting rid of check_region()
Date: Tue, 18 Jun 2002 15:39:24 +0200 [thread overview]
Message-ID: <20020618153924.H129@sexything> (raw)
In-Reply-To: <20020618150055.E129@sexything>
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
prev parent reply other threads:[~2002-06-18 13:30 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-06-18 13:00 [RFC] Getting rid of check_region() Morten Helgesen
2002-06-18 13:39 ` Morten Helgesen [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20020618153924.H129@sexything \
--to=morten.helgesen@nextframe.net \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.