From: "Matt Carlson" <mcarlson@broadcom•com>
To: "Rafael J. Wysocki" <rjw@sisk•pl>
Cc: "netdev@vger•kernel.org" <netdev@vger•kernel.org>,
"David Miller" <davem@davemloft•net>,
"Matthew Carlson" <mcarlson@broadcom•com>,
"Michael Chan" <mchan@broadcom•com>,
"Linux PM mailing list" <linux-pm@lists•linux-foundation.org>,
"Thomas Fjellstrom" <thomas@fjellstrom•ca>,
"Jay Cliburn" <jcliburn@gmail•com>,
"Chris Snook" <chris.snook@gmail•com>,
"Jie Yang" <jie.yang@atheros•com>
Subject: Re: [PATCH 1/3] tg3: Avoid setting power.can_wakeup for devices that cannot wake up
Date: Thu, 10 Feb 2011 12:48:44 -0800 [thread overview]
Message-ID: <20110210204844.GA4158@mcarlson.broadcom.com> (raw)
In-Reply-To: <201102101753.09504.rjw@sisk.pl>
On Thu, Feb 10, 2011 at 08:53:09AM -0800, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki <rjw@sisk•pl>
>
> The tg3 driver uses device_init_wakeup() in such a way that the
> device's power.can_wakeup flag may be set even though the PCI
> subsystem cleared it before, in which case the device cannot wake
> up the system from sleep states. Modify the driver to only change
> the power.can_wakeup flag if the device is not capable of generating
> wakeup signals.
>
> Signed-off-by: Rafael J. Wysocki <rjw@sisk•pl>
> ---
> drivers/net/tg3.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> Index: linux-2.6/drivers/net/tg3.c
> ===================================================================
> --- linux-2.6.orig/drivers/net/tg3.c
> +++ linux-2.6/drivers/net/tg3.c
> @@ -12403,9 +12403,11 @@ static void __devinit tg3_get_eeprom_hw_
> tp->tg3_flags3 |= TG3_FLG3_RGMII_EXT_IBND_TX_EN;
> }
> done:
> - device_init_wakeup(&tp->pdev->dev, tp->tg3_flags & TG3_FLAG_WOL_CAP);
> - device_set_wakeup_enable(&tp->pdev->dev,
> + if (tp->tg3_flags & TG3_FLAG_WOL_CAP)
> + device_set_wakeup_enable(&tp->pdev->dev,
> tp->tg3_flags & TG3_FLAG_WOL_ENABLE);
> + else
> + device_set_wakeup_capable(&tp->pdev->dev, false);
I did this because I couldn't see where 'can_wakeup' gets set. I don't
see a call to device_init_wakeup() that would be relevant to tg3
devices. I do see a couple calls to device_set_wakeup_capable() in
acpi/glue.c and acpi/scan.c. Is that the place?
> }
>
> static int __devinit tg3_issue_otp_command(struct tg3 *tp, u32 cmd)
This is something I was always curious about too. The TG3_FLAG_WOL_CAP
tracks whether or not the device can handle WOL. Is it safe to do away
with this flag and lean on the 'can_wakeup' flag instead? My concern is
that some other part of the system might enable that flag after
tg3_get_invariants() has run. If that happens and the device isn't
really WOL capable bad things can occur.
next prev parent reply other threads:[~2011-02-10 20:49 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-10 16:51 [PATCH 0/3] Net driver fixes related to power management Rafael J. Wysocki
2011-02-10 16:53 ` [PATCH 1/3] tg3: Avoid setting power.can_wakeup for devices that cannot wake up Rafael J. Wysocki
2011-02-10 20:48 ` Matt Carlson [this message]
2011-02-10 21:08 ` Rafael J. Wysocki
2011-02-11 0:00 ` Matt Carlson
2011-02-10 16:54 ` [PATCH 2/3] atl1c: Do not call device_init_wakeup() in atl1c_probe() Rafael J. Wysocki
2011-02-10 16:55 ` [PATCH 3/3] atl1: Do not use legacy PCI power management Rafael J. Wysocki
2011-02-11 19:39 ` [PATCH 0/3] Net driver fixes related to " David Miller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110210204844.GA4158@mcarlson.broadcom.com \
--to=mcarlson@broadcom$(echo .)com \
--cc=chris.snook@gmail$(echo .)com \
--cc=davem@davemloft$(echo .)net \
--cc=jcliburn@gmail$(echo .)com \
--cc=jie.yang@atheros$(echo .)com \
--cc=linux-pm@lists$(echo .)linux-foundation.org \
--cc=mchan@broadcom$(echo .)com \
--cc=netdev@vger$(echo .)kernel.org \
--cc=rjw@sisk$(echo .)pl \
--cc=thomas@fjellstrom$(echo .)ca \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox