Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 672012

Summary: dev-util/valgrind-3.14.0 add ~alpha ~sparc ~ia64 keywords
Product: Gentoo Linux Reporter: Craig Andrews <candrews>
Component: KeywordingAssignee: 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.