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: <20100914182707.GB15549@redhat.com> Date: Tue, 14 Sep 2010 20:27:07 +0200 From: "Michael S. Tsirkin" <mst@...hat.com> To: Shirley Ma <mashirle@...ibm.com> Cc: Arnd Bergmann <arnd@...db.de>, Avi Kivity <avi@...hat.com>, "Xin, Xiaohui" <xiaohui.xin@...el.com>, David Miller <davem@...emloft.net>, netdev@...r.kernel.org, kvm@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [RFC PATCH 2/2] macvtap: TX zero copy between guest and host kernel On Tue, Sep 14, 2010 at 10:02:25AM -0700, Shirley Ma wrote: > On Tue, 2010-09-14 at 18:29 +0200, Michael S. Tsirkin wrote: > > Precisely. This is what the patch from Xin Xiaohui does. That code > > already seems to do most of what you are trying to do, right? > > I thought host pins guest kernel buffer pages was good enough for TX > thought I didn't look up xiaohui's vhost asycn io patch in details. As others said, the harder issues for TX are in determining that it's safe to unpin the memory, and how much memory is it safe to pin to beging with. For RX we have some more complexity. > What's the performance data Xiaohui got from using kiocb? I haven't seen > any performance number from him yet. > > > The main thing missing seems to be macvtap integration, so that we can > > fall back > > on data copy if zero copy is unavailable? > > How hard would it be to basically link the mp and macvtap modules > > together to get us this functionality? Anyone? > > The simple integration is using macvtap + xiaohui's vhost asycn io > patch. I can make a try for TX only. > > Thanks > Shirley Well it's up to you of course, but this is not what I would try: if you look at the patchset vhost changes is not the largest part of it, so this sounds a bit like effort duplication. TX only is also much less interesting than full zero copy. I think that you should be able to simply combine the two drivers together, add an ioctl to enable/disable zero copy mode of operation. -- MST -- 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