[PATCH net] net: airoha: Do not read uninitialized fragment address in airoha_dev_xmit()

Lorenzo Bianconi lorenzo at kernel.org
Mon Apr 27 22:06:22 PDT 2026


> On Sat, 25 Apr 2026 16:10:51 +0200 Lorenzo Bianconi wrote:
> > - Does the TSO checksum calculation earlier in this function ensure the
> >   TCP header is in the linear portion of the SKB?
> >   This issue is not related to the current patch. Moreover, can we have a TSO
> >   packet where the tcp header is not in the linear area of the skb?
> 
> Don't think so.
> 
> > - If dma_map_single() fails partway through a multi-fragment packet and
> >   triggers this goto error_unmap, will it break the Tx ring contiguity?
> >   This issue is not related to the current patch. Moreover, I guess the hw is
> >   capable of managing out-of-order descriptors.
> > 
> > - Is it safe to map fragment data using dma_map_single() instead of
> >   skb_frag_dma_map()?
> >   This issue is not related to the current patch. I will post a dedicated patch
> >   for it.
> 
> I don't think it's an issue. Unless you support DMABUF zero-copy 
> the skb_frag helper is just for convenience. I wouldn't bother.

ack, let's skip it for the moment.

Regards,
Lorenzo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-mediatek/attachments/20260428/97dd05ea/attachment.sig>


More information about the Linux-mediatek mailing list