Page MenuHomeFreeBSD

Partially revert r248770.
ClosedPublic

Authored by dchagin on Apr 1 2021, 7:37 PM.

Details

Summary

nvme_ns_bio_process() is on the path where sleep is prohibited as
g_io_shedule_down() calls THREAD_NO_SLEEPNG() before geom->start().

Diff Detail

Repository
R10 FreeBSD src repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

I've never seen this lead to deadlock, but I'm also only ever using nvme with direct dispatch...

This revision is now accepted and ready to land.Apr 1 2021, 8:38 PM
In D29539#662228, @imp wrote:

I've never seen this lead to deadlock, but I'm also only ever using nvme with direct dispatch...

10x, seems that I'm first who use nvme under geom o_O

This revision was automatically updated to reflect the committed changes.