diff options
author | alc <alc@FreeBSD.org> | 2012-06-16 18:56:19 +0000 |
---|---|---|
committer | alc <alc@FreeBSD.org> | 2012-06-16 18:56:19 +0000 |
commit | 6eeaee04e4175a3faa0ce3f7778a3abce0d8d9a2 (patch) | |
tree | ebf0376a21131a0f41835773afe0ddda56d3efbe /sys/vm/swap_pager.c | |
parent | ca31dfb2b475f8026e45a8b10673993068ebc000 (diff) | |
download | FreeBSD-src-6eeaee04e4175a3faa0ce3f7778a3abce0d8d9a2.zip FreeBSD-src-6eeaee04e4175a3faa0ce3f7778a3abce0d8d9a2.tar.gz |
The page flag PGA_WRITEABLE is set and cleared exclusively by the pmap
layer, but it is read directly by the MI VM layer. This change introduces
pmap_page_is_write_mapped() in order to completely encapsulate all direct
access to PGA_WRITEABLE in the pmap layer.
Aesthetics aside, I am making this change because amd64 will likely begin
using an alternative method to track write mappings, and having
pmap_page_is_write_mapped() in place allows me to make such a change
without further modification to the MI VM layer.
As an added bonus, tidy up some nearby comments concerning page flags.
Reviewed by: kib
MFC after: 6 weeks
Diffstat (limited to 'sys/vm/swap_pager.c')
-rw-r--r-- | sys/vm/swap_pager.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/vm/swap_pager.c b/sys/vm/swap_pager.c index be999cc..3c902e3 100644 --- a/sys/vm/swap_pager.c +++ b/sys/vm/swap_pager.c @@ -1593,7 +1593,7 @@ swp_pager_async_iodone(struct buf *bp) * status, then finish the I/O ( which decrements the * busy count and possibly wakes waiter's up ). */ - KASSERT((m->aflags & PGA_WRITEABLE) == 0, + KASSERT(!pmap_page_is_write_mapped(m), ("swp_pager_async_iodone: page %p is not write" " protected", m)); vm_page_undirty(m); |