On 03/01/2018 02:15 PM, Peter Krempa wrote:
On Thu, Mar 01, 2018 at 14:08:29 +0100, Michal Privoznik wrote:
> Signed-off-by: --help <mprivozn(a)redhat.com>
Hmm.
> ---
> docs/news.xml | 102 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 102 insertions(+)
>
> diff --git a/docs/news.xml b/docs/news.xml
> index 86a0c8d18..53bf9a49c 100644
> --- a/docs/news.xml
> +++ b/docs/news.xml
> @@ -44,6 +44,28 @@
> using the <code>cachetune</code> element in
<code>cputune</code>.
> </description>
> </change>
> + <change>
> + <summary>
> + Allow opening secondary drivers
> + </summary>
> + <description>
> + Up until now it was possible to connect to only hypervisor drivers
> + (e.g. qemu:///system, lxc:///, vbox:///system, and so on). The
> + internal drivers (like network driver, node device driver, etc.) were
> + hidden from users and users could use them only indirectly. Starting
> + with this release new connection URIs are accepted. For instance
> + network:///system, storage:///system and so on.
> + </description>
Isn't this an internal change not really used for consumption of
clients?
Not really. Try it yourself:
virsh -c network:///system net-list --all
> + </change>
> + <change>
> + <summary>
> + virtlogd, virtlockd: Add support for admin protocol
> + </summary>
> + <description>
> + These two daemons now support admin protocol through which some admin
> + info can be gathered or some configuration tweaked on the fly.
> + </description>
> + </change>
> </section>
> <section title="Improvements">
> <change>
> @@ -82,8 +104,88 @@
> libxl: add support for setting clock offset and adjustment
> </summary>
> </change>
> + <change>
> + <summary>
> + Make port allocator global
> + </summary>
> + <description>
> + Up until now each driver had their own port allocator module. This
> + meant that info on port usage was not shared. Starting with this
> + release, the port allocator module is made global and therefore
> + drivers allocate ports from global pool.
> + </description>
> + </change>
> + <change>
> + <summary>
> + src: Enable building with GCC 8.0
> + </summary>
> + <description>
> + GCC 8.0 added more warnings which found some genuine problems with our
code.
> + </description>
I'm not sure whether that improved anything. Also wasn't that gcc 7?
It added a lot of cases into our switches which are now safer. The
problem with enums in switch() statements is we have to be 100% sure
value fits into the enum. For instance:
int x = VIR_DOMAIN_DEVICE_LAST + 1;
switch ((virDomainDeviceType) x) {
...
}
is obviously problematic.
And no, it's gcc 8.
Michal (aka --help).