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

Re: [libvirt] [PATCH v6 4/4] xlconfigtest: add tests for numa cell sibling distances

On 11/11/2017 06:42 AM, Wim ten Have wrote:
On Fri, 10 Nov 2017 13:59:25 -0700
Jim Fehlig <jfehlig suse com> wrote:

On 11/10/2017 01:55 PM, Jim Fehlig wrote:
On 11/02/2017 09:47 AM, Wim Ten Have wrote:
From: Wim ten Have <wim ten have oracle com>

Test a bidirectional xen-xl domxml to and from native for vnuma
support administration as brought under this patch series.

Added tests for the libxl_domain_config generator determining
vnuma conversion for XML-2-json and json-2-XML.

I think these should be split into two patches. Essentially patch 4 in V5 plus a
patch 5 with the new ibxl_domain_config generator test.

Signed-off-by: Wim ten Have <wim ten have oracle com>
Changes on v5:
- Added tests for libxl_domain_config generator.
- Introduced empty stubs for xc_physinfo(), xc_sharing_freed_pages(),
    xc_sharing_used_frames to satisfy libxl_get_physinfo() under test
    simulation for libxlxml2domconfigtest.c
   tests/libxlxml2domconfigdata/basic-hvm.json        | 95 +++++++++++++++++++++-
   tests/libxlxml2domconfigdata/basic-hvm.xml         | 66 ++++++++++++++-

And instead of adding vNUMA config to the _basic_ test files,
vnuma-hvm.{json,xml} seems more appropriate. I've split this out into a patch 5
and attached here for completeness.

IMO this series is ready to go in. I'll wait for your ACK on the doc changes I
suggested in patch 1.

Sorry, I meant to say I'll wait for your feedback on my suggested doc changes in
patch 1 before pushing this series.

   Hi Jim, I am good with your suggested changes.  So this message is to;  ACK

Thanks, I've pushed this series now. And thanks for your perseverance! I know it's taken a while to get this work committed, mostly due to my delays.

BTW, I've sent a patch to mention your new features in 3.10.0 news



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