mm/hmm: a simple question regarding devm_request_mem_region()

[Nicolin Chen <nicoleotsuka@gmail.com>]

Hello Jerome,

I started to looking at the mm/hmm code and having a question at the
devm_request_mem_region() call in the hmm_devmem_add() implementation:

>	addr = min((unsigned long)iomem_resource.end,
>		   (1UL << MAX_PHYSMEM_BITS) - 1);

The main question is here as I am a bit confused by this addr. The code
is trying to get an addr from the end of memory space. However, I have
tried on an ARM64 platform where ioport_resource.end is -1, so it takes
"(1UL << MAX_PHYSMEM_BITS) - 1" as the addr base, while this addr is way
beyond the actual main memory size that's available on my board. Is HMM
supposed to get an memory region like this? Would it be possible for you
to give some hint to help me understand it?

>	addr = addr - size + 1UL;
>
>	/*
>	 * FIXME add a new helper to quickly walk resource tree and find free
>	 * range
>	 *
>	 * FIXME what about ioport_resource resource ?
>	 */
>	for (; addr > size && addr >= iomem_resource.start; addr -= size) {
>		ret = region_intersects(addr, size, 0, IORES_DESC_NONE);
>		if (ret != REGION_DISJOINT)
>			continue;
>
>		devmem->resource = devm_request_mem_region(device, addr, size,

Thanks
Nicolin