Subject: Re: PR/34186 CVS commit: src/sys/arch/i386/i386
To: None <port-i386-maintainer@netbsd.org, gnats-admin@netbsd.org,>
From: Christos Zoulas <christos@zoulas.com>
List: netbsd-bugs
Date: 01/06/2007 16:40:02
The following reply was made to PR port-i386/34186; it has been noted by GNATS.
From: christos@zoulas.com (Christos Zoulas)
To: gnats-bugs@NetBSD.org, port-i386-maintainer@netbsd.org,
gnats-admin@netbsd.org, netbsd-bugs@netbsd.org,
Wolfgang.Stukenbrock@nagler-company.com
Cc:
Subject: Re: PR/34186 CVS commit: src/sys/arch/i386/i386
Date: Sat, 6 Jan 2007 11:35:07 -0500
On Jan 6, 3:30pm, pavel@netbsd.org (Pavel Cahyna) wrote:
-- Subject: PR/34186 CVS commit: src/sys/arch/i386/i386
| Log Message:
| PR port-i386/34186 by Wolfgang Stukenbrock:
| mapping of msgbuf during startup may map invalid physical adresses
|
| "If the last available physical memory segment on a system is less 16k,
| than the startup code that will map the kernel message buffer, will fail
| and map physical pages behind the last segment. This may either only lead
| to a message buffer without physical memory behind it, or to an
| overlapping message buffer with something else."
|
| Fix by allowing multiple physical memory segments to be used for msgbuf.
| Also remove some leftover msgbuf manipulation from pmap.c.
|
| Fix supplied by Wolfgang Stukenbrock in the PR, with some modifications
| from me, mainly to use the already existing constant VM_PHYSSEG_MAX as the
| static limit of number of msgbuf segments.
|
|
| To generate a diff of this commit:
| cvs rdiff -r1.588 -r1.589 src/sys/arch/i386/i386/machdep.c
| cvs rdiff -r1.194 -r1.195 src/sys/arch/i386/i386/pmap.c
|
| Please note that diffs are not public domain; they are subject to the
| copyright notices on the relevant files.
Don't we have the same problem on amd64?
christos