Allow calls to vm_pager_haspages with the object read locked
ClosedPublic
Actions

Authored by jeff on Aug 18 2019, 8:44 AM.

Details

Reviewers

kib
markj

Commits

rS351235: Permit vm_pager_has_page() to run with a shared lock. Introduce

Summary

Only the vnode pager required the exclusive lock and only because it wanted to drop and reacquire. I added macros to automate this which will likely be useful elsewhere as we do more read locking of objects.

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 25942
Build 24503: arc lint + arc unit

Event Timeline

jeff created this revision.Aug 18 2019, 8:44 AM

Harbormaster completed remote builds in B25942: Diff 60960.Aug 18 2019, 8:44 AM

jeff retitled this revision from Allow read locked objects to call has_pages to Allow calls to vm_pager_haspages with the object read locked.Aug 18 2019, 8:46 AM

jeff edited the summary of this revision. (Show Details)

jeff added reviewers: kib, markj.

markj accepted this revision.Aug 19 2019, 4:02 PM

This revision is now accepted and ready to land.Aug 19 2019, 4:02 PM

kib accepted this revision.Aug 19 2019, 6:23 PM

Closed by commit rS351235: Permit vm_pager_has_page() to run with a shared lock. Introduce (authored by jeff). · Explain WhyAug 19 2019, 10:25 PM

This revision was automatically updated to reflect the committed changes.

jeff added a commit: rS351235: Permit vm_pager_has_page() to run with a shared lock. Introduce.

Herald added a subscriber: imp. · View Herald TranscriptAug 19 2019, 10:25 PM