>>> On 19.12.17 at 10:21, <jgr...@suse.com> wrote:
> On 19/12/17 09:23, Jan Beulich wrote:
>>>>> On 18.12.17 at 23:22, <boris.ostrov...@oracle.com> wrote:
>>> +void __init arch_xen_balloon_init(struct resource *hostmem_resource)
>>> +{
>>> + struct xen_memory_map memmap;
>>> + int rc;
>>> + unsigned int i, last_guest_ram;
>>> + phys_addr_t max_addr = max_pfn << PAGE_SHIFT;
>>
>> PFN_PHYS() as right now you still have an issue on 32-bit.
>
> Why? PFN_PHYS is defined as:
>
> #define PFN_PHYS(x) ((phys_addr_t)(x) << PAGE_SHIFT)
Well, that's why I suggested its use (just like you did in your
own review). IOW - now I'm confused.
Jan
