diff options
author | alc <alc@FreeBSD.org> | 2006-07-01 23:43:54 +0000 |
---|---|---|
committer | alc <alc@FreeBSD.org> | 2006-07-01 23:43:54 +0000 |
commit | 62b562eed7986c25571417770adb496f2503e5d9 (patch) | |
tree | a8d7824cda178c199255c4be6d70fd89025f59af /sys/amd64 | |
parent | c05d97a8923e4260d738512928cda22e3204c923 (diff) | |
download | FreeBSD-src-62b562eed7986c25571417770adb496f2503e5d9.zip FreeBSD-src-62b562eed7986c25571417770adb496f2503e5d9.tar.gz |
Tidy up pmap_ts_referenced(): Eliminate excessive white space. Eliminate
an initialized but otherwise unused variable. Explicitly check a pointer
against NULL.
There are no functional changes. Checked by: md5
Diffstat (limited to 'sys/amd64')
-rw-r--r-- | sys/amd64/amd64/pmap.c | 12 |
1 files changed, 1 insertions, 11 deletions
diff --git a/sys/amd64/amd64/pmap.c b/sys/amd64/amd64/pmap.c index d5e618c..d95bbaa 100644 --- a/sys/amd64/amd64/pmap.c +++ b/sys/amd64/amd64/pmap.c @@ -3047,32 +3047,23 @@ pmap_ts_referenced(vm_page_t m) pv_entry_t pv, pvf, pvn; pmap_t pmap; pt_entry_t *pte; - pt_entry_t v; int rtval = 0; if (m->flags & PG_FICTITIOUS) return (rtval); - mtx_assert(&vm_page_queue_mtx, MA_OWNED); if ((pv = TAILQ_FIRST(&m->md.pv_list)) != NULL) { - pvf = pv; - do { pvn = TAILQ_NEXT(pv, pv_list); - TAILQ_REMOVE(&m->md.pv_list, pv, pv_list); - TAILQ_INSERT_TAIL(&m->md.pv_list, pv, pv_list); - pmap = PV_PMAP(pv); PMAP_LOCK(pmap); pte = pmap_pte(pmap, pv->pv_va); - - if (pte && ((v = pte_load(pte)) & PG_A) != 0) { + if (pte != NULL && (*pte & PG_A) != 0) { atomic_clear_long(pte, PG_A); pmap_invalidate_page(pmap, pv->pv_va); - rtval++; if (rtval > 4) { PMAP_UNLOCK(pmap); @@ -3082,7 +3073,6 @@ pmap_ts_referenced(vm_page_t m) PMAP_UNLOCK(pmap); } while ((pv = pvn) != NULL && pv != pvf); } - return (rtval); } |