From: Andrew Cooper <andrew.cooper3@citrix.com>
To: Simon Martin <furryfuttock@gmail.com>, xen-devel@lists.xen.org
Subject: Re: stubdom/libxc-x86_64 compile error with valgrind installed
Date: Mon, 21 Jul 2014 11:19:14 +0100 [thread overview]
Message-ID: <53CCE922.9080109@citrix.com> (raw)
In-Reply-To: <1216211935.20140721110014@gmail.com>
On 21/07/14 11:00, Simon Martin wrote:
> Hello all,
>
> I am building on Debian Wheezy and I have valgrind installed. About a
> month ago I started having compilation problems. I assume it must have
> something to do with Andrew's valgrind patch of June 10th.
>
> The problem seems to lie in the fact that valgrind is in /usr/include,
> however -nostdinc is specified on the compilation so we don't get it.
>
> I removed -nostdinc from stubdom/Makefile and it compiled correctly.
> Reading the gcc documentation this *should* be safe, as it will only
> search the default directories *after* the explicit include
> directories. That being said, there is probably a better fix for this.
>
> Regards.
>
The problem is that stubdoms are blindly using the regular environment
config.h without reconfiguring libxc for their reduced environment.
As a result, config.h is correctly stating that memcheck.h exists in the
regular environment, even when it is included into the reduced stubdom
environment.
While removing -nostdinc does fix your problem, it is not the correct
fix. memcheck.h is safe to use in this instance, but most header files
you find in /usr/include are not.
I am not sure what is the correct solution here is. Introducing an
explicit --{en,dis}able-valgrind to the main ./configure will still
break for anyone who chooses to enable it. I suspect a fix involves
someone draining the swamp which is the stubdom build system.
~Andrew
next prev parent reply other threads:[~2014-07-21 10:19 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-21 10:00 stubdom/libxc-x86_64 compile error with valgrind installed Simon Martin
2014-07-21 10:06 ` Ian Campbell
2014-07-21 11:17 ` Simon Martin
2014-07-21 10:19 ` Andrew Cooper [this message]
2014-07-21 10:33 ` Ian Campbell
2014-07-21 10:40 ` Andrew Cooper
2014-07-21 10:47 ` Ian Campbell
2014-07-21 10:56 ` Andrew Cooper
2014-07-21 11:06 ` Ian Campbell
2014-07-21 11:10 ` Ian Campbell
2014-07-21 11:14 ` Ian Jackson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=53CCE922.9080109@citrix.com \
--to=andrew.cooper3@citrix.com \
--cc=furryfuttock@gmail.com \
--cc=xen-devel@lists.xen.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.