This patch follows the spirit of D63594, and removes some null checks
for things which should be operating invariants. Specifically
{Read,Write}[GF]PR now no longer check whether the supplied buffers are
null, because they never are. After this, the Do*** versions of these
function no longer serve any purpose and are inlined into their callers.
Other cleanups are possible here too, but I am taking this one step at a
time because this involves a lot of architecture-specific code, which I
don't have the hardware to test on (I did do a build-test though).
This keeps the existing bug in place as the size should be rather sizeof(ioVec). But then sizeof(ioVec) is also not much useful as it is used only for PtraceDisplayBytes. It is probably considered as a different cleanup patch so this patch does not touch it.