Thread (5 messages) 5 messages, 3 authors, 2026-04-28

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

From: Lorenzo Bianconi <lorenzo@kernel.org>
Date: 2026-04-28 05:06:33
Also in: linux-arm-kernel, linux-mediatek

On Sat, 25 Apr 2026 16:10:51 +0200 Lorenzo Bianconi wrote:
quoted
- 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.
quoted
- 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

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help