Re: [PATCH] vmxnet3: must split too big fragments

2012-11-02 Thread David Miller
From: Eric Dumazet Date: Mon, 29 Oct 2012 18:30:49 +0100 > From: Eric Dumazet > > vmxnet3 has a 16Kbytes limit per tx descriptor, that happened to work > as long as we provided PAGE_SIZE fragments. > > Our stack can now build larger fragments, so we need to split them to > the 16kbytes boundar

Re: [PATCH] vmxnet3: must split too big fragments

2012-10-29 Thread Shreyas Bhatewara
> > Signed-of-by: Shreyas Bhatewara Pardon the typo. And also, thanks to Jongman for testing. Signed-off-by: Shreyas Bhatewara -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.ke

Re: [PATCH] vmxnet3: must split too big fragments

2012-10-29 Thread Shreyas Bhatewara
- Original Message - > From: Eric Dumazet > > vmxnet3 has a 16Kbytes limit per tx descriptor, that happened to work > as long as we provided PAGE_SIZE fragments. > > Our stack can now build larger fragments, so we need to split them to > the 16kbytes boundary. > > Signed-off-by: Eric

Re: [Pv-drivers] [PATCH] vmxnet3: must split too big fragments

2012-10-29 Thread Eric Dumazet
On Mon, 2012-10-29 at 10:52 -0700, Bhavesh Davda wrote: > LGTM. Thanks for doing this! Did you do any performance testing with this > patch? > > Reviewed-by: Bhavesh Davda Just to be clear : I coded the patch and compiled it, but didnt test it. Jongman did the tests ;) Thanks ! -- To unsubs

Re: [Pv-drivers] [PATCH] vmxnet3: must split too big fragments

2012-10-29 Thread Bhavesh Davda
Inc." , net...@vger.kernel.org, > linux-kernel@vger.kernel.org, "jongman heo" > > Sent: Monday, October 29, 2012 10:30:49 AM > Subject: [Pv-drivers] [PATCH] vmxnet3: must split too big fragments > > From: Eric Dumazet > > vmxnet3 has a 16Kbytes limit per tx de

[PATCH] vmxnet3: must split too big fragments

2012-10-29 Thread Eric Dumazet
From: Eric Dumazet vmxnet3 has a 16Kbytes limit per tx descriptor, that happened to work as long as we provided PAGE_SIZE fragments. Our stack can now build larger fragments, so we need to split them to the 16kbytes boundary. Signed-off-by: Eric Dumazet Reported-by: jongman heo Tested-by: jon