On Wed, 28 Sep 2022 14:33:36 +1000 David Gibson <david(a)gibson.dropbear.id.au> wrote:I can't get these warnings to trigger on the cppcheck versions I have, so remove them. If we find in future we need to replace these, they should be replaced with inline suppressions so its clear what's the section of code at issue.At least for TCP: - /* First buffer is to discard data, last one may be partially filled. */ - iov[-1].iov_len = already_sent; gone with commit 38fbfdbcb916 ("tcp: Get rid of iov with cached MSS, drop sendmmsg(), add deferred flush"). -- Stefano