Re: Entering freeze for libvirt-3.5.0

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

 



On Tue, Jul 04, 2017 at 01:03:52PM +0200, Andrea Bolognani wrote:
> On Sat, 2017-07-01 at 14:18 +0400, Roman Bogorodskiy wrote:
> > > qemuxml2argvtest fails consistently in my FreeBSD guest.
> > 
> > I guess that's caused by clang inlining functions that are mocked
> > (specifically, some numa related stuff); I think that was discussed
> > several times already. Anyway, it should work fine with '-O0' in CFLAGS.
> 
> Well, would you look at that. It does indeed work flawlessly
> when compiled without optimizations! :O
> 
> I'm not sure if that would be considered a reasonable
> compromise to get the test suite running on FreeBSD in the
> context of CI, though. I think it working reliably without
> messing with CFLAGS would be a requirement; others might
> disagree.

Hmm, I thought I fixed that problem when I introduce this patch:

  commit 728cacc8abed2b8de39e7b96fa42fde6850ec23a
  Author: Daniel P. Berrange <berrange@xxxxxxxxxx>
  Date:   Fri Apr 7 15:07:49 2017 +0100

    annotate all mocked functions with noinline


This made us annotate all mocked functions with noinline, which was
sufficient to make CLang builds pass tests on Ubuntu VMs.

Perhaps my syntax-check rule is missing some functions that still
need to be marked noinline to get BSD working ?

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux