Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

[unitrace] clarify installation artifacts and locations

Đang mở
  • Có gì không ổn?
#124 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
20/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
linux, ubuntu
Lĩnh vực
build-system, release

Hướng nghiên cứu

The layout to change is the install tree in the issue: bin/summary/ (including schemas/), etc/unitrace/metrics/config/, lib/libunitrace_tool.so, and the bin/addrasm.py, analyzeperfmetrics.py and mergetrace.py scripts. Find where the install step places these files in the repository's build or packaging scripts. Done means the maintainers have answered the four questions and the agreed paths are applied, so this needs a maintainer decision before any code change.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

On:

Below are unitrace installation artifacts:

/opt/install/
|-- bin
|   |-- addrasm.py
|   |-- analyzeperfmetrics.py
|   |-- mergetrace.py
|   |-- summary
|   |   |-- categorize.py
|   |   |-- schemas
|   |   |   |-- LLaMA.ini
|   |   |   `-- LLaMA.json
|   |   `-- summary.py
|   |-- unitrace
|   `-- uniview.py
|-- etc
|   `-- unitrace
|       `-- metrics
|           `-- config
|               |-- bmg
|               |   |-- ComputeBasic.txt
|               |   `-- MemoryProfile.txt
|               `-- pvc
|                   |-- ComputeBasic.txt
|                   |-- GpuOffload.txt
|                   |-- L1ProfileReads.txt
|                   |-- L1ProfileSlmBankConflicts.txt
|                   |-- L1ProfileWrites.txt
|                   `-- MemProfile.txt
`-- lib
    `-- libunitrace_tool.so

11 directories, 18 files

I have concerns that above layout will be acceptable to package unitrace for Linux distribution packaging:

  1. Some stuff installed into /usr/bin/ have too wide names without clear affiliation to unitrace. These are addrasm.py, analyzeperfmetrics.py, mergetrace.py. I would consider renaming them with some some kind of uni* prefix.
  2. What is the purpose of libunitrace_tool.so? It does not have associated development environment (header files) and there seem to be only one actual executable - unitrace. So why the library needs to be dynamic?
  3. Does user supposed to modify stuff landed in /etc/unitrace/metrics/config/? If yes - these are indeed configuration files. If not - then these are unitrace internal stuff and should be moved to /usr/lib/unitrace/ folder.
  4. /usr/bin/summary/* - this location is definitely not correct. This must be moved to /usr/lib/unitrace/

@androniychuk, do you have any other questions/comments for the above unitrace installation artifacts?

CC: @mschilling0

Ngôn ngữ chính
C++
Star
276
Fork
73
Không có pull request nào được merge trong 30 ngày

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của intel/pti-gpu

Tất cả issue của intel/pti-gpu

Issue tương tự

Thêm issue về C++

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.