Page MenuHomeFreeBSD

Use vm_page_unwire_noq() instead of vm_page_unwire(PQ_NONE).

Authored by markj on May 30 2019, 4:36 PM.



For managed pages, these two calls are not equivalent:
vm_page_unwire(PQ_NONE) will remove the page from its page queue, if
any, while vm_page_unwire_noq() leaves it alone. However, the existing
callers of vm_page_unwire(PQ_NONE) are applying it to fictitious or
unmanaged pages, so there should be no functional impact.

Diff Detail

rS FreeBSD src repository
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

kib added inline comments.
233 ↗(On Diff #58077)

Just a note that the page is managed. It is because it is freed immediately after, we can use vm_page_unwire_noq().

This revision is now accepted and ready to land.May 30 2019, 5:27 PM
233 ↗(On Diff #58077)

This call was added in D19247, so it will not be committed with the rest of this diff.

This revision was automatically updated to reflect the committed changes.