From patchwork Thu Jan 29 01:49:18 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Parag Warudkar X-Patchwork-Id: 20920 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by ozlabs.org (Postfix) with ESMTP id 40067DDFDC for ; Thu, 29 Jan 2009 12:49:40 +1100 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754808AbZA2Btd (ORCPT ); Wed, 28 Jan 2009 20:49:33 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753132AbZA2Btb (ORCPT ); Wed, 28 Jan 2009 20:49:31 -0500 Received: from yw-out-2324.google.com ([74.125.46.28]:9948 "EHLO yw-out-2324.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753104AbZA2Bta (ORCPT ); Wed, 28 Jan 2009 20:49:30 -0500 Received: by yw-out-2324.google.com with SMTP id 9so3134815ywe.1 for ; Wed, 28 Jan 2009 17:49:27 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:date:from:x-x-sender:to:cc :subject:in-reply-to:message-id:references:user-agent:mime-version :content-type; bh=tev8MEdSEwT0VsbT5rnNbuSYplhDDhXhBC4a6peB6e8=; b=RU6e0u/nk/svji9Xd7cLtlePdnvn2iQ04YHkQxI3GBcvu8rZ0mkOXjq+iFswOGLwYV yRXpwPkrjS70ssZ5MINocpgbd8hrcOyqLaaoih4mW1VoGqwNqlI7pkfUe2yDY7/8dB4J AbLMy7ieiFnsf9gtq6OzZC0qlm2hDikvDCI2c= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=date:from:x-x-sender:to:cc:subject:in-reply-to:message-id :references:user-agent:mime-version:content-type; b=TN+nDbpmc2UBPfps1hx/3nl8de3faBYBFngaOVAkoff6uLxmHbAhHxaZF3vqOc52JQ WyxnCQR2OelF8FJcxSxvjxK0UtHJ4Ec+iy7foOXAcz26kusVLzHDalHNN+y+0U8Jplgp Nkhi6/xOtQFNmumiTrYQekyR6haPxgc3GhJxc= Received: by 10.100.139.20 with SMTP id m20mr1511362and.147.1233193767688; Wed, 28 Jan 2009 17:49:27 -0800 (PST) Received: from parag-desktop.local ([65.29.109.147]) by mx.google.com with ESMTPS id d38sm704051and.49.2009.01.28.17.49.25 (version=TLSv1/SSLv3 cipher=RC4-MD5); Wed, 28 Jan 2009 17:49:26 -0800 (PST) Date: Wed, 28 Jan 2009 20:49:18 -0500 (EST) From: Parag Warudkar X-X-Sender: parag@parag-desktop To: Linus Torvalds cc: Parag Warudkar , netdev@vger.kernel.org, Linux Kernel Mailing List , "David S. Miller" , Andrew Morton Subject: Re: 2.6.29-rc3: tg3 dead after resume In-Reply-To: Message-ID: References: User-Agent: Alpine 2.00 (DEB 1167 2008-08-23) MIME-Version: 1.0 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On Wed, 28 Jan 2009, Linus Torvalds wrote: > For example, if we get the "dev->current_state" cache wrong, then we may > not actually end up changing it when we should, because we think we > already match the target state. I don't _think_ that is it, but that's the > kind of thing that could happen. > > Can you do a > > lspci -vvxxx -s [tg3-device] > > before-and-after suspend? Is there some state that looks like it got > corrupted? Sure, diff -u below. There are differences but not sure if they are abnormal or expected. Also, BTW, reverting the only tg3 specific commit - commit 9e9fd12dc0679643c191fc9795a3021807e77de4 Author: Matt Carlson Date: Mon Jan 19 16:57:45 2009 -0800 tg3: Fix firmware loading did not help. parag@parag-desktop:~$ diff -u lspci-pre-suspend lspci-post-suspend <4us, L1 unlimited ExtTag+ AttnBtn- AttnInd- PwrInd- RBE+ FLReset- @@ -36,15 +36,15 @@ 20: 00 00 00 00 00 00 00 00 00 00 00 00 3c 10 07 13 30: 00 00 04 20 48 00 00 00 00 00 00 00 03 01 00 00 40: 00 00 00 00 00 00 00 00 01 50 03 c0 08 20 00 64 -50: 03 58 fc 00 00 00 00 78 09 e8 78 00 7d c9 08 78 -60: 00 00 00 00 00 00 00 00 98 02 02 a0 00 00 18 76 -70: f2 10 00 00 c0 00 00 00 2c 00 00 00 00 00 00 00 -80: 3c 10 07 13 00 00 00 00 34 00 13 04 82 70 08 fc -90: 19 be 00 01 00 00 00 b7 00 00 00 00 14 00 00 00 -a0: 00 00 00 00 4c 01 00 00 00 00 00 00 3e 01 00 00 -b0: 00 00 00 00 00 00 00 36 00 00 00 00 00 00 00 00 +50: 03 58 fc 00 00 00 00 78 09 e8 78 00 7e cb 08 a8 +60: 00 00 00 00 00 00 00 00 9a 02 02 a0 00 00 00 10 +70: 72 10 00 00 c0 00 00 00 2c 00 00 00 00 00 00 00 +80: 3c 10 07 13 00 00 00 00 00 00 00 00 fe 70 08 fc +90: 11 be 00 00 00 00 00 00 00 00 00 00 00 00 00 00 +a0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 +b0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 c0: 00 00 00 00 00 80 00 00 0e 00 00 00 00 00 00 00 d0: 10 00 01 00 a0 8f 00 00 00 50 10 00 11 64 03 00 e0: 40 00 11 10 00 00 00 00 05 d0 81 00 0c f0 e0 fe -f0: 00 00 00 00 c9 41 00 00 00 00 00 00 00 00 00 00 +f0: 00 00 00 00 d1 41 00 00 00 00 00 00 00 00 00 00 Parag --- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html --- lspci-pre-suspend 2009-01-28 20:35:37.070584068 -0500 +++ lspci-post-suspend 2009-01-28 20:36:56.922471408 -0500 @@ -12,7 +12,7 @@ Capabilities: [50] Vital Product Data Capabilities: [58] Vendor Specific Information Capabilities: [e8] Message Signalled Interrupts: Mask- 64bit+ Queue=0/0 Enable+ - Address: 00000000fee0f00c Data: 41c9 + Address: 00000000fee0f00c Data: 41d1 Capabilities: [d0] Express (v1) Endpoint, MSI 00 DevCap: MaxPayload 128 bytes, PhantFunc 0, Latency L0s