> -----Original Message----- > From: Jan Beulich <jbeul...@suse.com> > Sent: 09 March 2020 09:59 > To: p...@xen.org > Cc: xen-devel@lists.xenproject.org; Paul Durrant <pdurr...@amazon.com>; > Andrew Cooper > <andrew.coop...@citrix.com>; George Dunlap <george.dun...@citrix.com>; Wei > Liu <w...@xen.org>; Roger Pau > Monné <roger....@citrix.com> > Subject: Re: [PATCH v4 2/6] x86 / p2m: remove page_list check in > p2m_alloc_table... > > On 09.03.2020 10:35, p...@xen.org wrote: > > From: Paul Durrant <pdurr...@amazon.com> > > > > ... and replace with a check of domain_tot_pages(). > > > > The check of page_list prevents the prior allocation of PGC_extra pages, > > whereas what the code is trying to verify is that the toolstack has not > > already RAM for the domain. > > > > Signed-off-by: Paul Durrant <p...@xen.org> > > Reviewed-by: Jan Beulich <jbeul...@suse.com>
Thnaks. > preferably with "remove" in the subject replaced by the "replace" > currently at the beginning of the description. (It also looks as > if there was "allocated" or some such missing near the end of the > description.) > Ok, I'll fix in v5. Paul _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel