On Tue, Dec 28, 2021 at 04:07:15PM +0800, Li Zhijian wrote: > + while (length > 0) { > + va = (u8 *)(uintptr_t)buf->addr + offset; > + bytes = buf->size - offset; > + > + if (bytes > length) > + bytes = length; > + > + arch_wb_cache_pmem(va, bytes); So why did we need to check that the va was pmem to call this? Jason