From mboxrd@z Thu Jan  1 00:00:00 1970
From: Julien Grall <julien.grall@linaro.org>
Subject: Re: [RFC PATCH] xen/arm: Add 4-level page table for
	stage 2 translation
Date: Mon, 05 May 2014 19:43:38 +0100
Message-ID: <5367DBDA.4030706@linaro.org>
References: <1398860114-10064-1-git-send-email-vijay.kilari@gmail.com>	<5360EF08.7090700@linaro.org>
	<CALicx6u405497bbgbaEzw2S6B6x0qbfAJH_EOAW6RQ+3ukBv-Q@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"; Format="flowed"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <CALicx6u405497bbgbaEzw2S6B6x0qbfAJH_EOAW6RQ+3ukBv-Q@mail.gmail.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Vijay Kilari <vijay.kilari@gmail.com>
Cc: Ian Campbell <Ian.Campbell@citrix.com>, Stefano Stabellini <stefano.stabellini@eu.citrix.com>, Prasun Kapoor <Prasun.Kapoor@caviumnetworks.com>, Vijaya Kumar K <vijaya.kumar@caviumnetworks.com>, xen-devel@lists.xen.org, Stefano Stabellini <stefano.stabellini@citrix.com>
List-Id: xen-devel@lists.xenproject.org

Hi Vijay,

On 03/05/14 09:09, Vijay Kilari wrote:
> On Wed, Apr 30, 2014 at 6:09 PM, Julien Grall <julien.grall@linaro.org> wrote:
>> Hello Vijaya,
>>
>> On 04/30/2014 01:15 PM, vijay.kilari@gmail.com wrote:
>>> -    first = __map_domain_page(p2m->first_level);
>>> -    dump_pt_walk(first, addr);
>>> -    unmap_domain_page(first);
>>> +    lookup = __map_domain_page(p2m->lookup_level);
>>> +    dump_pt_walk(lookup, addr);
>>
>> dump_pt_walk assume the table is the first level. I think you should
>> update dump_pt_walk to handle the zeroeth level.
>
>     It was misguide by dump_hyp_walk(), which is passing zeroeth level
> to dump_pt_walk() for ARM64. Will update it
>
>>>
>>> -#define VADDR_BITS              32
>>> -#define VADDR_MASK              (~0UL)
>>> -
>>
>> Why did you remove VADDR_{BITS,MASK}?
>
>    It is not used at all. at least for ARM

 From the commit message point of view, you are removing theses lines 
without no reason.

IHMO, it should be part of another patch.

Regards,

-- 
Julien Grall