mirror of
https://github.com/openwrt/openwrt.git
synced 2024-12-29 01:59:02 +00:00
1700424553
Some users noticed repeated resyncs at random intervals, which go away when the MEI driver is configured to use polling instead of interrupts. Debugging shows that this seems to be caused by concurrent calls to MEI_ReadMailbox (in the interrupt handler) and MEI_WriteMailbox. This appears to be mostly triggered when there is an interrupt for vectoring error reports. In polling mode, calls to MEI_ReadMailbox are protected by the same semaphore as is used in MEI_WriteMailbox. When interrupts are used, MEI_WriteMailbox appears to rely on MEI_DisableDeviceInt and MEI_EnableDeviceInt to provide mutual exclusion with the interrupt handler. These functions mask/unmask interrupts, and there is an additional check of the mask in the interrupt handler itself. However, this is not sufficient on systems with SMP, as the interrupt handler may be running in parallel, and could already be past the interrupt mask check at this point. This adds a lock to the interrupt handler, and also acquires this lock in MEI_DisableDeviceInt. This should make sure that after a call to MEI_DisableDeviceInt the interrupt is masked, and the interrupt handler is either not running, has alread finished its work, or is still before the interrupt mask check, and is thus going to detect the change. Tested-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> Signed-off-by: Jan Hoffmann <jan@3e8.eu>
46 lines
1.3 KiB
Diff
46 lines
1.3 KiB
Diff
--- a/src/drv_mei_cpe_common.c
|
|
+++ b/src/drv_mei_cpe_common.c
|
|
@@ -104,6 +104,8 @@ IFX_uint32_t MEI_FsmStateSetMsgPreAction
|
|
MEI_DEVCFG_DATA_T MEI_DevCfgData;
|
|
#endif
|
|
|
|
+static DEFINE_SPINLOCK(MEI_InterruptLock);
|
|
+
|
|
/* ============================================================================
|
|
Proc-FS and debug variable definitions
|
|
========================================================================= */
|
|
@@ -2134,6 +2136,9 @@ IFX_int32_t MEI_ProcessIntPerIrq(MEIX_CN
|
|
#if (MEI_SUPPORT_DEBUG_STREAMS == 1)
|
|
IFX_int_t extraDbgStreamLoop = 0;
|
|
#endif
|
|
+ unsigned long flags;
|
|
+
|
|
+ spin_lock_irqsave(&MEI_InterruptLock, flags);
|
|
|
|
/* get the actual chip device from the list and step through the VRX devices */
|
|
while(pNextXCntrl)
|
|
@@ -2167,6 +2172,8 @@ IFX_int32_t MEI_ProcessIntPerIrq(MEIX_CN
|
|
}
|
|
#endif
|
|
|
|
+ spin_unlock_irqrestore(&MEI_InterruptLock, flags);
|
|
+
|
|
return meiIntCnt;
|
|
}
|
|
|
|
@@ -2639,9 +2646,14 @@ IFX_int32_t MEI_MsgSendPreAction(
|
|
*/
|
|
IFX_void_t MEI_DisableDeviceInt(MEI_DEV_T *pMeiDev)
|
|
{
|
|
+ unsigned long flags;
|
|
+ spin_lock_irqsave(&MEI_InterruptLock, flags);
|
|
+
|
|
MEI_MaskInterrupts( &pMeiDev->meiDrvCntrl,
|
|
ME_ARC2ME_INTERRUPT_MASK_ALL);
|
|
|
|
+ spin_unlock_irqrestore(&MEI_InterruptLock, flags);
|
|
+
|
|
return;
|
|
}
|
|
|