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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sat, 2 Apr 2022 23:05:36 +0900 From: Dominique Martinet <asmadeus@...ewreck.org> To: Christian Schoenebeck <linux_oss@...debyte.com> Cc: v9fs-developer@...ts.sourceforge.net, netdev@...r.kernel.org, Eric Van Hensbergen <ericvh@...il.com>, Latchesar Ionkov <lucho@...kov.net>, Greg Kurz <groug@...d.org>, Vivek Goyal <vgoyal@...hat.com>, Nikolay Kichukov <nikolay@...um.net> Subject: Re: [PATCH v4 12/12] net/9p: allocate appropriate reduced message buffers Christian Schoenebeck wrote on Thu, Dec 30, 2021 at 02:23:18PM +0100: > So far 'msize' was simply used for all 9p message types, which is far > too much and slowed down performance tremendously with large values > for user configurable 'msize' option. > > Let's stop this waste by using the new p9_msg_buf_size() function for > allocating more appropriate, smaller buffers according to what is > actually sent over the wire. By the way, thinking of protocols earlier made me realize this won't work on RDMA transport... unlike virtio/tcp/xen, RDMA doesn't "mailbox" messages: there's a pool of posted buffers, and once a message has been received it looks for the header in the received message and associates it with the matching request, but there's no guarantee a small message will use a small buffer... This is also going to need some thought, perhaps just copying small buffers and recycling the buffer if a large one was used? but there might be a window with no buffer available and I'm not sure what'd happen, and don't have any RDMA hardware available to test this right now so this will be fun. I'm not shooting this down (it's definitely interesting), but we might need to make it optional until someone with RDMA hardware can validate a solution. -- Dominique
Powered by blists - more mailing lists