[libvirt] [PATCH 0/7] Couple of -Os fixes

Martin Kletzander mkletzan at redhat.com
Thu Mar 3 11:35:32 UTC 2016


On Thu, Mar 03, 2016 at 09:44:35AM +0100, Michal Privoznik wrote:
>So while trying to bring down the size of my nss module, one of the things I've
>tried was compiling with -Os instead of -O3. Interesting errors pop up. Here
>are the fixes. If you think I should squash some patches together, I will
>gladly do that.
>

I would squash together initialization patches and then patches that
deal with the inlines.  ACK series with the removed double sign-off in
6/7.

>Michal Privoznik (7):
>  virCommandAddEnv: Drop inline
>  virDomainNumatuneNodeSpecified: Drop inline
>  virDomainChrRemove: Initialize @ret
>  vbox: Initialize @rc
>  qemuDomainBlockCommit: Initialize @baseSource
>  virNetworkDefForwardIf: drop inline
>  virStorageBackendLogicalMatchPoolSource: Initialize @thisSource
>
> src/conf/domain_conf.c                | 2 +-
> src/conf/network_conf.c               | 8 ++++++++
> src/conf/network_conf.h               | 8 +-------
> src/conf/numa_conf.c                  | 2 +-
> src/libvirt_private.syms              | 1 +
> src/qemu/qemu_driver.c                | 2 +-
> src/storage/storage_backend_logical.c | 2 +-
> src/util/vircommand.c                 | 2 +-
> src/vbox/vbox_common.c                | 2 +-
> src/vbox/vbox_tmpl.c                  | 2 +-
> 10 files changed, 17 insertions(+), 14 deletions(-)
>
>--
>2.4.10
>
>--
>libvir-list mailing list
>libvir-list at redhat.com
>https://www.redhat.com/mailman/listinfo/libvir-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20160303/1756d304/attachment-0001.sig>


More information about the libvir-list mailing list