From: Pavel Machek <pavel@ucw.cz>
To: Bjorn Helgaas <bhelgaas@google.com>
Cc: Greg KH <greg@kroah.com>,
kernel list <linux-kernel@vger.kernel.org>,
Joe Lawrence <joe.lawrence@stratus.com>,
Myron Stowe <myron.stowe@redhat.com>
Subject: Re: /sys/module/pcie_aspm/parameters/policy not writable?
Date: Wed, 10 Jul 2013 15:29:50 +0200 [thread overview]
Message-ID: <20130710132950.GA3684@amd.pavel.ucw.cz> (raw)
In-Reply-To: <CAErSpo57qjTkGbNK5qLzJcKuqBGK5=R=raEdyL6M1d5O=FkK_Q@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1879 bytes --]
Hi!
> >> But:
> >> 1) it should not list unavailable options
> >>
> >> 2) operation not permitted seems like wrong error code for
> >> operation not supported.
> >
> > So I forcibly enabled ASPM, and now ping latencies are in normal
> > range... no matter how I set
> > /sys/module/pcie_aspm/parameters/policy. Strange.
> >
> > Any ideas what correct solution is?
> > Pavel
> > Signed-off-by: Pavel Machek <pavel@ucw.cz>
> > (but don't apply)
...
> > diff --git a/drivers/pci/pci-acpi.c b/drivers/pci/pci-acpi.c
> > index e4b1fb2..9a1b63e 100644
> > --- a/drivers/pci/pci-acpi.c
> > +++ b/drivers/pci/pci-acpi.c
> > @@ -382,7 +382,7 @@ static int __init acpi_pci_init(void)
> >
> > if (acpi_gbl_FADT.boot_flags & ACPI_FADT_NO_ASPM) {
> > printk(KERN_INFO"ACPI FADT declares the system doesn't support PCIe ASPM, so disable it\n");
> > - pcie_no_aspm();
> > +// pcie_no_aspm();
> > }
> >
> > ret = register_acpi_bus_type(&acpi_pci_bus);
>
> Hi Pavel,
>
> Interesting. Can you collect dmesg and "lspci -vvv" output for both
> cases (high ping latency and normal ping latency)?
Will do. Results are in attachment (200KB...)
> Also, how much
> difference does this make in ping latency?
The ping latency goes from 100msec range to <2msec.
> If ASPM is enabled for a
> device, e.g., your NIC, the link may be put in a low power state when
> the device is idle. It takes time to exit that low power state, of
> course, but I would expect that time to be in the microsecond time and
> probably not observable via ping.
I'd hope so. 100msec ping makes ssh unpleasant to use.
Thanks,
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
[-- Attachment #2: delme.gz --]
[-- Type: application/octet-stream, Size: 39205 bytes --]
next prev parent reply other threads:[~2013-07-10 13:29 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-09 1:26 Pavel Machek
2013-07-09 4:13 ` Greg KH
2013-07-09 9:14 ` Pavel Machek
2013-07-09 9:49 ` Pavel Machek
2013-07-09 10:10 ` Pavel Machek
2013-07-09 16:25 ` Bjorn Helgaas
2013-07-10 13:29 ` Pavel Machek [this message]
2013-07-10 19:57 ` Bjorn Helgaas
2013-07-10 22:21 ` Wyborny, Carolyn
2013-07-10 22:57 ` Bjorn Helgaas
2013-07-11 17:45 ` Wyborny, Carolyn
2013-07-12 11:52 ` Pavel Machek
2013-07-12 11:03 ` Pavel Machek
2013-07-12 11:11 ` Pavel Machek
2013-07-19 17:46 ` Bjorn Helgaas
2013-07-24 15:19 ` Wyborny, Carolyn
2013-07-28 13:51 ` Pavel Machek
2013-08-01 14:55 ` Wyborny, Carolyn
2013-08-01 15:55 ` Wyborny, Carolyn
2013-08-02 0:39 ` Pavel Machek
2013-08-02 14:58 ` Wyborny, Carolyn
2013-07-31 23:53 ` Bjorn Helgaas
2013-08-01 14:57 ` Wyborny, Carolyn
2013-08-01 19:33 ` Bjorn Helgaas
2013-08-01 20:00 ` Pavel Machek
2013-08-01 20:27 ` Bjorn Helgaas
2013-08-02 1:02 ` Pavel Machek
2013-08-02 2:13 ` Pavel Machek
2013-08-02 13:48 ` Bjorn Helgaas
2013-08-01 6:19 ` Jeff Kirsher
2013-07-09 16:37 ` Greg KH
2013-07-09 17:15 ` Pavel Machek
2013-07-10 4:24 ` Robert Hancock
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=20130710132950.GA3684@amd.pavel.ucw.cz \
--to=pavel@ucw.cz \
--cc=bhelgaas@google.com \
--cc=greg@kroah.com \
--cc=joe.lawrence@stratus.com \
--cc=linux-kernel@vger.kernel.org \
--cc=myron.stowe@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
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
all inboxes | Powered by JetHome®