Re: [PATCH] gianfar:don't add FCB length to hard_header_len

2012-05-25 Thread Paul Gortmaker
On 12-05-25 03:51 PM, Joe Perches wrote: > On Fri, 2012-05-25 at 11:58 -0400, Paul Gortmaker wrote: >> But you really shouldn't need the hardware to validate this kind of >> patch anyways -- aside from your code flow change in the irq routine of >> gianfar_ptp, you should have been simply able to c

Re: [PATCH] gianfar:don't add FCB length to hard_header_len

2012-05-25 Thread Joe Perches
On Fri, 2012-05-25 at 11:58 -0400, Paul Gortmaker wrote: > But you really shouldn't need the hardware to validate this kind of > patch anyways -- aside from your code flow change in the irq routine of > gianfar_ptp, you should have been simply able to check for object file > equivalence before and

Re: [PATCH] gianfar:don't add FCB length to hard_header_len

2012-05-25 Thread Paul Gortmaker
[Re: [PATCH] gianfar:don't add FCB length to hard_header_len] On 24/05/2012 (Thu 09:16) Joe Perches wrote: > On Thu, 2012-05-24 at 17:04 +0200, Jan Ceuleers wrote: > > On 05/22/2012 09:18 PM, David Miller wrote: > > > From: Jiajun Wu > > > Date: Tue, 22 May 2012

Re: [PATCH] gianfar:don't add FCB length to hard_header_len

2012-05-24 Thread Jan Ceuleers
On 05/24/2012 06:16 PM, Joe Perches wrote: > On Thu, 2012-05-24 at 17:04 +0200, Jan Ceuleers wrote: >> On 05/22/2012 09:18 PM, David Miller wrote: >>> Someone needs to go through this driver when net-next opens up >>> and fix all of the indentation in this driver. >> >> May I give that a go? > > I

Re: [PATCH] gianfar:don't add FCB length to hard_header_len

2012-05-24 Thread Jan Ceuleers
On 05/22/2012 09:18 PM, David Miller wrote: > From: Jiajun Wu > Date: Tue, 22 May 2012 17:00:48 +0800 > >> FCB(Frame Control Block) isn't the part of netdev hard header. >> Add FCB to hard_header_len will make GRO fail at MAC comparision stage. >> >> Signed-off-by: Jiajun Wu > > Applied, thanks

Re: [PATCH] gianfar:don't add FCB length to hard_header_len

2012-05-24 Thread Joe Perches
On Thu, 2012-05-24 at 17:04 +0200, Jan Ceuleers wrote: > On 05/22/2012 09:18 PM, David Miller wrote: > > From: Jiajun Wu > > Date: Tue, 22 May 2012 17:00:48 +0800 > > > >> FCB(Frame Control Block) isn't the part of netdev hard header. > >> Add FCB to hard_header_len will make GRO fail at MAC comp

Re: [PATCH] gianfar:don't add FCB length to hard_header_len

2012-05-22 Thread David Miller
From: Jiajun Wu Date: Tue, 22 May 2012 17:00:48 +0800 > FCB(Frame Control Block) isn't the part of netdev hard header. > Add FCB to hard_header_len will make GRO fail at MAC comparision stage. > > Signed-off-by: Jiajun Wu Applied, thanks. Someone needs to go through this driver when net-next

[PATCH] gianfar:don't add FCB length to hard_header_len

2012-05-22 Thread Jiajun Wu
FCB(Frame Control Block) isn't the part of netdev hard header. Add FCB to hard_header_len will make GRO fail at MAC comparision stage. Signed-off-by: Jiajun Wu --- drivers/net/ethernet/freescale/gianfar.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/net/ether