hi, > Hi, thanks for review. > > On Thu, Nov 25, 2010 at 01:58:04AM +0000, YAMAMOTO Takashi wrote: >> hi, >> >> - what's VM_PHYSSEG_OP_PG? > > It's to lookup vm_physseg by "struct vm_page *", relying on that > "struct vm_page *[]" is allocated linearly. It'll be used to remove > vm_page::phys_addr as we talked some time ago.
i'm not sure if commiting this unused uncommented code now helps it. some try-and-benchmark cycles might be necessary given that vm_page <-> paddr conversion could be performace critical. YAMAMOTO Takashi