This does look like a nice cleanup, but I think we also need to rename the function now that it does not return the nvmet_ns but only sets it in the nvmet_req structure. What about nvmet_req_find_namespace?