From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ian Campbell <ian.campbell@citrix.com>
Subject: Re: [PATCH V13 1/7] xen/arm: p2m changes for mem_access
 support
Date: Thu, 12 Mar 2015 17:11:38 +0000
Message-ID: <1426180298.32572.70.camel@citrix.com>
References: <1425677073-13729-1-git-send-email-tklengyel@sec.in.tum.de>
	<1425677073-13729-2-git-send-email-tklengyel@sec.in.tum.de>
	<55018D24.9030603@linaro.org> <1426168561.32572.5.camel@citrix.com>
	<55019E62.2000506@citrix.com> <5501C541.2060804@linaro.org>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <5501C541.2060804@linaro.org>
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: Julien Grall <julien.grall@linaro.org>
Cc: wei.liu2@citrix.com, stefano.stabellini@eu.citrix.com, Andrew Cooper <andrew.cooper3@citrix.com>, tim@xen.org, xen-devel@lists.xen.org, stefano.stabellini@citrix.com, jbeulich@suse.com, keir@xen.org, Tamas K Lengyel <tklengyel@sec.in.tum.de>, ian.jackson@eu.citrix.com
List-Id: xen-devel@lists.xenproject.org

On Thu, 2015-03-12 at 16:56 +0000, Julien Grall wrote:
> On 12/03/15 14:10, Andrew Cooper wrote:
> > On 12/03/15 13:56, Ian Campbell wrote:
> >> On Thu, 2015-03-12 at 12:57 +0000, Julien Grall wrote:
> >>> Hi Tamas,
> >>>
> >>> On 06/03/15 21:24, Tamas K Lengyel wrote:
> >>>> @@ -1090,6 +1098,8 @@ void p2m_teardown(struct domain *d)
> >>>>  
> >>>>      p2m_free_vmid(d);
> >>>>  
> >>>> +    radix_tree_destroy(&p2m->mem_access_settings, NULL);
> >>>> +
> >>>>      spin_unlock(&p2m->lock);
> >>>>  }
> >>>>  
> >>>> @@ -1115,6 +1125,10 @@ int p2m_init(struct domain *d)
> >>>>      p2m->max_mapped_gfn = 0;
> >>>>      p2m->lowest_mapped_gfn = ULONG_MAX;
> >>>>  
> >>>> +    p2m->default_access = p2m_access_rwx;
> >>>> +    p2m->mem_access_enabled = false;
> >>> false is defined for bool not bool_t.
> >>> Please use 0 here.
> >> I'm not convinced, false is false whatever you assign it to.
> > 
> > C specified that false expands to the constant 0, and true to the
> > constant 1.
> > 
> > They are fine for use with any integral type.
> 
> It's still mixing types... which are defined in different headers.

Whatever, if you care so much please send a patch to fix it. I'm not
going to reject Tamas' patches on this basis.

Ian.