On 20/12/2018 16:33, Paul Durrant wrote: > This patch adds domain and vcpu init hooks for viridian features. The init > hooks do not yet do anything; they will be added to by subsequent patches. > > Signed-off-by: Paul Durrant <[email protected]>
Please can we start by fixing the current, broken, initialisation and destruction logic ? To get rid of DOMCTL_setmaxvcpus, we need the *_destroy() logic to be fully idempotent. Also, viridian_domain_deinit() should not call into viridian_vcpu_deinit(), and viridian_vcpu_deinit() shouldn't be faking up a write to the assist page. AFAICT, the deinit path is all entirely pointless at the moment and can be deleted. ~Andrew _______________________________________________ Xen-devel mailing list [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
