All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ssb: Make bus registration failure not be silent
@ 2010-04-23 18:17 Larry Finger
  0 siblings, 0 replies; only message in thread
From: Larry Finger @ 2010-04-23 18:17 UTC (permalink / raw)
  To: John W Linville, Michael Buesch; +Cc: b43-dev, linux-wireless

In kernel Bugzilla #15825, a case where routine ssb_bus_pcibus_register()
fails silently is reported. The cause for the failure is still being
investigated; however, the diagnosis would be easier if the failure
had been reported.

Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
---

John,

This patch has low priority - 2.6.35. Whatever fixes are needed after the
source of the problem is discovered will likely be 2.6.34 and stable material.

Larry
---

Michael,

Please be aware that the failure in the SSB registration leads to a system
freeze when ssb is unloaded. I did a quick check but could not locate a
routine that fails to tear down all structures on an error return, but that
search will be easier once we know which routine is failing. The OP for the
Bugzilla entry has been given a test patch to further identify the problem.

Larry
---

Index: wireless-testing/drivers/ssb/main.c
===================================================================
--- wireless-testing.orig/drivers/ssb/main.c
+++ wireless-testing/drivers/ssb/main.c
@@ -834,6 +834,9 @@ int ssb_bus_pcibus_register(struct ssb_b
 	if (!err) {
 		ssb_printk(KERN_INFO PFX "Sonics Silicon Backplane found on "
 			   "PCI device %s\n", dev_name(&host_pci->dev));
+	} else {
+		ssb_printk(KERN_ERR PFX "Failed to register PCI version"
+			   " of SSB with error %d\n", err);
 	}
 
 	return err;

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2010-04-23 18:17 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-04-23 18:17 [PATCH] ssb: Make bus registration failure not be silent Larry Finger

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.