Raising BASEPRI on M7 port (v9.0.0) is done as below: ~~~

static portFORCE_INLINE void vPortRaiseBASEPRI( void )

{

uint32t ulNewBASEPRI = configMAXSYSCALLINTERRUPTPRIORITY;

__asm

{

	/* Set BASEPRI to the max syscall priority to effect a critical

	section. */

	cpsid i

	msr basepri, ulNewBASEPRI

	dsb

	isb

	cpsie i

}

}

~~~ Could you explain the need for a DSB?

Additionally, I understand that ISB is used to ensure that when this function returns, the BASEPRI change is observed. However this makes sense only if ISB is placed after "cpsie i" instruction so that the PRIMASK clear is also observed.

I have also looked into this (http://infocenter.arm.com/help/topic/com.arm.doc.dai0321a/DAI0321Aprogrammingguidememorybarriersform_profile.pdf) but I could not find the exact case in the guidelines.

The cpsid/cpsie instructions are a workaround to a silicon errata and are only required if the revision of the core is r01p, otherwise use the Cortex-M4F port layer. I imagine the errata docs will be the only place they are mentioned.

Thank you for your answer.

What about the need for DSB? Is it required after the MSR instruction that sets BASEPRI?

(To help our discussion I am pasting the CM4F port that is suitable for our CM7 (non r0p1))

~~~ static portFORCEINLINE void vPortRaiseBASEPRI( void ) { uint32t ulNewBASEPRI = configMAXSYSCALLINTERRUPT_PRIORITY;

__asm
{
	/* Set BASEPRI to the max syscall priority to effect a critical
	section. */
	msr basepri, ulNewBASEPRI
	dsb
	isb
}

} ~~~

On the M3 and M4 it is actually after the next instruction: "MSR side-effects are visible after one further instruction is executed". I believe it is different on the M7 though and will have to look it up. The M4F port was updated so it could also be used on an M7.

So, for M3/4 at least, removing the DSB instruction means that the side-effects will be visible after the ISB instruction, which is what we want. So it looks that practically there is no need for a DSB.

I will be waiting for your findings on M7 as well. Many thanks again for the help.

Any news on this?