RE: Re: [PATCH v24 2/4] scsi: ufs: L2P map management for HPB read

From: Daejun Park
Date: Thu Feb 25 2021 - 23:54:52 EST


> > +static int ufshpb_init_mem_wq(void)
> > +{
> > + int ret;
> > + unsigned int pool_size;
> > +
> > + ufshpb_mctx_cache = kmem_cache_create("ufshpb_mctx_cache",
> > + sizeof(struct
> > ufshpb_map_ctx),
> > + 0, 0, NULL);
> > + if (!ufshpb_mctx_cache) {
> > + pr_err("ufshpb: cannot init mctx cache\n");
> > + return -ENOMEM;
> > + }
> > +
> > + pool_size = PAGE_ALIGN(ufshpb_host_map_kbytes * 1024) /
> > PAGE_SIZE;
> > + pr_info("%s:%d ufshpb_host_map_kbytes %u pool_size %u\n",
> > + __func__, __LINE__, ufshpb_host_map_kbytes,
> > pool_size);
> > +
>
> I think print function name is not proper while booting.
> And one HPB is associated with one HBA, if there are two UFS
> controllers, how can I differentiate them?

I will use dev_info instead of pr_info.

Thanks,
Daejun

> Bean
>
>
>