From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shirley Ma Subject: Re: [PATCH V5 2/6 net-next] netdevice.h: Add zero-copy flag in netdevice Date: Wed, 18 May 2011 10:00:28 -0700 Message-ID: <1305738028.32080.66.camel@localhost.localdomain> References: <1305671318.10756.49.camel@localhost.localdomain> <20110518103819.GL7589@redhat.com> <20110518111734.GO7589@redhat.com> <1305729507.32080.6.camel@localhost.localdomain> <20110518154746.GA21378@redhat.com> <1305734857.32080.53.camel@localhost.localdomain> <20110518163633.GB22001@redhat.com> <1305737140.32080.59.camel@localhost.localdomain> <20110518165138.GD22001@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: =?UTF-8?Q?Micha=C5=82_Miros=C5=82aw?= , Ben Hutchings , David Miller , Eric Dumazet , Avi Kivity , Arnd Bergmann , netdev@vger.kernel.org, kvm@vger.kernel.org, linux-kernel@vger.kernel.org To: "Michael S. Tsirkin" Return-path: In-Reply-To: <20110518165138.GD22001@redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On Wed, 2011-05-18 at 19:51 +0300, Michael S. Tsirkin wrote: > > > Yes, I agree. I think for tcpdump, we really need to copy the > data > > > anyway, to avoid guest changing it in between. So we do that and > then > > > use the copy everywhere, release the old one. Hmm? > > > > Yes. Old one use zerocopy, new one use copy data. > > > > Thanks > > Shirley > > No, that's wrong, as they might become different with a > malicious guest. As long as we copied already, lets realease > the data and have everyone use the copy. Ok, I will patch pskb_expand_head to test it out. Shirley