]> xenbits.xensource.com Git - libvirt.git/commit
libxl: Clarify that 'xenlight' should only be used internally
authorJim Fehlig <jfehlig@suse.com>
Mon, 4 May 2020 21:20:37 +0000 (15:20 -0600)
committerJim Fehlig <jfehlig@suse.com>
Tue, 5 May 2020 20:53:37 +0000 (14:53 -0600)
commit836ea91d4ed174f2bb65ef894a8d6daf0675e743
tree514ebb2423f9f63a388725ac6d7c40e254f469d4
parentd218a9c263190681aa8ee7f0ec5f3c298339d635
libxl: Clarify that 'xenlight' should only be used internally

The libxl driver has suffered an identity crisis since its introduction.
It took on the name 'libxl' since at the time libvirt already contained
a 'xen' driver for the old Xen toolstack implementation. 'libxl' is short
for libxenlight, which is often called xenlight. Unfortunately all forms
of the name are used in the libxl driver.

The only remaining use of the 'xenlight' form is when interacting with
the host device manager, which is difficult to change since it would
cause problems when upgrading the driver.

Rename the #define to make it clear the 'xenlight' form is internal and
add a comment describing why the name exists and that its use should be
discouraged.

Signed-off-by: Jim Fehlig <jfehlig@suse.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
src/libxl/libxl_conf.h
src/libxl/libxl_domain.c
src/libxl/libxl_driver.c