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

Re: [RFC v2 6/8] tools/arm: Introduce force_assign_without_iommu option to xl.cfg


  • To: Oleksii Moisieiev <Oleksii_Moisieiev@xxxxxxxx>
  • From: Anthony PERARD <anthony.perard@xxxxxxxxxx>
  • Date: Thu, 17 Feb 2022 14:52:44 +0000
  • Authentication-results: esa2.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Nick Rosbrook <rosbrookn@xxxxxxxxxxxx>, Juergen Gross <jgross@xxxxxxxx>, Paul Durrant <paul@xxxxxxx>
  • Delivery-date: Thu, 17 Feb 2022 14:52:56 +0000
  • Ironport-data: A9a23:AdtuIqiryZIHr8LIZcsdAgTkX161JxYKZh0ujC45NGQN5FlHY01je htvUD2FO/yNM2L9eIp+Po7k9U5SvJKAnd81GQU+/n82QyMb9cadCdqndUqhZCn6wu8v7a5EA 2fyTvGacajYm1eF/k/F3oAMKRCQ7InQLlbGILes1htZGEk0GE/NtTo5w7Rj2tQx3oDja++wk YiaT/P3aQfNNwFcagr424rbwP+4lK2v0N+wlgVWicFj5DcypVFMZH4sDfjZw0/DaptVBoaHq 9Prl9lVyI97EyAFUbtJmp6jGqEDryW70QKm0hK6UID66vROS7BbPg/W+5PwZG8O4whlkeydx /1o9r2rVh53L5bWgeoaeTxSEyFcDaFJreqvzXiX6aR/zmXDenrohf5vEFs3LcsT/eMf7WNmr KJCbmpXN1ba2rzwkOnTpupE36zPKOHiOp8fvXdxiynUF/88TbjIQrnQ5M8e1zA17ixLNaiCO 5FFNGUzBPjGSztkNHMKAo1ipbiHl0v6Vn5HhlWJo5NitgA/yyQuieOwYbI5YOeiX8JOlFyfo WHA1234BB8dL9GUjzGC9xqEnfTTlCn2XIYTEryQ9fNwhlCXgGsJB3U+TVKmpdGph0j4XMhQQ 2QW9TAptrMa71GwQ5/2WBjQiHyZuh8RXfJAHut87xuCooLE7gDcCmUaQzppbN09qNRwVTEsz kWOnd7iGXpoqrL9YVWZ+7SPrDKFHC8UNm4PZCIaZQYd6tylq4Y25jrNRNt+FK++jvXuBCr9h TuNqUAWmLE7ncMNkaKh8jj6bymE/8aTCFRvv0OOAzzjvlgRiJOZi5KAzBv69ulZcpeiVECPu 1IhkcieztACAsTY/MCSe9klELas7veDFTTTh19zApUsnwiQF26fkZN4u28nehowWioQUXqwO RKI51sNjHNGFCbyNcdKj5SN59PGJEQKPfDsTbjqY9VHefCdnyfXrXg1NSZ8M40A+XXAcJ3T2 7/HIK5A7l5AUMyLKQZaoc9HjtfHIQhkmAvuqWjTlUjP7FZnTCf9pU05GFWPdPsly6iPvR/Y9 d1SX+PTlUkCCLahPHWKqtFORbzvEZTdLcqpwyCwXrTeSjeK5Ul7U6OBqV/fU9cNc1tpehfgo SjmBx4wJKvXjnzbMwSaAk2Pm5u0NauTWUkTZHR2VX7xgiBLSd/2sM83KstmFZF6pbcL5aMlE JE4lzCoX60npsLvoG9GM/EQbeVKKXyWuO55F3H0OmdhIc87FlehFx2NVlKHyRTixxGf7aMWy 4BMHCuCKXbabwg9XsvQdty1yFa94SoUlO5oBhOaKdhPYkT8toNtLnWp3PMwJsgNLzTFxyebi FnKUUtJ+7GVrt9n6sTNiICFs5ytT7l0EH1FEjSJ9r2xLyTboDaumNcSTOaScDnBf2ro46H+N /5NxvTxPaRfzlZHuoZxCZhxyqc664e9rrNW1F08Tn7Kc06qGvVrJXzfhZtDsahEx7l4vwqqW x3QpokGaOvRYM68SQwfPgsoaOiHxMo4oDiK4KRnOlj+6Q924KGDDRdYMS6ThXEPN7ByKo4kn 7ss4ZZE9wylhxM2Gd+alSQIpX+UJ3kNXqh75JEXBIjn1lgixl1YOMGODyb35NeEaslWM1lsK TiR3fKQi7NZz0vEUnwyCXmSgrYN2cVQ4EhHnA0YOlCEutvZnftmjhRe/AM+QhlR0hgagfl4P XJmNhEtKKiDl9uyaBOvg4x499l9OSCk
  • Ironport-hdrordr: A9a23:GQrDDaDsgaHY57nlHegjsceALOsnbusQ8zAXPh9KJCC9I/bzqy nxpp8mPEfP+U4ssQIb6Ku90ci7MDjhHPtOjbX5Uo3SODUO1FHIEGgm1/qa/9SCIVy1ygc+79 YGT0EWMrSZYjZHZITBkW+F+r0bsbq6GdWT9ILjJgBWPGNXgs9bjztRO0K+KAlbVQNGDZ02GN 63/cxcvQetfnwRc4CSGmQFd/KrnayGqLvWJTo9QzI34giHij2lrJTgFQKD4xsYWzRThZ8/7G n+lRDj7KnLiYD19vac7R6c031loqqg9jJxPr3OtiHTEESvtu+cXvUhZ1RFhkFxnAjg0idvrD CGmWZbAy060QKtQojym2qh5+Co6kdT11byjVCfmnftusr/WXYzDNdAn5tQdl/D51Mnp8wU6t M844u1jesiMfr7plWL2zEIbWAbqmOk5X451eIDhX1WVoUTLLdXsIwE5UtQVJMNBjjz5owrGP RnSJi03ocfTXqKK3TC+mV/yt2lWXo+Wh+AX0gZo8SQlzxbhmpwwUcUzNEW2n0A6JU+QZ9Z4P msCNUgqJheCssNKa5tDuYIRsW6TmTLXBLXKWqXZU/qEakWUki93qIfII9Flt1CVKZ4sqfaqa 6xI2+w71RCBH4GIff+raF2zg==
  • Ironport-sdr: lBTkOqFGFD+ObEvNJy0UneyyktmIazivQ5/YSeDK1KA+Eqy7qShmT/ZtoAQiaHltaY2tY7Wv8p XjO0RGQBhbuhbva/h5lp0DvvYLtW9y06DVz+FS8qRrPX91UzOoEPipOf2U+KfkKMGibor0H1KM NcRa23N4EwaE1cEF/GjhCmcQ0prd4KQjUl9H2wGqYIMb7keVWUpPuDDNfDWcLTIpctsJmIQcjD ocMQ9E2qlbDEoVG4NvWXnxLRpBaluiJ900Utr9IJTNrTFgN/Lt6yaEWxLHPEMsbyjdcWQ0eTHX NTl//i4RIL3Ato7S6q59jsTZ
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Tue, Feb 08, 2022 at 06:00:12PM +0000, Oleksii Moisieiev wrote:
> If set, Xen is allowed to assign the devices even if they are not under
> IOMMU.
> Can be confugired from dom.cfg in the following format:
> force_assign_without_iommu = 1
> 
> This parameter has the same purpose as xen,force-assign-without-iommu
> property in dom0less archtecture.
> 
> Signed-off-by: Oleksii Moisieiev <oleksii_moisieiev@xxxxxxxx>
> ---
> diff --git a/tools/libs/light/libxl_types.idl 
> b/tools/libs/light/libxl_types.idl
> index 2a42da2f7d..1080966c33 100644
> --- a/tools/libs/light/libxl_types.idl
> +++ b/tools/libs/light/libxl_types.idl
> @@ -564,6 +564,7 @@ libxl_domain_build_info = Struct("domain_build_info",[
>      ("apic",             libxl_defbool),
>      ("dm_restrict",      libxl_defbool),
>      ("tee",              libxl_tee_type),
> +    ("force_assign_without_iommu", libxl_defbool),

As you are making changes to libxl's API, could you add a LIBXL_HAVE_*
macro in "tools/include/libxl.h", they are plenty of example there about
adding new fields in "libxl_domain_build_info".

Thanks,

-- 
Anthony PERARD



 


Rackspace

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