On Tue, Jan 30, 2024 at 1:40 PM Huang, Ying <ying.huang@xxxxxxxxx> wrote: > > Kairui Song <ryncsn@xxxxxxxxx> writes: > > > From: Kairui Song <kasong@xxxxxxxxxxx> > > > > No feature change, simply move the routine to a standalone function to > > be re-used later. The error path handling is copied from the "out_page" > > label, to make the code change minimized for easier reviewing. > > The error processing for mem_cgroup_swapin_charge_folio() failure is > changed a little. That looks OK for me. But you need to make it > explicit in change log. Especially, it's not "no feature change" > strictly. Yes, you are correct, I thought it was hardly observable for users, so ignored that, let me fix the commit message then. Thanks for the suggestion. > > -- > Best Regards, > Huang, Ying