On 06/01/2022 08:28, Jason Gunthorpe wrote: > 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? Sorry, i didn't get you. I didn't check whether va is pmem, since only MR registered with PERSISTENCE(only pmem can register this access flag) can reach here. Thanks Zhijian > > Jason