Re: [Xen-devel] [PATCH v2] x86/hvm: Fix altp2m_vcpu_enable_notify error handling

2017-11-15 Thread Jan Beulich
>>> On 15.11.17 at 14:47, wrote: > The altp2m_vcpu_enable_notify subop handler might skip calling > rcu_unlock_domain() after rcu_lock_current_domain(). Albeit since both > rcu functions are no-ops when run on the current domain, this doesn't > really have repercussions. > > The second change is

[Xen-devel] [PATCH v2] x86/hvm: Fix altp2m_vcpu_enable_notify error handling

2017-11-15 Thread Adrian Pop
The altp2m_vcpu_enable_notify subop handler might skip calling rcu_unlock_domain() after rcu_lock_current_domain(). Albeit since both rcu functions are no-ops when run on the current domain, this doesn't really have repercussions. The second change is adding a missing break that would have potent