All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/3] bnx2x: free workqueue when driver fail to register
@ 2009-05-06  9:22 Stanislaw Gruszka
  2009-05-06 12:04 ` Eilon Greenstein
  0 siblings, 1 reply; 3+ messages in thread
From: Stanislaw Gruszka @ 2009-05-06  9:22 UTC (permalink / raw)
  To: netdev; +Cc: Eilon Greenstein

Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
---
 drivers/net/bnx2x_main.c |    9 ++++++++-
 1 files changed, 8 insertions(+), 1 deletions(-)

diff --git a/drivers/net/bnx2x_main.c b/drivers/net/bnx2x_main.c
index ad5ef25..ff2d9f6 100644
--- a/drivers/net/bnx2x_main.c
+++ b/drivers/net/bnx2x_main.c
@@ -11412,13 +11412,20 @@ static struct pci_driver bnx2x_pci_driver = {
 
 static int __init bnx2x_init(void)
 {
+	int ret;
+
 	bnx2x_wq = create_singlethread_workqueue("bnx2x");
 	if (bnx2x_wq == NULL) {
 		printk(KERN_ERR PFX "Cannot create workqueue\n");
 		return -ENOMEM;
 	}
 
-	return pci_register_driver(&bnx2x_pci_driver);
+	ret = pci_register_driver(&bnx2x_pci_driver);
+	if (ret) {
+		printk(KERN_ERR PFX "Cannot register driver\n");
+		destroy_workqueue(bnx2x_wq);
+	}
+	return ret;
 }
 
 static void __exit bnx2x_cleanup(void)
-- 
1.5.5.6


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/3] bnx2x: free workqueue when driver fail to register
  2009-05-06  9:22 [PATCH 1/3] bnx2x: free workqueue when driver fail to register Stanislaw Gruszka
@ 2009-05-06 12:04 ` Eilon Greenstein
  2009-05-08 21:52   ` David Miller
  0 siblings, 1 reply; 3+ messages in thread
From: Eilon Greenstein @ 2009-05-06 12:04 UTC (permalink / raw)
  To: Stanislaw Gruszka, David Miller; +Cc: netdev, gertner

On Wed, 2009-05-06 at 02:22 -0700, Stanislaw Gruszka wrote:
> Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
Acked-by: Eilon Greenstein <eilong@broadcom.com>

Thanks Stanislaw!




^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/3] bnx2x: free workqueue when driver fail to register
  2009-05-06 12:04 ` Eilon Greenstein
@ 2009-05-08 21:52   ` David Miller
  0 siblings, 0 replies; 3+ messages in thread
From: David Miller @ 2009-05-08 21:52 UTC (permalink / raw)
  To: eilong; +Cc: sgruszka, netdev, gertner

From: "Eilon Greenstein" <eilong@broadcom.com>
Date: Wed, 6 May 2009 15:04:28 +0300

> On Wed, 2009-05-06 at 02:22 -0700, Stanislaw Gruszka wrote:
>> Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
> Acked-by: Eilon Greenstein <eilong@broadcom.com>

Applied to net-next-2.6

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2009-05-08 21:52 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-05-06  9:22 [PATCH 1/3] bnx2x: free workqueue when driver fail to register Stanislaw Gruszka
2009-05-06 12:04 ` Eilon Greenstein
2009-05-08 21:52   ` David Miller

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.