|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-changelog] [xen staging] x86/p2m: use available local variable in guest_physmap_add_entry()
commit d917077b8d0adb78b27d4282a096d98278c52243
Author: Jan Beulich <jbeulich@xxxxxxxx>
AuthorDate: Fri Apr 3 17:17:29 2020 +0200
Commit: Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Fri Apr 3 17:17:29 2020 +0200
x86/p2m: use available local variable in guest_physmap_add_entry()
The domain is being passed in - no need to obtain it from p2m->domain.
Also drop a pointless cast and simplify expressions while touching this
code anyway.
Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
---
xen/arch/x86/mm/p2m.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/xen/arch/x86/mm/p2m.c b/xen/arch/x86/mm/p2m.c
index 51b9e2584d..d6e9a4d279 100644
--- a/xen/arch/x86/mm/p2m.c
+++ b/xen/arch/x86/mm/p2m.c
@@ -902,7 +902,7 @@ guest_physmap_add_entry(struct domain *d, gfn_t gfn, mfn_t
mfn,
if ( p2m_is_shared(ot) )
{
/* Do an unshare to cleanly take care of all corner cases. */
- rc = mem_sharing_unshare_page(p2m->domain, gfn_x(gfn_add(gfn, i)));
+ rc = mem_sharing_unshare_page(d, gfn_x(gfn) + i);
if ( rc )
{
p2m_unlock(p2m);
@@ -919,8 +919,7 @@ guest_physmap_add_entry(struct domain *d, gfn_t gfn, mfn_t
mfn,
* Foreign domains are okay to place an event as they
* won't go to sleep.
*/
- (void)mem_sharing_notify_enomem(p2m->domain,
- gfn_x(gfn_add(gfn, i)), false);
+ mem_sharing_notify_enomem(d, gfn_x(gfn) + i, false);
return rc;
}
omfn = p2m->get_entry(p2m, gfn_add(gfn, i),
--
generated by git-patchbot for /home/xen/git/xen.git#staging
_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/xen-changelog
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |