On 04.02.2022 10:28, Roger Pau Monné wrote: > On Wed, Feb 02, 2022 at 04:29:37PM +0100, Jan Beulich wrote: >> On 02.02.2022 16:14, Roger Pau Monné wrote: >>> On Tue, Jan 04, 2022 at 10:41:53AM +0100, Jan Beulich wrote: >>>> Do away with the "pod_target_out_unlock" label. In particular by folding >>>> if()-s, the logic can be expressed with less code (and no goto-s) this >>>> way. >>>> >>>> Limit scope of "p2m", constifying it at the same time. >>> >>> Is this stale? I cannot find any reference to a p2m variable in the >>> chunks below. >> >> Indeed it is, leftover from rebasing over the introduction of >> p2m_pod_get_mem_target() in what is now patch 1. Dropped. > > I'm happy with this change with the commit adjusted: > > Acked-by: Roger Pau Monné <roger....@citrix.com>
Thanks. > Not sure if you can commit this now regardless of patch 1? I think it can be moved ahead; there looks to be only a minor contextual dependency. Jan
