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

Re: [PATCH v8 00/16] acquire_resource size and external IPT monitoring


  • To: Oleksandr <olekstysh@xxxxxxxxx>, Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • Date: Mon, 1 Feb 2021 13:07:04 +0000
  • 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=FMI2Cy8yTr/93ZwxRqgzifnxTLwEw3jggpXvj+TaX8A=; b=K86yXtw5ICke9R1qrHgWO6xzb6FZxI5Hf9r/errSJ5W4k6gFoXOCPkcVqxegfmH70dT6vKfXJsnreiJ3WSV1kDAyAOPyUiN5MaId2fgjV/a1+5bTa4MrJ3co54EHltcYbwqGjqmsRH9G4o9v0drt9u1uyhbDbQS3FgEQnJXMRZBi6LObiCpq5xP+S9jr36prvoR3vMsoxnapiR6wsBL+VBYrLsL99+Gp6nrvezqZUd3WHu2mD151woXX4iy6JkL73njg27o3dvyf6BQV8iXXMjKvUn5uhM+9Z0LbH4XNViVcO7Vfnkx1GmH+CaqCN16XmEFuZedqBGXYqDDhoP7o6g==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=IS4AA/PhMz9kK1BO+rDai1Alu/oBuUa/KjH3FKHxNM72G30/uCbzX5Rz1q08FrkKmgcWUiFvTGxInGiqRtzuphuebv7JBJDbyb5QxYNmLQ0Q8eLrvBkulGke2sUn0j4IW1IiFeTCx+w+CwVnYlYDRBYp5R51VhNFahHRkpEHFTcMkN8R3XxuMiDjmiIpYBRhGn1aH8OS2Ut/PnF48lojfm6I/wHqUU+/wv+QuA5ABLYRONumhtkbDhCo9/Ud9qmDPlj6DayCrVKcKW0l3mPUSezSBuhxMliNHWyG8lI9SskBTc+BTvvib/sPvg4b8/ZNDK3BbLH4EyH0T9709kbWFQ==
  • Authentication-results: esa6.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: Jan Beulich <JBeulich@xxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Ian Jackson <iwj@xxxxxxxxxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, Jun Nakajima <jun.nakajima@xxxxxxxxx>, Kevin Tian <kevin.tian@xxxxxxxxx>, Michał Leszczyński <michal.leszczynski@xxxxxxx>, Tamas K Lengyel <tamas@xxxxxxxxxxxxx>
  • Delivery-date: Mon, 01 Feb 2021 13:07:27 +0000
  • Ironport-sdr: lMwtFkoNGmUbZN9rg2tLiIEAVyJSXXObP4HHjA3+LYjHoYaIdaBM/wliyUi+vQg59ThY9IJhwE 1rjIonmuIKysZs8UUb1FiE2KIYrZtXpRYQuhg8JD2hEBTetpoBBdDX0lMDov5BnLtioAyN4C/x /JQPPH/sZx1Plyi9HmvhS6jqCST8R338Kvrz0GQ/KtV4X0C99mNQpN/RkCaTaAXREA5i82pq7X YB3rCVRiKyFVTEC0OcFfANLU/UY39e10us3nebPNG4H+shx8EwcmgkpEzvo6B+5umXpPeNKPE9 S2I=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 01/02/2021 12:34, Oleksandr wrote:
>
> On 30.01.21 04:58, Andrew Cooper wrote:
>
> Hi Andrew
>
>> Combined series (as they are dependent).  First, the resource size
>> fixes, and
>> then the external IPT monitoring built on top.
>>
>> Posting in full for reference, but several patches are ready to go
>> in.  Those
>> in need of review are patch 6, 8 and 12.
>>
>> See individual patches for changes.  The major work was rebasing over
>> the
>> ARM/IOREQ series which moved a load of code which this series was
>> bugfixing.
>
> Looks like that some of these patches have been already merged. So I
> preliminary tested current staging
> (9dc687f155a57216b83b17f9cde55dd43e06b0cd x86/debug: fix page-overflow
> bug in dbg_rw_guest_mem) on Arm *with* IOREQ enabled.
>
> I didn't notice any regressions with IOREQ on Arm))

Fantastic!

Tamas and I did do extended testing on the subset which got committed,
before it went in, and it is all fixing of corner cases, rather than
fundamentally changing how things worked.


One query I did leave on IRC, and hasn't had an answer.

What is the maximum number of vcpus in an ARM guest?  You moved an
x86-ism "max 128 vcpus" into common code.

~Andrew



 


Rackspace

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