From owner-freebsd-mobile@FreeBSD.ORG  Wed Jun 20 13:51:57 2007
Return-Path: <owner-freebsd-mobile@FreeBSD.ORG>
X-Original-To: mobile@freebsd.org
Delivered-To: freebsd-mobile@FreeBSD.ORG
Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52])
	by hub.freebsd.org (Postfix) with ESMTP id 6043D16A47B
	for <mobile@freebsd.org>; Wed, 20 Jun 2007 13:51:57 +0000 (UTC)
	(envelope-from freebsd@gm.nunu.org)
Received: from py-out-1112.google.com (py-out-1112.google.com [64.233.166.180])
	by mx1.freebsd.org (Postfix) with ESMTP id CDDF113C4CC
	for <mobile@freebsd.org>; Wed, 20 Jun 2007 13:51:56 +0000 (UTC)
	(envelope-from freebsd@gm.nunu.org)
Received: by py-out-1112.google.com with SMTP id a29so375157pyi
	for <mobile@freebsd.org>; Wed, 20 Jun 2007 06:51:55 -0700 (PDT)
Received: by 10.35.67.18 with SMTP id u18mr1252315pyk.1182347515559;
	Wed, 20 Jun 2007 06:51:55 -0700 (PDT)
Received: by 10.35.71.8 with HTTP; Wed, 20 Jun 2007 06:51:55 -0700 (PDT)
Message-ID: <626eb4530706200651s255e2ff2u80d70d2d887c8e4c@mail.gmail.com>
Date: Wed, 20 Jun 2007 22:51:55 +0900
From: "Hidetoshi Shimokawa" <simokawa@FreeBSD.ORG>
Sender: freebsd@gm.nunu.org
To: "Paolo Pisati" <piso@freebsd.org>
In-Reply-To: <20070618183627.GA2183@tin.it>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
References: <46734A3E.3010903@gddsn.org.cn>
	<626eb4530706160525o54234621s2de098e64cdeb4fe@mail.gmail.com>
	<20070616.173002.-457443410.imp@bsdimp.com>
	<626eb4530706161858l5f60d67ej55874cb9348e649b@mail.gmail.com>
	<20070618183627.GA2183@tin.it>
X-Google-Sender-Auth: a81d699bd8b52af7
Cc: mobile@freebsd.org, wsk@gddsn.org.cn, current@freebsd.org
Subject: Re: kernel panic with pccard insert on recent 7.0 CURRENT
X-BeenThere: freebsd-mobile@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Mobile computing with FreeBSD <freebsd-mobile.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-mobile>, 
	<mailto:freebsd-mobile-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-mobile>
List-Post: <mailto:freebsd-mobile@freebsd.org>
List-Help: <mailto:freebsd-mobile-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-mobile>,
	<mailto:freebsd-mobile-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Wed, 20 Jun 2007 13:51:57 -0000

On 6/19/07, Paolo Pisati <piso@freebsd.org> wrote:
> On Sun, Jun 17, 2007 at 10:58:12AM +0900, Hidetoshi Shimokawa wrote:
> > And INTR_FILTER doesn't seem well-tested at least for
> > handling of stray interrupts for filter only IRQs.
> > I need the following patch to workaroung the problem.
> >
> > http://people.freebsd.org/~simokawa/tmp/kern_intr.c-20070617.patch
> > on which platform?

amd64

> cause, right now, if there's a stray interrupt we disable the
> irq line if the interrut disable function is hooked to
> ie_disab:

My patch may be wrong. But it seems too restrictive to disable the
interrupt forever
only one stray interrupt. Drivers could return _STRAY even if it is the source.

> kern_intr.c::intr_event_handle():
>
>         ...
>         if (thread & FILTER_HANDLED) {
>                 if (ie->ie_eoi != NULL)
>                         ie->ie_eoi(ie->ie_source);
>         } else {
>                 if (ie->ie_disab != NULL)
>                         ie->ie_disab(ie->ie_source);
>         }
>         ...
>
> bye,
> P.
>
>


-- 
/\ Hidetoshi Shimokawa
\/  simokawa@FreeBSD.ORG