git: kernel - pmap (mainly i386) - fix TLB race and other issues

Matthew Dillon dillon at crater.dragonflybsd.org
Wed Dec 30 11:39:27 PST 2009


commit 5926987a8c7f76518fb332134f1f6c2f3db89862
Author: Matthew Dillon <dillon at apollo.backplane.com>
Date:   Wed Dec 30 09:10:18 2009 -0800

    kernel - pmap (mainly i386) - fix TLB race and other issues
    
    * Temporary fix for a TLB race.  Due to threading a process pmap
      may be mapped to multiple cpus.  The APTmap is built into the
      per-process pmap and used for temporary page table mappings.
    
      The i386 pmap code was only invalidating the TLB if the APTmap
      had to be updated, but did not consider the possibility that
      it would be accessed from a different cpu than the one it was
      originally installed in.
    
      Add a simple global cpumask to avoid the degenerate case.
      This is a bit of overkill.  A followup commit will move
      APTmap into the globaldata structure as a more permanent
      fix.
    
      The vkernel and x86_64 pmap code do not have this issue.  The
      vkernel pmap code already uses a per-globaldata pmap and
      the x86_64 pmap code uses the direct phys map.
    
    * Add a ton of debugging code articulated with PMAP_DEBUG in
      case I ever have to do this again.  Default disabled.
    
    * Fix a few minor issues in the pmap code.
    
      (1) The pv in pmap_remove_entry() is not optional.
    
      (2) Add a few minor missing pmap->pm_generation increments.
    
      (3) Add a missing critical section.
    
      (4) Remove an unused #define.
    
    * Zalloc code.  No specific bugs but make sure the ZONE_INTERRUPT
      code acquires the MP lock when/if it needs to expand the mapping.
    
    Reported-by: YONETANI Tomokazu <y0netan1 at crater.dragonflybsd.org>

Summary of changes:
 sys/platform/pc32/i386/pmap.c        |  180 +++++++++++++++++++++++++++++-----
 sys/platform/pc32/include/pmap.h     |   19 ++++-
 sys/platform/pc64/x86_64/pmap.c      |   38 +++++---
 sys/platform/vkernel/platform/pmap.c |   30 ++++--
 sys/vm/vm_zone.c                     |   16 +++-
 5 files changed, 228 insertions(+), 55 deletions(-)

http://gitweb.dragonflybsd.org/dragonfly.git/commitdiff/5926987a8c7f76518fb332134f1f6c2f3db89862


-- 
DragonFly BSD source repository





More information about the Commits mailing list