[PATCH V4 1/9] block: allow bvec for zone append get pages

Chaitanya Kulkarni Chaitanya.Kulkarni at wdc.com
Thu Dec 3 21:43:10 EST 2020


On 12/2/20 00:55, Christoph Hellwig wrote:
> On Tue, Dec 01, 2020 at 10:22:19PM -0800, Chaitanya Kulkarni wrote:
>> Remove the bvec check in the bio_iov_iter_get_pages() for
>> REQ_OP_ZONE_APPEND so that we can reuse the code and build iter from
>> bvec.
> We should do the same optimization for bvec pages that the normal path
> does.  That being said using bio_iov_iter_get_pages in nvmet does not
> 	make any sense to me whatsover.
>
Are you referring to the inline bvec ? then yes, I'll add it in next
version.

I did not understand bio_iov_iter_get_pages() comment though.

Reimplementing the bio loop over sg with the use of bio_add_hw_page() seems

repetition of the code which we already have in bio_iov_iter_get_pages().


Can you please explain why bio_iov_iter_get_pages() not the right way ?





More information about the Linux-nvme mailing list