Commit 69376502 authored by Linas Vepstas's avatar Linas Vepstas Committed by Paul Mackerras

[PATCH] ppc64: misc minor cleanup

02-eeh-minor-cleanup.patch

This patch performs some minor cleanup of the eeh.c file, including:
-- trim some trailing whitespace
-- remove extraneous #includes
-- use the macro PCI_DN uniformly, instead of the void pointer chase.
-- typos in comments
-- improved debug printk's
Signed-off-by: default avatarLinas Vepstas <linas@linas.org>
Signed-off-by: default avatarPaul Mackerras <paulus@samba.org>
parent ae65a391
...@@ -17,16 +17,15 @@ ...@@ -17,16 +17,15 @@
* Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
*/ */
#include <linux/bootmem.h>
#include <linux/init.h> #include <linux/init.h>
#include <linux/list.h> #include <linux/list.h>
#include <linux/mm.h>
#include <linux/notifier.h> #include <linux/notifier.h>
#include <linux/pci.h> #include <linux/pci.h>
#include <linux/proc_fs.h> #include <linux/proc_fs.h>
#include <linux/rbtree.h> #include <linux/rbtree.h>
#include <linux/seq_file.h> #include <linux/seq_file.h>
#include <linux/spinlock.h> #include <linux/spinlock.h>
#include <asm/atomic.h>
#include <asm/eeh.h> #include <asm/eeh.h>
#include <asm/io.h> #include <asm/io.h>
#include <asm/machdep.h> #include <asm/machdep.h>
...@@ -49,8 +48,8 @@ ...@@ -49,8 +48,8 @@
* were "empty": all reads return 0xff's and all writes are silently * were "empty": all reads return 0xff's and all writes are silently
* ignored. EEH slot isolation events can be triggered by parity * ignored. EEH slot isolation events can be triggered by parity
* errors on the address or data busses (e.g. during posted writes), * errors on the address or data busses (e.g. during posted writes),
* which in turn might be caused by dust, vibration, humidity, * which in turn might be caused by low voltage on the bus, dust,
* radioactivity or plain-old failed hardware. * vibration, humidity, radioactivity or plain-old failed hardware.
* *
* Note, however, that one of the leading causes of EEH slot * Note, however, that one of the leading causes of EEH slot
* freeze events are buggy device drivers, buggy device microcode, * freeze events are buggy device drivers, buggy device microcode,
...@@ -256,18 +255,17 @@ static void __pci_addr_cache_insert_device(struct pci_dev *dev) ...@@ -256,18 +255,17 @@ static void __pci_addr_cache_insert_device(struct pci_dev *dev)
dn = pci_device_to_OF_node(dev); dn = pci_device_to_OF_node(dev);
if (!dn) { if (!dn) {
printk(KERN_WARNING "PCI: no pci dn found for dev=%s\n", printk(KERN_WARNING "PCI: no pci dn found for dev=%s\n", pci_name(dev));
pci_name(dev));
return; return;
} }
/* Skip any devices for which EEH is not enabled. */ /* Skip any devices for which EEH is not enabled. */
pdn = dn->data; pdn = PCI_DN(dn);
if (!(pdn->eeh_mode & EEH_MODE_SUPPORTED) || if (!(pdn->eeh_mode & EEH_MODE_SUPPORTED) ||
pdn->eeh_mode & EEH_MODE_NOCHECK) { pdn->eeh_mode & EEH_MODE_NOCHECK) {
#ifdef DEBUG #ifdef DEBUG
printk(KERN_INFO "PCI: skip building address cache for=%s\n", printk(KERN_INFO "PCI: skip building address cache for=%s - %s\n",
pci_name(dev)); pci_name(dev), pdn->node->full_name);
#endif #endif
return; return;
} }
...@@ -410,16 +408,16 @@ int eeh_unregister_notifier(struct notifier_block *nb) ...@@ -410,16 +408,16 @@ int eeh_unregister_notifier(struct notifier_block *nb)
* @dn: device node to read * @dn: device node to read
* @rets: array to return results in * @rets: array to return results in
*/ */
static int read_slot_reset_state(struct device_node *dn, int rets[]) static int read_slot_reset_state(struct pci_dn *pdn, int rets[])
{ {
int token, outputs; int token, outputs;
struct pci_dn *pdn = dn->data;
if (ibm_read_slot_reset_state2 != RTAS_UNKNOWN_SERVICE) { if (ibm_read_slot_reset_state2 != RTAS_UNKNOWN_SERVICE) {
token = ibm_read_slot_reset_state2; token = ibm_read_slot_reset_state2;
outputs = 4; outputs = 4;
} else { } else {
token = ibm_read_slot_reset_state; token = ibm_read_slot_reset_state;
rets[2] = 0; /* fake PE Unavailable info */
outputs = 3; outputs = 3;
} }
...@@ -496,7 +494,7 @@ static void eeh_event_handler(void *dummy) ...@@ -496,7 +494,7 @@ static void eeh_event_handler(void *dummy)
/** /**
* eeh_token_to_phys - convert EEH address token to phys address * eeh_token_to_phys - convert EEH address token to phys address
* @token i/o token, should be address in the form 0xE.... * @token i/o token, should be address in the form 0xA....
*/ */
static inline unsigned long eeh_token_to_phys(unsigned long token) static inline unsigned long eeh_token_to_phys(unsigned long token)
{ {
...@@ -522,7 +520,7 @@ static inline unsigned long eeh_token_to_phys(unsigned long token) ...@@ -522,7 +520,7 @@ static inline unsigned long eeh_token_to_phys(unsigned long token)
* will query firmware for the EEH status. * will query firmware for the EEH status.
* *
* Returns 0 if there has not been an EEH error; otherwise returns * Returns 0 if there has not been an EEH error; otherwise returns
* a non-zero value and queues up a solt isolation event notification. * a non-zero value and queues up a slot isolation event notification.
* *
* It is safe to call this routine in an interrupt context. * It is safe to call this routine in an interrupt context.
*/ */
...@@ -542,7 +540,7 @@ int eeh_dn_check_failure(struct device_node *dn, struct pci_dev *dev) ...@@ -542,7 +540,7 @@ int eeh_dn_check_failure(struct device_node *dn, struct pci_dev *dev)
if (!dn) if (!dn)
return 0; return 0;
pdn = dn->data; pdn = PCI_DN(dn);
/* Access to IO BARs might get this far and still not want checking. */ /* Access to IO BARs might get this far and still not want checking. */
if (!pdn->eeh_capable || !(pdn->eeh_mode & EEH_MODE_SUPPORTED) || if (!pdn->eeh_capable || !(pdn->eeh_mode & EEH_MODE_SUPPORTED) ||
...@@ -562,7 +560,7 @@ int eeh_dn_check_failure(struct device_node *dn, struct pci_dev *dev) ...@@ -562,7 +560,7 @@ int eeh_dn_check_failure(struct device_node *dn, struct pci_dev *dev)
atomic_inc(&eeh_fail_count); atomic_inc(&eeh_fail_count);
if (atomic_read(&eeh_fail_count) >= EEH_MAX_FAILS) { if (atomic_read(&eeh_fail_count) >= EEH_MAX_FAILS) {
/* re-read the slot reset state */ /* re-read the slot reset state */
if (read_slot_reset_state(dn, rets) != 0) if (read_slot_reset_state(pdn, rets) != 0)
rets[0] = -1; /* reset state unknown */ rets[0] = -1; /* reset state unknown */
eeh_panic(dev, rets[0]); eeh_panic(dev, rets[0]);
} }
...@@ -576,7 +574,7 @@ int eeh_dn_check_failure(struct device_node *dn, struct pci_dev *dev) ...@@ -576,7 +574,7 @@ int eeh_dn_check_failure(struct device_node *dn, struct pci_dev *dev)
* function zero of a multi-function device. * function zero of a multi-function device.
* In any case they must share a common PHB. * In any case they must share a common PHB.
*/ */
ret = read_slot_reset_state(dn, rets); ret = read_slot_reset_state(pdn, rets);
if (!(ret == 0 && rets[1] == 1 && (rets[0] == 2 || rets[0] == 4))) { if (!(ret == 0 && rets[1] == 1 && (rets[0] == 2 || rets[0] == 4))) {
__get_cpu_var(false_positives)++; __get_cpu_var(false_positives)++;
return 0; return 0;
...@@ -635,7 +633,6 @@ EXPORT_SYMBOL(eeh_dn_check_failure); ...@@ -635,7 +633,6 @@ EXPORT_SYMBOL(eeh_dn_check_failure);
* @token i/o token, should be address in the form 0xA.... * @token i/o token, should be address in the form 0xA....
* @val value, should be all 1's (XXX why do we need this arg??) * @val value, should be all 1's (XXX why do we need this arg??)
* *
* Check for an eeh failure at the given token address.
* Check for an EEH failure at the given token address. Call this * Check for an EEH failure at the given token address. Call this
* routine if the result of a read was all 0xff's and you want to * routine if the result of a read was all 0xff's and you want to
* find out if this is due to an EEH slot freeze event. This routine * find out if this is due to an EEH slot freeze event. This routine
...@@ -680,7 +677,7 @@ static void *early_enable_eeh(struct device_node *dn, void *data) ...@@ -680,7 +677,7 @@ static void *early_enable_eeh(struct device_node *dn, void *data)
u32 *device_id = (u32 *)get_property(dn, "device-id", NULL); u32 *device_id = (u32 *)get_property(dn, "device-id", NULL);
u32 *regs; u32 *regs;
int enable; int enable;
struct pci_dn *pdn = dn->data; struct pci_dn *pdn = PCI_DN(dn);
pdn->eeh_mode = 0; pdn->eeh_mode = 0;
...@@ -732,7 +729,7 @@ static void *early_enable_eeh(struct device_node *dn, void *data) ...@@ -732,7 +729,7 @@ static void *early_enable_eeh(struct device_node *dn, void *data)
/* This device doesn't support EEH, but it may have an /* This device doesn't support EEH, but it may have an
* EEH parent, in which case we mark it as supported. */ * EEH parent, in which case we mark it as supported. */
if (dn->parent && dn->parent->data if (dn->parent && PCI_DN(dn->parent)
&& (PCI_DN(dn->parent)->eeh_mode & EEH_MODE_SUPPORTED)) { && (PCI_DN(dn->parent)->eeh_mode & EEH_MODE_SUPPORTED)) {
/* Parent supports EEH. */ /* Parent supports EEH. */
pdn->eeh_mode |= EEH_MODE_SUPPORTED; pdn->eeh_mode |= EEH_MODE_SUPPORTED;
...@@ -793,13 +790,11 @@ void __init eeh_init(void) ...@@ -793,13 +790,11 @@ void __init eeh_init(void)
for (phb = of_find_node_by_name(NULL, "pci"); phb; for (phb = of_find_node_by_name(NULL, "pci"); phb;
phb = of_find_node_by_name(phb, "pci")) { phb = of_find_node_by_name(phb, "pci")) {
unsigned long buid; unsigned long buid;
struct pci_dn *pci;
buid = get_phb_buid(phb); buid = get_phb_buid(phb);
if (buid == 0 || phb->data == NULL) if (buid == 0 || PCI_DN(phb) == NULL)
continue; continue;
pci = phb->data;
info.buid_lo = BUID_LO(buid); info.buid_lo = BUID_LO(buid);
info.buid_hi = BUID_HI(buid); info.buid_hi = BUID_HI(buid);
traverse_pci_devices(phb, early_enable_eeh, &info); traverse_pci_devices(phb, early_enable_eeh, &info);
...@@ -828,11 +823,13 @@ void eeh_add_device_early(struct device_node *dn) ...@@ -828,11 +823,13 @@ void eeh_add_device_early(struct device_node *dn)
struct pci_controller *phb; struct pci_controller *phb;
struct eeh_early_enable_info info; struct eeh_early_enable_info info;
if (!dn || !dn->data) if (!dn || !PCI_DN(dn))
return; return;
phb = PCI_DN(dn)->phb; phb = PCI_DN(dn)->phb;
if (NULL == phb || 0 == phb->buid) { if (NULL == phb || 0 == phb->buid) {
printk(KERN_WARNING "EEH: Expected buid but found none\n"); printk(KERN_WARNING "EEH: Expected buid but found none for %s\n",
dn->full_name);
dump_stack();
return; return;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment