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

Bug#1065695: marked as done (libhsa-runtime-dev: endianness detection in fails on arm64 and ppc64el)

Your message dated Mon, 18 Mar 2024 18:57:01 -0600
with message-id <80c5a169-33e6-4c33-aba2-6aa607afc7dd@slerp.xyz>
and subject line Re: endianness detection fixed
has caused the Debian Bug report #1065695,
regarding libhsa-runtime-dev: endianness detection in fails on arm64 and ppc64el
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
1065695: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1065695
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: libhsa-runtime-dev
Version: 5.7.1-1
Severity: normal
X-Debbugs-Cc: cgmb@slerp.xyz

Dear Maintainer,

The endianness detection logic in hsa/hsa.h fails on arm64 and ppc64el,
which leads to build failures in rccl on those platforms [1].

The current logic is:

    // Try to detect CPU endianness
    #if !defined(LITTLEENDIAN_CPU) && !defined(BIGENDIAN_CPU)
    #if defined(__i386__) || defined(__x86_64__) || defined(_M_IX86) || \
        defined(_M_X64)
    #define LITTLEENDIAN_CPU
    #endif
    #endif

This code should probably check if __BYTE_ORDER__ is defined, and if its value
is __ORDER_LITTLE_ENDIAN__ or __ORDER_BIG_ENDIAN__ before giving up.

Sincerely,
Cory Bloor

[1]: https://buildd.debian.org/status/fetch.php?pkg=rccl&arch=ppc64el&ver=5.4.3-2%7Eexp1&stamp=1709357300&raw=0

-- System Information:
Debian Release: trixie/sid
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 6.6.15-amd64 (SMP w/32 CPU threads; PREEMPT)
Locale: LANG=C, LC_CTYPE=C.UTF-8 (charmap=UTF-8), LANGUAGE not set
Shell: /bin/sh linked to /usr/bin/dash
Init: unable to detect

Versions of packages libhsa-runtime-dev depends on:
ii  libhsa-runtime64-1  5.7.1-1

libhsa-runtime-dev recommends no packages.

libhsa-runtime-dev suggests no packages.

-- no debconf information

--- End Message ---
--- Begin Message --- 
Version: 5.7.1-2
A check for __BYTE_ORDER__ has been added to the hsa/hsa.h header in 5.7.1-2, which has been uploaded to unstable. I believe this fixes the problem for arm64 and ppc64el with GCC and Clang. 

Sincerely,
Cory Bloor

--- End Message ---
Reply to: