[PATCH 01/23] block: factor out a bvec_set_page helper

David Howells dhowells at redhat.com
Mon Jan 30 02:33:36 PST 2023


Christoph Hellwig <hch at lst.de> wrote:

> +static inline void bvec_set_page(struct bio_vec *bv, struct page *page,
> +		unsigned int len, unsigned int offset)

Could you swap len and offset around?  It reads better offset first.  You move
offset into the page and then do something with len bytes.

David




More information about the Linux-nvme mailing list