[PR #71] [CLOSED] Yuval/lbm1 5419 add uncorr #128

Closed
opened 2026-01-31 21:29:53 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/qemu/qemu/pull/71
Author: @yuval-lb
Created: 8/30/2018
Status: Closed

Base: masterHead: yuval/LBM1-5419_add_uncorr


📝 Commits (2)

  • a0185b1 add uncorrectalbe bitmap to NVMeNamespace
  • 5e52d8f nvme.c uncorrectable logic

📊 Changes

2 files changed (+49 additions, -0 deletions)

View changed files

📝 hw/block/nvme.c (+48 -0)
📝 hw/block/nvme.h (+1 -0)

📄 Description

We want to add uncorrectable feature to qemu-nvme.

  1. create bitmap to follow which block will raise NVME_UNRECOVERED_READ
  2. add NVME_CMD_WRITE_UNCOR - set range of blocks to raise "unrecovered" error
  3. add in nvme_rw - if its read check if the block is uncorrectable and raise error
    - if it write - write reset the block to be correctable.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/qemu/qemu/pull/71 **Author:** [@yuval-lb](https://github.com/yuval-lb) **Created:** 8/30/2018 **Status:** ❌ Closed **Base:** `master` ← **Head:** `yuval/LBM1-5419_add_uncorr` --- ### 📝 Commits (2) - [`a0185b1`](https://github.com/qemu/qemu/commit/a0185b197f526dc7326e2e6edd4e510189be58c0) add uncorrectalbe bitmap to NVMeNamespace - [`5e52d8f`](https://github.com/qemu/qemu/commit/5e52d8f7e8a85afa78cd87c3a22be772fc4d1d97) nvme.c uncorrectable logic ### 📊 Changes **2 files changed** (+49 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `hw/block/nvme.c` (+48 -0) 📝 `hw/block/nvme.h` (+1 -0) </details> ### 📄 Description We want to add uncorrectable feature to qemu-nvme. 1. create bitmap to follow which block will raise NVME_UNRECOVERED_READ 2. add NVME_CMD_WRITE_UNCOR - set range of blocks to raise "unrecovered" error 3. add in nvme_rw - if its read check if the block is uncorrectable and raise error - if it write - write reset the block to be correctable. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 21:29:53 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/qemu#128