On 11/6/18 12:31 PM, Aaron Lu wrote: > We have multiple places of freeing a page, most of them doing similar > things and a common function can be used to reduce code duplicate. > > It also avoids bug fixed in one function but left in another. > > Signed-off-by: Aaron Lu <aaron.lu@xxxxxxxxx> Acked-by: Vlastimil Babka <vbabka@xxxxxxx> Thanks! > --- > v3: Vlastimil mentioned the possible performance loss by using > page_ref_sub_and_test(page, 1) for put_page_testzero(page), since > we aren't sure so be safe by keeping page ref decreasing code as > is, only move freeing page part to a common function.