On Sat, Jun 01, 2013 at 02:34:46PM +1000, Benjamin Herrenschmidt wrote:
>On Thu, 2013-05-30 at 16:24 +0800, Gavin Shan wrote:
>> The patch intends to add debugfs entry powerpc/EEH/PHBx so that
>> the administrator can inject EEH errors to specified PCI host
>> bridge for testing purpose.
>
>Use a better naming for the debugfs files. Something like
>eeh_err_inject/pciNNNN, to be consistent with the general naming
>of PHBs in the system.
>
>However, maybe it would be better to instead having something along
>the lines of a directory per PHB with a file in it for error injection ?
>
>That way we can stick more things in there that can become handy for
>debugging / diagnostics, such as register dumps etc...
>
>
Yeah, I think it would be more reasonable to have one directory per PHB.
I'll make it in next version:
powerpc/pciNNNN/err_inject
Thanks,
Gavin
>> Signed-off-by: Gavin Shan <sha...@linux.vnet.ibm.com>
>> ---
>> arch/powerpc/platforms/powernv/eeh-ioda.c | 36
>> ++++++++++++++++++++++++++++-
>> 1 files changed, 35 insertions(+), 1 deletions(-)
>>
>> diff --git a/arch/powerpc/platforms/powernv/eeh-ioda.c
>> b/arch/powerpc/platforms/powernv/eeh-ioda.c
>> index ec5c524..4cc9db7 100644
>> --- a/arch/powerpc/platforms/powernv/eeh-ioda.c
>> +++ b/arch/powerpc/platforms/powernv/eeh-ioda.c
>> @@ -22,6 +22,7 @@
>>
>> #include <linux/bootmem.h>
>> #include <linux/delay.h>
>> +#include <linux/debugfs.h>
>> #include <linux/init.h>
>> #include <linux/io.h>
>> #include <linux/irq.h>
>> @@ -43,6 +44,29 @@
>> #include "powernv.h"
>> #include "pci.h"
>>
>> +static struct dentry *ioda_eeh_dbgfs = NULL;
>> +
>> +static int ioda_eeh_dbgfs_set(void *data, u64 val)
>> +{
>> + struct pci_controller *hose = data;
>> + struct pnv_phb *phb = hose->private_data;
>> +
>> + out_be64(phb->regs + 0xD10, val);
>> + return 0;
>> +}
>> +
>> +static int ioda_eeh_dbgfs_get(void *data, u64 *val)
>> +{
>> + struct pci_controller *hose = data;
>> + struct pnv_phb *phb = hose->private_data;
>> +
>> + *val = in_be64(phb->regs + 0xD10);
>> + return 0;
>> +}
>> +
>> +DEFINE_SIMPLE_ATTRIBUTE(ioda_eeh_dbgfs_ops, ioda_eeh_dbgfs_get,
>> + ioda_eeh_dbgfs_set, "0x%llx\n");
>> +
>> /**
>> * ioda_eeh_post_init - Chip dependent post initialization
>> * @hose: PCI controller
>> @@ -54,10 +78,20 @@
>> static int ioda_eeh_post_init(struct pci_controller *hose)
>> {
>> struct pnv_phb *phb = hose->private_data;
>> + char name[16];
>> +
>> + /* Create EEH debugfs root if possible */
>> + if (!ioda_eeh_dbgfs)
>> + ioda_eeh_dbgfs = debugfs_create_dir("EEH",
>> powerpc_debugfs_root);
>>
>> /* FIXME: Enable it for PHB3 later */
>> - if (phb->type == PNV_PHB_IODA1)
>> + if (phb->type == PNV_PHB_IODA1) {
>> + sprintf(name, "PHB%d", hose->global_number);
>> + debugfs_create_file(name, 0600, ioda_eeh_dbgfs,
>> + hose, &ioda_eeh_dbgfs_ops);
>> +
>> phb->eeh_enabled = 1;
>> + }
>>
>> return 0;
>> }
>
>
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
