Re: [PATCH] pci/runtime-pm: respect devices autosuspend timeout on config access

2012-11-06 Thread David Airlie
- Original Message - > From: "Huang Ying" > To: "David Airlie" > Cc: linux-...@vger.kernel.org, linux-kernel@vger.kernel.org, "Bjorn Helgaas" > , "Rafael J. > Wysocki" > Sent: Wednesday, 7 November, 2012 4:26:25 PM > Sub

Re: [PATCH] pci/runtime-pm: respect devices autosuspend timeout on config access

2012-11-06 Thread Huang Ying
On Wed, 2012-11-07 at 01:15 -0500, David Airlie wrote: > > > > > > Cc: Huang Ying > > > Cc: Bjorn Helgaas > > > Cc: Rafael J. Wysocki > > > Signed-off-by: Dave Airlie > > > --- > > > drivers/pci/pci-sysfs.c | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git

Re: [PATCH] pci/runtime-pm: respect devices autosuspend timeout on config access

2012-11-06 Thread David Airlie
> > > > Cc: Huang Ying > > Cc: Bjorn Helgaas > > Cc: Rafael J. Wysocki > > Signed-off-by: Dave Airlie > > --- > > drivers/pci/pci-sysfs.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/pci/pci-sysfs.c b/drivers/pci/pci-sysfs.c > > index

Re: [PATCH] pci/runtime-pm: respect devices autosuspend timeout on config access

2012-11-06 Thread Huang Ying
On Wed, 2012-11-07 at 15:30 +1000, Dave Airlie wrote: > So I've been adding runtime pm to nouveau/radeon, and on X start it does a > lot of pci accesses. Now because the pm on these devices is equivalent > to D3cold, we have to resume them which involves a heavy latency due to > POSTing the cards.

[PATCH] pci/runtime-pm: respect devices autosuspend timeout on config access

2012-11-06 Thread Dave Airlie
So I've been adding runtime pm to nouveau/radeon, and on X start it does a lot of pci accesses. Now because the pm on these devices is equivalent to D3cold, we have to resume them which involves a heavy latency due to POSTing the cards. The driver configures the autosuspend timeout to 5s for this

[PATCH] pci/runtime-pm: respect devices autosuspend timeout on config access

2012-11-06 Thread Dave Airlie
So I've been adding runtime pm to nouveau/radeon, and on X start it does a lot of pci accesses. Now because the pm on these devices is equivalent to D3cold, we have to resume them which involves a heavy latency due to POSTing the cards. The driver configures the autosuspend timeout to 5s for this

Re: [PATCH] pci/runtime-pm: respect devices autosuspend timeout on config access

2012-11-06 Thread Huang Ying
On Wed, 2012-11-07 at 15:30 +1000, Dave Airlie wrote: So I've been adding runtime pm to nouveau/radeon, and on X start it does a lot of pci accesses. Now because the pm on these devices is equivalent to D3cold, we have to resume them which involves a heavy latency due to POSTing the cards. The

Re: [PATCH] pci/runtime-pm: respect devices autosuspend timeout on config access

2012-11-06 Thread David Airlie
Cc: Huang Ying ying.hu...@intel.com Cc: Bjorn Helgaas bhelg...@google.com Cc: Rafael J. Wysocki r...@sisk.pl Signed-off-by: Dave Airlie airl...@redhat.com --- drivers/pci/pci-sysfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pci/pci-sysfs.c

Re: [PATCH] pci/runtime-pm: respect devices autosuspend timeout on config access

2012-11-06 Thread Huang Ying
On Wed, 2012-11-07 at 01:15 -0500, David Airlie wrote: Cc: Huang Ying ying.hu...@intel.com Cc: Bjorn Helgaas bhelg...@google.com Cc: Rafael J. Wysocki r...@sisk.pl Signed-off-by: Dave Airlie airl...@redhat.com --- drivers/pci/pci-sysfs.c | 2 +- 1 file changed, 1

Re: [PATCH] pci/runtime-pm: respect devices autosuspend timeout on config access

2012-11-06 Thread David Airlie
: [PATCH] pci/runtime-pm: respect devices autosuspend timeout on config access On Wed, 2012-11-07 at 01:15 -0500, David Airlie wrote: Cc: Huang Ying ying.hu...@intel.com Cc: Bjorn Helgaas bhelg...@google.com Cc: Rafael J. Wysocki r...@sisk.pl Signed-off-by: Dave Airlie airl