From owner-cvs-all@FreeBSD.ORG  Mon Mar  5 09:10:17 2007
Return-Path: <owner-cvs-all@FreeBSD.ORG>
X-Original-To: cvs-all@FreeBSD.org
Delivered-To: cvs-all@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52])
	by hub.freebsd.org (Postfix) with ESMTP id B3CD016A404;
	Mon,  5 Mar 2007 09:10:17 +0000 (UTC) (envelope-from bde@FreeBSD.org)
Received: from repoman.freebsd.org (repoman.freebsd.org [69.147.83.41])
	by mx1.freebsd.org (Postfix) with ESMTP id A57CB13C4A5;
	Mon,  5 Mar 2007 09:10:17 +0000 (UTC) (envelope-from bde@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.13.6/8.13.6) with ESMTP id l259AHuN081080;
	Mon, 5 Mar 2007 09:10:17 GMT (envelope-from bde@repoman.freebsd.org)
Received: (from bde@localhost)
	by repoman.freebsd.org (8.13.6/8.13.4/Submit) id l259AHY8081078;
	Mon, 5 Mar 2007 09:10:17 GMT (envelope-from bde)
Message-Id: <200703050910.l259AHY8081078@repoman.freebsd.org>
From: Bruce Evans <bde@FreeBSD.org>
Date: Mon, 5 Mar 2007 09:10:17 +0000 (UTC)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Cc: 
Subject: cvs commit: src/sys/i386/isa clock.c
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Mon, 05 Mar 2007 09:10:17 -0000

bde         2007-03-05 09:10:17 UTC

  FreeBSD src repository

  Modified files:
    sys/i386/isa         clock.c 
  Log:
  Partial fix for a bug in rev.1.231.  If suspend/resume clobbers the
  RTC state, then it may clobber the RTC index register, so the index
  register must be restored before using it to restore control registers
  in rtc_restore().
  
  The following problems remain:
  - rtc_restore() is only called if pmtimer is configured.  Buggy
    suspend/resumes are more likely to clobber the index register than
    a control register, so pmtimer is more needed than it used to be.
  - pmtimer doesn't exist for amd64.
  - Restoring of the RTC state may race with rtcintr().  If an RTC
    interrupt is handled before the state is restored, then rtcin(RTC_INTR)
    in rtcintr() may read from the wrong register, so rtcintr() may spin
    forever.  This may be mitigated by the most common state clobbering
    being to turn off RTC interrupts.
  
  Revision  Changes    Path
  1.235     +1 -0      src/sys/i386/isa/clock.c