Thread (4 messages) 4 messages, 2 authors, 2018-10-29

Re: [PATCH] lightnvm: pblk: fix chunk close trace event check

From: Matias Bjørling <hidden>
Date: 2018-10-26 18:49:36
Also in: lkml

On 10/26/2018 10:12 AM, Hans Holmberg wrote:
quoted hunk ↗ jump to hunk
The check for chunk closes suffers from an off-by-one issue, leading
to chunk close events not being traced. Fix this.

Fixes: 4c44abf43d00 ("lightnvm: pblk: add trace events for chunk states")

Signed-off-by: Hans Holmberg <redacted>
---

  drivers/lightnvm/pblk-core.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/lightnvm/pblk-core.c b/drivers/lightnvm/pblk-core.c
index 6944aac43b01..6581c35f51ee 100644
--- a/drivers/lightnvm/pblk-core.c
+++ b/drivers/lightnvm/pblk-core.c
@@ -531,7 +531,7 @@ void pblk_check_chunk_state_update(struct pblk *pblk, struct nvm_rq *rqd)
  		if (caddr == 0)
  			trace_pblk_chunk_state(pblk_disk_name(pblk),
  							ppa, NVM_CHK_ST_OPEN);
-		else if (caddr == chunk->cnlb)
+		else if (caddr == (chunk->cnlb - 1))
  			trace_pblk_chunk_state(pblk_disk_name(pblk),
  							ppa, NVM_CHK_ST_CLOSED);
  	}
Thanks Hans, Applied for 4.21/5.1. Note that you're sending from your 
owntronix email account. Checkpatch complains that sender and author 
doesn't match.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help