[dpdk-dev] [PATCH] Request for comments on ixgbe TSO support

jigsaw jigsaw at gmail.com
Fri Oct 4 19:54:31 CEST 2013


Hi Stephen,


>>This will work for local generated packets but overlapping existing field won't work well for forwarding.
So adding a new mss field in mbuf could be the way out? or I
misunderstand something.

>> What we want to be able to do is to take offload (jumbo) packets in with from virtio
Sorry I don't understand why TSO is connected to virtio. Could you
give more details here?
Are you suggesting this TSO patch overlaps your work, or it should be
based on your work?


thx &
rgds,
-Qinglai

On Fri, Oct 4, 2013 at 8:40 PM, Stephen Hemminger
<stephen at networkplumber.org> wrote:
> On Fri,  4 Oct 2013 20:06:52 +0300
> Qinglai Xiao <jigsaw at gmail.com> wrote:
>
>> This patch is a draft of TSO on 82599. That is, it is not expected to be
>> accepted as is.
>> The problem is where to put the mss field. In this patch, the mss is put in
>> the union of hash in rte_pktmbuf. It is not the best place, but it is quite
>> convenient, since hash is not used in TX procedure.
>> The idea is to avoid increasing sizeof(struct rte_pktmbuf), while keeping mss
>> easy to access.
>>
>> However, the hash is also misleading, coz mss has nothing to do with Rx hash.
>> A more formal way could be rename hash as below:
>>
>>       union {
>>               uint32_t data;
>>               struct rx_hash hash;
>>               uint32_t tx_mss;
>>       } misc;
>>
>> It is gonna be a major change coz it affects the core data structure.
>>
>> Any comments will be appreciated.
>>
>> Qinglai Xiao (1):
>>   ixgbe: TCP/UDP segment offload support on 82599.
>>
>>  lib/librte_mbuf/rte_mbuf.h        |    6 +++++-
>>  lib/librte_pmd_ixgbe/ixgbe_rxtx.c |   32 +++++++++++++++++++++++++++++---
>>  2 files changed, 34 insertions(+), 4 deletions(-)
>>
>
> This will work for local generated packets but overlapping existing
> field won't work well for forwarding.
>
> What we want to be able to do is to take offload (jumbo) packets in
> with from virtio (need better driver support which I am doing), and then
> send them through to network devices.
>


More information about the dev mailing list