[PATCH 1/6] x86/p2m: tidy p2m_add_foreign() a little

2020-12-15 Thread Jan Beulich
Drop a bogus ASSERT() - we don't typically assert incoming domain pointers to be non-NULL, and there's no particular reason to do so here. Replace the open-coded DOMID_SELF check by use of rcu_lock_remote_domain_by_id(), at the same time covering the request being made with the current domain's ac

Re: [PATCH 1/6] x86/p2m: tidy p2m_add_foreign() a little

2020-12-17 Thread Andrew Cooper
On 15/12/2020 16:25, Jan Beulich wrote: > Drop a bogus ASSERT() - we don't typically assert incoming domain > pointers to be non-NULL, and there's no particular reason to do so here. > > Replace the open-coded DOMID_SELF check by use of > rcu_lock_remote_domain_by_id(), at the same time covering th

Re: [PATCH 1/6] x86/p2m: tidy p2m_add_foreign() a little

2020-12-18 Thread Jan Beulich
On 17.12.2020 20:03, Andrew Cooper wrote: > On 15/12/2020 16:25, Jan Beulich wrote: >> Drop a bogus ASSERT() - we don't typically assert incoming domain >> pointers to be non-NULL, and there's no particular reason to do so here. >> >> Replace the open-coded DOMID_SELF check by use of >> rcu_lock_re