From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?ISO-8859-1?Q?S=E9guier_R=E9gis?= Subject: Re: [PATCH] net: via-velocity.c fix sleep-with-spinlock bug during MTU change Date: Fri, 20 Jun 2008 12:24:02 +0200 Message-ID: <485B8542.7060204@e-teleport.net> References: <20080603134019.df2f51d4.akpm@linux-foundation.org> <20080603215155.GA9712@electric-eye.fr.zoreil.com> <20080604215909.GA17446@electric-eye.fr.zoreil.com> <20080614212341.GA12976@electric-eye.fr.zoreil.com> <4856DA62.7080803@e-teleport.net> <20080617214538.GA6388@electric-eye.fr.zoreil.com> <48584AA4.8040801@e-teleport.net> <48593E35.1010409@e-teleport.net> <20080618201012.GA4019@electric-eye.fr.zoreil.com> <48598784.40306@e-teleport.net> <20080619204312.GA2160@electric-eye.fr.zoreil.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Andrew Morton , Arjan van de Ven , alan@redhat.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org To: Francois Romieu Return-path: Received: from webmail.e-teleport.net ([213.190.83.68]:37642 "EHLO mail.imsnet.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752121AbYFTKYE (ORCPT ); Fri, 20 Jun 2008 06:24:04 -0400 In-Reply-To: <20080619204312.GA2160@electric-eye.fr.zoreil.com> Sender: netdev-owner@vger.kernel.org List-ID: =46rancois Romieu a =E9crit : > S=E9guier R=E9gis : > =20 >>> I have updated http://userweb.kernel.org/~romieu/via-velocity/2.6.2= 6-rc6/ >>> =20 > [...] > =20 >> Patch 0001 doesn't work, >> >> if I don't make a mistake skb->len is not modified in skb_pad{,to}wh= en the=20 >> padding is applied. In this case, we need to attribute ETH_ZLEN to l= en. >> =20 > > Yes, that's what the driver did beforehand. > > The updated serie is at the usual place. Does it perform better ? > > =20 Ok for the mtu change, no error messages, I try multiples mtu=20 (1400,2000,4500,9000). But if I generate a paquet with a size more than around 3825, the drive= r=20 don't send it and no paquet is send after, work again after an interfac= e=20 down/up and until no paquet > 3825. --=20 R=E9gis