Neither the tid or _mem_req seems to be used. These can get nuked, right?

It's not necessary, but maybe an assert here that the "mode" parameter indicates a fetch?

Why should this be removed?

It's not part of your change, but while you're dicing up this function lets get rid of this commented out code.

This looks more involved than the original version of the code below. Why does it need to do more work?

star->start

Ah... This is why that block of code above can be removed?

This if and the one above it really are begging to be merged. If the condition turns into a big mess then never mind, but otherwise I think we definitely want to do that.

done

done

It's not gone, it's moved... Need to check that the physical address still is what we think it is after translation. 

it's really dated.. done

you're right buildInst should have been used and then it will be ~ the same size

done

yes