Should we just drive these flags into the walk() function, rather than set them here?   It seems cleaner to me to have a single set of flags that the caller gets to set, rather than setting the page-in flags here based on our flags.  I don't feel strongly.