[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Xen-devel] [PATCH v3] memory_hotplug: Free pages as higher order
- To: Oscar Salvador <osalvador@xxxxxxxxxxxxxxxxxx>
- From: Arun KS <arunks@xxxxxxxxxxxxxx>
- Date: Thu, 27 Sep 2018 13:39:41 +0530
- Cc: mhocko@xxxxxxxx, vatsa@xxxxxxxxxxxxxx, linux-mm@xxxxxxxxx, getarunks@xxxxxxxxx, kys@xxxxxxxxxxxxx, boris.ostrovsky@xxxxxxxxxx, sthemmin@xxxxxxxxxxxxx, xen-devel@xxxxxxxxxxxxxxxxxxxx, haiyangz@xxxxxxxxxxxxx, yasu.isimatu@xxxxxxxxx, dan.j.williams@xxxxxxxxx, akpm@xxxxxxxxxxxxxxxxxxxx, vbabka@xxxxxxx, osalvador@xxxxxxx, jgross@xxxxxxxx, aaron.lu@xxxxxxxxx, malat@xxxxxxxxxx, gregkh@xxxxxxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, vinmenon@xxxxxxxxxxxxxx, jrdr.linux@xxxxxxxxx, devel@xxxxxxxxxxxxxxxxxxxxxx, iamjoonsoo.kim@xxxxxxx, mgorman@xxxxxxxxxxxxxxxxxxx, kirill.shutemov@xxxxxxxxxxxxxxx
- Delivery-date: Thu, 27 Sep 2018 08:10:01 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 2018-09-27 12:39, Oscar Salvador wrote:
On Thu, Sep 27, 2018 at 12:28:50PM +0530, Arun KS wrote:
+ __free_pages_boot_core(page, order);
Hi,
I am not sure, but if we are going to use that function from the
memory-hotplug code,
we might want to rename that function to something more generic?
The word "boot" suggests that this is only called from the boot stage.
I ll rename it to __free_pages_core()
And what about the prefetch operations?
I saw that you removed them in your previous patch and that had some
benefits [1].
Should we remove them here as well?
Sure. Will update this as well.
Thanks,
Arun
[1] https://patchwork.kernel.org/patch/10613359/
Thanks
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel
|