Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 16 Feb 2004 12:03:45 -0700 (MST)
From:      Scott Long <scottl@freebsd.org>
To:        Robert Watson <rwatson@FreeBSD.org>
Cc:        Dag-Erling Smorgrav <des@FreeBSD.org>
Subject:   Re: cvs commit: src/sys/vm vm_kern.c
Message-ID:  <20040216120153.S80753@pooker.samsco.home>
In-Reply-To: <Pine.NEB.3.96L.1040216135731.63057M-100000@fledge.watson.org>
References:  <Pine.NEB.3.96L.1040216135731.63057M-100000@fledge.watson.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, 16 Feb 2004, Robert Watson wrote:
>
> On Mon, 16 Feb 2004, Dag-Erling Smorgrav wrote:
>
> >   Log:
> >   Don't panic if we fail to satisfy an M_WAITOK request; return 0 instead.
> >   The calling code will either handle that gracefully or cause a page fault.
>
> Also, this turns an easily understood and widely documented kernel panic
> message into a page fault.  Prior to this, users could google for the
> message and find documentation on increasing the size of the kernel
> address space.  Now, it requires facility with the source code in order to
> figure out what it is going on (since the page fault trace won't include
> the memory allocation).
>

Agreed on all points.  I thought that there was along discussion on this
in the last 1-2 weeks and that all of these issues were brought up.
Please back this out, and lets focus on getting the sematics that you need
for procfs rather than just ruining the sematics for everything else.

Scott



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040216120153.S80753>