Assertion failed (driver->state == DRIVER_STARTING) at file 'devman.c', line 521

[Jakub Jermář]

After applying the attached patch on top of mainline,784, devamn hit the following assertion:

void set_driver_phone(driver_t *driver, sysarg_t phone)
{
        fibril_mutex_lock(&driver->driver_mutex);
        assert(driver->state == DRIVER_STARTING);
        driver->phone = phone;
        fibril_mutex_unlock(&driver->driver_mutex);
}

I am also attaching a screenshot that shows what devman had been doing before it hit the assertion.

[Vojtech Horky]

I am not sure whether this is not completely different issue. But this assertion is hit when the driver specifies wrong name. I encountered this assertion when I copied skeleton of an existing driver and forgot to change driver_t.name to reflect the new name.

Completely off topic note: such assertion could be very dangerous because malicious client can "murder" devman by simply sending a register message with name of already running driver.

[Jakub Jermář]

This ticket is no longer relevant as there have been major changes to devman in the meantime and the set_driver_phone() along with the assert has been removed in mainline,924.