On Tue, Aug 16 2022, Heng Qi <[email protected]> wrote:

> From: Xuan Zhuo <[email protected]>
>
> The purpose of this feature is to split the header and the payload of
> the packet.
>
> |                    receive buffer                                    |
> |                       0th descriptor             | 1th descriptor    |
> | virtnet hdr | mac | ip hdr | tcp hdr|<-- hold -->|   payload         |
>
> We can use a buffer plus a separate page when allocating the receive
> buffer. In this way, we can ensure that all payloads can be
> independently in a page, which is very beneficial for the zerocopy
> implemented by the upper layer.
>
> Signed-off-by: Xuan Zhuo <[email protected]>
> Signed-off-by: Heng Qi <[email protected]>
> Reviewed-by: Kangjie Xu <[email protected]>
> ---
> v7:
>       1. Fix some presentation issues.
>       2. Use "split transport header". @Jason Wang
>       3. Clarify some paragraphs. @Cornelia Huck
>       4. determine the device what to do if it does not perform header split 
> on a packet.
>
> v6:
>       1. Fix some syntax issues. @Cornelia Huck
>       2. Clarify some paragraphs. @Cornelia Huck
>       3. Determine the device what to do if it does not perform header split 
> on a packet.
>
> v5:
>       1. Determine when hdr_len is credible in the process of rx
>       2. Clean up the use of buffers and descriptors
>       3. Clarify the meaning of used lenght if the first descriptor is 
> skipped in the case of merge
>
> v4:
>       1. fix typo @Cornelia Huck @Jason Wang
>       2. do not split header for IP fragmentation packet. @Jason Wang
>
> v3:
>       1. Fix some syntax issues
>       2. Fix some terminology issues
>       3. It is not unified with ip alignment, so ip alignment is not included
>       4. Make it clear that the device must support four types, in the case 
> of successful negotiation.
>
>  conformance.tex |   2 ++
>  content.tex     | 102 
> ++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 104 insertions(+)

I do not have any further comments on the change, let's see what the
networking folks think.

[Do we require patches to be posted to virtio-comment, or is virtio-dev
enough? I'm a bit unsure right now.]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to