Thread (106 messages) 106 messages, 6 authors, 2022-01-08

Re: [PATCH 07/48] iov_iter: Convert iter_xarray to use folios

From: Matthew Wilcox <willy@infradead.org>
Date: 2021-12-23 14:31:31
Also in: linux-fsdevel

On Wed, Dec 22, 2021 at 10:57:28PM -0800, Christoph Hellwig wrote:
quoted
+		size_t offset = offset_in_folio(folio, start + __off);	\
+		if (xas_retry(&xas, folio))			\
 			continue;				\
+		if (WARN_ON(xa_is_value(folio)))		\
 			break;					\
+		if (WARN_ON(folio_test_hugetlb(folio)))		\
 			break;					\
+		while (offset < folio_size(folio)) {		\
Nit: I'd be tempted to use a for loop on offset here.
Dave, this is your code originally ... any opinions?
Otherwise looks good:

Reviewed-by: Christoph Hellwig <hch@lst.de>
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help