cvs commit: src/sys/vm swap_pager.c

John Baldwin jhb at freebsd.org
Thu Jul 31 17:42:29 UTC 2008


On Thursday 31 July 2008 05:06:51 am Kostik Belousov wrote:
> On Wed, Jul 30, 2008 at 09:12:15PM +0000, John Baldwin wrote:
> > jhb         2008-07-30 21:12:15 UTC
> > 
> >   FreeBSD src repository
> > 
> >   Modified files:
> >     sys/vm               swap_pager.c 
> >   Log:
> >   SVN rev 181019 on 2008-07-30 21:12:15Z by jhb
> >   
> >   If the kernel has run out of metadata for swap, then explicitly panic()
> >   instead of emitting a warning before deadlocking.
> >   
> >   MFC after:      1 month
> >   
> >   Revision  Changes    Path
> >   1.303     +1 -1      src/sys/vm/swap_pager.c
> 
> Hmm, would it make sense to attempt to sync the filesystems ? We
> panic due to resource exhaustion as opposed to some data consistency
> violation. Sure, sync might deadlock too, because we may be in the low
> memory condition.

There is already a 'sync on panic' option for the brave. :)  What would really 
be a better fix is if this condition could be treated the same as running out 
of swap space (i.e. go kill a process and keep running).  I would have done 
that in the first place instead of adding the printf if it was easy. :)

-- 
John Baldwin


More information about the cvs-src mailing list