On Tue, Apr 23, 2024 at 04:32:14PM +0200, Jan Beulich wrote: > Drop the inner scope that was left from earlier if/else removal. Take > the opportunity and make the paging_unlock() invocation common to > success and error paths, though.
TBH I'm not sure I prefer the fact to continue function execution after an error is found, I specially dislike that you have to add a !rc check to the nestedhvm conditional block, and because anything that we further add to the function would also need a !rc check. > > Signed-off-by: Jan Beulich <jbeul...@suse.com> Acked-by: Roger Pau Monné <roger....@citrix.com> Albeit I do prefer the extra call to paging_unlock() and early return from the function in case of error. Thanks, Roger.
