NVMe fixes
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest0 of 10 patches 0 repliesNew
Patches in v1 No applies recorded yet
None of the 10 patches the cover letter announces are in this archive.
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [PATCHv2-4.5 00/10] NVMe fixes
Quoted 8 lines · Keith Busch
Just want to point out a couple issues. Both are easy fixes and will send in a few minutes.
First, admin queues don't have namespaces, so the new namespace state check isn't always correct. I haven't been able to synthesize a test that gets an admin command down this path in the disabled queue condition, but it's clearly not correct.
Quoted 4 lines
The second issue potentially existed all along, but it's a littlie easier to hit now: we need to lock out shutdown from manipulating registers while they're being setup. This was hit by forcing controller reset from the shelll in a loop, then "rmmod nvme" at the same time.
I already applied these for 4.5, just didn't send the ack before I had done some testing. JFYI.
Signature · 1 line
Jens Axboe