Thread (5 messages) 5 messages, 3 authors, 2021-03-16

RE: [PATCH] nvmet-tcp: Fix receive data digest calculation for multiple h2cdata PDUs

From: Grupi, Elad <hidden>
Date: 2021-03-16 15:25:08

Sure. I will send new patch shortly.

Elad

-----Original Message-----
From: Sagi Grimberg <sagi@grimberg.me> 
Sent: Tuesday, 16 March 2021 8:30
To: Grupi, Elad; linux-nvme@lists.infradead.org; Christoph Hellwig; Keith Busch; Chaitanya Kulkarni
Subject: Re: [PATCH] nvmet-tcp: Fix receive data digest calculation for multiple h2cdata PDUs


[EXTERNAL EMAIL] 

Hi Sagi,
Hey Elad,
Can you please explain why you avoid calling nvmet_tcp_unmap_pdu_iovec on data_digest PDU?
I don't recall any other case that unmap is being called on good path.
You are correct. We need to either add unmap in the data_digest path here or when completing the data digest reception.. Probably better here.

Care to send a patch or should I?
_______________________________________________
Linux-nvme mailing list
Linux-nvme@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-nvme
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help