Re: [Xen-devel] [Patch for staging 2/2] x86: remove an ASSERT to avoid crash when destroy a domain.

2017-08-04 Thread Jan Beulich
>>> Yi Sun 08/04/17 11:45 AM >>> >--- a/xen/arch/x86/psr.c >+++ b/xen/arch/x86/psr.c >@@ -1294,9 +1294,7 @@ static void psr_free_cos(struct domain *d) >{ >unsigned int socket, cos; > >- ASSERT(socket_info); >- >-if ( !d->arch.psr_cos_ids ) >+if (

[Xen-devel] [Patch for staging 2/2] x86: remove an ASSERT to avoid crash when destroy a domain.

2017-08-04 Thread Yi Sun
In 'psr_free_cos', we should not use 'ASSERT(socket_info)' because the 'socket_info' is allocated only if 'psr' boot parameter is set. So remove it and use 'psr_alloc_feat_enabled' to check if 'socket_info' is valid or not to avoid crash. Signed-off-by: Yi Sun ---