vm_fault: Factor out per-object operations into vm_fault_object()

[markj]

Summary

No functional change intended.

Obtained from: jeff (object_concurrency patches)

[markj]

Rebase.

[markj]

Push OBJ_DEAD checking into vm_fault_object() as well.

[kib]

sys/vm/vm_fault.c

1540

I liked the old /* break to PAGE HAS BEEN FOUND. */ comment. I wonder if you could just paste it there.

1541

This is from vm_fault_getpages()?

[markj]

sys/vm/vm_fault.c

1540

Well, it sounds somewhat obvious now that the lookup loop shrunk from 400LOC (in stable/12) to 45LOC. This fragment could be written like this:

res = vm_fault_object(&fs, &behind, &ahead);
if (res == FAULT_SOFT)
    break; /* page has been found */
if (res == FAULT_HARD) {
    faultcount = behind + 1 + ahead;
    hardfault = true;
    break; /* page has been found */
}
switch (res) {
    <handle other statuses>
}

Or do you prefer to fully restore the old "PAGE HAS BEEN FOUND" label? I don't have strong feelings about it.

1541

Yes.

[kib]

sys/vm/vm_fault.c

1540

I used this comment for orientation in the file. I remembered to search for it to find the core anchor basically splitting the fault handling into 'before we have a page' (lookup/validation) and 'after' (doing pmap work and post-accounting).

May be I can use FAULT_SOFT/FAULT_HARD for this after your change.