Re: [PATCH 00/34] Make CONFIG_HVM work

[Wei Liu <wei.liu2@citrix.com>]

On Mon, Aug 20, 2018 at 10:13:07AM +0100, Andrew Cooper wrote:
> On 17/08/2018 16:12, Wei Liu wrote:
> > This series goes through x86 code to make CONFIG_HVM work.
> >
> > With this series, it is possible to build Xen with PV support only.
> >
> > Running `xl info` on a host with PV only Xen:
> >
> > root@lcy2-dt108:~# xl info
> > host                   : lcy2-dt108
> > release                : 4.17.0-0.bpo.1-amd64
> > version                : #1 SMP Debian 4.17.8-1~bpo9+1 (2018-07-23)
> > machine                : x86_64
> > nr_cpus                : 8
> > max_cpu_id             : 7
> > nr_nodes               : 1
> > cores_per_socket       : 4
> > threads_per_core       : 2
> > cpu_mhz                : 3504.057
> > hw_caps                :
> > bfebfbff:77faf3ff:2c100800:00000121:0000000f:009c6fbf:00000000:00000100
> > virt_caps              : hvm_directio
> > total_memory           : 32589
> > free_memory            : 4158
> > sharing_freed_memory   : 0
> > sharing_used_memory    : 0
> > outstanding_claims     : 0
> > free_cpus              : 0
> > xen_major              : 4
> > xen_minor              : 12
> > xen_extra              : -unstable
> > xen_version            : 4.12-unstable
> > xen_caps               : xen-3.0-x86_64 xen-3.0-x86_32p
> > xen_scheduler          : credit
> > xen_pagesize           : 4096
> > platform_params        : virt_start=0xffff800000000000
> > xen_changeset          : Fri Aug 17 12:53:34 2018 +0100 git:382ad34e4e
> > xen_commandline        : placeholder loglvl=all guest_loglvl=all
> > com2=115200,8n1 ucode=scan console=com2,vga console_to_ring
> > sync_console hvm_fep
> > cc_compiler            : gcc (Debian 6.3.0-18+deb9u1) 6.3.0 20170516
> > cc_compile_by          : wei
> > cc_compile_domain      : uk.xensource.com
> > cc_compile_date        : Fri Aug 17 14:41:56 BST 2018
> > build_id               : 3989ecb7693aa02f6ecc748a951ed444cc70ba94
> > xend_config_format     : 4
> >
> > The hvm_directio flag is not accurate. See the last patch for
> > discussion.
> >
> > The major goal at the moment is to get something that works first,
> > then refine code structure later.  Currently CONFIG_HVM is littered in
> > individual files. In the future some of the code could / should be
> > moved to files under hvm/ for cleaner split.
> >
> > I ran some basic PV / PVSHIM VM life cycle tests and XTF PV tests, all
> > worked.
> >
> > $ ls -l xen # PV only, non-debug
> > -rwxrwxr-x 1 wei wei 1957436 Aug 17 15:32 xen
> > $ ls -l xen # default build, non-debug
> > -rwxrwxr-x 1 wei wei 2379388 Aug 17 15:39 xen
> >
> > The PV only Xen is ~17.8% smaller in size.
> 
> Hmm - only that little?  I'm somewhat surprised.  I guess it will be
> equally telling to see the delta for an HVM-only Xen.

There is in fact more code that should be put under CONFIG_HVM, but
that's something for later patches.

> 
> Either way, to get this going in the right direction, Patches 1-4 are
> trivial.  Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>

Thanks.

Wei.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel