Loading...
iokit/Kernel/IOInterruptEventSource.cpp xnu-4903.241.1 xnu-4903.270.47
--- xnu/xnu-4903.241.1/iokit/Kernel/IOInterruptEventSource.cpp
+++ xnu/xnu-4903.270.47/iokit/Kernel/IOInterruptEventSource.cpp
@@ -2,7 +2,7 @@
  * Copyright (c) 1998-2014 Apple Inc. All rights reserved.
  *
  * @APPLE_OSREFERENCE_LICENSE_HEADER_START@
- * 
+ *
  * This file contains Original Code and/or Modifications of Original Code
  * as defined in and that are subject to the Apple Public Source License
  * Version 2.0 (the 'License'). You may not use this file except in
@@ -11,10 +11,10 @@
  * unlawful or unlicensed copies of an Apple operating system, or to
  * circumvent, violate, or enable the circumvention or violation of, any
  * terms of an Apple operating system software license agreement.
- * 
+ *
  * Please obtain a copy of the License at
  * http://www.opensource.apple.com/apsl/ and read it before using this file.
- * 
+ *
  * The Original Code and all software distributed under the License are
  * distributed on an 'AS IS' basis, WITHOUT WARRANTY OF ANY KIND, EITHER
  * EXPRESS OR IMPLIED, AND APPLE HEREBY DISCLAIMS ALL SUCH WARRANTIES,
@@ -22,7 +22,7 @@
  * FITNESS FOR A PARTICULAR PURPOSE, QUIET ENJOYMENT OR NON-INFRINGEMENT.
  * Please see the License for the specific language governing rights and
  * limitations under the License.
- * 
+ *
  * @APPLE_OSREFERENCE_LICENSE_HEADER_END@
  */
 
@@ -72,258 +72,277 @@
 OSMetaClassDefineReservedUnused(IOInterruptEventSource, 6);
 OSMetaClassDefineReservedUnused(IOInterruptEventSource, 7);
 
-bool IOInterruptEventSource::init(OSObject *inOwner,
-				  Action inAction,
-				  IOService *inProvider,
-				  int inIntIndex)
-{
-    bool res = true;
-
-    if ( !super::init(inOwner, (IOEventSourceAction) inAction) )
-        return false;
-
-    reserved = IONew(ExpansionData, 1);
-
-    if (!reserved) {
-        return false;
-    }
-
-    bzero(reserved, sizeof(ExpansionData));
-
-    provider = inProvider;
-    producerCount = consumerCount = 0;
-    autoDisable = explicitDisable = false;
-    intIndex = ~inIntIndex;
-
-    // Assumes inOwner holds a reference(retain) on the provider
-    if (inProvider) {
-        if (IA_ANY_STATISTICS_ENABLED) {
-            /*
-             * We only treat this as an "interrupt" if it has a provider; if it does,
-             * set up the objects necessary to track interrupt statistics.  Interrupt
-             * event sources without providers are most likely being used as simple
-             * event source in order to poke at workloops and kick off work.
-             *
-             * We also avoid try to avoid interrupt accounting overhead if none of
-             * the statistics are enabled.
-             */
-            reserved->statistics = IONew(IOInterruptAccountingData, 1);
-
-            if (!reserved->statistics) {
-                /*
-                 * We rely on the free() routine to clean up after us if init fails
-                 * midway.
-                 */
-                return false;
-            }
-
-            bzero(reserved->statistics, sizeof(IOInterruptAccountingData));
-
-            reserved->statistics->owner = this;
-        }
-
-        res = (kIOReturnSuccess == registerInterruptHandler(inProvider, inIntIndex));
-
-	if (res) {
-	    intIndex = inIntIndex;
-        }
-    }
-
-    IOStatisticsInitializeCounter();
-
-    return res;
-}
-
-IOReturn IOInterruptEventSource::registerInterruptHandler(IOService *inProvider,
-				  int inIntIndex)
-{
-    IOReturn ret;
-    int intType;
-    IOInterruptAction intHandler;
-
-    ret = inProvider->getInterruptType(inIntIndex, &intType);
-    if (kIOReturnSuccess != ret)
-	return (ret);
-
-    autoDisable = (intType == kIOInterruptTypeLevel);
-    if (autoDisable) {
-	intHandler = OSMemberFunctionCast(IOInterruptAction,
-	    this, &IOInterruptEventSource::disableInterruptOccurred);
-    }
-    else
-	intHandler = OSMemberFunctionCast(IOInterruptAction,
-	    this, &IOInterruptEventSource::normalInterruptOccurred);
-
-    ret = provider->registerInterrupt(inIntIndex, this, intHandler);
-
-    /*
-     * Add statistics to the provider.  The setWorkLoop convention should ensure
-     * that we always go down the unregister path before we register (outside of
-     * init()), so we don't have to worry that we will invoke addInterruptStatistics
-     * erroneously.
-     */
-    if ((ret == kIOReturnSuccess) && (reserved->statistics)) {
-        /*
-         * Stash the normal index value, for the sake of debugging.
-         */
-        reserved->statistics->interruptIndex = inIntIndex;
-
-        /*
-         * We need to hook the interrupt information up to the provider so that it
-         * can find the statistics for this interrupt when desired.  The provider is
-         * responsible for maintaining the reporter for a particular interrupt, and
-         * needs a handle on the statistics so that it can request that the reporter
-         * be updated as needed.  Errors are considered "soft" for the moment (it
-         * will either panic, or fail in a way such that we can still service the
-         * interrupt).
-         */
-        provider->addInterruptStatistics(reserved->statistics, inIntIndex);
-
-        /*
-         * Add the statistics object to the global list of statistics objects; this
-         * is an aid to debugging (we can trivially find statistics for all eligible
-         * interrupts, and dump them; potentially helpful if the system is wedged
-         * due to interrupt activity).
-         */
-        interruptAccountingDataAddToList(reserved->statistics);
-    }
-
-    return (ret);
+bool
+IOInterruptEventSource::init(OSObject *inOwner,
+    Action inAction,
+    IOService *inProvider,
+    int inIntIndex)
+{
+	bool res = true;
+
+	if (!super::init(inOwner, (IOEventSourceAction) inAction)) {
+		return false;
+	}
+
+	reserved = IONew(ExpansionData, 1);
+
+	if (!reserved) {
+		return false;
+	}
+
+	bzero(reserved, sizeof(ExpansionData));
+
+	provider = inProvider;
+	producerCount = consumerCount = 0;
+	autoDisable = explicitDisable = false;
+	intIndex = ~inIntIndex;
+
+	// Assumes inOwner holds a reference(retain) on the provider
+	if (inProvider) {
+		if (IA_ANY_STATISTICS_ENABLED) {
+			/*
+			 * We only treat this as an "interrupt" if it has a provider; if it does,
+			 * set up the objects necessary to track interrupt statistics.  Interrupt
+			 * event sources without providers are most likely being used as simple
+			 * event source in order to poke at workloops and kick off work.
+			 *
+			 * We also avoid try to avoid interrupt accounting overhead if none of
+			 * the statistics are enabled.
+			 */
+			reserved->statistics = IONew(IOInterruptAccountingData, 1);
+
+			if (!reserved->statistics) {
+				/*
+				 * We rely on the free() routine to clean up after us if init fails
+				 * midway.
+				 */
+				return false;
+			}
+
+			bzero(reserved->statistics, sizeof(IOInterruptAccountingData));
+
+			reserved->statistics->owner = this;
+		}
+
+		res = (kIOReturnSuccess == registerInterruptHandler(inProvider, inIntIndex));
+
+		if (res) {
+			intIndex = inIntIndex;
+		}
+	}
+
+	IOStatisticsInitializeCounter();
+
+	return res;
+}
+
+IOReturn
+IOInterruptEventSource::registerInterruptHandler(IOService *inProvider,
+    int inIntIndex)
+{
+	IOReturn ret;
+	int intType;
+	IOInterruptAction intHandler;
+
+	ret = inProvider->getInterruptType(inIntIndex, &intType);
+	if (kIOReturnSuccess != ret) {
+		return ret;
+	}
+
+	autoDisable = (intType == kIOInterruptTypeLevel);
+	if (autoDisable) {
+		intHandler = OSMemberFunctionCast(IOInterruptAction,
+		    this, &IOInterruptEventSource::disableInterruptOccurred);
+	} else {
+		intHandler = OSMemberFunctionCast(IOInterruptAction,
+		    this, &IOInterruptEventSource::normalInterruptOccurred);
+	}
+
+	ret = provider->registerInterrupt(inIntIndex, this, intHandler);
+
+	/*
+	 * Add statistics to the provider.  The setWorkLoop convention should ensure
+	 * that we always go down the unregister path before we register (outside of
+	 * init()), so we don't have to worry that we will invoke addInterruptStatistics
+	 * erroneously.
+	 */
+	if ((ret == kIOReturnSuccess) && (reserved->statistics)) {
+		/*
+		 * Stash the normal index value, for the sake of debugging.
+		 */
+		reserved->statistics->interruptIndex = inIntIndex;
+
+		/*
+		 * We need to hook the interrupt information up to the provider so that it
+		 * can find the statistics for this interrupt when desired.  The provider is
+		 * responsible for maintaining the reporter for a particular interrupt, and
+		 * needs a handle on the statistics so that it can request that the reporter
+		 * be updated as needed.  Errors are considered "soft" for the moment (it
+		 * will either panic, or fail in a way such that we can still service the
+		 * interrupt).
+		 */
+		provider->addInterruptStatistics(reserved->statistics, inIntIndex);
+
+		/*
+		 * Add the statistics object to the global list of statistics objects; this
+		 * is an aid to debugging (we can trivially find statistics for all eligible
+		 * interrupts, and dump them; potentially helpful if the system is wedged
+		 * due to interrupt activity).
+		 */
+		interruptAccountingDataAddToList(reserved->statistics);
+	}
+
+	return ret;
 }
 
 void
 IOInterruptEventSource::unregisterInterruptHandler(IOService *inProvider,
-				  int inIntIndex)
-{
-    if (reserved->statistics) {
-        interruptAccountingDataRemoveFromList(reserved->statistics);
-        provider->removeInterruptStatistics(reserved->statistics->interruptIndex);
-    }
-
-    provider->unregisterInterrupt(inIntIndex);
+    int inIntIndex)
+{
+	if (reserved->statistics) {
+		interruptAccountingDataRemoveFromList(reserved->statistics);
+		provider->removeInterruptStatistics(reserved->statistics->interruptIndex);
+	}
+
+	provider->unregisterInterrupt(inIntIndex);
 }
 
 
 IOInterruptEventSource *
 IOInterruptEventSource::interruptEventSource(OSObject *inOwner,
-					     Action inAction,
-					     IOService *inProvider,
-					     int inIntIndex)
-{
-    IOInterruptEventSource *me = new IOInterruptEventSource;
-
-    if (me && !me->init(inOwner, inAction, inProvider, inIntIndex)) {
-        me->release();
-        return 0;
-    }
-
-    return me;
+    Action inAction,
+    IOService *inProvider,
+    int inIntIndex)
+{
+	IOInterruptEventSource *me = new IOInterruptEventSource;
+
+	if (me && !me->init(inOwner, inAction, inProvider, inIntIndex)) {
+		me->release();
+		return 0;
+	}
+
+	return me;
 }
 
 IOInterruptEventSource *
 IOInterruptEventSource::interruptEventSource(OSObject *inOwner,
-					     IOService *inProvider,
-					     int inIntIndex,
-					     ActionBlock inAction)
-{
-    IOInterruptEventSource * ies;
-    ies = IOInterruptEventSource::interruptEventSource(inOwner, (Action) NULL, inProvider, inIntIndex);
-    if (ies) ies->setActionBlock((IOEventSource::ActionBlock) inAction);
-
-    return ies;
-}
-
-void IOInterruptEventSource::free()
-{
-    if (provider && intIndex >= 0)
-        unregisterInterruptHandler(provider, intIndex);
-
-    if (reserved) {
-        if (reserved->statistics) {
-            IODelete(reserved->statistics, IOInterruptAccountingData, 1);
-        }
-
-        IODelete(reserved, ExpansionData, 1);
-    }
-
-    super::free();
-}
-
-void IOInterruptEventSource::enable()
-{
-    if (provider && intIndex >= 0) {
-        provider->enableInterrupt(intIndex);
-        explicitDisable = false;
-        enabled = true;
-    }
-}
-
-void IOInterruptEventSource::disable()
-{
-    if (provider && intIndex >= 0) {
-        provider->disableInterrupt(intIndex);
-        explicitDisable = true;
-        enabled = false;
-    }
-}
-
-void IOInterruptEventSource::setWorkLoop(IOWorkLoop *inWorkLoop)
-{
-    if (inWorkLoop) super::setWorkLoop(inWorkLoop);
-
-    if (provider) {
+    IOService *inProvider,
+    int inIntIndex,
+    ActionBlock inAction)
+{
+	IOInterruptEventSource * ies;
+	ies = IOInterruptEventSource::interruptEventSource(inOwner, (Action) NULL, inProvider, inIntIndex);
+	if (ies) {
+		ies->setActionBlock((IOEventSource::ActionBlock) inAction);
+	}
+
+	return ies;
+}
+
+void
+IOInterruptEventSource::free()
+{
+	if (provider && intIndex >= 0) {
+		unregisterInterruptHandler(provider, intIndex);
+	}
+
+	if (reserved) {
+		if (reserved->statistics) {
+			IODelete(reserved->statistics, IOInterruptAccountingData, 1);
+		}
+
+		IODelete(reserved, ExpansionData, 1);
+	}
+
+	super::free();
+}
+
+void
+IOInterruptEventSource::enable()
+{
+	if (provider && intIndex >= 0) {
+		provider->enableInterrupt(intIndex);
+		explicitDisable = false;
+		enabled = true;
+	}
+}
+
+void
+IOInterruptEventSource::disable()
+{
+	if (provider && intIndex >= 0) {
+		provider->disableInterrupt(intIndex);
+		explicitDisable = true;
+		enabled = false;
+	}
+}
+
+void
+IOInterruptEventSource::setWorkLoop(IOWorkLoop *inWorkLoop)
+{
+	if (inWorkLoop) {
+		super::setWorkLoop(inWorkLoop);
+	}
+
+	if (provider) {
+		if (!inWorkLoop) {
+			if (intIndex >= 0) {
+				/*
+				 * It isn't necessarily safe to wait until free() to unregister the interrupt;
+				 * our provider may disappear.
+				 */
+				unregisterInterruptHandler(provider, intIndex);
+				intIndex = ~intIndex;
+			}
+		} else if ((intIndex < 0) && (kIOReturnSuccess == registerInterruptHandler(provider, ~intIndex))) {
+			intIndex = ~intIndex;
+		}
+	}
+
 	if (!inWorkLoop) {
-	    if (intIndex >= 0) {
-                /*
-                 * It isn't necessarily safe to wait until free() to unregister the interrupt;
-                 * our provider may disappear.
-                 */
-                unregisterInterruptHandler(provider, intIndex);
-		intIndex = ~intIndex;
-	    }
-	} else if ((intIndex < 0) && (kIOReturnSuccess == registerInterruptHandler(provider, ~intIndex))) {
-	    intIndex = ~intIndex;
-	}
-    }
-
-    if (!inWorkLoop) super::setWorkLoop(inWorkLoop);
-}
-
-const IOService *IOInterruptEventSource::getProvider() const
-{
-    return provider;
-}
-
-int IOInterruptEventSource::getIntIndex() const
-{
-    return intIndex;
-}
-
-bool IOInterruptEventSource::getAutoDisable() const
-{
-    return autoDisable;
-}
-
-bool IOInterruptEventSource::checkForWork()
-{
-    uint64_t startSystemTime = 0;
-    uint64_t endSystemTime = 0;
-    uint64_t startCPUTime = 0;
-    uint64_t endCPUTime = 0;
-    unsigned int cacheProdCount = producerCount;
-    int numInts = cacheProdCount - consumerCount;
-    IOInterruptEventAction intAction = (IOInterruptEventAction) action;
-    ActionBlock intActionBlock = (ActionBlock) actionBlock;
+		super::setWorkLoop(inWorkLoop);
+	}
+}
+
+const IOService *
+IOInterruptEventSource::getProvider() const
+{
+	return provider;
+}
+
+int
+IOInterruptEventSource::getIntIndex() const
+{
+	return intIndex;
+}
+
+bool
+IOInterruptEventSource::getAutoDisable() const
+{
+	return autoDisable;
+}
+
+bool
+IOInterruptEventSource::checkForWork()
+{
+	uint64_t startSystemTime = 0;
+	uint64_t endSystemTime = 0;
+	uint64_t startCPUTime = 0;
+	uint64_t endCPUTime = 0;
+	unsigned int cacheProdCount = producerCount;
+	int numInts = cacheProdCount - consumerCount;
+	IOInterruptEventAction intAction = (IOInterruptEventAction) action;
+	ActionBlock intActionBlock = (ActionBlock) actionBlock;
 	bool trace = (gIOKitTrace & kIOTraceIntEventSource) ? true : false;
-	
-    IOStatisticsCheckForWork();
-	
-	if ( numInts > 0 )
-	{
-		if (trace)
+
+	IOStatisticsCheckForWork();
+
+	if (numInts > 0) {
+		if (trace) {
 			IOTimeStampStartConstant(IODBG_INTES(IOINTES_ACTION),
-						 VM_KERNEL_ADDRHIDE(intAction), VM_KERNEL_ADDRHIDE(owner),
-						 VM_KERNEL_ADDRHIDE(this), VM_KERNEL_ADDRHIDE(workLoop));
+			    VM_KERNEL_ADDRHIDE(intAction), VM_KERNEL_ADDRHIDE(owner),
+			    VM_KERNEL_ADDRHIDE(this), VM_KERNEL_ADDRHIDE(workLoop));
+		}
 
 		if (reserved->statistics) {
 			if (IA_GET_STATISTIC_ENABLED(kInterruptAccountingSecondLevelSystemTimeIndex)) {
@@ -336,8 +355,11 @@
 		}
 
 		// Call the handler
-		if (kActionBlock & flags) (intActionBlock)(this, numInts);
-		else                      (*intAction)(owner, this, numInts);
+		if (kActionBlock & flags) {
+			(intActionBlock)(this, numInts);
+		} else {
+			(*intAction)(owner, this, numInts);
+		}
 
 		if (reserved->statistics) {
 			if (IA_GET_STATISTIC_ENABLED(kInterruptAccountingSecondLevelCountIndex)) {
@@ -354,23 +376,23 @@
 				IA_ADD_VALUE(&reserved->statistics->interruptStatistics[kInterruptAccountingSecondLevelSystemTimeIndex], endSystemTime - startSystemTime);
 			}
 		}
-		
-		if (trace)
+
+		if (trace) {
 			IOTimeStampEndConstant(IODBG_INTES(IOINTES_ACTION),
-						VM_KERNEL_ADDRHIDE(intAction), VM_KERNEL_ADDRHIDE(owner),
-						VM_KERNEL_ADDRHIDE(this), VM_KERNEL_ADDRHIDE(workLoop));
-		
+			    VM_KERNEL_ADDRHIDE(intAction), VM_KERNEL_ADDRHIDE(owner),
+			    VM_KERNEL_ADDRHIDE(this), VM_KERNEL_ADDRHIDE(workLoop));
+		}
+
 		consumerCount = cacheProdCount;
-		if (autoDisable && !explicitDisable)
+		if (autoDisable && !explicitDisable) {
 			enable();
-	}
-	
-	else if ( numInts < 0 )
-	{
-		if (trace)
+		}
+	} else if (numInts < 0) {
+		if (trace) {
 			IOTimeStampStartConstant(IODBG_INTES(IOINTES_ACTION),
-						VM_KERNEL_ADDRHIDE(intAction), VM_KERNEL_ADDRHIDE(owner),
-						VM_KERNEL_ADDRHIDE(this), VM_KERNEL_ADDRHIDE(workLoop));
+			    VM_KERNEL_ADDRHIDE(intAction), VM_KERNEL_ADDRHIDE(owner),
+			    VM_KERNEL_ADDRHIDE(this), VM_KERNEL_ADDRHIDE(workLoop));
+		}
 
 		if (reserved->statistics) {
 			if (IA_GET_STATISTIC_ENABLED(kInterruptAccountingSecondLevelSystemTimeIndex)) {
@@ -381,10 +403,13 @@
 				startCPUTime = thread_get_runtime_self();
 			}
 		}
-		
+
 		// Call the handler
-		if (kActionBlock & flags) (intActionBlock)(this, numInts);
-		else                      (*intAction)(owner, this, numInts);
+		if (kActionBlock & flags) {
+			(intActionBlock)(this, numInts);
+		} else {
+			(*intAction)(owner, this, numInts);
+		}
 
 		if (reserved->statistics) {
 			if (IA_GET_STATISTIC_ENABLED(kInterruptAccountingSecondLevelCountIndex)) {
@@ -401,80 +426,90 @@
 				IA_ADD_VALUE(&reserved->statistics->interruptStatistics[kInterruptAccountingSecondLevelSystemTimeIndex], endSystemTime - startSystemTime);
 			}
 		}
-		
-		if (trace)
+
+		if (trace) {
 			IOTimeStampEndConstant(IODBG_INTES(IOINTES_ACTION),
-							VM_KERNEL_ADDRHIDE(intAction), VM_KERNEL_ADDRHIDE(owner),
-							VM_KERNEL_ADDRHIDE(this), VM_KERNEL_ADDRHIDE(workLoop));
-		
+			    VM_KERNEL_ADDRHIDE(intAction), VM_KERNEL_ADDRHIDE(owner),
+			    VM_KERNEL_ADDRHIDE(this), VM_KERNEL_ADDRHIDE(workLoop));
+		}
+
 		consumerCount = cacheProdCount;
-		if (autoDisable && !explicitDisable)
+		if (autoDisable && !explicitDisable) {
 			enable();
-	}
-	
-    return false;
-}
-
-void IOInterruptEventSource::normalInterruptOccurred
-    (void */*refcon*/, IOService */*prov*/, int /*source*/)
+		}
+	}
+
+	return false;
+}
+
+void
+IOInterruptEventSource::normalInterruptOccurred
+(void */*refcon*/, IOService */*prov*/, int /*source*/)
 {
 	bool trace = (gIOKitTrace & kIOTraceIntEventSource) ? true : false;
-	
-    IOStatisticsInterrupt();
-    producerCount++;
-	
-	if (trace)
-	    IOTimeStampStartConstant(IODBG_INTES(IOINTES_SEMA), VM_KERNEL_ADDRHIDE(this), VM_KERNEL_ADDRHIDE(owner));
-
-    if (reserved->statistics) {
-        if (IA_GET_STATISTIC_ENABLED(kInterruptAccountingFirstLevelCountIndex)) {
-            IA_ADD_VALUE(&reserved->statistics->interruptStatistics[kInterruptAccountingFirstLevelCountIndex], 1);
-        }
-    }
-	
-    signalWorkAvailable();
-	
-	if (trace)
-	    IOTimeStampEndConstant(IODBG_INTES(IOINTES_SEMA), VM_KERNEL_ADDRHIDE(this), VM_KERNEL_ADDRHIDE(owner));
-}
-
-void IOInterruptEventSource::disableInterruptOccurred
-    (void */*refcon*/, IOService *prov, int source)
+
+	IOStatisticsInterrupt();
+	producerCount++;
+
+	if (trace) {
+		IOTimeStampStartConstant(IODBG_INTES(IOINTES_SEMA), VM_KERNEL_ADDRHIDE(this), VM_KERNEL_ADDRHIDE(owner));
+	}
+
+	if (reserved->statistics) {
+		if (IA_GET_STATISTIC_ENABLED(kInterruptAccountingFirstLevelCountIndex)) {
+			IA_ADD_VALUE(&reserved->statistics->interruptStatistics[kInterruptAccountingFirstLevelCountIndex], 1);
+		}
+	}
+
+	signalWorkAvailable();
+
+	if (trace) {
+		IOTimeStampEndConstant(IODBG_INTES(IOINTES_SEMA), VM_KERNEL_ADDRHIDE(this), VM_KERNEL_ADDRHIDE(owner));
+	}
+}
+
+void
+IOInterruptEventSource::disableInterruptOccurred
+(void */*refcon*/, IOService *prov, int source)
 {
 	bool trace = (gIOKitTrace & kIOTraceIntEventSource) ? true : false;
-	
-    prov->disableInterrupt(source);	/* disable the interrupt */
-	
-    IOStatisticsInterrupt();
-    producerCount++;
-	
-	if (trace)
-	    IOTimeStampStartConstant(IODBG_INTES(IOINTES_SEMA), VM_KERNEL_ADDRHIDE(this), VM_KERNEL_ADDRHIDE(owner));
-
-    if (reserved->statistics) {
-        if (IA_GET_STATISTIC_ENABLED(kInterruptAccountingFirstLevelCountIndex)) {
-            IA_ADD_VALUE(&reserved->statistics->interruptStatistics[kInterruptAccountingFirstLevelCountIndex], 1);
-        }
-    }
-    
-    signalWorkAvailable();
-	
-	if (trace)
-	    IOTimeStampEndConstant(IODBG_INTES(IOINTES_SEMA), VM_KERNEL_ADDRHIDE(this), VM_KERNEL_ADDRHIDE(owner));
-}
-
-void IOInterruptEventSource::interruptOccurred
-    (void *refcon, IOService *prov, int source)
-{
-    if (autoDisable && prov)
-        disableInterruptOccurred(refcon, prov, source);
-    else
-        normalInterruptOccurred(refcon, prov, source);
-}
-
-IOReturn IOInterruptEventSource::warmCPU
-    (uint64_t abstime)
-{
-
+
+	prov->disableInterrupt(source); /* disable the interrupt */
+
+	IOStatisticsInterrupt();
+	producerCount++;
+
+	if (trace) {
+		IOTimeStampStartConstant(IODBG_INTES(IOINTES_SEMA), VM_KERNEL_ADDRHIDE(this), VM_KERNEL_ADDRHIDE(owner));
+	}
+
+	if (reserved->statistics) {
+		if (IA_GET_STATISTIC_ENABLED(kInterruptAccountingFirstLevelCountIndex)) {
+			IA_ADD_VALUE(&reserved->statistics->interruptStatistics[kInterruptAccountingFirstLevelCountIndex], 1);
+		}
+	}
+
+	signalWorkAvailable();
+
+	if (trace) {
+		IOTimeStampEndConstant(IODBG_INTES(IOINTES_SEMA), VM_KERNEL_ADDRHIDE(this), VM_KERNEL_ADDRHIDE(owner));
+	}
+}
+
+void
+IOInterruptEventSource::interruptOccurred
+(void *refcon, IOService *prov, int source)
+{
+	if (autoDisable && prov) {
+		disableInterruptOccurred(refcon, prov, source);
+	} else {
+		normalInterruptOccurred(refcon, prov, source);
+	}
+}
+
+IOReturn
+IOInterruptEventSource::warmCPU
+(uint64_t abstime)
+{
 	return ml_interrupt_prewarm(abstime);
 }