[PATCH] nvme/lightnvm: Log using the ctrl named device

Matias Bjørling m at bjorling.me
Mon May 2 01:06:48 PDT 2016


On 05/01/2016 05:25 AM, Sagi Grimberg wrote:
> Align with the rest of the nvme subsystem.
>
> Signed-off-by: Sagi Grimberg <sagi at grimberg.me>
> ---
>   drivers/nvme/host/lightnvm.c | 12 ++++++------
>   1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/nvme/host/lightnvm.c b/drivers/nvme/host/lightnvm.c
> index 9461dd639acd..a03a3e8febdf 100644
> --- a/drivers/nvme/host/lightnvm.c
> +++ b/drivers/nvme/host/lightnvm.c
> @@ -367,7 +367,7 @@ static int nvme_nvm_get_l2p_tbl(struct nvm_dev *nvmdev, u64 slba, u32 nlb,
>   		ret = nvme_submit_sync_cmd(ns->ctrl->admin_q,
>   				(struct nvme_command *)&c, entries, len);
>   		if (ret) {
> -			dev_err(ns->ctrl->dev, "L2P table transfer failed (%d)\n",
> +			dev_err(ns->ctrl->device, "L2P table transfer failed (%d)\n",
>   									ret);
>   			ret = -EIO;
>   			goto out;
> @@ -441,27 +441,27 @@ static int nvme_nvm_get_bb_tbl(struct nvm_dev *nvmdev, struct ppa_addr ppa,
>   	ret = nvme_submit_sync_cmd(ctrl->admin_q, (struct nvme_command *)&c,
>   								bb_tbl, tblsz);
>   	if (ret) {
> -		dev_err(ctrl->dev, "get bad block table failed (%d)\n", ret);
> +		dev_err(ctrl->device, "get bad block table failed (%d)\n", ret);
>   		ret = -EIO;
>   		goto out;
>   	}
>
>   	if (bb_tbl->tblid[0] != 'B' || bb_tbl->tblid[1] != 'B' ||
>   		bb_tbl->tblid[2] != 'L' || bb_tbl->tblid[3] != 'T') {
> -		dev_err(ctrl->dev, "bbt format mismatch\n");
> +		dev_err(ctrl->device, "bbt format mismatch\n");
>   		ret = -EINVAL;
>   		goto out;
>   	}
>
>   	if (le16_to_cpu(bb_tbl->verid) != 1) {
>   		ret = -EINVAL;
> -		dev_err(ctrl->dev, "bbt version not supported\n");
> +		dev_err(ctrl->device, "bbt version not supported\n");
>   		goto out;
>   	}
>
>   	if (le32_to_cpu(bb_tbl->tblks) != nr_src_blks) {
>   		ret = -EINVAL;
> -		dev_err(ctrl->dev, "bbt unsuspected blocks returned (%u!=%u)",
> +		dev_err(ctrl->device, "bbt unsuspected blocks returned (%u!=%u)",
>   				le32_to_cpu(bb_tbl->tblks), nr_src_blks);
>   		goto out;
>   	}
> @@ -494,7 +494,7 @@ static int nvme_nvm_set_bb_tbl(struct nvm_dev *nvmdev, struct nvm_rq *rqd,
>   	ret = nvme_submit_sync_cmd(ns->ctrl->admin_q, (struct nvme_command *)&c,
>   								NULL, 0);
>   	if (ret)
> -		dev_err(ns->ctrl->dev, "set bad block table failed (%d)\n", ret);
> +		dev_err(ns->ctrl->device, "set bad block table failed (%d)\n", ret);
>   	return ret;
>   }
>
>

Thanks Sagi. I have a couple of patches on top of lightnvm.c that 
conflicts. Will it be okay with you if I carry it through my for-next 
branch that I send to Jens?



More information about the Linux-nvme mailing list