cvs commit: src/sys/amd64/amd64 db_trace.c exception.S genassym.c trap.c

Joseph Koshy jkoshy at FreeBSD.org
Sat Aug 27 16:03:40 GMT 2005


jkoshy      2005-08-27 16:03:40 UTC

  FreeBSD src repository

  Modified files:
    sys/amd64/amd64      db_trace.c exception.S genassym.c trap.c 
  Log:
  - Special-case NMI handling on the AMD64.
  
    On entry or exit from the kernel the 'alltraps' and 'doreti' code
    used taken by normal traps disables interrupts to protect the
    critical sections where it is setting up %gs.
  
    This protection is insufficient in the presence of NMIs since NMIs
    can be taken even when the processor has disabled normal interrupts.
    Thus the NMI handler needs to actually read MSR_GBASE on entry to
    the kernel to determine whether a swap of %gs using 'swapgs' is
    needed.  However, reads of MSRs are expensive and integrating this
    check into the 'alltraps'/'doreti' path would penalize normal
    interrupts.
  
  - Teach DDB about the 'nmi_calltrap' symbol.
  
  Reviewed by:    bde, peter (older versions of this change)
  
  Revision  Changes    Path
  1.69      +2 -1      src/sys/amd64/amd64/db_trace.c
  1.127     +76 -2     src/sys/amd64/amd64/exception.S
  1.156     +2 -0      src/sys/amd64/amd64/genassym.c
  1.290     +2 -1      src/sys/amd64/amd64/trap.c


More information about the cvs-all mailing list