netdev - Re: [Xen-devel] [3.15-rc3] Bisected: xen-netback mangles packets between two guests on a bridge since merge of "TX grant mapping with SKBTX_DEV

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <653156457.20140502160632@eikelenboom.it>
Date:	Fri, 2 May 2014 16:06:32 +0200
From:	Sander Eikelenboom <linux@...elenboom.it>
To:	Zoltan Kiss <zoltan.kiss@...rix.com>
CC:	Eric Dumazet <eric.dumazet@...il.com>, netdev@...r.kernel.org,
	<xen-devel@...ts.xen.org>, Ian Campbell <Ian.Campbell@...rix.com>,
	"David S. Miller" <davem@...emloft.net>
Subject: Re: [Xen-devel] [3.15-rc3] Bisected: xen-netback mangles packets between two guests on a bridge since merge of "TX grant mapping with SKBTX_DEV_ZEROCOPY instead of copy" series.


Friday, May 2, 2014, 4:00:40 PM, you wrote:

>>> Let's see what that does different and if that gives an insight in what is going
>>> wrong.
>>
>> The debug patch that i used is attached.
>> Here is some of the output, skb->data_len size sometimes varies slightly as does skb->truesize (is that allowed to become > 64k ?).
> Yes, truesize can be over 64K, as it includes the aligned size of the 
> data itself (64k max), struct skbuff and struct shared_info.
> However my calculation for the truesize might be wrong: I count all the 
> frags as full sized pages (skb->truesize += i * PAGE_SIZE), but the last 
> page is not necessarily used entirely by that frag.
> I had the idea that truesize should include the size of the whole page 
> as we reserved it with alloc_page for this skb, but it might not be true.
Btw. with you "skb->>truesize -= nskb->data_len;" version truesize should 
> end up around 128k, something entirely wrong. Can you try with 
"skb->>truesize -= nskb->data_len;" in that line?
> My repro env is underway in the meantime ...

Erhmm i don't get this last part, first you entitle
"skb->truesize -= nskb->data_len;" as being "entirely wrong" (which is probably true)

Then you want me to try just that ? .. or did you mean something else ? 

> Regards,

> Zoli


--
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