[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #64 from Honggang LI --- https://release-monitoring.org/project/98355/ Create this project for upstream release monitoring for rshim. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 Michal Schmidt changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |RAWHIDE Last Closed||2020-05-06 20:30:50 --- Comment #63 from Michal Schmidt --- The package is successfully built and tagged "f33". It will be installable with dnf in Fedora Rawhide once a daily compose of Rawhide finishes and propagates to mirrors. $ koji latest-pkg f33 rshim Build Tag Built by rshim-2.0.4-1.fc33f33 lsun You may want to push rshim also as an update to Fedora 32, the current stable release. You'd do that by creating the f32 branch in Fedora dist-git: fedpkg request-branch --repo rshim f32 I don't remember if the new branch would be created empty, or as a fork of master. You can do a git merge or cherry-pick between branches. Then you can do a build while having the f32 branch checked out: fedpkg build Then you can publish your build as a package update for Fedora 32 using Bodhi (read https://fedoraproject.org/wiki/Bodhi). Getting rshim into RHEL is out of scope for this Fedora review (unless you're interested in EPEL, https://fedoraproject.org/wiki/EPEL). We should continue in RHEL bug 1744737. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #62 from l...@mellanox.com --- Michal / Honggang, This is my first Fedora package. Any advice what the next steps would be? Such as how to get it into fedora 33 or Redhat, how to make it visible to 'dnf' install, etc. Thanks! -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #61 from l...@mellanox.com --- Thanks a lot for the help!!! The changes have been pushed. The 'fedpkg build' also seem passed. https://src.fedoraproject.org/rpms/rshim $ fedpkg import ~/rshim-2.0.4-1.fc31.src.rpm $ git commit -m "Initial import (#1814682)." $ git push $ fedpkg build https://koji.fedoraproject.org/koji/taskinfo?taskID=44157911 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #60 from Gwyn Ciesla --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/rshim -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 Michal Schmidt changed: What|Removed |Added Flags|fedora-review? |fedora-review+ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 Michal Schmidt changed: What|Removed |Added Flags|fedora-review+ |fedora-review? --- Comment #59 from Michal Schmidt --- (In reply to lsun from comment #57) > It said "The review is not approved by the assignee of the Bugzilla bug". Oh, I see. The server side checks who set the fedora-review+ flag. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #58 from l...@mellanox.com --- [lsun@fedora ~]$ fedpkg -v request-repo rshim 1814682 https://pagure.io/releng/fedora-scm-requests/issue/24793 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #57 from l...@mellanox.com --- This was the issue that was rejected. https://pagure.io/releng/fedora-scm-requests/issue/24679 It said "The review is not approved by the assignee of the Bugzilla bug". I just submitted a new request: https://pagure.io/releng/fedora-scm-requests/issue/24793 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #56 from Michal Schmidt --- The "fedora-review" flag is already set to "+" and it should be the only flag that matters for this purpose. Please try again, double-check that the Bugzilla number is correct. If it still does not work, try it with verbose output and paste here the output: fedpkg -v request-repo rshim 1814682 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #55 from l...@mellanox.com --- Michal, thanks for adding me packager group! I tried to do "fedpkg request-repo ...", but got rejected. It complained that this bugzilla hasn't been approved yet. Do you need to approve this one first? Or any further comments or advice what the next step would be? Thanks! -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 Honggang LI changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #54 from Honggang LI --- I had run basic functional tests for rshim. Confirmed it works as expected and the output of fedora-review tool looks good for me. Set the 'fedora-review+' flag. Michal, please add lsun into packager group. = Package Review == Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated [ ] = Manual review needed = MUST items = C/C++: [ ]: Package does not contain kernel modules. PASS [ ]: Package contains no static executables. PASS [x]: If your application is a C or C++ application you must list a BuildRequires against gcc, gcc-c++ or clang. [x]: Header files in -devel subpackage, if present. [x]: Package does not contain any libtool archives (.la) [x]: Rpath absent or only used for internal libs. Generic: [ ]: Package is licensed with an open-source compatible license and meets other legal requirements as defined in the legal section of Packaging Guidelines. PASS [ ]: License field in the package spec file matches the actual license. Note: Checking patched sources after %prep for licenses. Licenses found: "*No copyright* GPL (v2)", "Unknown or generated", "GPL (v2)". 23 files have unknown license. Detailed output of licensecheck in /home/honli/review/1814682-rshim/licensecheck.txt PASS. It is a review tool issue, as those files have "SPDX-License-Identifier: GPL-2.0-only" tag. [ ]: License file installed when any subpackage combination is installed. PASS. [ ]: Package must own all directories that it creates. Note: Directories without known owners: /usr/lib/systemd, /usr/lib/systemd/system PASS [ ]: %build honors applicable compiler flags or justifies otherwise. PASS [ ]: Package contains no bundled libraries without FPC exception. PASS [ ]: Changelog in prescribed format. PASS [ ]: Sources contain only permissible code or content. PASS [ ]: Package contains desktop file if it is a GUI application. PASS. It is not a GUI application. [ ]: Development files must be in a -devel package PASS. no devel [ ]: Package uses nothing in %doc for runtime. PASS [ ]: Package consistently uses macros (instead of hard-coded directory names). PASS [ ]: Package is named according to the Package Naming Guidelines. PASS [ ]: Package does not generate any conflict. PASS [ ]: Package obeys FHS, except libexecdir and /usr/target. PASS [ ]: If the package is a rename of another package, proper Obsoletes and Provides are present. PASS [ ]: Requires correct, justified where necessary. PASS [ ]: Spec file is legible and written in American English. PASS [ ]: Package contains systemd file(s) if in need. PASS. start/stop rshim service confirmed the systemd service file works. [ ]: Useful -debuginfo package or justification otherwise. PASS [ ]: Package is not known to require an ExcludeArch tag. PASS [ ]: Large documentation must go in a -doc subpackage. Large could be size (~1MB) or number of files. Note: Documentation size is 10240 bytes in 1 files. PASS [ ]: Package complies to the Packaging Guidelines PASS [x]: Package successfully compiles and builds into binary rpms on at least one supported primary architecture. [x]: Package installs properly. [x]: Rpmlint is run on all rpms the build produces. Note: No rpmlint messages. [x]: If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package is included in %license. [x]: Package requires other packages for directories it uses. [x]: Package does not own files or directories owned by other packages. [x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT [x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the beginning of %install. [x]: Macros in Summary, %description expandable at SRPM build time. [x]: Dist tag is present. [x]: Package does not contain duplicates in %files. [x]: Permissions on files are set properly. [x]: Package must not depend on deprecated() packages. [x]: Package use %makeinstall only when make install DESTDIR=... doesn't work. [x]: Package is named using only allowed ASCII characters. [x]: Package does not use a name that already exists. [x]: Package is not relocatable. [x]: Sources used to build the package match the upstream source, as provided in the spec URL. [x]: Spec file name must match the spec package %{name}, in the format %{name}.spec. [x]: systemd_post is invoked in %post, systemd_preun in %preun, and systemd_postun in %postun for Systemd service files. Note:
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #53 from Honggang LI --- (In reply to lsun from comment #49) > > Why does the rshim.service use killall in the first place? There is: > > > KillMode=process > > > > Why this mode? Do you need child processes to be left running in the cgroup > > after the service is stopped? > > If yes, commenting on it in the unit file would be nice. > > It's a copy/paste error. I would like to stop all processes when service > stops. > > > > > > ExecStop=/usr/bin/killall -SIGKILL rshim > > > > Referencing processes to kill by name is not good. It would kill unrelated > > processes with the same name. > > Is none of systemd's kill modes suitable for stopping the service without > > additional help? > > And why SIGKILL? Does it not stop on SIGTERM? > > Will update in next posted version with the following: > - Remove the "Requires: psmisc" > - Remove "ExecStop". > - Use "KillMode=control-group" > - Support SIGTERM > > [Service] > Restart=always > Type=forking > ExecStart=-/usr/sbin/rshim $OPTIONS > KillMode=control-group Confirmed rshim.service start/stop works with the new systemd service file. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #52 from Honggang LI --- (In reply to Alaa Hleihel (Mellanox) from comment #51) > Hi Honggang, > You can take the setup back and continue your testing/review. I confirmed the network traffic between the aarch64 machine and SmartNIC works via ssh. The rshim user space driver works for me. Thanks -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #51 from Alaa Hleihel (Mellanox) --- Thanks a lot, Liming. I tested rshim-2.0.4-1 and I see that everything is functioning well now using the default RHEL-8.1 kernel 4.18.0-147.el8.aarch64. Hi Honggang, You can take the setup back and continue your testing/review. Regards, Alaa # ls /dev/rshim0/ boot console misc rshim # cat /dev/rshim0/misc DISPLAY_LEVEL 0 (0:basic, 1:advanced, 2:log) BOOT_MODE 1 (0:rshim, 1:emmc, 2:emmc-boot-swap) BOOT_TIMEOUT100 (seconds) SW_RESET0 (1: reset) DEV_NAMEpcie-01:00.2 Access to the SoC console works well, I used this command: # sudo minicom --color on --baudrate 115200 --device /dev/rshim0/console File transfer using the tmfifo interface is also working well: # ip -s addr show tmfifo_net0 123: tmfifo_net0: mtu 1500 qdisc fq_codel state UNKNOWN group default qlen 1000 link/ether 00:1a:ca:ff:ff:02 brd ff:ff:ff:ff:ff:ff inet 192.168.100.1/24 brd 192.168.100.255 scope global tmfifo_net0 valid_lft forever preferred_lft forever inet6 fe80::21a:caff:feff:ff02/64 scope link valid_lft forever preferred_lft forever RX: bytes packets errors dropped overrun mcast 2362587328770 0 0 0 TX: bytes packets errors dropped carrier collsns 615240797 406494 0 56 0 0 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #50 from l...@mellanox.com --- Thanks all for the comments. 2.0.4 has been uploaded trying to solve these issues. Spec URL: https://github.com/Mellanox/rshim-user-space/releases/download/rshim-2.0.4/rshim.spec SRPM URL: https://github.com/Mellanox/rshim-user-space/releases/download/rshim-2.0.4/rshim-2.0.4-1.fc31.src.rpm koji build URL: https://koji.fedoraproject.org/koji/taskinfo?taskID=43638819 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #49 from l...@mellanox.com --- Thanks Michal for the comments. Please see my response below. (In reply to Michal Schmidt from comment #47) > (In reply to lsun from comment #32) > > As for the "systemd-units", it appears to be required by koji. I got some > > failures like below once removed it. So I'll leave it for now (please > > correct me if I am incorrect). > > https://koji.fedoraproject.org/koji/taskinfo?taskID=43163313 > > OK, in that case leaving a BuildRequires there is acceptable. > Note that the "systemd-units" package was merged into the main "systemd" > package in 2012 (before Fedora 18). Fedora packaging guidelines removed the > last reference to "systemd-units" in 2016, keeping "BuildRequires: systemd" > as the preferred way. > > There is another option. You can remove the BR and instead tell your > configure script to not autodetect the systemd units directory, using: > %configure --with-systemdsystemunitdir=%{_unitdir} > The minor advantage of this would be a smaller buildroot. > > Use whatever option you prefer there. Will update in next post and use "BuildRequires: systemd" as suggested. > > (In reply to lsun from comment #46) > > Spec URL: > > https://github.com/Mellanox/rshim-user-space/releases/download/rshim-2.0.3/rshim.spec > > > Requires: psmisc > > Why does the rshim.service use killall in the first place? There is: > > KillMode=process > > Why this mode? Do you need child processes to be left running in the cgroup > after the service is stopped? > If yes, commenting on it in the unit file would be nice. It's a copy/paste error. I would like to stop all processes when service stops. > > > ExecStop=/usr/bin/killall -SIGKILL rshim > > Referencing processes to kill by name is not good. It would kill unrelated > processes with the same name. > Is none of systemd's kill modes suitable for stopping the service without > additional help? > And why SIGKILL? Does it not stop on SIGTERM? Will update in next posted version with the following: - Remove the "Requires: psmisc" - Remove "ExecStop". - Use "KillMode=control-group" - Support SIGTERM [Service] Restart=always Type=forking ExecStart=-/usr/sbin/rshim $OPTIONS KillMode=control-group > > > Requires: kernel-modules-extra > > This is always going to be imperfect, because nothing guarantees that the > installed package corresponds to the actually running kernel (different > versions, variants like -debug, custom unpackaged kernels, ...). > There is precedent for depending on kernel-modules-extra in Fedora packages > (usbip, xl2tpd, ...) though. > I would just make it more explicit which module the package needs by instead > using this: > > Requires: kmod(cuse.ko) > # Hint for dnf to prefer kernel-modules-extra over > kernel-debug-modules-extra: > Suggests: kernel-modules-extra Will update in next posted version as suggested: Requires: kmod(cuse.ko) Suggests: kernel-modules-extra > > > BTW, is it necessary to call 'system("modprobe cuse");' in src/rshim.c? I > would expect the module to get autoloaded during the call to > cuse_lowlevel_setup. It seems not loading 'cuse' automatically. Below is what I got on CentOS-7 when calling cuse_lowlevel_setup() with cuse not loaded. "cuse: device not found, try 'modprobe cuse' first". -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #48 from Alaa Hleihel (Mellanox) --- (In reply to Alaa Hleihel (Mellanox) from comment #37) > 3. Red Hat needs to enable CONFIG_DEVMEM on aarch64 builds as well (it's the > only arch that has this config disabled). > Update: enabling CONFIG_DEVMEM is no longer required. rshim-2.0.3 uses the device resources directly instead of /dev/mem -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #47 from Michal Schmidt --- (In reply to lsun from comment #32) > As for the "systemd-units", it appears to be required by koji. I got some > failures like below once removed it. So I'll leave it for now (please > correct me if I am incorrect). > https://koji.fedoraproject.org/koji/taskinfo?taskID=43163313 OK, in that case leaving a BuildRequires there is acceptable. Note that the "systemd-units" package was merged into the main "systemd" package in 2012 (before Fedora 18). Fedora packaging guidelines removed the last reference to "systemd-units" in 2016, keeping "BuildRequires: systemd" as the preferred way. There is another option. You can remove the BR and instead tell your configure script to not autodetect the systemd units directory, using: %configure --with-systemdsystemunitdir=%{_unitdir} The minor advantage of this would be a smaller buildroot. Use whatever option you prefer there. (In reply to lsun from comment #46) > Spec URL: > https://github.com/Mellanox/rshim-user-space/releases/download/rshim-2.0.3/rshim.spec > Requires: psmisc Why does the rshim.service use killall in the first place? There is: > KillMode=process Why this mode? Do you need child processes to be left running in the cgroup after the service is stopped? If yes, commenting on it in the unit file would be nice. > ExecStop=/usr/bin/killall -SIGKILL rshim Referencing processes to kill by name is not good. It would kill unrelated processes with the same name. Is none of systemd's kill modes suitable for stopping the service without additional help? And why SIGKILL? Does it not stop on SIGTERM? > Requires: kernel-modules-extra This is always going to be imperfect, because nothing guarantees that the installed package corresponds to the actually running kernel (different versions, variants like -debug, custom unpackaged kernels, ...). There is precedent for depending on kernel-modules-extra in Fedora packages (usbip, xl2tpd, ...) though. I would just make it more explicit which module the package needs by instead using this: Requires: kmod(cuse.ko) # Hint for dnf to prefer kernel-modules-extra over kernel-debug-modules-extra: Suggests: kernel-modules-extra BTW, is it necessary to call 'system("modprobe cuse");' in src/rshim.c? I would expect the module to get autoloaded during the call to cuse_lowlevel_setup. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #46 from l...@mellanox.com --- Thanks for help the debugging! Below is updated version trying to solve these issues. Spec URL: https://github.com/Mellanox/rshim-user-space/releases/download/rshim-2.0.3/rshim.spec SRPM URL: https://github.com/Mellanox/rshim-user-space/releases/download/rshim-2.0.3/rshim-2.0.3-1.fc31.src.rpm koji build URL: https://koji.fedoraproject.org/koji/taskinfo?taskID=43397845 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #45 from Alaa Hleihel (Mellanox) --- Update: We found the issue, the device was not enabled, so we couldn't read from it... [root@qualcomm-amberwing-rep2-01 pcie_read_test]# lspci - -d :c2d2 :01:00.2 DMA controller: Mellanox Technologies MT416842 BlueField SoC management interfac (prog-if 00 [8237]) Subsystem: Mellanox Technologies Device 0082 Control: I/O- Mem- BusMaster- SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B- DisINTx- Status: Cap+ 66MHz- UDF- FastB2B- ParErr- DEVSEL=fast >TAbort- SERR- 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #44 from Alaa Hleihel (Mellanox) --- Hi Liming, No, I am not using Fedora, I am using RHEL-8.1 (kernel-modules-extra-4.18.0-147.el8.aarch64). > kernel-modules-extra-5.7.0-0.rc0.git8.1.fc33.x86_64 This output was from Honggang, he just was checking the location of the cuse driver. > On the server where the problem exists, have we tried whether the kernel > module based driver works or not? If not working, the SmartNIC FW might be > stuck. We might need power-cycle to recover. Thanks! Yes, I did, and the kernel rshim driver worked fine on the same system. Reboot and power-cycle didn't help when using the user-space rshim. Note that I have an x86_64 system at Mellanox lab with RHEL-8.1 and everything worked well. For now, I see that the issue happens only on this aarch64 system (I don't have another arm system to try). Also, I've tried building upstream kernel v5.6 using the RHEL-8.1 .config file (+ enabling CONFIG_DEVMEM) and I got the same issue. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #43 from l...@mellanox.com --- Alaa, I downloaded Fedora-33 (Rawhide) x86_64 version and tried it like below. It worked well for me. Below is what I captured. The difference is that it's the x86 version. (I don't have setup to try f33 aarch64 version yet). On the server where the problem exists, have we tried whether the kernel module based driver works or not? If not working, the SmartNIC FW might be stuck. We might need power-cycle to recover. Thanks! # uname -r 5.7.0-0.rc0.git8.1.fc33.x86_64 # cat /etc/redhat-release Fedora release 33 (Rawhide) # cat /dev/rshim0/misc DISPLAY_LEVEL 0 (0:basic, 1:advanced, 2:log) BOOT_MODE 1 (0:rshim, 1:emmc, 2:emmc-boot-swap) BOOT_TIMEOUT100 (seconds) SW_RESET0 (1: reset) DEV_NAMEpcie-00:08.0 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #42 from l...@mellanox.com --- Alaa, Looks like this server is running kernel-modules-extra-5.7.0. What OS is it? Is it Fedora-33? Is there a way I could download it and verify it? I just tried this driver on Fedora 31 and CentOS 8.1. All worked well for me. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #41 from l...@mellanox.com --- (In reply to Alaa Hleihel (Mellanox) from comment #37) > So to recap my previous long comment: > > 1. I think the rshim RPM package should Require the package which will > provide the 'cuse' kernel module, for RHEL-8 that's "kernel-modules-extra". > What about Fedora? > > 2. The rshim RPM package should Require "psmisc" package since it uses > 'killall' utility. > > 3. Red Hat needs to enable CONFIG_DEVMEM on aarch64 builds as well (it's the > only arch that has this config disabled). > > 4. Liming, are you familiar with such an issue where accessing the device > hangs (this does not happen when using the kernel module rshim)? Thanks Alaa! So far we have verified on CentOS-7, RedHat-7 and ubuntu-18 on x86 machine. For #1 & #2. Yes, I'll add the dependency. This module is part of the kernel package in CentOS/RedHat 7. Looks like it has separate package in Fedora 31 and Redhat-8. Fro #4, the hung issue is new to me. I'll install centos 8 today and try to reproduce it. Thanks! -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #40 from Alaa Hleihel (Mellanox) --- (In reply to Alaa Hleihel (Mellanox) from comment #38) > For issue #4, I see that there are upstream fixes in the "fuse" driver that > fixes a deadlock issue. > Those fixes were already backported to RHEL-8.2, will update my kernel and > retest. didn't help.. with kernel-debug, from the first terminal I ran "# cat /dev/rshim0/misc", and at the same time from another terminal I ran "# sudo minicom --color on --baudrate 115200 --device /dev/rshim0/console", then got: (I lowered hung_task_timeout_secs threshold to 10 seconds): [ 812.368606] INFO: task cat:5758 blocked for more than 10 seconds. [ 812.373943] Not tainted 4.18.0-193.el8.aarch64+debug #1 [ 812.379721] "echo 0 > /proc/sys/kernel/hung_task_timeout_secs" disables this message. [ 812.387436] cat D26640 5758 5687 0x0209 [ 812.392996] Call trace: [ 812.395347] __switch_to+0x1a0/0x258 [ 812.398983] __schedule+0x918/0x2120 [ 812.402455] schedule+0xf4/0x3a8 [ 812.405722] request_wait_answer+0x29c/0x530 [fuse] [ 812.410669] fuse_simple_request+0x408/0x970 [fuse] [ 812.415440] fuse_direct_io+0xd88/0x18c8 [fuse] [ 812.419997] cuse_read_iter+0xdc/0x110 [cuse] [ 812.424249] new_sync_read+0x358/0x4b0 [ 812.427976] __vfs_read+0xc4/0xf8 [ 812.431358] vfs_read+0xe0/0x290 [ 812.434486] ksys_read+0xcc/0x178 [ 812.437786] __arm64_sys_read+0x70/0xa0 [ 812.441704] el0_svc_handler+0x160/0x388 [ 812.445511] el0_svc+0x8/0xc [ 812.448459] Showing all locks held in the system: [ 812.454560] 1 lock held by khungtaskd/292: [ 812.458709] #0: 2000139abda0 (rcu_read_lock){}, at: debug_show_all_locks+0xd8/0x358 [ 812.467068] 2 locks held by agetty/1645: [ 812.471036] #0: 80143f62aa90 (>ldisc_sem){}, at: ldsem_down_read+0x48/0x58 [ 812.479096] #1: 20002e6722e0 (>atomic_read_lock){+.+.}, at: n_tty_read+0x1ac/0x13f8 [ 812.489254] = -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #39 from Honggang LI --- (In reply to Alaa Hleihel (Mellanox) from comment #37) > So to recap my previous long comment: > > 1. I think the rshim RPM package should Require the package which will > provide the 'cuse' kernel module, for RHEL-8 that's "kernel-modules-extra". > What about Fedora? Same as RHEL-8. localhost ~]$ find /lib/modules/ -name '*cuse*' /lib/modules/5.7.0-0.rc0.git8.1.fc33.x86_64/extra/fs/fuse/cuse.ko.xz localhost ~]$ rpm -qf /lib/modules/5.7.0-0.rc0.git8.1.fc33.x86_64/extra/fs/fuse/cuse.ko.xz kernel-modules-extra-5.7.0-0.rc0.git8.1.fc33.x86_64 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #38 from Alaa Hleihel (Mellanox) --- For issue #4, I see that there are upstream fixes in the "fuse" driver that fixes a deadlock issue. Those fixes were already backported to RHEL-8.2, will update my kernel and retest. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #37 from Alaa Hleihel (Mellanox) --- So to recap my previous long comment: 1. I think the rshim RPM package should Require the package which will provide the 'cuse' kernel module, for RHEL-8 that's "kernel-modules-extra". What about Fedora? 2. The rshim RPM package should Require "psmisc" package since it uses 'killall' utility. 3. Red Hat needs to enable CONFIG_DEVMEM on aarch64 builds as well (it's the only arch that has this config disabled). 4. Liming, are you familiar with such an issue where accessing the device hangs (this does not happen when using the kernel module rshim)? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #36 from Alaa Hleihel (Mellanox) --- Hi, I logged in to the system and found the following issues: 1. rshim service start fails: Apr 12 02:39:06 qualcomm-amberwing-rep2-01.khw4.lab.eng.bos.redhat.com rshim[4799]: Probing pcie-01:00.2 Apr 12 02:39:06 qualcomm-amberwing-rep2-01.khw4.lab.eng.bos.redhat.com rshim[4799]: create rshim pcie-01:00.2 Apr 12 02:39:06 qualcomm-amberwing-rep2-01.khw4.lab.eng.bos.redhat.com rshim[4799]: Failed to map RShim registers [root@qualcomm-amberwing-rep2-01 ~]# rshim -f modprobe: FATAL: Module cuse not found in directory /lib/modules/4.18.0-147.el8.aarch64 Probing pcie-01:00.2 create rshim pcie-01:00.2 Failed to map RShim registers The reason that a required module is not installed on the system: [root@qualcomm-amberwing-rep2-01 ~]# modinfo cuse modinfo: ERROR: Module cuse not found. The fix is: # dnf install -y kernel-modules-extra Then the module will be available: [root@qualcomm-amberwing-rep2-01 ~]# modinfo cuse filename: /lib/modules/4.18.0-147.el8.aarch64/kernel/fs/fuse/cuse.ko.xz 2. rshim service stop fails: Apr 12 02:35:57 qualcomm-amberwing-rep2-01.khw4.lab.eng.bos.redhat.com systemd[1]: Stopping rshim driver for BlueField SoC... Apr 12 02:35:57 qualcomm-amberwing-rep2-01.khw4.lab.eng.bos.redhat.com systemd[4383]: rshim.service: Failed to execute command: No such file or directory Apr 12 02:35:57 qualcomm-amberwing-rep2-01.khw4.lab.eng.bos.redhat.com systemd[4383]: rshim.service: Failed at step EXEC spawning /usr/bin/killall: No such file or directory ^^^ Apr 12 02:35:57 qualcomm-amberwing-rep2-01.khw4.lab.eng.bos.redhat.com systemd[1]: rshim.service: Control process exited, code=exited status=203 Apr 12 02:36:55 qualcomm-amberwing-rep2-01.khw4.lab.eng.bos.redhat.com sshd[4384]: Connection closed by 10.35.206.44 port 60160 [preauth] Apr 12 02:36:59 qualcomm-amberwing-rep2-01.khw4.lab.eng.bos.redhat.com sshd[4386]: Accepted password for root from 10.35.206.44 port 60162 ssh2 Apr 12 02:36:59 qualcomm-amberwing-rep2-01.khw4.lab.eng.bos.redhat.com systemd-logind[1469]: New session 5 of user root. Apr 12 02:36:59 qualcomm-amberwing-rep2-01.khw4.lab.eng.bos.redhat.com systemd[1]: Started Session 5 of user root. Apr 12 02:36:59 qualcomm-amberwing-rep2-01.khw4.lab.eng.bos.redhat.com sshd[4386]: pam_unix(sshd:session): session opened for user root by (uid=0) Apr 12 02:37:27 qualcomm-amberwing-rep2-01.khw4.lab.eng.bos.redhat.com systemd[1]: rshim.service: State 'stop-sigterm' timed out. Killing. Apr 12 02:37:27 qualcomm-amberwing-rep2-01.khw4.lab.eng.bos.redhat.com systemd[1]: rshim.service: Killing process 4363 (rshim) with signal SIGKILL. Apr 12 02:37:27 qualcomm-amberwing-rep2-01.khw4.lab.eng.bos.redhat.com systemd[1]: rshim.service: Failed with result 'exit-code'. Apr 12 02:37:27 qualcomm-amberwing-rep2-01.khw4.lab.eng.bos.redhat.com systemd[1]: Stopped rshim driver for BlueField SoC. The fix is: # dnf install -y psmisc 3. Even after fixing the above, we still fail to load everything: [root@qualcomm-amberwing-rep2-01 ~]# rshim -f Probing pcie-01:00.2 create rshim pcie-01:00.2 Failed to map RShim registers From strace on "rshim -f": write(1, "Probing pcie-01:00.2\n", 21Probing pcie-01:00.2 ) = 21 write(1, "create rshim pcie-01:00.2\n", 26create rshim pcie-01:00.2 ) = 26 openat(AT_FDCWD, "/dev/mem", O_RDWR|O_SYNC) = -1 ENOENT (No such file or directory) ^^ ^^ mmap(NULL, 1048576, PROT_READ|PROT_WRITE, MAP_SHARED, -1, 0x8010030) = -1 EBADF (Bad file descriptor) write(1, "Failed to map RShim registers\n", 30Failed to map RShim registers ) = 30 That's because CONFIG_DEVMEM is not enabled in the kernel: [root@qualcomm-amberwing-rep2-01 ~]# grep CONFIG_DEVMEM /boot/config-4.18.0-147.el8.aarch64 # CONFIG_DEVMEM is not set --> Note; I see that this config is disabled only on aarch64 in RHEL-8. I created a kernel with this config enabled, and then it worked. [root@qualcomm-amberwing-rep2-01 ~]# ls -l /dev/mem crw-r-. 1 root kmem 1, 1 Apr 12 2020 /dev/mem [root@qualcomm-amberwing-rep2-01 ~]# systemctl start rshim [root@qualcomm-amberwing-rep2-01 ~]# systemctl status rshim ● rshim.service - rshim driver for BlueField SoC Loaded: loaded (/usr/lib/systemd/system/rshim.service; disabled; vendor preset: disabled) Active: active (running) since Sun 2020-04-12 05:36:57 EDT; 4s ago Docs: man:rshim(8) Process: 5783 ExecStart=/usr/sbin/rshim $OPTIONS (code=exited, status=0/SUCCESS) Main PID: 5784 (rshim) Tasks: 6
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 Alaa Hleihel (Mellanox) changed: What|Removed |Added Flags|needinfo?(ahleihel@redhat.c | |om) | --- Comment #35 from Alaa Hleihel (Mellanox) --- (In reply to Honggang LI from comment #34) > Hi, Alaa > > I tried to test rshim user space driver with machine > qualcomm-amberwing-rep2-01 . > But I can't get the /dev/rshim* file. Could you please have a look? > > thanks Hi Honggang LI, Sure, I will log in to the system and check why it's not working. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 Honggang LI changed: What|Removed |Added Flags||needinfo?(ahleihel@redhat.c ||om) --- Comment #34 from Honggang LI --- Hi, Alaa I tried to test rshim user space driver with machine qualcomm-amberwing-rep2-01 . But I can't get the /dev/rshim* file. Could you please have a look? thanks -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #33 from l...@mellanox.com --- Thanks all for the comments! Below are the updated info (2.0.2) trying to solve the comments. I kept the "BuildRequires: systemd-units" in order to pass koji build. Spec URL: https://github.com/Mellanox/rshim-user-space/releases/download/rshim-2.0.2/rshim.spec SRPM URL: https://github.com/Mellanox/rshim-user-space/releases/download/rshim-2.0.2/rshim-2.0.2-1.fc31.src.rpm koji build URL: https://koji.fedoraproject.org/koji/taskinfo?taskID=43163450 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #32 from l...@mellanox.com --- (In reply to Michal Schmidt from comment #31) > Thank you Honggang for the detailed review results. > I disagree with the overall PASSED result, though the package is almost > there. > > > Liming, > > please fix this item flagged by fedora-review: > > [!]: Uses parallel make %{?_smp_mflags} macro. > The recommended fix is to use the "%make_build" macro. > > You still have this in the %install section: > > %__spec_install_post > Please remove it. > > About these BRs: > > BuildRequires: systemd-units > > BuildRequires: systemd-rpm-macros > The first one seems unnecessary. > systemd-rpm-macros should be sufficient. Thanks Michal! I'll use %make_build and remove the %__spec_install_post in next post. As for the "systemd-units", it appears to be required by koji. I got some failures like below once removed it. So I'll leave it for now (please correct me if I am incorrect). https://koji.fedoraproject.org/koji/taskinfo?taskID=43163313 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 Michal Schmidt changed: What|Removed |Added Status|NEW |ASSIGNED Flags|fedora-review+ |fedora-review? --- Comment #31 from Michal Schmidt --- Thank you Honggang for the detailed review results. I disagree with the overall PASSED result, though the package is almost there. Liming, please fix this item flagged by fedora-review: > [!]: Uses parallel make %{?_smp_mflags} macro. The recommended fix is to use the "%make_build" macro. You still have this in the %install section: > %__spec_install_post Please remove it. About these BRs: > BuildRequires: systemd-units > BuildRequires: systemd-rpm-macros The first one seems unnecessary. systemd-rpm-macros should be sufficient. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 Honggang LI changed: What|Removed |Added Flags||fedora-review+ --- Comment #30 from Honggang LI --- The srpm package PASSED review. Set the 'fedora-review+' flag for it. Package Review == Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated [ ] = Manual review needed = MUST items = C/C++: [ ]: Package does not contain kernel modules. honli: PASS [ ]: Package contains no static executables. honli: PASS [x]: If your application is a C or C++ application you must list a BuildRequires against gcc, gcc-c++ or clang. [x]: Header files in -devel subpackage, if present. [x]: Package does not contain any libtool archives (.la) [x]: Rpath absent or only used for internal libs. Generic: [ ]: Package is licensed with an open-source compatible license and meets other legal requirements as defined in the legal section of Packaging Guidelines. honli: PASS. See next comment. [ ]: License field in the package spec file matches the actual license. Note: Checking patched sources after %prep for licenses. Licenses found: "*No copyright* GPL (v2)", "Unknown or generated". 19 files have unknown license. Detailed output of licensecheck in /home/honli/1814682-rshim/licensecheck.txt honli: PASS. I had checked all files. Only three files do not have license tag. rshim-2.0.1/README.md, rshim-2.0.1/man/rshim.8, and rshim-2.0.1/bootstrap.sh does not have license tag. It is fine for the first two files as no one add license tag for such kind of files. rshim-2.0.1/bootstrap.sh is a dummy bash script. It is better to add a SPDX-License-Identifier in the head of it. But it is also acceptable to ignore it. [ ]: License file installed when any subpackage combination is installed. honli: PASS. [ ]: Package must own all directories that it creates. Note: Directories without known owners: /usr/lib/systemd/system, /usr/lib/systemd honli: PASS. Both directories are co-own by many packages. [ ]: %build honors applicable compiler flags or justifies otherwise. honli: PASS. Default compiler flags used. [ ]: Package contains no bundled libraries without FPC exception. honli: PASS. No bundled library. [ ]: Changelog in prescribed format. honli: PASS [ ]: Sources contain only permissible code or content. honli: PASS [ ]: Package contains desktop file if it is a GUI application. honli: PASS. No desktop file. [ ]: Development files must be in a -devel package honli: PASS. No development files. [ ]: Package uses nothing in %doc for runtime. honli: PASS. [ ]: Package consistently uses macros (instead of hard-coded directory names). honli: PASS [ ]: Package is named according to the Package Naming Guidelines. honli: PASS [ ]: Package does not generate any conflict. honli: PASS [ ]: Package obeys FHS, except libexecdir and /usr/target. honli: PASS [ ]: If the package is a rename of another package, proper Obsoletes and Provides are present. honli: PASS [ ]: Requires correct, justified where necessary. honli: PASS [ ]: Spec file is legible and written in American English. honli: PASS [ ]: Package contains systemd file(s) if in need. honli: PASS [ ]: Useful -debuginfo package or justification otherwise. honli: PASS [ ]: Package is not known to require an ExcludeArch tag. honli: PASS [ ]: Large documentation must go in a -doc subpackage. Large could be size (~1MB) or number of files. Note: Documentation size is 10240 bytes in 1 files. honli: PASS [ ]: Package complies to the Packaging Guidelines honli: PASS [x]: Package successfully compiles and builds into binary rpms on at least one supported primary architecture. [x]: Package installs properly. [x]: Rpmlint is run on all rpms the build produces. Note: No rpmlint messages. [x]: If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package is included in %license. [x]: Package requires other packages for directories it uses. [x]: Package does not own files or directories owned by other packages. [x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT [x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the beginning of %install. [x]: Macros in Summary, %description expandable at SRPM build time. [x]: Dist tag is present. [x]: Package does not contain duplicates in %files. [x]: Permissions on files are set properly. [x]: Package must not depend on deprecated() packages. [x]: Package use %makeinstall only when make install DESTDIR=... doesn't work. [x]: Package is named using only allowed ASCII characters. [x]: Package does not use a name that already exists. [x]: Package is not relocatable. [x]: Sources used to build the
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 Honggang LI changed: What|Removed |Added Flags|needinfo?(mschmidt@redhat.c | |om) | --- Comment #29 from Honggang LI --- (In reply to lsun from comment #28) > Michal (and Honggang), > > Any advices what the next step would be? Do I need to provide new version to The package is good. We were waiting for hardware to test the package. Unfortunately, we are unlikely get the hardware after a week waiting. We will skip the test of this package for Fedora Rawhide. Redhat QE will test it for RHEL, when this package get into RHEL. > fix anything? Michal, Honggang mentioned that I need to be added into fedora > packager group (comment 15 above). mschmidt will add you into the packager group. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 l...@mellanox.com changed: What|Removed |Added Flags||needinfo?(mschmidt@redhat.c ||om) --- Comment #28 from l...@mellanox.com --- Michal (and Honggang), Any advices what the next step would be? Do I need to provide new version to fix anything? Michal, Honggang mentioned that I need to be added into fedora packager group (comment 15 above). Is it something you could help or any suggestions? Or are we waiting for the verification be complete first? This is my first fedora package. I'll need some guidance and a 'sponsor' according to the wiki https://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group. Thanks! Liming -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #27 from Honggang LI --- (In reply to Alaa Hleihel (Mellanox) from comment #26) > Did you check if you can use the other machines in Beaker? Yes, I sent email to machine owner to ask for access of rdma-qe-04 and rdma-snic-01 . -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #26 from Alaa Hleihel (Mellanox) --- (In reply to Honggang LI from comment #25) > I'm waiting for a pair of external host and SmartNIC to test rshim. The one > I used is unavailable now. Hi Honggang, Did you check if you can use the other machines in Beaker? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #25 from Honggang LI --- I'm waiting for a pair of external host and SmartNIC to test rshim. The one I used is unavailable now. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #24 from l...@mellanox.com --- >>> modprobe: FATAL: Module cuse not found in directory >>> /lib/modules/4.18.0-188.2.el8. bz1655714.v1.aarch64 Honggang, I was a little concerned about the "modprobe: FATAL: Module cuse not found" error message you mentioned earlier. Once you try it on hpe-mantis-01 (the external host machine) and see the same error messages, you could check the Linux configuration CONFIG_CUSE (see below) to see whether it's enabled or not. The user-space driver relies on it to create the /dev/rshimX/. It's usually enabled by default in centos, ubuntu, and default arm64 config. But I am not sure the kernel configuration you're using. CONFIG_FUSE_FS=m CONFIG_CUSE=m Thanks! -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #23 from Alaa Hleihel (Mellanox) --- Thanks! I see is pbunyan's setup :) * mellanox-bluefield-01.khw4.lab.eng.bos.redhat.com This is indeed the BlueFeild card, that we also refer to as the SmartNIC. The 'external' host that host this card is also an ARM64 system in this case (which doesn't matter much for us), it is: hpe-mantis-01.khw4.lab.eng.bos.redhat.com I used to connect to the SmartNIC console from the mantis host using this command: # minicom --baudrate 115200 --device /dev/ttyUSB0 Anyway, you already got a direct SSH connection to this SmartNIC (ssh to mellanox-bluefield-01.khw4.lab.eng.bos.redhat.com host). So to test out the new rshim user-space package, you need to login to the mantis host and install it there. But first, note that you need to remove the rshim kernel module that is currently installed and running there (RPM package rshim-1.16-0.ga7ad4e6_4.18.0_80.el8.aarch64.aarch64) [root@hpe-mantis-01 ~]# lsmod | grep rshim rshim_net 262144 0 rshim_pcie262144 0 rshim 262144 2 rshim_pcie,rshim_net [root@hpe-mantis-01 ~]# (In reply to Honggang LI from comment #22) > (In reply to Alaa Hleihel (Mellanox) from comment #20) > > > Connecting to the SmartNIC is possible via couple of interfaces: > > 1. minicom to the UART/USB interface (if cables are connected) > > 2. using rshim driver on the external host, we get the /dev/rshimX/console > > > Do you mean kernel space driver? or user space driver? Either one will provide this console. But we are moving away from the kernel implementation to the user-space one, so let's focus on the new user-space driver. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #22 from Honggang LI --- (In reply to Alaa Hleihel (Mellanox) from comment #20) > Connecting to the SmartNIC is possible via couple of interfaces: > 1. minicom to the UART/USB interface (if cables are connected) > 2. using rshim driver on the external host, we get the /dev/rshimX/console Do you mean kernel space driver? or user space driver? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #21 from Honggang LI --- (In reply to Alaa Hleihel (Mellanox) from comment #20) > (In reply to Honggang LI from comment #19) > > (In reply to Alaa Hleihel (Mellanox) from comment #18) > > > > > See also https://bugzilla.redhat.com/show_bug.cgi?id=1655736#c7 > > > > on SmartNIC: > > # ip add add 192.168.100.2/24 dev tmfifo_net0 > > # ip -6 addr add 2001::192:168:100:2/112 dev tmfifo_net0 > > > > What does that mean "on SmartNIC"? How can I "login" the SmartNIC? > > I'm confused, weren't you trying to run rshim tool on the SmartNIC before? > > [root@mellanox-bluefield-01 p]# uname -r > What is "mellanox-bluefield-01" host? is this the SmartNIC (BlueField) or > the external host that is hosting the BlueField card? mellanox-bluefield-01.khw4.lab.eng.bos.redhat.com > > > How can I "login" the SmartNIC? > > Connecting to the SmartNIC is possible via couple of interfaces: > 1. minicom to the UART/USB interface (if cables are connected) > 2. using rshim driver on the external host, we get the /dev/rshimX/console > that we can connected to with screen or minicom tools. > > > Can I have access to this system? I sent the accout/pw via email. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #20 from Alaa Hleihel (Mellanox) --- (In reply to Honggang LI from comment #19) > (In reply to Alaa Hleihel (Mellanox) from comment #18) > > > See also https://bugzilla.redhat.com/show_bug.cgi?id=1655736#c7 > > on SmartNIC: > # ip add add 192.168.100.2/24 dev tmfifo_net0 > # ip -6 addr add 2001::192:168:100:2/112 dev tmfifo_net0 > > What does that mean "on SmartNIC"? How can I "login" the SmartNIC? I'm confused, weren't you trying to run rshim tool on the SmartNIC before? > [root@mellanox-bluefield-01 p]# uname -r What is "mellanox-bluefield-01" host? is this the SmartNIC (BlueField) or the external host that is hosting the BlueField card? > How can I "login" the SmartNIC? Connecting to the SmartNIC is possible via couple of interfaces: 1. minicom to the UART/USB interface (if cables are connected) 2. using rshim driver on the external host, we get the /dev/rshimX/console that we can connected to with screen or minicom tools. Can I have access to this system? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #19 from Honggang LI --- (In reply to Alaa Hleihel (Mellanox) from comment #18) > See also https://bugzilla.redhat.com/show_bug.cgi?id=1655736#c7 on SmartNIC: # ip add add 192.168.100.2/24 dev tmfifo_net0 # ip -6 addr add 2001::192:168:100:2/112 dev tmfifo_net0 What does that mean "on SmartNIC"? How can I "login" the SmartNIC? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 Alaa Hleihel (Mellanox) changed: What|Removed |Added Flags|needinfo?(l...@mellanox.com | |) | --- Comment #18 from Alaa Hleihel (Mellanox) --- (In reply to Honggang LI from comment #17) > I tried to test rshim with an aarch64 machine. But I never got the > /dev/rsh*/* files. How can I test rshim? thank > > > [root@mellanox-bluefield-01 p]# rshim -f -l 4 > I see that you are trying to load the rshim driver on the BlueField system, which is wrong. - rshim user-space driver is used on the external host that hosts the BlueField card. - on the BlueField, we have a module named mlxbf_tmfifo that creates the corresponding interface that links to the rshim interface on the external host. See also https://bugzilla.redhat.com/show_bug.cgi?id=1655736#c7 > [root@mellanox-bluefield-01 p]# uname -r > 4.18.0-188.2.el8.bz1655714.v1.aarch64 This kernel build has the mlxbf_tmfifo driver. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 Honggang LI changed: What|Removed |Added Flags||needinfo?(l...@mellanox.com ||) --- Comment #17 from Honggang LI --- I tried to test rshim with an aarch64 machine. But I never got the /dev/rsh*/* files. How can I test rshim? thank [root@mellanox-bluefield-01 p]# rshim -f -l 4 modprobe: FATAL: Module cuse not found in directory /lib/modules/4.18.0-188.2.el8.bz1655714.v1.aarch64 Probing pcie-03:00.2 create rshim pcie-03:00.2 BAR[0] unassigned, run 'lspci -v' ^Cepoll_wait failed; Interrupted system call ^Cepoll_wait failed; Interrupted system call ^Cepoll_wait failed; Interrupted system call ^Cepoll_wait failed; Interrupted system call ^Cepoll_wait failed; Interrupted system call ^Cepoll_wait failed; Interrupted system call ^Cepoll_wait failed; Interrupted system call ^C^Cepoll_wait failed; Interrupted system call ^Z [1]+ Stopped rshim -f -l 4 [root@mellanox-bluefield-01 p]# ps -ef | grep rshim root 112134877 2 07:10 pts/000:00:00 rshim -f -l 4 root 112394877 0 07:11 pts/000:00:00 grep --color=auto rshim [root@mellanox-bluefield-01 p]# kill -9 11213 [root@mellanox-bluefield-01 p]# [1]+ Killed rshim -f -l 4 [root@mellanox-bluefield-01 p]# [root@mellanox-bluefield-01 p]# lspci -v 00:00.0 PCI bridge: Mellanox Technologies MT416842 BlueField SoC Crypto enabled (prog-if 00 [Normal decode]) Flags: bus master, fast devsel, latency 0, IRQ 75 Bus: primary=00, secondary=01, subordinate=03, sec-latency=0 I/O behind bridge: -0fff [size=4K] Memory behind bridge: -001f [size=2M] Prefetchable memory behind bridge: 00e2-00e204ff [size=80M] Capabilities: [60] Express Root Port (Slot-), MSI 00 Capabilities: [e0] MSI: Enable+ Count=4/4 Maskable+ 64bit+ Capabilities: [40] Power Management version 3 Capabilities: [100] Advanced Error Reporting Capabilities: [1c0] #19 Capabilities: [230] Access Control Services Capabilities: [320] #27 Capabilities: [370] #26 Capabilities: [430] Downstream Port Containment Kernel driver in use: pcieport 01:00.0 PCI bridge: Mellanox Technologies MT416842 Family [BlueField SoC PCIe Bridge] (prog-if 00 [Normal decode]) Flags: bus master, fast devsel, latency 0, IRQ 78 Bus: primary=01, secondary=02, subordinate=03, sec-latency=0 I/O behind bridge: -0fff [size=4K] Memory behind bridge: -001f [size=2M] Prefetchable memory behind bridge: 00e2-00e204ff [size=80M] Capabilities: [60] Express Upstream Port, MSI 00 Capabilities: [e0] MSI: Enable+ Count=1/4 Maskable+ 64bit+ Capabilities: [40] Power Management version 3 Capabilities: [100] Advanced Error Reporting Capabilities: [1c0] #19 Capabilities: [320] #27 Capabilities: [370] #26 Kernel driver in use: pcieport 02:02.0 PCI bridge: Mellanox Technologies MT416842 Family [BlueField SoC PCIe Bridge] (prog-if 00 [Normal decode]) Flags: bus master, fast devsel, latency 0, IRQ 79 Bus: primary=02, secondary=03, subordinate=03, sec-latency=0 I/O behind bridge: -0fff [size=4K] Memory behind bridge: -001f [size=2M] Prefetchable memory behind bridge: 00e2-00e204ff [size=80M] Capabilities: [60] Express Downstream Port (Slot-), MSI 00 Capabilities: [e0] MSI: Enable+ Count=2/4 Maskable+ 64bit+ Capabilities: [40] Power Management version 3 Capabilities: [100] Advanced Error Reporting Capabilities: [1c0] #19 Capabilities: [230] Access Control Services Capabilities: [320] #27 Capabilities: [370] #26 Capabilities: [430] Downstream Port Containment Kernel driver in use: pcieport 03:00.0 Ethernet controller: Mellanox Technologies MT416842 BlueField integrated ConnectX-5 network controller Subsystem: Mellanox Technologies Device 0029 Flags: bus master, fast devsel, latency 0, IRQ 81 Memory at e2 (64-bit, prefetchable) [size=32M] Memory at e20400 (64-bit, prefetchable) [size=2M] Expansion ROM at e0 [disabled] [size=1M] Capabilities: [60] Express Endpoint, MSI 00 Capabilities: [48] Vital Product Data Capabilities: [9c] MSI-X: Enable+ Count=64 Masked- Capabilities: [c0] Vendor Specific Information: Len=18 Capabilities: [40] Power Management version 3 Capabilities: [100] Advanced Error Reporting Capabilities: [150] Alternative Routing-ID Interpretation (ARI)
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #16 from l...@mellanox.com --- Thanks Honggang! The 'TAINTED_SCALAR +1' is an easy fix. I'll make it in next version. The "SLEEP +2" is the same as "Error: SLEEP (CWE-367): [#def26]" in last build, which is ok since this function happens during driver probe which could take while. (In reply to Honggang LI from comment #14) ... > Task URL: https://cov01.lab.eng.brq.redhat.com/covscanhub/task/164815/ > Comment: > > > Added (+), Fixed (-) > > SLEEP +2 > TAINTED_SCALAR+1 ... > * > > > Newly introduced defects > List of Defects > > Error: TAINTED_SCALAR (CWE-20): [#def1] ... > > Error: SLEEP (CWE-367): [#def2] ... > > Error: SLEEP (CWE-367): [#def3] ... -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 Honggang LI changed: What|Removed |Added Assignee|ho...@redhat.com|mschm...@redhat.com --- Comment #15 from Honggang LI --- Michal Assign this bug to you, as I don't have the permission to add lsun into the fedora packager group. thanks -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #14 from Honggang LI --- (In reply to lsun from comment #13) > Thanks all for the comments! > Below are the updated info (2.0.1) trying to solve these comments. > > Spec URL: > https://github.com/Mellanox/rshim-user-space/releases/download/rshim-2.0.1/ > rshim.spec > SRPM URL: > https://github.com/Mellanox/rshim-user-space/releases/download/rshim-2.0.1/ > rshim-2.0.1-1.fc31.src.rpm * Task method: VersionDiffBuild Task URL: https://cov01.lab.eng.brq.redhat.com/covscanhub/task/164815/ Comment: Added (+), Fixed (-) SLEEP +2 TAINTED_SCALAR+1 BUFFER_SIZE_WARNING -1 CLANG_WARNING -4 LOCK -2 OVERRUN -1 PW.BAD_PRINTF_FORMAT_STRING -3 RESOURCE_LEAK -1 SIZEOF_MISMATCH -1 SLEEP -4 TAINTED_SCALAR-7 * Newly introduced defects List of Defects Error: TAINTED_SCALAR (CWE-20): [#def1] rshim-2.0.1/src/rshim.c:2209: tainted_argument: Calling function "rshim_fd_full_read" taints argument "index". rshim-2.0.1/src/rshim.c:2211: tainted_data: Using tainted variable "index" as an index into an array "rshim_devs". rshim-2.0.1/src/rshim.c:2211: remediation: Ensure that tainted values are properly sanitized, by checking that their values are within a permissible range. # 2209| rc = rshim_fd_full_read(rshim_work_fd[0], , sizeof(index)); # 2210| if (rc == sizeof(index)) { # 2211|-> bd = rshim_devs[index]; # 2212| if (bd) # 2213| rshim_work_handler(bd); Error: SLEEP (CWE-367): [#def2] rshim-2.0.1/src/rshim_pcie.c:392: lock_acquire: Calling function "pthread_mutex_lock" acquires lock "bd->mutex". rshim-2.0.1/src/rshim_pcie.c:400: sleep: Call to "rshim_register" might sleep while holding lock "bd->mutex". # 398| */ # 399| rshim_lock(); # 400|-> ret = rshim_register(bd); # 401| if (ret) { # 402| rshim_unlock(); Error: SLEEP (CWE-367): [#def3] rshim-2.0.1/src/rshim_pcie_lf.c:524: lock_acquire: Calling function "pthread_mutex_lock" acquires lock "bd->mutex". rshim-2.0.1/src/rshim_pcie_lf.c:532: sleep: Call to "rshim_register" might sleep while holding lock "bd->mutex". # 530| */ # 531| rshim_lock(); # 532|-> ret = rshim_register(bd); # 533| if (ret) { # 534| rshim_unlock(); > koji build URL: https://koji.fedoraproject.org/koji/taskinfo?taskID=42727191 > > Below are the unsolved coverity warnings and explanations (based on Comment > 12): > > Error: LOCK (CWE-667): [#def2] > [lsun] Function rshim_write_delayed() has lock held outside by the caller. > No need to unlock it on return. > > Error: LOCK (CWE-667): [#def3] > [lsun] same. > > Error: MISSING_LOCK (CWE-667): [#def4] > [lsun] This is the RSH_DEV_TYPE_TMFIFO case and is called from > rshim_fifo_input()and rshim_fifo_output(). In both cases the ringlock are > already held. > > Error: MISSING_LOCK (CWE-667): [#def5] > [lsun] This one is called from the 'RSH_EVENT_ATTACH' handling, which > already has the lock held when calling rshim_notify(bd, RSH_EVENT_ATTACH, 0). > > Error: MISSING_LOCK (CWE-667): [#def6] > [lsun] same > > Error: SLEEP (CWE-367): [#def26] > rshim_register(bd); > [lsun] This one is called during device probe at early stage. The sleep > purpose is to detect if any other driver has already attached to the same > rshim device (since it could be attached from USB or PCIe via different host > machine). The probing takes some time. The sleep here is ok. > > Error: MISSING_LOCK (CWE-667): [#def24] > Error: MISSING_LOCK (CWE-667): [#def25] > [lsun] same as [#def4 above. Thanks for the comments. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 l...@mellanox.com changed: What|Removed |Added Flags|needinfo?(l...@mellanox.com | |) | --- Comment #13 from l...@mellanox.com --- Thanks all for the comments! Below are the updated info (2.0.1) trying to solve these comments. Spec URL: https://github.com/Mellanox/rshim-user-space/releases/download/rshim-2.0.1/rshim.spec SRPM URL: https://github.com/Mellanox/rshim-user-space/releases/download/rshim-2.0.1/rshim-2.0.1-1.fc31.src.rpm koji build URL: https://koji.fedoraproject.org/koji/taskinfo?taskID=42727191 Below are the unsolved coverity warnings and explanations (based on Comment 12): Error: LOCK (CWE-667): [#def2] [lsun] Function rshim_write_delayed() has lock held outside by the caller. No need to unlock it on return. Error: LOCK (CWE-667): [#def3] [lsun] same. Error: MISSING_LOCK (CWE-667): [#def4] [lsun] This is the RSH_DEV_TYPE_TMFIFO case and is called from rshim_fifo_input()and rshim_fifo_output(). In both cases the ringlock are already held. Error: MISSING_LOCK (CWE-667): [#def5] [lsun] This one is called from the 'RSH_EVENT_ATTACH' handling, which already has the lock held when calling rshim_notify(bd, RSH_EVENT_ATTACH, 0). Error: MISSING_LOCK (CWE-667): [#def6] [lsun] same Error: SLEEP (CWE-367): [#def26] rshim_register(bd); [lsun] This one is called during device probe at early stage. The sleep purpose is to detect if any other driver has already attached to the same rshim device (since it could be attached from USB or PCIe via different host machine). The probing takes some time. The sleep here is ok. Error: MISSING_LOCK (CWE-667): [#def24] Error: MISSING_LOCK (CWE-667): [#def25] [lsun] same as [#def4 above. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #12 from Michal Schmidt --- Honggang provided his test results in: https://people.redhat.com/honli/bz1814682/rshim-2.0-1.fc31.tar.xz -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #11 from Michal Schmidt --- I reproduced it on a Fedora system. You can install csmock: dnf install csmock And apply the "clang" checker on your source RPM: csmock -t clang -r fedora-rawhide-x86_64 -o output/ ./rshim-2.0-1.fc31.src.rpm You'll find "scan-results.*" in the "output/" directory. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #10 from l...@mellanox.com --- (In reply to Honggang LI from comment #8) > (In reply to lsun from comment #4) > > > Thanks Honggang for the comments. I am working on it and will post new > > version after fixing these warnings. > > Please fix those issue for this fedora package review. When we import this > package for RHEL-8, > we will have to fix those issue. Because it is a mandatory task for RHEL > package import. > > Fix those issues earlier will save time for us in the future. > > Thanks Yes, definitely. I have been working on it and planned to submit a new version today. One question about the coverity warnings: Is there a way I could run koji to get the same output? Or is it possible you could give me some details about this warning: "4 CLANG_WARNING"? I ran coverity locally but couldn't get this one. Thanks! -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #9 from l...@mellanox.com --- (In reply to Honggang LI from comment #7) > 1814682-rshim]$ rpm -qpl results/rshim-2.0-1.fc33.x86_64.rpm | grep man > /usr/share/man/man1/bfrshim.1.gz > ^ > > 1814682-rshim]$ rpm -qpl results/rshim-2.0-1.fc33.x86_64.rpm | grep bin > /usr/bin/bfrshim > > > ./src/rshim.c:2092: rc = system("modprobe cuse"); > ./src/rshim_net.c:56: rc = system("modprobe tun"); > > The binary 'bfrshim' may execute 'modprobe' command, which requires > administrator permission. > I suggest to move 'bfrshim' into the '/usr/sbin' directory, and install the > manpage into man-8 section instead of man-1. This binary does need root permission to run. I'll update it as suggested. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #8 from Honggang LI --- (In reply to lsun from comment #4) > Thanks Honggang for the comments. I am working on it and will post new > version after fixing these warnings. Please fix those issue for this fedora package review. When we import this package for RHEL-8, we will have to fix those issue. Because it is a mandatory task for RHEL package import. Fix those issues earlier will save time for us in the future. Thanks -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #7 from Honggang LI --- 1814682-rshim]$ rpm -qpl results/rshim-2.0-1.fc33.x86_64.rpm | grep man /usr/share/man/man1/bfrshim.1.gz ^ 1814682-rshim]$ rpm -qpl results/rshim-2.0-1.fc33.x86_64.rpm | grep bin /usr/bin/bfrshim ./src/rshim.c:2092: rc = system("modprobe cuse"); ./src/rshim_net.c:56: rc = system("modprobe tun"); The binary 'bfrshim' may execute 'modprobe' command, which requires administrator permission. I suggest to move 'bfrshim' into the '/usr/sbin' directory, and install the manpage into man-8 section instead of man-1. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #6 from l...@mellanox.com --- (In reply to Michal Schmidt from comment #3) > The github project name is "rshim-user-space", the package name is "rshim", > the daemon binary and the systemd service are called "bfrshim". > Would it be too late to make them the same? Just a proposal. Feel free to > disagree. It's good suggestion. A little history that this package is user-space driver and is intended to replace the kernel module implementation which has name 'rshim' on GitHub. Both are called 'rshim' for easy upgrade since they have exactly the same functionality and user interface. RPM version < 2.0 is for kernel modules implementation, >= 2.0 is for user space implementation. I'll rename bfrshim to 'rshim' directly to avoid confusion on this one. Thanks! -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #5 from l...@mellanox.com --- (In reply to Michal Schmidt from comment #2) > Commenting on a few lines in rshim.spec: > > > URL: https://github.com/mellanox/rshim-user-space > > Seeing the name "rshim-user-space" made me curious if there's a kernel part > and I found: > https://github.com/mellanox/rshim > "BlueField RSHIM driver" > > What is the relation between the kernel and the userspace drivers? > Was the kernel driver the original approach which later got obsoleted by the > userspace implementation? Yes, the kernel driver is the original approach, which has upstreaming difficulties due to mixed functionalities. This 'rshim-user-space' driver intends to replace the kernel driver with exactly the same user interface. > > > Obsoletes: %{name} < 2.0 > > This line says: rshim-2.0 obsoletes rshim < 2.0. That seems redundant. Will remove it. > > > %global with_systemd %(if ( test -d "%{_unitdir}" > /dev/null); then echo > > -n '1'; else echo -n '0'; fi) > > Why not assume systemd always? The driver might be used on some older OS version (like centos 6) which doesn't have systemd. The check here is to prevent source RPM build issues for such case. This package will also be included in Mellanox OFED package which needs to support different build environments. > > > %prep > > rm -fr %{name}-%{version} > > mkdir %{name}-%{version} > > tar -axf %{SOURCE0} -C %{name}-%{version} --strip-components 1 > > %setup -q -D -T > > Would this simpler way work too?: > %prep > %setup -n rshim-user-space-%{name}-%{version} > > Or maybe, since you are the upstream maintainer, you could change the > upstream tarball generation to avoid the odd directory name? I had a little trouble with the name. The current tarball with such top-level odd name was generated by GitHub automatically when a release/tag is created. But you're right. I could actually upload the tarball and use it instead. Will make the change in next post. > > > %install > > %makeinstall -C src INSTALL_DIR="%{buildroot}%{_bindir}" > > %if "%{with_systemd}" == "1" > > %{__install} -d %{buildroot}%{_unitdir} > > %{__install} -m 0644 bfrshim.service %{buildroot}%{_unitdir} > > %endif > > %{__install} -d %{buildroot}%{_mandir}/man1 > > %{__install} -m 0644 man/bfrshim.1 %{buildroot}%{_mandir}/man1 > > Why don't you let %makeinstall install these? Will update it in next post. > > > %__spec_install_post > > This looks like duplicating rpm-build's work. Will update it in next post. > > > %post > > %if "%{with_systemd}" == "1" > > systemctl daemon-reload > > systemctl enable bfrshim.service > > %endif > > No, you need to use the macros from the packaging guidelines: > https://docs.fedoraproject.org/en-US/packaging-guidelines/Scriptlets/ > #_systemd Will update it in next post. > > > %files > > %license LICENSE > > %defattr(-,root,root,-) > > This looks like the default, so the %defattr line is redundant. Will remove it in next post. > > > %%doc README.md > > Double % sign, typo? Will fix it in next post. > > > %changelog > > * Fri Mar 13 2020 Liming Sun - 2.0-1 > >- Update the spec file according to fedora packaging-guidelines > > * Mon Dec 16 2019 Liming Sun > >- Initial packaging > > Please add an empty line between changelog entries. Will fix it in next post. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #4 from l...@mellanox.com --- (In reply to Honggang LI from comment #1) > http://people.redhat.com/honli/bz1814682/ > > 1 BUFFER_SIZE_WARNING > 4 CLANG_WARNING > 2 LOCK > 5 MISSING_LOCK > 1 OVERRUN > 1 RESOURCE_LEAK > 1 SIZEOF_MISMATCH > 5 SLEEP > 7 TAINTED_SCALAR > > Hi, please fix the issues found by Coverity. If you think an issue is false > positive, please explain it. Thanks Honggang for the comments. I am working on it and will post new version after fixing these warnings. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 --- Comment #3 from Michal Schmidt --- The github project name is "rshim-user-space", the package name is "rshim", the daemon binary and the systemd service are called "bfrshim". Would it be too late to make them the same? Just a proposal. Feel free to disagree. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 Michal Schmidt changed: What|Removed |Added CC||mschm...@redhat.com --- Comment #2 from Michal Schmidt --- Commenting on a few lines in rshim.spec: > URL: https://github.com/mellanox/rshim-user-space Seeing the name "rshim-user-space" made me curious if there's a kernel part and I found: https://github.com/mellanox/rshim "BlueField RSHIM driver" What is the relation between the kernel and the userspace drivers? Was the kernel driver the original approach which later got obsoleted by the userspace implementation? > Obsoletes: %{name} < 2.0 This line says: rshim-2.0 obsoletes rshim < 2.0. That seems redundant. > %global with_systemd %(if ( test -d "%{_unitdir}" > /dev/null); then echo -n > '1'; else echo -n '0'; fi) Why not assume systemd always? > %prep > rm -fr %{name}-%{version} > mkdir %{name}-%{version} > tar -axf %{SOURCE0} -C %{name}-%{version} --strip-components 1 > %setup -q -D -T Would this simpler way work too?: %prep %setup -n rshim-user-space-%{name}-%{version} Or maybe, since you are the upstream maintainer, you could change the upstream tarball generation to avoid the odd directory name? > %install > %makeinstall -C src INSTALL_DIR="%{buildroot}%{_bindir}" > %if "%{with_systemd}" == "1" > %{__install} -d %{buildroot}%{_unitdir} > %{__install} -m 0644 bfrshim.service %{buildroot}%{_unitdir} > %endif > %{__install} -d %{buildroot}%{_mandir}/man1 > %{__install} -m 0644 man/bfrshim.1 %{buildroot}%{_mandir}/man1 Why don't you let %makeinstall install these? > %__spec_install_post This looks like duplicating rpm-build's work. > %post > %if "%{with_systemd}" == "1" > systemctl daemon-reload > systemctl enable bfrshim.service > %endif No, you need to use the macros from the packaging guidelines: https://docs.fedoraproject.org/en-US/packaging-guidelines/Scriptlets/#_systemd > %files > %license LICENSE > %defattr(-,root,root,-) This looks like the default, so the %defattr line is redundant. > %%doc README.md Double % sign, typo? > %changelog > * Fri Mar 13 2020 Liming Sun - 2.0-1 >- Update the spec file according to fedora packaging-guidelines > * Mon Dec 16 2019 Liming Sun >- Initial packaging Please add an empty line between changelog entries. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 Honggang LI changed: What|Removed |Added Flags||needinfo?(l...@mellanox.com ||) --- Comment #1 from Honggang LI --- http://people.redhat.com/honli/bz1814682/ 1 BUFFER_SIZE_WARNING 4 CLANG_WARNING 2 LOCK 5 MISSING_LOCK 1 OVERRUN 1 RESOURCE_LEAK 1 SIZEOF_MISMATCH 5 SLEEP 7 TAINTED_SCALAR Hi, please fix the issues found by Coverity. If you think an issue is false positive, please explain it. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 Honggang LI changed: What|Removed |Added CC||ho...@redhat.com Assignee|nob...@fedoraproject.org|ho...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC
https://bugzilla.redhat.com/show_bug.cgi?id=1814682 Alaa Hleihel (Mellanox) changed: What|Removed |Added CC||ahlei...@redhat.com Blocks||1655882, 1744737 Doc Type|--- |If docs needed, set a value -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org