On 11/5/2024 11:18 PM, Keith Busch wrote: > + status = nvmet_copy_to_sgl(req, 0, logs, 1024); > + kfree(logs); > +out: > + nvmet_req_complete(req, 0); Shouldn't this be: nvmet_req_complete(req, status)?