[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 04/12] AMD/IOMMU: guest IOMMU support is for HVM only


  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Date: Thu, 29 Apr 2021 17:14:06 +0200
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=x0Bbs3LphpW85oFXqn3E6en/H/xTRIyg7eLQBHAY4fA=; b=gsaKOAPj4nXtHZMSY/OX6x1NbY5S/bDjnKoOaqgQQN+inwf3zR7Fz451VGuV9O3rxLONB91jneOBg5Jdmzehqx/uJuvQ488N+XAoKcL4dliD2xi7nv15LogkO2j9oBl+AD9SEoCBZKB/olGOpEUqcSOs/9Vv/t/CRUpVbP0hzDotSbaJ7OoOprY6HN4E9OdlFVwVCSqWUWxF2SYvLBF9WmqLTKH78RRgeWa+BY99WWelEGnbFhGXwSzbbAbReV9PHe+kdUp8oX9+EdHBjMtnNOa2o3JnPMpAEXBwwcb6ktBY3iKiz6sgULCFnFmJPy5G/zcX8q0qwNWiyeKS6CvQWQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=fRzm7bf0grBwkzaTr8zl0R40iid5h0ika5V8OOlD/zwX8faVNe4sDAFiRdsxrwciDCedC8SXPJ+00L2qQzQdWhqksbGSdQALpP3ez3qLtuaV7ZPee2U3Wteu1tIeVTo6JdZSr2obslxlct1ZQ11/wElhJW8giBD0AEYwVAt2AGI3sASjNM6i1DTdqWc06LCggIIa2Xp5BmcsKAJVcl+535Jh65Rk0rCU55l8pbfoFJrKYFe3bZWHj9TUNet2jvBzCsbFlorZKd7PHBej7nTJFwYu9ZltBb8qyRYaqhUWOdD6rI/mzwrgksjK89e8XTeiGPOe4coW9lyZ3eiOGloboA==
  • Authentication-results: esa1.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, "Andrew Cooper" <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>
  • Delivery-date: Thu, 29 Apr 2021 15:14:22 +0000
  • Ironport-hdrordr: A9a23:yvqVBKMlVRJHasBcT3Pw55DYdL4zR+YMi2QD/3taDTRIb82VkN 2vlvwH1RnyzA0cQm0khMroAse9aFvm39pQ7ZMKNbmvGDPntmyhMZ144eLZrAHIMxbVstRQ3a IIScRDIfXtEFl3itv76gGkE9AmhOKK6rysmP229RdQZCtBApsQiTtRIACdD0FwWU1qBYAhEo Cd+8pAoFObCAkqR+68AWQIWPWGmsbCk4jobQVDKxks7gSPij3A0s+HLzGz2BACXzRThYoz6G StqX2C2oyPkdGejiXd2Wja8ohMlLLaq+drKcSQhqEuW1DRoymyYoAJYczngBkUp6WV5E8ugJ 3wpX4bTrtOwlfwWk3wnhf3wQnn118Vmgzf4HuVm2Hqr8C8ZB9SMbs4uatjfhHU61UtsbhHuc ohtQ/p1Os0fGb9tR/w6NTSWxZhmlDcmwtYrccpg2FCSoxbUbdNrOUkjTNoOa0dFyH34p1PKp gWMOjg4p9tADSnRkGclGxuzNuwZ280DxeLT2MT0/blogR+rTRXyVAVy9cYmWpF3JUhS4Nc7+ CBCahwkqpSJ/VmIp5VNaMke4+aG2bNSRXDPCa7JknmLrgOPzbop4Ts6Ls4yem2cPUzvdUPsa WEdGkdmX85ekroB8HL9oZM6ArxTGK0Wimo4t1C5rBi04eMB4bDAGmmchQDgsGgq/IQDonwQP CoIq9bBPflMC/HBZtJ5QvjQJNfQENuEPE9i5IeYRajs8jLIorluqjwa/DIPofgFj4iRyfRGX 0GcD/vJNhRz0yiV3Pi6SKhHk/FSwjax9ZdAaLa9+8cxMwmLYtXqDUYjly/+4WqJFR5w+gLVX o7BImivrKwpGGw82qNxX5uIABhAkFc56ild3tLoAQNIn7laLprgaTZRUlimF+8YjNvRcLfFw BS435t/7isEpCWzSc+T/WqL3ydlHlWgH6RVZ8Tlumi6K7eC9IFJ6djfJY0ORTAFhRzlwovgn xEchU4SkjWES6rr76kgpwSDOT2bMJ9nw+vHM5RpRvkxAehjPBqYkFecy+lUMaRjwprbSFTnE dN/6gWh6fFpSyiMlIlgOMzMERFbUOeBL4uNnXCWKxk3pTQPC1gR2aDgjKXzzU+YHDj+Ukpim v9FiGMYv3QDl1BundX77by/DpPBxegVnM1Tko/nZx2FGzAtHo26+ONa6ap+0a6a1cJwIgmQX v4SApXBjkr68G81RaTljrHKG4vwY82OPfBSJ45davI53+rIIqUtK0PEvNO5qx5PNT2vuJja5 PHRyalaBfDT8850Q2coXgofBRuoH4/iPXyxVnL6nO70HNXO4uaHH1WA5UgZ/eS4GjvS6zWjN FXjdcpsfCxNWu0QNic0q3TZyNCLBSWgWPedZBelblk+YYJ8J10FN3ndBGN8ldt9hA3Nt31m0 MTW74T2sGLBqZfO+gpPxtE9V8onumVJEQlsgbKEvYzFGtd+0PzDpes2f70srIhDU2KmRvoNX Se+yNb+e3ZXyHr789tN4sAZUBXYlM78nJs4aercJDREhyjc4h4jReHG074VL9WU66eH7oM6j 58/tGThueSMw71whrZszc+AqVA9Q+cMI+PKTPJPe5D6NqhP1uQxoOs/c6olT/yDQKBVH5wv/ wMSWUgKuJZijcji4Ur0i+9DozPy3hV7Wd20HVAjV7i2o+v/WHBO1pJWDep2qlrYQ==
  • Ironport-sdr: +mMeZz95DOHkLV/bWuJ6pKTZnRPfVOYcXl5YBYoxiZeTIbUL/IKvZmwBN1VxAJeWlNhjxy2nPW nJ73s7uRjeOxM65ISs9kmm0VAUMNRYldZJHdLAO/S+Zmh7e5nkbYUXMYDXiRt86ePP28smazPK T+0Rs681ZLxMnDBBqJ+E0izNnjLNrQYQ4Xp0OTfAhxpQQGtgqi/nOdhSxgcz1k1RV69ebvguhA JwqiFYeHnI5mNnMBM3dVZ5eUqbmMdvJRw3O70oGEYaDBB5ws0IDbIPdCGnKd0pPYFp9xgoo7lY HRU=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Mon, Apr 12, 2021 at 04:07:50PM +0200, Jan Beulich wrote:
> Generally all of this is dead code anyway, but there's a caller of
> guest_iommu_add_ppr_log(), and the code itself calls
> p2m_change_type_one(), which is about to become HVM-only. Hence this
> code, short of deleting it altogether, needs to become properly HVM-
> only as well.
> 
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

Reviewed-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>

Thanks, Roger.



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.