if (queue->hdr_digest && !req->offset) - nvme_tcp_hdgst(queue->snd_hash, pdu, sizeof(*pdu)); + *(__le32 *)(pdu + 1) = nvme_tcp_hdgst(pdu, sizeof(*pdu));
I'd move this assignment to a helper nvme_tcp_set_hdgst(), especially as it has two call-sites. Other than that, Reviewed-by: Sagi Grimberg <sagi@xxxxxxxxxxx