Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 22 Jul 2006 13:25:33 +0200 (CEST)
From:      flag@oltrelinux.com
To:        "Scott Long" <scottl@samsco.org>
Cc:        Perforce Change Reviews <perforce@freebsd.org>, Paolo Pisati <piso@freebsd.org>
Subject:   Re: PERFORCE change 102049 for review
Message-ID:  <24452.87.17.204.226.1153567533.squirrel@krisma.oltrelinux.com>
In-Reply-To: <44C13A13.7070307@samsco.org>
References:  <200607211021.k6LALOmp039359@repoman.freebsd.org> <44C13A13.7070307@samsco.org>

next in thread | previous in thread | raw e-mail | index | archive | help
> Paolo Pisati wrote:
>> http://perforce.freebsd.org/chv.cgi?CH=102049
>>
>> Change 102049 by piso@piso_longino on 2006/07/21 10:21:18
>>
>> 	Axe INTR_FAST.
>>
>> Affected files ...
>>
>> .. //depot/projects/soc2006/intr_filter/dev/aac/aac.c#4 edit
>> .. //depot/projects/soc2006/intr_filter/dev/adlink/adlink.c#4 edit
>> .. //depot/projects/soc2006/intr_filter/dev/cy/cy_isa.c#4 edit
>> .. //depot/projects/soc2006/intr_filter/dev/cy/cy_pci.c#5 edit
>>
>> Differences ...
>>
>> ==== //depot/projects/soc2006/intr_filter/dev/aac/aac.c#4 (text+ko) ====
>>
>> @@ -312,7 +312,7 @@
>>  		}
>>  	} else {
>>  		if (bus_setup_intr(sc->aac_dev, sc->aac_irq,
>> -				   INTR_FAST|INTR_TYPE_BIO,
>> +				   INTR_TYPE_BIO,
>>  				   aac_fast_intr, NULL,
>>  				   sc, &sc->aac_intr)) {
>>  			device_printf(sc->aac_dev,
>>
>
> AAC uses the same INTR_FAST + taskqueue scheme that if_em did.
>
> Scott

yep, but in this case i kept it like a filter_only + private taskqueue
driver, so no need to worry about MPSAFE.

P.




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?24452.87.17.204.226.1153567533.squirrel>