Date: Tue, 24 Aug 2010 17:48:22 +0000 (UTC) From: Andriy Gapon <avg@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs arc.c Message-ID: <201008241748.o7OHmhrJ044295@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
avg 2010-08-24 17:48:22 UTC
FreeBSD src repository
Modified files:
sys/cddl/contrib/opensolaris/uts/common/fs/zfs arc.c
Log:
SVN rev 211762 on 2010-08-24 17:48:22Z by avg
zfs arc_reclaim_thread: no need to call arc_reclaim_needed when
resetting needfree
needfree is checked at the very start of arc_reclaim_needed.
This change makes code easier to follow and maintain in face of
potential changed in arc_reclaim_needed.
Also, put the whole sub-block under _KERNEL because needfree can be set
only in kernel code.
To do: rename needfree to something else to aovid confusion with
OpenSolaris global variable of the same name which is used in the same
code, but has different meaning (page deficit).
Note: I have an impression that locking around accesses to this variable
as well as mutual notifications between arc_reclaim_thread and
arc_lowmem are not proper.
MFC after: 1 week
Revision Changes Path
1.43 +3 -3 src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/arc.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201008241748.o7OHmhrJ044295>