Re: [PATCH] powernv/pci: Add PHB register dump debugfs handle

2016-07-26 Thread Michael Ellerman
Russell Currey writes: > On Tue, 2016-07-26 at 11:45 +1000, Michael Ellerman wrote: >> Quoting Russell Currey (2016-07-22 15:23:36) >> >> DEFINE_SIMPLE_ATTRIBUTE(fops_foo, NULL, foo_set, "%llu\n"); >> >> That requires that you write "1" to the file to trigger the reg dump. >

Re: [PATCH] powernv/pci: Add PHB register dump debugfs handle

2016-07-25 Thread Russell Currey
On Tue, 2016-07-26 at 11:45 +1000, Michael Ellerman wrote: > Quoting Russell Currey (2016-07-22 15:23:36) > > > > On EEH events the kernel will print a dump of relevant registers. > > If EEH is unavailable (i.e. CONFIG_EEH is disabled, a new platform > > doesn't have EEH support, etc) this

Re: [PATCH] powernv/pci: Add PHB register dump debugfs handle

2016-07-25 Thread Michael Ellerman
Quoting Russell Currey (2016-07-22 15:23:36) > On EEH events the kernel will print a dump of relevant registers. > If EEH is unavailable (i.e. CONFIG_EEH is disabled, a new platform > doesn't have EEH support, etc) this information isn't readily available. > > Add a new debugfs handler to trigger

Re: [PATCH] powernv/pci: Add PHB register dump debugfs handle

2016-07-25 Thread Michael Ellerman
Tyrel Datwyler writes: > On 07/21/2016 11:36 PM, Gavin Shan wrote: >> On Fri, Jul 22, 2016 at 03:23:36PM +1000, Russell Currey wrote: >>> diff --git a/arch/powerpc/platforms/powernv/pci-ioda.c >>> b/arch/powerpc/platforms/powernv/pci-ioda.c >>> index 891fc4a..ada2f3c

Re: [PATCH] powernv/pci: Add PHB register dump debugfs handle

2016-07-25 Thread Russell Currey
On Fri, 2016-07-22 at 16:36 +1000, Gavin Shan wrote: > On Fri, Jul 22, 2016 at 03:23:36PM +1000, Russell Currey wrote: > > > > On EEH events the kernel will print a dump of relevant registers. > > If EEH is unavailable (i.e. CONFIG_EEH is disabled, a new platform > > doesn't have EEH support,

Re: [PATCH] powernv/pci: Add PHB register dump debugfs handle

2016-07-25 Thread Gavin Shan
On Mon, Jul 25, 2016 at 10:53:49AM -0700, Tyrel Datwyler wrote: >On 07/21/2016 11:36 PM, Gavin Shan wrote: >> On Fri, Jul 22, 2016 at 03:23:36PM +1000, Russell Currey wrote: >>> On EEH events the kernel will print a dump of relevant registers. >>> If EEH is unavailable (i.e. CONFIG_EEH is

Re: [PATCH] powernv/pci: Add PHB register dump debugfs handle

2016-07-25 Thread Tyrel Datwyler
On 07/21/2016 11:36 PM, Gavin Shan wrote: > On Fri, Jul 22, 2016 at 03:23:36PM +1000, Russell Currey wrote: >> On EEH events the kernel will print a dump of relevant registers. >> If EEH is unavailable (i.e. CONFIG_EEH is disabled, a new platform >> doesn't have EEH support, etc) this information

Re: [PATCH] powernv/pci: Add PHB register dump debugfs handle

2016-07-22 Thread Gavin Shan
On Fri, Jul 22, 2016 at 03:23:36PM +1000, Russell Currey wrote: >On EEH events the kernel will print a dump of relevant registers. >If EEH is unavailable (i.e. CONFIG_EEH is disabled, a new platform >doesn't have EEH support, etc) this information isn't readily available. > >Add a new debugfs

[PATCH] powernv/pci: Add PHB register dump debugfs handle

2016-07-21 Thread Russell Currey
On EEH events the kernel will print a dump of relevant registers. If EEH is unavailable (i.e. CONFIG_EEH is disabled, a new platform doesn't have EEH support, etc) this information isn't readily available. Add a new debugfs handler to trigger a PHB register dump, so that this information can be