On Thu, Jan 20, 2022 at 09:10:20PM +0800, alexs@xxxxxxxxxx wrote: > From: Alex Shi <alexs@xxxxxxxxxx> > > This function could be full replaced by folio_is_file_lru, so no reason > to keep a duplicate function. This is not a helpful way to do this kind of replacement. Instead of choosing a function to remove and doing a blind replacement, choose a call site and convert the whole calling function to use folios. Once you've removed all callers, you can remove the wrapper function. Also, a number of changes here will conflict with patches I've already posted. Try doing change_pte_range() in mprotect.c to get a feel for how to convert a function entirely to folios.