From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH 2/3] ucc_geth: do not touch net queue in adjust_link phylib callback Date: Mon, 21 Jul 2008 08:30:51 -0700 (PDT) Message-ID: <20080721.083051.39701172.davem@davemloft.net> References: <20080721151735.GB7989@polina.dev.rtsoft.ru> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: jeff@garzik.org, netdev@vger.kernel.org To: avorontsov@ru.mvista.com Return-path: Received: from 74-93-104-97-Washington.hfc.comcastbusiness.net ([74.93.104.97]:42631 "EHLO sunset.davemloft.net" rhost-flags-OK-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1753886AbYGUPav (ORCPT ); Mon, 21 Jul 2008 11:30:51 -0400 In-Reply-To: <20080721151735.GB7989@polina.dev.rtsoft.ru> Sender: netdev-owner@vger.kernel.org List-ID: From: Anton Vorontsov Date: Mon, 21 Jul 2008 19:17:35 +0400 > If the net queue has not been started, we'll get this nice oops > and non-working ethernet: ... > Since the driver is using phylib (which is doing netif_carrier_on/off()), > we should simply remove netif_tx_schedule_all() from adjust_link(). > > Signed-off-by: Anton Vorontsov Also applied, thanks.