From hselasky@c2i.net  Thu May 27 11:38:44 2010
Return-Path: <hselasky@c2i.net>
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id A2043106566C
	for <FreeBSD-gnats-submit@freebsd.org>; Thu, 27 May 2010 11:38:44 +0000 (UTC)
	(envelope-from hselasky@c2i.net)
Received: from swip.net (mailfe10.tele2.se [212.247.155.33])
	by mx1.freebsd.org (Postfix) with ESMTP id 3382E8FC18
	for <FreeBSD-gnats-submit@freebsd.org>; Thu, 27 May 2010 11:38:43 +0000 (UTC)
Received: from [188.126.201.140] (account mc467741@c2i.net HELO laptop002.hselasky.homeunix.org)
  by mailfe10.swip.net (CommuniGate Pro SMTP 5.2.19)
  with ESMTPA id 1196229477 for FreeBSD-gnats-submit@freebsd.org; Thu, 27 May 2010 13:38:41 +0200
Message-Id: <201005271335.55559.hselasky@c2i.net>
Date: Thu, 27 May 2010 13:35:55 +0200
From: Hans Petter Selasky <hselasky@c2i.net>
To: FreeBSD-gnats-submit@freebsd.org
Subject: Panic in pccard.c

>Number:         147127
>Category:       kern
>Synopsis:       [pccard] [patch] Fix panic in pccard.c
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    hselasky
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Thu May 27 11:40:02 UTC 2010
>Closed-Date:    
>Last-Modified:  Thu Jun 14 05:55:24 UTC 2012
>Originator:     Hans Petter Selasky
>Release:        FreeBSD 9.0-CURRENT i386
>Organization:
>Environment:

>Description:
When inserting a PCCARD a panic can occur if the PCCARD generate interrupts
before the driver is loaded.

>How-To-Repeat:
>Fix:

--- dev/pccard/pccard.c (revision 208027)
+++ dev/pccard/pccard.c (local)
@@ -1258,7 +1258,10 @@
 pccard_intr(void *arg)
 {
        struct pccard_function *pf = (struct pccard_function*) arg;
-       
+
+        if (pf->intr_handler == NULL)
+                return;                 /* can happen during PCCARD insertion 
*/
+       
        pf->intr_handler(pf->intr_handler_arg); 
 }
>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-bugs->eadler 
Responsible-Changed-By: eadler 
Responsible-Changed-When: Thu May 10 18:18:52 UTC 2012 
Responsible-Changed-Why:  
I'll take it. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=147127 
State-Changed-From-To: open->analyzed 
State-Changed-By: eadler 
State-Changed-When: Sun May 13 16:00:22 UTC 2012 
State-Changed-Why:  
awaiting approval 

http://www.freebsd.org/cgi/query-pr.cgi?pr=147127 
State-Changed-From-To: analyzed->open 
State-Changed-By: eadler 
State-Changed-When: Wed Jun 13 05:43:27 UTC 2012 
State-Changed-Why:  
needs more information, seems to be the wrong thing 

http://www.freebsd.org/cgi/query-pr.cgi?pr=147127 

From: Hans Petter Selasky <hselasky@c2i.net>
To: eadler@freebsd.org,
 freebsd-gnats-submit@freebsd.org
Cc:  
Subject: Re: kern/147127: [pccard] [patch] Fix panic in pccard.c
Date: Wed, 13 Jun 2012 08:30:09 +0200

 On Wednesday 13 June 2012 07:43:28 eadler@freebsd.org wrote:
 > Synopsis: [pccard] [patch] Fix panic in pccard.c
 > 
 > State-Changed-From-To: analyzed->open
 > State-Changed-By: eadler
 > State-Changed-When: Wed Jun 13 05:43:27 UTC 2012
 > State-Changed-Why:
 > needs more information, seems to be the wrong thing
 > 
 > http://www.freebsd.org/cgi/query-pr.cgi?pr=147127
 
 I can try to reproduce using 9-stable. It might take some time.
 
 --HPS
Responsible-Changed-From-To: eadler->hselasky 
Responsible-Changed-By: eadler 
Responsible-Changed-When: Thu Jun 14 05:55:23 UTC 2012 
Responsible-Changed-Why:  
over to committer 

http://www.freebsd.org/cgi/query-pr.cgi?pr=147127 
>Unformatted:
