[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2023-01-17 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 Ali Erdinc Koroglu changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|---

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2023-01-17 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #23 from Fedora Admin user for bugzilla script actions --- The Pagure repository was created at https://src.fedoraproject.org/rpms/wult -- You are receiving this mail because: You are always notified about changes to this produ

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2023-01-17 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 Jaroslav Škarvada changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #22 fr

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2023-01-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #21 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5239245 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2099771-wult/fedora-rawhide-

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2023-01-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #20 from Jakub Kadlčík --- Created attachment 1938563 --> https://bugzilla.redhat.com/attachment.cgi?id=1938563&action=edit The .spec file difference from Copr build 5227725 to 5239245 -- You are receiving this mail because: Y

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2023-01-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #19 from Ali Erdinc Koroglu --- Thank you for the comments, changes are upstreamed and new version released. SPEC Url: https://download.copr.fedorainfracloud.org/results/aekoroglu/fedora/fedora-rawhide-x86_64/05239153-wult/wult.sp

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2023-01-16 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #18 from Jaroslav Škarvada --- The package cannot be installed: Error: Problem: conflicting requests - nothing provides python3-wult(x86-64) = 1.10.53-1.fc38 needed by wult-1.10.53-1.fc38.x86_64 It's because now the python3-wu

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2023-01-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #17 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5227725 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2099771-wult/fedora-rawhide-

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2023-01-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #16 from Jakub Kadlčík --- Created attachment 1937839 --> https://bugzilla.redhat.com/attachment.cgi?id=1937839&action=edit The .spec file difference from Copr build 5223855 to 5227725 -- You are receiving this mail because: Y

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2023-01-13 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #15 from Ali Erdinc Koroglu --- Hello again, - I added those CFLAGS to fix rpminspect annocheck tests but you're right and changed makefile as you recommended. - Wult has some C and Python codes that's why I would like to have this

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2023-01-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #14 from Jaroslav Škarvada --- (In reply to Ali Erdinc Koroglu from comment #12) - Regarding the patch, maybe upstream could allow installation without drivers/helpers, but for now I am OK with the patch. - %make_build CFLAGS="%op

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2023-01-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #13 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5223855 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2099771-wult/fedora-rawhide-

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2023-01-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #12 from Ali Erdinc Koroglu --- Hello again, thanks for the comments. SPEC Url: https://download.copr.fedorainfracloud.org/results/aekoroglu/fedora/fedora-rawhide-x86_64/05223603-wult/wult.spec SRPM Url: https://download.copr.fedo

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2023-01-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #11 from Jaroslav Škarvada --- (In reply to Jaroslav Škarvada from comment #10) > - %make_build CFLAGS="%optflags -fpic -pie -g" -C helpers/ndlrunner IMHO -fpic is also there implicitly via the redhat-hardened-cc1. -- You are r

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2023-01-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #10 from Jaroslav Škarvada --- Sorry for the delay. Package Review == Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated [ ] = Manual review needed Issues: === - Latest version is packaged

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2022-11-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 Petr Pisar changed: What|Removed |Added Flags||fedora-review? -- You are receiving th

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2022-10-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 Jaroslav Škarvada changed: What|Removed |Added CC||jskar...@redhat.com Sta

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2022-10-20 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #9 from Ali Erdinc Koroglu --- New version SPEC Url : https://download.copr.fedorainfracloud.org/results/aekoroglu/fedora/fedora-rawhide-x86_64/04961963-wult/wult.spec SRPM Url : https://download.copr.fedorainfracloud.org/results/

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2022-09-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #8 from Ali Erdinc Koroglu --- Finally my colleagues released a new version of Wult which does not require a kernel driver (uses eBPF instead) any more. SPEC Url: https://download.copr.fedorainfracloud.org/results/aekoroglu/fedora

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2022-07-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #7 from Ali Erdinc Koroglu --- Hello again, discussion [1] will take some time so I excluded driver sources from the package. 1: https://lists.fedoraproject.org/archives/list/packag...@lists.fedoraproject.org/thread/75LZKVSVSTMSJR

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2022-07-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #6 from Ali Erdinc Koroglu --- Wult developers recognize the importance of upstream and they are themselves upstream kernel developers. They do work towards adding the functionality required for wult to upstream, but this is going

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2022-07-06 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #5 from Zbigniew Jędrzejewski-Szmek --- + package name is OK ~ latest version (1.10.9 was released a few hours ago) + license is acceptable for Fedora (BSD) + BR/R/P look OK + builds and installs OK wult-devel ships a kernel drive

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2022-07-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 Bug 2099771 depends on bug 2099789, which changed state. Bug 2099789 Summary: Review Request: pepc - Power, Energy, and Performance configuration tool https://bugzilla.redhat.com/show_bug.cgi?id=2099789 What|Removed

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2022-07-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #4 from Ali Erdinc Koroglu --- Hi there, I changed the spec file as you recommended. SPEC Url: https://download.copr.fedorainfracloud.org/results/aekoroglu/intel-fedora/fedora-rawhide-x86_64/04597858-wult/wult.spec SRPM Url: https

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2022-07-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 --- Comment #3 from Zbigniew Jędrzejewski-Szmek --- "Requires: wult" → add " = %{version}-%{release}" "debug_package %{nil}" → If the debug package breaks build, that usually hides other issues. But here it seems to build fine if that is remo

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2022-07-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 Zbigniew Jędrzejewski-Szmek changed: What|Removed |Added Depends On||2099789 --- Comment #2

[Bug 2099771] Review Request: wult - Tool for measuring Intel CPU C-state wake latency

2022-07-05 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2099771 Zbigniew Jędrzejewski-Szmek changed: What|Removed |Added CC||zbys...@in.waw.pl