Thread (11 messages) 11 messages, 5 authors, 2021-01-19

Re: [PATCH] gcc-plugins: simplify GCC plugin-dev capability test

From: Masahiro Yamada <masahiroy@kernel.org>
Date: 2021-01-19 19:16:06
Also in: linux-hardening, linux-kbuild, lkml

On Wed, Jan 20, 2021 at 2:48 AM Thierry Reding [off-list ref] wrote:
On Fri, Dec 18, 2020 at 08:33:37PM +0000, Jon Hunter wrote:
quoted
On 18/12/2020 17:54, Linus Torvalds wrote:
quoted
On Fri, Dec 18, 2020 at 7:33 AM Jon Hunter [off-list ref] wrote:
quoted
However, if you are saying that this is a problem/bug with our builders,
then of course we will have to get this fixed.
This seems to be a package dependency problem with the gcc plugins -
they clearly want libgmp, but apparently the package hasn't specified
that dependency.

If this turns out to be a big problem, I guess we can't simplify the
plugin check after all.

We historically just disabled gcc-plugins if that header didn't build,
which obviously meant that it "worked" for people, but it also means
that clearly the coverage can't have been as good as it could/should
be.

So if it's as simple as just installing the GNU multiprecision
libraries ("gmp-devel" on most rpm-based systems, "libgmp-dev" on most
debian systems), then I think that's the right thing to do. You'll get
a working build again, and equally importantly, your build servers
will actually do a better job of covering the different build options.

Thanks. I have reported this issue to the team that administers the
builders. So hopefully, they will install the necessary packages for us
now.
Just to close the loop on this, the builders now have libgmp-dev and
libmpc-dev packages installed and the builds are passing without the
workaround we had used.

Thierry

I was slightly concerned about your question:

"In case where CC != HOSTCC, it's possible that CC was not built against
the same version of GMP/MPC as HOSTCC. And even HOSTCC might not
necessarily have been built against the versions provided by libgmp-dev
or libmpc-dev. I'm not overly familiar with GMP/MPC, so perhaps if these
headers are reasonably stable, this is not all that important. But if it
is, then which version of GMP/MPC do we need? The version that CC was
built against, or the version that HOSTCC was built against?"



I do not have a good insight about this.

I am not sure if it is perfectly OK to
use gmp.h from HOSTCC when it was not bundled with CC.

The version difference might not be a significant issue, though...





-- 
Best Regards
Masahiro Yamada
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help