On Fri, Jun 9, 2017 at 6:49 PM, Michal Hocko <mhocko@xxxxxxxxxx> wrote: > On Fri 09-06-17 17:51:24, Wei Yang wrote: >> Hi, Michal >> >> I am not that familiar with hotplug and trying to catch up the issue >> and your solution. >> >> One potential issue I found is we don't check the physical boundary >> when add_memory_resource(). >> >> For example, on x86-64, only 64T physical memory is supported currently. >> Looks it is expanded after 5-level pagetable is introduced. While there is >> still some limitations on this. But we don't check the boundary I think. >> >> During the bootup, this is ensured by the max_pfn which is guaranteed to >> be under MAX_ARCH_PFN. I don't see some limitation on this when doing >> hotplug. > > This might be true and I would have to double check but this rework > doesn't change anything in that regards. Or do I miss something? Ah, yes, I believe your patch set don't touch this area. This is just related to hotplug. > -- > Michal Hocko > SUSE Labs -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>
