> -----Original Message----- > Subject: [PATCH v5 1/2] arm/p2m: Rework p2m_init() > > From: Andrew Cooper <andrew.coop...@citrix.com> > > p2m_init() is mostly trivial initialisation, but has two fallible operations > which are on either side of the backpointer trigger for teardown to take > actions. > > p2m_free_vmid() is idempotent with a failed p2m_alloc_vmid(), so rearrange > p2m_init() to perform all trivial setup, then set the backpointer, then > perform all fallible setup. > > This will simplify a future bugfix which needs to add a third fallible > operation. > > No practical change. > > Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com> > Reviewed-by: Julien Grall <jgr...@amazon.com> > Reviewed-by: Bertrand Marquis <bertrand.marq...@arm.com>
Release-acked-by: Henry Wang <henry.w...@arm.com> Kind regards, Henry
