Matthew Wilcox <willy@xxxxxxxxxxxxx> wrote: > > This could be page_offset(page), which reads better to me: > > ret = afs_store_data(vnode, &iter, page_offset(page) + f, true); True. It gets converted to folio_pos() in patch #5 - will that do? David -- Linux-cachefs mailing list Linux-cachefs@xxxxxxxxxx https://listman.redhat.com/mailman/listinfo/linux-cachefs