Take semaphore when fatal error occur in coredump with CONFIG_DEBUG_COREDUMP_BACKEND_FLASH_PARTITION on

[liutingyue]

When enable CONFIG_DEBUG_COREDUMP and use flash partition as backend, the flash backend output call partition_open() and take semaphore .

If CONFIG_ASSERT enabled，the assertion log is shown as follows:

ASSERTION FAIL [((arch_is_in_isr() == 0) || ((timeout).ticks == (((k_timeout_t) {0})).ticks))] @ WEST_TOPDIR/zephyr/kernel/sem.c:136

/**
 * @brief Open the flash partition.
 *
 * @return Same as flash_area_open().
 */
static int partition_open(void)
{
	int ret;

	(void)k_sem_take(&flash_sem, FLASH_BACKEND_SEM_TIMEOUT);

	ret = flash_area_open(FLASH_PARTITION_ID, &backend_ctx.flash_area);
	if (ret != 0) {
		LOG_ERR("Error opening flash partition for coredump!");

		backend_ctx.flash_area = NULL;
		k_sem_give(&flash_sem);
	}

	return ret;
}

[github-actions]

Hi @liutingyue! We appreciate you submitting your first issue for our open-source project. 🌟

Even though I'm a bot, I can assure you that the whole community is genuinely grateful for your time and effort. 🤖💙

[github-actions]

This issue has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this issue will automatically be closed in 14 days. Note, that you can always re-open a closed issue at any time.

[jennifervlaar]

I have recently encountered this issue while working with Zephyr, is there an ETA of when the bug will be fixed or a workaround that anyone has found?

[liutingyue]

The official team does not seem to have any plans to fix the problem in a short time. My personal suggestion is that the conditions for determining whether it is in ISR can be processed separately.

[mrkhldn]

There's some discussion of the same issue here, #59116, but I don't think we arrived at a good answer.