Hi jan > -----Original Message----- > From: Jan Beulich <jbeul...@suse.com> > Sent: Tuesday, August 9, 2022 3:59 PM > To: Penny Zheng <penny.zh...@arm.com> > Cc: Wei Chen <wei.c...@arm.com>; Andrew Cooper > <andrew.coop...@citrix.com>; George Dunlap <george.dun...@citrix.com>; > Julien Grall <jul...@xen.org>; Stefano Stabellini <sstabell...@kernel.org>; > Wei Liu <w...@xen.org>; xen-devel@lists.xenproject.org > Subject: Re: [PATCH v9 8/8] xen: retrieve reserved pages on > populate_physmap > > On 09.08.2022 09:53, Penny Zheng wrote: > >> -----Original Message----- > >> From: Xen-devel <xen-devel-boun...@lists.xenproject.org> On Behalf Of > >> Jan Beulich > >> Sent: Monday, July 25, 2022 11:44 PM > >> > >> On 20.07.2022 07:46, Penny Zheng wrote: > >>> When a static domain populates memory through populate_physmap at > >>> runtime, it shall retrieve reserved pages from resv_page_list to > >>> make sure that guest RAM is still restricted in statically > >>> configured memory > >> regions. > >>> This commit also introduces a new helper acquire_reserved_page to > >>> make > >> it work. > >>> > >>> Signed-off-by: Penny Zheng <penny.zh...@arm.com> > >>> --- > >>> v9 changes: > >>> - Use ASSERT_ALLOC_CONTEXT() in acquire_reserved_page > >>> - Add free_staticmem_pages to undo prepare_staticmem_pages when > >>> assign_domstatic_pages fails > >> > >> May I suggest to re-consider naming of the various functions? Undoing > >> what "prepare" did by "free" is, well, counterintuitive. > >> > > > > How about change the name "prepare_staticmem_pages" to > "allocate_staticmem_pages"? > > Perhaps - if what the function does really resembles allocation in some way. > So far I wasn't really certain in that regard, and hence I was wondering > whether "prepare" doesn't better describe what it does, but then its inverse > also doesn't really "free" anything. >
Hmmmm, “prepare” with “destroy” in its inverse? Do you have any suggestion in mind? > Jan