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

Re: [PATCH v2 00/14] x86/mm: large parts of P2M code and struct p2m_domain are HVM-only


  • To: Jan Beulich <JBeulich@xxxxxxxx>
  • From: George Dunlap <George.Dunlap@xxxxxxxxxx>
  • Date: Fri, 1 Apr 2022 12:47:20 +0000
  • Accept-language: en-US
  • 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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=I3CEogWUl/Iv8X8F0n37ujgeDvEvj/P33m3aUItDlyc=; b=O8wsQttnjxEC9jItiALWdl8TgMKgKPUEOrIFa/jR2jhhsDbr5O4ON85Ax2CiokSMREgCjlHvhJ9GeYFofa8FzQOYHW8EyJR8q35pn7gcZE4HFmagTkjcXfC83ScZaiuQRfksSffYdqPycTYbGIkDtUbwm0EuMsmLA8a0mNsM/1ZMFfdfzmRSymT/ZSLWu89PcKmsKia1YJHYAqoC8rHGj70cU5X4s+lJNdFb6zBqw2eWi3ksfa+awsymTpmIk1Ok5dJobBdsTCf52CSg9TYlcw/jvDpuF6eLed+ZHTQcleEZ5apSCx7/II95pv69N8lGDsUifuy+5cgEo0J1te9gmQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=MSfQKTGuZAFbHUANbwx1UCtHDDEGNFxuhDZO4FTw84T3IyEmXkxODEuNm2lx1JwOR+6ts/3uH9qRNad2dK6A44AjUo5Cpx/8LmKbujwWLsywGSSLnq0r9dqOdOEowiL8xldez4+2SWNvPp82oSYIIOLIXXzzZxAVM2uPkTTzfAGkItW/u3PbUwxfyqKf2lxBd5uyiQnSpGN8azTqJJyPhpJycl85pDsdoKVEgklbasDx2tE30CbZ0YYoPSIpBXbhRhMG79oHP9E6XjpCZbMiHM4r8WjGALtdC4ZbI25GIZLyvWFZNzyZzkeEFsXyTYkZtKL/22FYFObVtYoCupiNhw==
  • Authentication-results: esa6.hc3370-68.iphmx.com; dkim=hardfail (body hash did not verify [final]) header.i=@citrix.onmicrosoft.com
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, "Andrew Cooper" <Andrew.Cooper3@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Roger Pau Monne <roger.pau@xxxxxxxxxx>
  • Delivery-date: Fri, 01 Apr 2022 12:47:32 +0000
  • Ironport-data: A9a23:Emb4Oq2WFfh2+6ZZofbD5VFxkn2cJEfYwER7XKvMYbSIYQITYwd3j TtIBzjCf73ffDO2KOnCW/2w909T7MDUxtU3HQI9+ythEyNEoJLMXInAIhyhMS3Nc5GZEB87v 8wTOoHJIpk9EiHR+R3xbrGx/SUni/rZG+T2YAKo1lidYCc9IMt2oU4zy4bV+7JVvOVVIz9hm Pv5rpGBZwb6hzMoajtF5v7d8Uhksfqrs2NA4FFmPv4S5w6CmyEZAqxEKPDqJRMUYGX18s1W5 Qrn5Ovklo8M1051UrtJqp6iLgtSBOS60TGm0hK6YYD76vR5jnF0g/5T2MY0Mx8N0W3Vx4kpk r2hiLTrIesXFvyU8Agie0Ew/xFWZcWqL5eefBBTGeTKp6H3WyOEL8dGVSnaDqVBkgpDOklc9 ORwFdw4Rkvra9RaYl6MYrIEaswLdKEHNW6E051q5Wmx4f0OGfgvT0hWjDPxMfhZas1mRJ7ji 8QlhTVHUhvPQi1UJkwuDIsjrtaKpDrvcDtVgQfAzUY3yzC7IA1Z1bHsNJzee8CQRNUTlUGdz o7E1z2nWFdAboXZkGfbtCL37gPMtXqTtIY6O7C/7PFnxnGOwGgaEDUdVEehoOn/gUm7Mz5aA xJLpHR08PJpnKCtZtanXBinnHKJhAJCBOZ7Eao8rzGt4bWBtm51AUBbF2UcObTKrvQeWjgCx lKP2dTzClRHoLCTDH6Q6LqQhTezIjQOa38PYzceSgkI6MWlp5s85jrtZNt+FK++jvXuBCr9h TuNqUADa6471JBRkf/hpBae3mzq9sOhohMJChv/YG2qzzNieJWcY5W122n5z7VQL4CUZwzU1 JQboPS24OcLBJCLsSWCRuQRAb2kj8q43C3gbU1HRMd4qWn0k5K3VcUJuWwleh81WioRUWWxC HI/rz+983O60JGCSaZsK7y8BM0xpUQLPYS0D6uEBjaij3UYSeNmwM2MTRPIt4wOuBJ1+U3aB Xt9WZzzZZr9If47pAdav89HjdcWKtkWnAs/v6zTwRW9yqa5b3WIU7oDO1bmRrlnsPPb/V+Pq 4cOapbiJ/BjvAvWOHa/HWk7dw1iEJTGLcqu95w/mhCrfGKK513N+9eOmOh8KuSJboxel/vS/ 2HVZ6Or4AGXuJEzEi3TMioLQOq2Bf5X9CtnVQRxbQfA8yVyOu6HsfZAH6bbiJF6rYSPO9YvF KJbEyhBa9wSIgn6F8M1Nsel9tU4JEX23Wpj/UONOVACQnKpfCSUkvfMdQrz7igeSC2xsMo1u bq70Q3HB5EEQmxf4Az+ManHI4+Z1ZTFpN9PYg==
  • Ironport-hdrordr: A9a23:2ZK+JqPb1g8Gj8BcT3D155DYdb4zR+YMi2TDiHofdfUFSKClfp 6V8cjzjSWE9Qr4WBkb6LW90dq7MA/hHP9OkMUs1NKZMjUO11HYVb2KgbGSoAEIXheOjNK1tp 0QPJSWaueAdWSS5PySiGLTcrYdKZu8gduVbI/lvi5QpGpRGsVdBnJCe2Cm+zpNNW577PQCZf ihz/sCgwDlVWUcb8y9CHVAdfPEvcf3mJXvZgNDLwI76SGV5AnYpoLSIly95FMzQjlPybAt/S zuiAri/JiutPm911v1y3LT1ZJLg9Hso+EzRfBky/JlagkEuDzYJriJaIfy+QzdZ9vfrGrCpe O84CvI+f4DrE85MFvF5ycFkDOQrwrGo0WSt2Nwx0GT+fARDQhKe/ZplMZXdADU5FEnu8w52K VX33iBv54SFh/Ymj/hjuK4HC2DBiKP0AsfeMMo/jViuLElGfdsRE0kjTdoOYZFGDi/5JEsEe FoAs2Z7PFKcUmCZ3ScumV02tSjUnk6Ax/DGyE5y4Go+ikTmGo8w1oTxcQZkHtF/JUhS4Nc7+ CBNqhzjrlBQsIfcKo4DuYcRsm8DHDLXHv3QSivCEWiELtCN2PGqpbx7rlw7Oa2eIYQxJ93g5 jFWEMwjx9FR6svM7z44HRmyGG+fIzmZ0Wf9ih33ekNhoHB
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHYKM3WxRAA+J80qEyImVUxoLvSJazbO6OA
  • Thread-topic: [PATCH v2 00/14] x86/mm: large parts of P2M code and struct p2m_domain are HVM-only


> On Feb 23, 2022, at 3:55 PM, Jan Beulich <JBeulich@xxxxxxxx> wrote:
> 
> The primary goal of this series is to leave p2m.c with, as its leading
> comment suggests, just code for "physical-to-machine mappings for
> automatically-translated domains". This requires splitting a few
> functions, with their non-HVM parts moved elsewhere.
> 
> There aren't many changes in v2, mostly from re-basing. See individual
> patches for details.
> 
> 01: x86/P2M: rename p2m_remove_page()
> 02: x86/P2M: introduce p2m_{add,remove}_page()
> 03: x86/mm: move guest_physmap_{add,remove}_page()
> 04: x86/mm: split set_identity_p2m_entry() into PV and HVM parts
> 05: x86/P2M: p2m_{alloc,free}_ptp() and p2m_alloc_table() are HVM-only
> 06: x86/P2M: PoD, altp2m, and nested-p2m are HVM-only
> 07: x86/P2M: split out init/teardown functions
> 08: x86/P2M: p2m_get_page_from_gfn() is HVM-only
> 09: x86/P2M: derive a HVM-only variant from __get_gfn_type_access()
> 10: x86/p2m: re-arrange {,__}put_gfn()
> 11: shr_pages field is MEM_SHARING-only
> 12: paged_pages field is MEM_PAGING-only
> 13: x86/P2M: p2m.c is HVM-only
> 14: x86/P2M: the majority for struct p2m_domain's fields are HVM-only

OK, I think every patch has an R-b from me on it now — let me know if I missed 
anything.

 -George

Attachment: signature.asc
Description: Message signed with OpenPGP


 


Rackspace

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