[Bug 2209759] Review Request: rocclr - ROCm Compute Language Runtime

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

 



https://bugzilla.redhat.com/show_bug.cgi?id=2209759

Tom Rix <trix@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |tstellar@xxxxxxxxxx
             Status|NEW                         |ASSIGNED
              Flags|                            |needinfo?(tstellar@redhat.c
                   |                            |om)



--- Comment #2 from Tom Rix <trix@xxxxxxxxxx> ---
The documentation could be improved by adding
BuildRequires: doxygen
Lots of docs will be generated.

A change to fix the no executable stack 
diff --git a/rocclr.spec b/rocclr.spec
index 0752e05..73fb54e 100644
--- a/rocclr.spec
+++ b/rocclr.spec
@@ -10,7 +10,7 @@

 Name:           rocclr
 Version:        %{rocm_version}
-Release:        1%{?dist}
+Release:        2%{?dist}
 Summary:        ROCm Compute Language Runtime
 Url:            https://github.com/ROCm-Developer-Tools/clr
 License:        MIT
@@ -152,6 +152,7 @@ sed -i "s|\(/usr/bin/\)env perl|\1perl|"
HIP-rocm-%{version}/bin/hipcc.pl

 %build
 %cmake \
+    -DCMAKE_SHARED_LINKER_FLAGS=-Wl,-z,noexecstack \
     -DHIP_COMMON_DIR=$(realpath HIP-rocm-%{version}) \
     -DCMAKE_INSTALL_LIBDIR=%{_lib} \
     -DHIP_PLATFORM=amd \
@@ -251,5 +252,8 @@ fi
 %exclude %{_datadir}/hip/samples

 %changelog
+* Sun May 28 2023 Tom Rix <trix@xxxxxxxxxx> - 5.5.0-2
+- Add noexecstack linker option
+
 * Tue May 16 2023 Jeremy Newton <alexjnewt at hotmail dot com> - 5.5.0-1
 - Initial package

A couple of things. in the config step of this package

-- ROCm Agent Enumurator Not Found
BuildRequires and Requires should have rocminfo package

I am testing by building rocBLAS, so far i have had to
export DEVICE_LIB_PATH=/usr/lib64/amdgcn/bitcode

there should be a automatic connect between where rocm-device-libs installs
this bitcode dir
and where hipcc/clang picks them up with '--rocm-device-lib-path' 

Tom Stellar what do you think ?


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2209759
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux