Bug 672012

Summary:	dev-util/valgrind-3.14.0 add ~alpha ~sparc ~ia64 keywords
Product:	Gentoo Linux	Reporter:	Craig Andrews <candrews>
Component:	Keywording	Assignee:	Anthony Basile <blueness>
Status:	RESOLVED CANTFIX
Severity:	normal	CC:	candrews, slyfox, wizardedit
Priority:	Normal	Flags:	stable-bot: sanity-check+
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:	dev-util/valgrind-3.14.0 ~alpha ~sparc ~ia64	Runtime testing required:	---

Description Craig Andrews gentoo-dev

2018-11-26 19:02:22 UTC

Please add the ~alpha keyword to dev-util/valgrind-3.14.0

Comment 1 Craig Andrews gentoo-dev

2018-11-26 19:04:55 UTC

and also ~sparc ~ia64

Comment 2 Sergei Trofimovich (RETIRED) gentoo-dev

2018-11-26 20:20:21 UTC

Please provide a rationale behind the request.

As implemented valgrind is a jit-compiler from target into a host instruction set. Each platform has to be supported explicitly as a sizable chunk of code: instruction decoder, compiler,syscall ABI description and rudimentary ELF loader.

Currently supported platforms are defined in
    https://sourceware.org/git/?p=valgrind.git;a=blob;f=configure.ac;h=289514ff15d1481d36dcd91130b992d81f239808;hb=HEAD#l221

Namely:
      i?86
      x86_64)
      powerpc64
      powerpc64le
      powerpc
      s390x
      armv7*
      aarch64*
      mips
      mipsel
      mipsisa32r2
      mips64*
      mipsisa64*

Note: all of the arches you requested are unsupported today.

Comment 3 Craig Andrews gentoo-dev

2018-11-26 20:23:15 UTC

Rationale is that the tests for media-libs/kvazaar use valgrind and I'm trying to get them to work (bug 671146)

We can just disable those tests on those architectures, no problem.

I thought this would be a simple keywording request, not a massive effort for upstream :)

Comment 4 Sergei Trofimovich (RETIRED) gentoo-dev

2018-11-26 20:28:29 UTC

USE=valgrind is already use.mask-ed on those targets. If you can make a subset of tests optional for valgrind (and you usually have to do it anyway) you should be fine.