[PATCH v3] PCI: create revision file in sysfs

2016-11-18 Thread Bjorn Helgaas
On Fri, Nov 11, 2016 at 02:37:23PM +, Emil Velikov wrote: > From: Emil Velikov > > Currently the revision isn't available via sysfs/libudev thus if one > wants to know the value they need to read through the config file. > > This in itself wakes/powers up the device, causing unwanted delay >

[PATCH v3] PCI: create revision file in sysfs

2016-11-18 Thread Daniel Vetter
On Fri, Nov 18, 2016 at 10:42:07AM +0100, Daniel Vetter wrote: > On Thu, Nov 17, 2016 at 08:40:10PM -0600, Bjorn Helgaas wrote: > > On Fri, Nov 18, 2016 at 10:42:20AM +0900, Michel Dänzer wrote: > > > On 18/11/16 08:48 AM, Bjorn Helgaas wrote: > > > > > > > > Popping the stack all the way back to

[PATCH v3] PCI: create revision file in sysfs

2016-11-18 Thread Michel Dänzer
On 18/11/16 08:48 AM, Bjorn Helgaas wrote: > > Popping the stack all the way back to Emil's Nov 8 message: > > When using the Mesa drivers alongside firefox [1] (since Mesa 13.0), > glxinfo (Mesa 10.0) and others, all the GPUs* will be awaken, > causing unwanted delays and increased power u

[PATCH v3] PCI: create revision file in sysfs

2016-11-18 Thread Daniel Vetter
On Thu, Nov 17, 2016 at 08:40:10PM -0600, Bjorn Helgaas wrote: > On Fri, Nov 18, 2016 at 10:42:20AM +0900, Michel Dänzer wrote: > > On 18/11/16 08:48 AM, Bjorn Helgaas wrote: > > > > > > Popping the stack all the way back to Emil's Nov 8 message: > > > > > > When using the Mesa drivers alongsi

[PATCH v3] PCI: create revision file in sysfs

2016-11-18 Thread Alex Deucher
On Fri, Nov 18, 2016 at 4:48 AM, Daniel Vetter wrote: > On Fri, Nov 18, 2016 at 10:42:07AM +0100, Daniel Vetter wrote: >> On Thu, Nov 17, 2016 at 08:40:10PM -0600, Bjorn Helgaas wrote: >> > On Fri, Nov 18, 2016 at 10:42:20AM +0900, Michel Dänzer wrote: >> > > On 18/11/16 08:48 AM, Bjorn Helgaas w

[PATCH v3] PCI: create revision file in sysfs

2016-11-18 Thread Bjorn Helgaas
On Fri, Nov 18, 2016 at 10:48:46AM +0100, Daniel Vetter wrote: > On Fri, Nov 18, 2016 at 10:42:07AM +0100, Daniel Vetter wrote: > > On Thu, Nov 17, 2016 at 08:40:10PM -0600, Bjorn Helgaas wrote: > > > On Fri, Nov 18, 2016 at 10:42:20AM +0900, Michel Dänzer wrote: > > > > On 18/11/16 08:48 AM, Bjor

[PATCH v3] PCI: create revision file in sysfs

2016-11-17 Thread Bjorn Helgaas
On Fri, Nov 18, 2016 at 10:42:20AM +0900, Michel Dänzer wrote: > On 18/11/16 08:48 AM, Bjorn Helgaas wrote: > > > > Popping the stack all the way back to Emil's Nov 8 message: > > > > When using the Mesa drivers alongside firefox [1] (since Mesa 13.0), > > glxinfo (Mesa 10.0) and others, all

[PATCH v3] PCI: create revision file in sysfs

2016-11-17 Thread Bjorn Helgaas
On Thu, Nov 17, 2016 at 03:44:02PM +0100, Lukas Wunner wrote: > On Wed, Nov 16, 2016 at 02:58:11PM -0600, Bjorn Helgaas wrote: > > On Mon, Nov 14, 2016 at 07:40:03PM +0100, Daniel Vetter wrote: > > > On Fri, Nov 11, 2016 at 02:37:23PM +, Emil Velikov wrote: > > > > From: Emil Velikov > > > >

[PATCH v3] PCI: create revision file in sysfs

2016-11-17 Thread Lukas Wunner
On Wed, Nov 16, 2016 at 02:58:11PM -0600, Bjorn Helgaas wrote: > On Mon, Nov 14, 2016 at 07:40:03PM +0100, Daniel Vetter wrote: > > On Fri, Nov 11, 2016 at 02:37:23PM +, Emil Velikov wrote: > > > From: Emil Velikov > > > > > > Currently the revision isn't available via sysfs/libudev thus if o

[PATCH v3] PCI: create revision file in sysfs

2016-11-17 Thread Alex Deucher
On Thu, Nov 17, 2016 at 9:35 AM, Bjorn Helgaas wrote: > On Thu, Nov 17, 2016 at 08:28:50AM -0500, Alex Deucher wrote: >> On Wed, Nov 16, 2016 at 3:58 PM, Bjorn Helgaas wrote: >> > On Mon, Nov 14, 2016 at 07:40:03PM +0100, Daniel Vetter wrote: >> >> Given that waking a gpu can take somewhere betwe

[PATCH v3] PCI: create revision file in sysfs

2016-11-17 Thread Bjorn Helgaas
On Thu, Nov 17, 2016 at 08:28:50AM -0500, Alex Deucher wrote: > On Wed, Nov 16, 2016 at 3:58 PM, Bjorn Helgaas wrote: > > On Mon, Nov 14, 2016 at 07:40:03PM +0100, Daniel Vetter wrote: > >> Given that waking a gpu can take somewhere between ages and forever, and > >> that we read the pci revisions

[PATCH v3] PCI: create revision file in sysfs

2016-11-17 Thread Alex Deucher
On Wed, Nov 16, 2016 at 3:58 PM, Bjorn Helgaas wrote: > [+cc Sinan, Lukas] > > Hi Daniel, > > On Mon, Nov 14, 2016 at 07:40:03PM +0100, Daniel Vetter wrote: >> On Fri, Nov 11, 2016 at 02:37:23PM +, Emil Velikov wrote: >> > From: Emil Velikov >> > >> > Currently the revision isn't available vi

[PATCH v3] PCI: create revision file in sysfs

2016-11-16 Thread Sinan Kaya
On 11/16/2016 3:58 PM, Bjorn Helgaas wrote: > [+cc Sinan, Lukas] > > Hi Daniel, > > On Mon, Nov 14, 2016 at 07:40:03PM +0100, Daniel Vetter wrote: >> On Fri, Nov 11, 2016 at 02:37:23PM +, Emil Velikov wrote: >>> From: Emil Velikov >>> >>> Currently the revision isn't available via sysfs/libu

[PATCH v3] PCI: create revision file in sysfs

2016-11-16 Thread Bjorn Helgaas
[+cc Sinan, Lukas] Hi Daniel, On Mon, Nov 14, 2016 at 07:40:03PM +0100, Daniel Vetter wrote: > On Fri, Nov 11, 2016 at 02:37:23PM +, Emil Velikov wrote: > > From: Emil Velikov > > > > Currently the revision isn't available via sysfs/libudev thus if one > > wants to know the value they need

[PATCH v3] PCI: create revision file in sysfs

2016-11-14 Thread Daniel Vetter
On Fri, Nov 11, 2016 at 02:37:23PM +, Emil Velikov wrote: > From: Emil Velikov > > Currently the revision isn't available via sysfs/libudev thus if one > wants to know the value they need to read through the config file. > > This in itself wakes/powers up the device, causing unwanted delay >

[PATCH v3] PCI: create revision file in sysfs

2016-11-11 Thread Emil Velikov
From: Emil Velikov Currently the revision isn't available via sysfs/libudev thus if one wants to know the value they need to read through the config file. This in itself wakes/powers up the device, causing unwanted delay since it can be quite costly. There are at least two userspace components