lkml.org 
[lkml]   [2016]   [Jan]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 10/25] lightnvm: put block back to gc list on its reclaim fail
    Date
    From: Wenwei Tao <ww.tao0320@gmail.com>

    We delete a block from the gc list before reclaim it, so
    put it back to the list on its reclaim fail, otherwise
    this block will not get reclaimed and be programmable
    in the future.

    Signed-off-by: Wenwei Tao <ww.tao0320@gmail.com>
    Signed-off-by: Matias Bjørling <m@bjorling.me>
    ---
    drivers/lightnvm/rrpc.c | 18 ++++++++++++++----
    1 file changed, 14 insertions(+), 4 deletions(-)

    diff --git a/drivers/lightnvm/rrpc.c b/drivers/lightnvm/rrpc.c
    index 5df4a69..bee2352 100644
    --- a/drivers/lightnvm/rrpc.c
    +++ b/drivers/lightnvm/rrpc.c
    @@ -381,16 +381,26 @@ static void rrpc_block_gc(struct work_struct *work)
    struct rrpc *rrpc = gcb->rrpc;
    struct rrpc_block *rblk = gcb->rblk;
    struct nvm_dev *dev = rrpc->dev;
    + struct nvm_lun *lun = rblk->parent->lun;
    + struct rrpc_lun *rlun = &rrpc->luns[lun->id - rrpc->lun_offset];

    + mempool_free(gcb, rrpc->gcb_pool);
    pr_debug("nvm: block '%lu' being reclaimed\n", rblk->parent->id);

    if (rrpc_move_valid_pages(rrpc, rblk))
    - goto done;
    + goto put_back;
    +
    + if (nvm_erase_blk(dev, rblk->parent))
    + goto put_back;

    - nvm_erase_blk(dev, rblk->parent);
    rrpc_put_blk(rrpc, rblk);
    -done:
    - mempool_free(gcb, rrpc->gcb_pool);
    +
    + return;
    +
    +put_back:
    + spin_lock(&rlun->lock);
    + list_add_tail(&rblk->prio, &rlun->prio_list);
    + spin_unlock(&rlun->lock);
    }

    /* the block with highest number of invalid pages, will be in the beginning
    --
    2.1.4
    \
     
     \ /
      Last update: 2016-01-12 08:21    [W:4.150 / U:0.652 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site