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