From: Anton Blanchard Date: Tue, 17 Apr 2012 19:16:48 +0000 (+0000) Subject: powerpc: Use WARN instead of dump_stack when printing EEH error backtrace X-Git-Tag: v3.5-rc1~132^2~30 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=14fb1fa6e266fb9c4622b210d2636a2004a47e2b;p=profile%2Fivi%2Fkernel-x86-ivi.git powerpc: Use WARN instead of dump_stack when printing EEH error backtrace When we get an EEH error we just print a backtrace with dump_stack which is rather cryptic. We really should print something before spewing out the backtrace. Also switch from dump_stack to WARN so we get more information about the fail - what modules were loaded, what process was running etc. This was useful information when debugging a recent EEH subsystem bug. The standard WARN output should also get picked up by monitoring tools like kerneloops. The register dump is of questionable value here but I figured it was better to use something standard and not roll my own. Signed-off-by: Anton Blanchard Signed-off-by: Benjamin Herrenschmidt --- diff --git a/arch/powerpc/platforms/pseries/eeh.c b/arch/powerpc/platforms/pseries/eeh.c index a75e37dc41a..ecd394cf34e 100644 --- a/arch/powerpc/platforms/pseries/eeh.c +++ b/arch/powerpc/platforms/pseries/eeh.c @@ -489,7 +489,7 @@ int eeh_dn_check_failure(struct device_node *dn, struct pci_dev *dev) * a stack trace will help the device-driver authors figure * out what happened. So print that out. */ - dump_stack(); + WARN(1, "EEH: failure detected\n"); return 1; dn_unlock: