Safely Temporarily Disable AM335x Ethernet Receive Interrupts

J Burke

I've modeled my ethernet code after the Starterware LWIP interface examples (although I'm not using LWIP for various reasons).

During the time the Receive interrupt process is called, I handle the incoming packets -- using some variables that are used in other parts of the system.

So, I need to be able to "protect" these variables and temporarily disable the receive interrupt.

I've tried using something like:

//Disable the Global Interrupts
intStatus = IntDisable();

/* Disable the interrupts for channel 0 and for control core 0 */
CPSWCPDMATxIntDisable(me->cpdma_base, 0);
CPSWWrCoreIntDisable(me->wrpr_base, 0, 0, CPSW_CORE_INT_TX_PULSE);

CPSWCPDMARxIntDisable(me->cpdma_base, 0);
CPSWWrCoreIntDisable(me->wrpr_base, 0, 0, CPSW_CORE_INT_RX_PULSE);

//Enable the Global Interrupts
IntEnable(intStatus);

or even just

//Disable the Global Interrupts
intStatus = IntDisable();

/* Disable the interrupts for channel 0 and for control core 0 */
CPSWWrCoreIntDisable(me->wrpr_base, 0, 0, CPSW_CORE_INT_TX_PULSE);

CPSWWrCoreIntDisable(me->wrpr_base, 0, 0, CPSW_CORE_INT_RX_PULSE);

//Enable the Global Interrupts
IntEnable(intStatus);

but this seems to cause crashes.

Is this the correct way to temporarily disable the interrupts?

over 7 years ago

0 Lalindra Jayatilleke over 7 years ago

TI__Mastermind 30365 points

J Burke,

I'm unsure on what version Starterware you are on, but if you look in C:\ti\pdk_am335x_1_0_2\packages\ti\csl\csl_cpIntcAux.h could you use CSL_CPINTC_disableAllHostInterrupt and CSL_CPINTC_enableAllHostInterrupt ?

Lali

0 J Burke over 7 years ago in reply to Lalindra Jayatilleke

Intellectual 350 points

Unfortunately I can't disable all interrupts (for other necessary interrupts in the system)

I'm looking for just something to disable the ethernet interrupts only.

I'm using starterware version 2.00.01.01.

0 Lalindra Jayatilleke over 7 years ago in reply to J Burke

TI__Mastermind 30365 points

J Burke,
Wouldn't you have to re-enable ints after the below line?

/* Disable the interrupts for channel 0 and for control core 0 */
CPSWWrCoreIntDisable(me->wrpr_base, 0, 0, CPSW_CORE_INT_TX_PULSE);

CPSWWrCoreIntDisable(me->wrpr_base, 0, 0, CPSW_CORE_INT_RX_PULSE);

Could you do a blanket disabling of ints in your code and re-enabling to see if you crash. This will indicate if its a system problem or something isolated to the Ethernet ints.

Lali

0 J Burke over 7 years ago in reply to Lalindra Jayatilleke

Intellectual 350 points

Yes, I do enable them later with:

CPSWWrCoreIntEnable(me->wrpr_base, 0, 0, CPSW_CORE_INT_TX_PULSE);

CPSWWrCoreIntEnable(me->wrpr_base, 0, 0, CPSW_CORE_INT_RX_PULSE);

It's a bit tricky for me to do a blanket stop of all interrupts helps, since this causes other problems with missed interrupts.

I just wondered if there was a "known" safe way to stop and restart Ethernet interrupts.

0 Lalindra Jayatilleke over 7 years ago in reply to J Burke

TI__Mastermind 30365 points

J Burke,

Any chance this thread would shed some light? https://e2e.ti.com/support/microcontrollers/stellaris_arm/f/471/t/284029

The last post has some INT related details. Sorry that I'm not much help to you on this. LWIP isn't supported in Starterware as of late.

Lali

0 J Burke over 7 years ago in reply to Lalindra Jayatilleke

Intellectual 350 points

Unfortunately that doesn't really help. The solutions offered seem to be "use an RTOS" or "have each message wait for a response."

Since I can do neither, that doesn't really work.

Processors

Processors forum

Safely Temporarily Disable AM335x Ethernet Receive Interrupts