cvs commit: src/sys/alpha/alpha pmap.c src/sys/i386/i386 pmap.c

[Poul-Henning Kamp]

phk         2004/06/02 00:04:21 PDT

  FreeBSD src repository

  Modified files:        (Branch: RELENG_4)
    sys/alpha/alpha      pmap.c 
    sys/i386/i386        pmap.c 
  Log:
  There is a comma missing in the table initializing the
  pmap_prefault_pageorder array.  This has two effects:
  
  1. The resulting bogus contents of the array thwarts part of the
  optimization effect pmap_prefault() is supposed to have.
  
  2. The resulting array is only 7 elements long (auto-sized), while
  pmap_prefault() expects it to be the intended 8 elements.  So this
  function in fact accesses memory beyond the end of the array.
  Fortunately though, if the data at this location is out of bounds it
  will be ignored.
  
  This bug dates back more than 6 years.  It has been introduced in
  revision 1.178.
  
  Submitted by:   Uwe Doering <gemini at geminix.org>
  PR:     67460
  
  Revision    Changes    Path
  1.35.2.11   +1 -1      src/sys/alpha/alpha/pmap.c
  1.250.2.26  +1 -1      src/sys/i386/i386/pmap.c