lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 03 May 2012 01:50:18 -0400 (EDT) From: David Miller <davem@...emloft.net> To: alexander.duyck@...il.com Cc: eric.dumazet@...il.com, alexander.h.duyck@...el.com, netdev@...r.kernel.org, edumazet@...gle.com, jeffrey.t.kirsher@...el.com Subject: Re: [PATCH 2/2] tcp: cleanup tcp_try_coalesce From: Alexander Duyck <alexander.duyck@...il.com> Date: Wed, 02 May 2012 22:41:36 -0700 > I think the part that has me confused is how being more precise about > removing from truesize gets in the way of detecting abuses of > truesize. It seems like it should be more as good as or better then > the original approach of just using skb->len. You can only trim from truesize if you can be absolutely certain that you have removed any reference in the fraglist, or the page'd head, to the entire "block" of data. If the skb still refers to even just one byte in a particular block, we must still charge the entire block in the truesize. For example, NIU has three pools of power-of-2 blocks of data it maintainers and the device pulls from to build incoming packets. So if the chip used one of the 2048 byte buffers, we charge the entire 2048 bytes even of the packet is much smaller. Conversely this means we cannot trim the 2048 part of the truesize of that SKB unless we had some mechanism to know for certain 1) what the block size of the underlying data is and 2) that we've removed all references to that. Currently this is not really possible, so we therefore defer truesize adjustments. Behaving otherwise is dangerous, because then we'd potentially end up with a lot of memory used, but not actually accounted for by anyone. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists