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, 20 Dec 2007 03:44:37 -0800 (PST) From: David Miller <davem@...emloft.net> To: haoki@...hat.com Cc: herbert@...dor.apana.org.au, netdev@...r.kernel.org, tyasui@...hat.com, mhiramat@...hat.com, satoshi.oshima.fk@...achi.com, billfink@...dspring.com, andi@...stfloor.org, johnpol@....mipt.ru, shemminger@...ux-foundation.org, yoshfuji@...ux-ipv6.org, yumiko.sugita.yf@...achi.com Subject: Re: [PATCH 4/4] [UDP]: memory accounting in IPv4 From: Hideo AOKI <haoki@...hat.com> Date: Mon, 17 Dec 2007 21:38:47 -0500 > This patch adds UDP memory usage accounting in IPv4. > > Send buffer accounting is performed by IP layer, because skbuff is > allocated in the layer. > > Receive buffer is charged, when the buffer successfully received. > Destructor of the buffer does uncharging and reclaiming, when the > buffer is freed. To set destructor at proper place, we use > __udp_queue_rcv_skb() instead of sock_queue_rcv_skb(). To maintain > consistency of memory accounting, socket lock is used to free receive > buffer in udp_recvmsg(). > > New packet will be add to backlog when the socket is used by user. > > Cc: Satoshi Oshima <satoshi.oshima.fk@...achi.com> > signed-off-by: Takahiro Yasui <tyasui@...hat.com> > signed-off-by: Masami Hiramatsu <mhiramat@...hat.com> > signed-off-by: Hideo Aoki <haoki@...hat.com> We can't accept these changes, even once the other issues are fixed, until IPV6 is supported as well. It's pointless to support proper UDP memory accounting only in IPV4 and not in IPV6 as well. -- 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