On Tue, 1 Feb 2022, Bart Van Assche wrote:
> On 2/1/22 10:33, Mikulas Patocka wrote:
> > +static inline blk_status_t nvme_setup_read_token(struct nvme_ns *ns, struct
> > request *req)
> > +{
> > + struct bio *bio = req->bio;
> > + struct nvme_copy_token *token =
> > page_to_virt(bio->bi_io_vec[0].bv_page) + bio->bi_io_vec[0].bv_offset;
>
> Hmm ... shouldn't this function use bvec_kmap_local() instead of
> page_to_virt()?
>
> Thanks,
>
> Bart.
.bv_page is allocated only in blkdev_issue_copy with alloc_page. So,
page_to_virt works.
But you are right that bvec_kmap_local may be nicer.
Mikulas
