WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] [XM-TEST] Use explicit (IP id==packet size) on h

To: niv@xxxxxxxxxx
Subject: Re: [Xen-devel] [PATCH] [XM-TEST] Use explicit (IP id==packet size) on hping2 tests to avoid problems with IP id==0
From: James Dykman <dykman@xxxxxxxxxx>
Date: Thu, 18 May 2006 17:54:53 -0400
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Thu, 18 May 2006 14:55:19 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <446CE7EA.6020601@xxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
niv@xxxxxxxxxxxxxxxxxxxxxxx wrote on 05/18/2006 05:32:26 PM:

> James Dykman wrote:
> > We have run into some xm-test network test failures in the tests that 
use 
> > hping2 that are caused by an IP id of 0:
> > 
> > - hping2 uses a packet socket, so must do its own fragmentation when 
> > sending messages >1500 bytes.
> > - When fragmenting, hping2 uses the low byte of the PID as the IP id 
in 
> > each of the fragmented IP headers.
> > - The kernel will assign an IP id if it sees the IP id ==  0 in the 
header 
> > that hping2 passes down 
> >    (net/ipv4/raw.c:raw_send_hdrinc())
> > So if the low byte of the PID is 0, every fragment will get assigned a 

> > unique IP id by the kernel, and 
> > reassembly will fail at the destination.
> > 
> > Jim
> > ----------------------------------
> > Use explicit (IP id==packet size) on hping2 tests to avoid problems 
with 
> > IP id==0
> 
> Jim, will this not be a cause potential corruption - if we have incoming
> fragments belonging to different packets with the same ipid
> there is no way for the reassembly code to distinguish them.
> Checksum failures might catch some, but that is no guarantee of
> safe delivery.
> 

That possibility exists in the tests as they are today... hping2 is 
assigning IP ids based on
its PID, completely independent of the kernel.

If you want to solve that problem TOO, then we need to replace hping2. Any 
suggestions?

Jim

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel