| 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
| ||
|
Message-Id: <20140731.215907.860867918938148695.davem@davemloft.net> Date: Thu, 31 Jul 2014 21:59:07 -0700 (PDT) From: David Miller <davem@...emloft.net> To: kys@...rosoft.com Cc: netdev@...r.kernel.org, linux-kernel@...r.kernel.org, devel@...uxdriverproject.org, olaf@...fle.de, apw@...onical.com, jasowang@...hat.com Subject: Re: [PATCH 1/1] Drivers: net-next: hyperv: Increase the size of the sendbuf region From: "K. Y. Srinivasan" <kys@...rosoft.com> Date: Wed, 30 Jul 2014 18:35:49 -0700 > For forwarding scenarios, it will be useful to allocate larger > sendbuf. Make the necessary adjustments to permit this. > > Signed-off-by: K. Y. Srinivasan <kys@...rosoft.com> This needs more information. You're increasing the size by 16 times, 1MB --> 16MB, thus less cache locality. You're also now using vmalloc() memory, thus more TLB misses and thrashing. This must have a negative impact on performance, and you have to test for that and quantify it when making a change as serious as this one. You also haven't gone into detail as to why forwarding scenerios require more buffer space, than say thousands of local sockets sending bulk TCP data. I'm not applying this, it needs a lot more work. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists