Date: Sat, 04 Jun 2005 13:31:37 -0400 From: Jake Burkholder <jake@locore.ca> To: Andre Guibert de Bruet <andy@siliconlandmark.com> Cc: current@freebsd.org Subject: Re: [RFC] [PATCH] VM & VFS changes Message-ID: <42A1E579.1020704@locore.ca> In-Reply-To: <20050601000203.D69811@lexi.siliconlandmark.com> References: <20050601000203.D69811@lexi.siliconlandmark.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Andre Guibert de Bruet wrote: > Hi, > > I would like to present a unified set of VM and VFS diffs for wider > scrutiny and testing. I have been running these diffs since May 10th > across daily reboots with no issues. > > [...] >Index: vm/swap_pager.c >=================================================================== >RCS file: /home/ncvs/src/sys/vm/swap_pager.c,v >retrieving revision 1.273 >diff -u -r1.273 swap_pager.c >--- vm/swap_pager.c 20 May 2005 21:26:05 -0000 1.273 >+++ vm/swap_pager.c 24 May 2005 02:21:30 -0000 >@@ -2193,6 +2193,60 @@ > return (error); > } > > [...] > > >+ >+ mtx_lock(&sw_dev_mtx); >+ TAILQ_REMOVE(&swtailq, sp, sw_list); >+ >+ sp = TAILQ_FIRST(&swtailq); >+ free(sp, M_VMPGDATA); >+ } > > It looks like the free should be before sp = TAILQ_FIRST() instead of after. Jake
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?42A1E579.1020704>