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: Fri, 04 May 2012 17:26:40 -0700 From: Alexander Duyck <alexander.h.duyck@...el.com> To: netdev@...r.kernel.org Cc: davem@...emloft.net, eric.dumazet@...il.com, jeffrey.t.kirsher@...el.com Subject: [PATCH 0/3] First pass of cleanups for pskb_expand_head After looking over the tcp coalesing and GRO code a couple of days ago it occurred to me that pskb_expand_head has a few flaws. A few of which are addressed in this patch series. This change set takes care of some of the minor cleanup items. One thing that caught my eye is the fact the memmove code in the fast-path is likely no longer doing any thing but burning cycles on a call that doesn't actually move any memory. The other change is a follow on to that to drop the fastpath variable which really just means if the skb is cloned or not. The final change in this set just adds an inline for getting the end offset since there were multiple places where we were computing end - head to get the offset and if we are storing it as an offset it makes more sense to just pull the actual value. There are a few more items that I will try to get to next week. The big one is the fact that pskb_expand_head can mess up the truesize since it can allocate a new head but never updates the truesize. I plan on adding a helper function for the cases where we are just using it unshare the head so I can identify the places where we are actually modifying the size. --- Alexander Duyck (3): skb: Add inline helper for getting the skb end offset from head skb: Drop "fastpath" variable for skb_cloned check in pskb_expand_head skb: Drop bad code from pskb_expand_head drivers/atm/ambassador.c | 2 + drivers/atm/idt77252.c | 2 + drivers/net/wimax/i2400m/usb-rx.c | 2 + drivers/staging/octeon/ethernet-tx.c | 2 + include/linux/skbuff.h | 12 ++++++++- net/core/skbuff.c | 46 ++++++++++------------------------ 6 files changed, 29 insertions(+), 37 deletions(-) -- Thanks, Alex -- 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