Opened 3 years ago

Closed 2 years ago

Last modified 2 years ago

#16894 closed bug (fixed)

nvme: release_spinlock: lock already released!

Reported by: kallisti5 Owned by: waddlesplash
Priority: normal Milestone: R1/beta4
Component: Drivers/Disk/NVMe Version: R1/beta2
Keywords: Cc:
Blocked By: Blocking: #16366, #17408
Platform: All

Description

nvme_interrupt_handler int_io_interrupt_handler x86_hardware_interrupt

hrev55024, x86_64

Attachments (1)

photo_2021-04-01_15-17-59.jpg (135.9 KB ) - added by kallisti5 3 years ago.

Download all attachments as: .zip

Change History (9)

by kallisti5, 3 years ago

comment:1 by waddlesplash, 3 years ago

This may actually be a race condition or other bug in the ConditionVariable code itself. It would not be the first time.

comment:2 by kallisti5, 3 years ago

just happened again. hrev55098, x86_64

Doing a git pull --rebase on haikuwebkit

comment:3 by kallisti5, 3 years ago

There appears to be some test here:

src/tests/add-ons/kernel/kernelland_emu/condition_variable.cpp

wonder if it would/could sniff out a bug.

comment:4 by kallisti5, 2 years ago

I saw this again, doing the same task lol... git pull --rebase on haikuwebkit.

hrev55643, x86_64

<kernel_x86_64> panic
<kernel_x86_64> ConditionVariable::_NotifyLocked
<kernel_x86_64> ConditionVariable::_Notify
<nvme_disk> nvme_interrupt_handler
<kernel_x86_64> x86_hardware_interrupt
<kernel_x86_64> int_bottom

Closing #17408 as a duplicate

comment:5 by kallisti5, 2 years ago

Blocking: 17408 added

comment:7 by waddlesplash, 2 years ago

Milestone: UnscheduledR1/beta4
Resolution: fixed
Status: newclosed

Fix merged in hrev55671.

comment:8 by waddlesplash, 2 years ago

Blocking: 16366 added
Note: See TracTickets for help on using tickets.