DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] FW: BUG in IP FRAGMENTATION
@ 2014-10-15 11:06 Dey, Souvik
  2014-10-15 11:23 ` Thomas Monjalon
  0 siblings, 1 reply; 5+ messages in thread
From: Dey, Souvik @ 2014-10-15 11:06 UTC (permalink / raw)
  To: dev; +Cc: Patil, PraveenKumar


Hi,
                In DPDK1.6 do we support overlapped fragments while doing reassembly. Also why we don't consider the first or last fragment to be out-of-order. Are this known limitations in DPDK or they are not working due to some bugs in the code ?

--
Regards,
Souvik

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] FW: BUG in IP FRAGMENTATION
  2014-10-15 11:06 [dpdk-dev] FW: BUG in IP FRAGMENTATION Dey, Souvik
@ 2014-10-15 11:23 ` Thomas Monjalon
  2014-10-15 11:52   ` Dey, Souvik
  0 siblings, 1 reply; 5+ messages in thread
From: Thomas Monjalon @ 2014-10-15 11:23 UTC (permalink / raw)
  To: Dey, Souvik; +Cc: dev, Patil, PraveenKumar

ME TOO, I HAVE A BUG WITH CAPS LOCK ;)

2014-10-15 11:06, Dey, Souvik:
> In DPDK1.6 do we support overlapped fragments while doing reassembly.
> Also why we don't consider the first or last fragment to be out-of-order.
> Are this known limitations in DPDK or they are not working due to some
> bugs in the code ?

Please test the latest version and explain how you see the bug.

Thanks
-- 
Thomas

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] FW: BUG in IP FRAGMENTATION
  2014-10-15 11:23 ` Thomas Monjalon
@ 2014-10-15 11:52   ` Dey, Souvik
  2014-10-15 12:18     ` Ananyev, Konstantin
  0 siblings, 1 reply; 5+ messages in thread
From: Dey, Souvik @ 2014-10-15 11:52 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: dev, Patil, PraveenKumar

We have not directly tried to use the 1.7 code with fragmentation apis, but we did run through the 1.7 code and did not find any much difference between 1.6 and 1.7 code. 
I had wrongly mentioned out-of-order in my previous mail. Actually out-of-order is working fine but we are facing issues with overlapping and duplicate fragments.
 In the 1.7 fragmentation code also in file ip_frag_internals.c , function ip_frag_process we also see this comment

	/*
	 * errorneous packet: either exceeed max allowed number of fragments,
	 * or duplicate first/last fragment encountered.
	 */

Which indirectly suggest that the handling of duplicate first/last fragment is taken as error. Same with overlapping fragment we could not find any piece of code which will be doing it.

--
Regards,
Souvik

-----Original Message-----
From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] 
Sent: Wednesday, October 15, 2014 4:54 PM
To: Dey, Souvik
Cc: dev@dpdk.org; Patil, PraveenKumar
Subject: Re: [dpdk-dev] FW: BUG in IP FRAGMENTATION

ME TOO, I HAVE A BUG WITH CAPS LOCK ;)

2014-10-15 11:06, Dey, Souvik:
> In DPDK1.6 do we support overlapped fragments while doing reassembly.
> Also why we don't consider the first or last fragment to be out-of-order.
> Are this known limitations in DPDK or they are not working due to some 
> bugs in the code ?

Please test the latest version and explain how you see the bug.

Thanks
--
Thomas

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] FW: BUG in IP FRAGMENTATION
  2014-10-15 11:52   ` Dey, Souvik
@ 2014-10-15 12:18     ` Ananyev, Konstantin
  2014-10-15 12:30       ` Dey, Souvik
  0 siblings, 1 reply; 5+ messages in thread
From: Ananyev, Konstantin @ 2014-10-15 12:18 UTC (permalink / raw)
  To: Dey, Souvik, Thomas Monjalon; +Cc: dev, Patil, PraveenKumar



> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Dey, Souvik
> Sent: Wednesday, October 15, 2014 12:53 PM
> To: Thomas Monjalon
> Cc: dev@dpdk.org; Patil, PraveenKumar
> Subject: Re: [dpdk-dev] FW: BUG in IP FRAGMENTATION
> 
> We have not directly tried to use the 1.7 code with fragmentation apis, but we did run through the 1.7 code and did not find any much
> difference between 1.6 and 1.7 code.
> I had wrongly mentioned out-of-order in my previous mail. Actually out-of-order is working fine but we are facing issues with
> overlapping and duplicate fragments.
>  In the 1.7 fragmentation code also in file ip_frag_internals.c , function ip_frag_process we also see this comment
> 
> 	/*
> 	 * errorneous packet: either exceeed max allowed number of fragments,
> 	 * or duplicate first/last fragment encountered.
> 	 */
> 
> Which indirectly suggest that the handling of duplicate first/last fragment is taken as error. Same with overlapping fragment we could
> not find any piece of code which will be doing it.

Yes, that's right.
If we encounter a duplicate and/or overlapping fragment we treat it  as an error.

> 
> --
> Regards,
> Souvik
> 
> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com]
> Sent: Wednesday, October 15, 2014 4:54 PM
> To: Dey, Souvik
> Cc: dev@dpdk.org; Patil, PraveenKumar
> Subject: Re: [dpdk-dev] FW: BUG in IP FRAGMENTATION
> 
> ME TOO, I HAVE A BUG WITH CAPS LOCK ;)
> 
> 2014-10-15 11:06, Dey, Souvik:
> > In DPDK1.6 do we support overlapped fragments while doing reassembly.
> > Also why we don't consider the first or last fragment to be out-of-order.
> > Are this known limitations in DPDK or they are not working due to some
> > bugs in the code ?
> 
> Please test the latest version and explain how you see the bug.
> 
> Thanks
> --
> Thomas

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] FW: BUG in IP FRAGMENTATION
  2014-10-15 12:18     ` Ananyev, Konstantin
@ 2014-10-15 12:30       ` Dey, Souvik
  0 siblings, 0 replies; 5+ messages in thread
From: Dey, Souvik @ 2014-10-15 12:30 UTC (permalink / raw)
  To: Ananyev, Konstantin, Thomas Monjalon; +Cc: dev, Patil, PraveenKumar

Any plans to support this in future in DPDK fragmentation code ?

-----Original Message-----
From: Ananyev, Konstantin [mailto:konstantin.ananyev@intel.com] 
Sent: Wednesday, October 15, 2014 5:49 PM
To: Dey, Souvik; Thomas Monjalon
Cc: dev@dpdk.org; Patil, PraveenKumar
Subject: RE: [dpdk-dev] FW: BUG in IP FRAGMENTATION



> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Dey, Souvik
> Sent: Wednesday, October 15, 2014 12:53 PM
> To: Thomas Monjalon
> Cc: dev@dpdk.org; Patil, PraveenKumar
> Subject: Re: [dpdk-dev] FW: BUG in IP FRAGMENTATION
> 
> We have not directly tried to use the 1.7 code with fragmentation 
> apis, but we did run through the 1.7 code and did not find any much difference between 1.6 and 1.7 code.
> I had wrongly mentioned out-of-order in my previous mail. Actually 
> out-of-order is working fine but we are facing issues with overlapping and duplicate fragments.
>  In the 1.7 fragmentation code also in file ip_frag_internals.c , 
> function ip_frag_process we also see this comment
> 
> 	/*
> 	 * errorneous packet: either exceeed max allowed number of fragments,
> 	 * or duplicate first/last fragment encountered.
> 	 */
> 
> Which indirectly suggest that the handling of duplicate first/last 
> fragment is taken as error. Same with overlapping fragment we could not find any piece of code which will be doing it.

Yes, that's right.
If we encounter a duplicate and/or overlapping fragment we treat it  as an error.

> 
> --
> Regards,
> Souvik
> 
> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com]
> Sent: Wednesday, October 15, 2014 4:54 PM
> To: Dey, Souvik
> Cc: dev@dpdk.org; Patil, PraveenKumar
> Subject: Re: [dpdk-dev] FW: BUG in IP FRAGMENTATION
> 
> ME TOO, I HAVE A BUG WITH CAPS LOCK ;)
> 
> 2014-10-15 11:06, Dey, Souvik:
> > In DPDK1.6 do we support overlapped fragments while doing reassembly.
> > Also why we don't consider the first or last fragment to be out-of-order.
> > Are this known limitations in DPDK or they are not working due to 
> > some bugs in the code ?
> 
> Please test the latest version and explain how you see the bug.
> 
> Thanks
> --
> Thomas

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2014-10-15 12:23 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-10-15 11:06 [dpdk-dev] FW: BUG in IP FRAGMENTATION Dey, Souvik
2014-10-15 11:23 ` Thomas Monjalon
2014-10-15 11:52   ` Dey, Souvik
2014-10-15 12:18     ` Ananyev, Konstantin
2014-10-15 12:30       ` Dey, Souvik

DPDK patches and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/dev/0 dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dev dev/ https://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git