Hi Julien Sorry for so long to respond, just back from a long holiday. 😉
> -----Original Message----- > From: Julien Grall <jul...@xen.org> > Sent: Thursday, July 1, 2021 1:44 AM > To: Penny Zheng <penny.zh...@arm.com>; xen-devel@lists.xenproject.org; > sstabell...@kernel.org; jbeul...@suse.com > Cc: Bertrand Marquis <bertrand.marq...@arm.com>; Wei Chen > <wei.c...@arm.com> > Subject: Re: [PATCH 2/9] xen/arm: introduce PGC_reserved > > Hi Penny, > > On 07/06/2021 03:43, Penny Zheng wrote: > > In order to differentiate pages of static memory, from those allocated > > from heap, this patch introduces a new page flag PGC_reserved to tell. > > I would prefer if this patch is folded in the patch first using it. This will > be > easier to understand how this flag will be used. > > Cheers, Sure, I'll re-organize this commit. > > > > > Signed-off-by: Penny Zheng <penny.zh...@arm.com> > > --- > > changes v2: > > - remove unused reserved field in struct page_info > > - remove unused helper page_get_reserved_owner and > > page_set_reserved_owner > > --- > > xen/include/asm-arm/mm.h | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/xen/include/asm-arm/mm.h b/xen/include/asm-arm/mm.h > index > > 0b7de3102e..7034fae1b6 100644 > > --- a/xen/include/asm-arm/mm.h > > +++ b/xen/include/asm-arm/mm.h > > @@ -108,6 +108,9 @@ struct page_info > > /* Page is Xen heap? */ > > #define _PGC_xen_heap PG_shift(2) > > #define PGC_xen_heap PG_mask(1, 2) > > + /* Page is reserved */ > > +#define _PGC_reserved PG_shift(3) > > +#define PGC_reserved PG_mask(1, 3) > > /* ... */ > > /* Page is broken? */ > > #define _PGC_broken PG_shift(7) > > > > -- > Julien Grall Cheers -- Penny Zheng
