Hi Andrew,
Andrew Lunn <and...@lunn.ch> writes:
>> /* Transmit function may have to reallocate the original SKB */
>> - nskb = p->xmit(skb, dev);
>> + nskb = p->dp->ds->dst->tag_ops->xmit(skb, dev);
>
> This is also the hot path for DSA transmit. Do we really want to do 4
> extra pointer dereferences a million times per second, compared to one
> copy during setup?
Yep I get the idea. It felt weird to copy structure members like this
and not at least reusing the dsa_device_ops structure.
Thanks,
Vivien
