[Bug 1814682] Review Request: rshim - rshim driver for Mellanox BlueField SoC

2020-05-06 Thread bugzilla
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

2020-05-06 Thread bugzilla
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

2020-05-06 Thread bugzilla
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

2020-05-06 Thread bugzilla
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

2020-05-06 Thread bugzilla
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

2020-05-06 Thread bugzilla
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

2020-05-06 Thread bugzilla
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

2020-05-05 Thread bugzilla
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

2020-05-05 Thread bugzilla
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

2020-05-05 Thread bugzilla
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

2020-05-04 Thread bugzilla
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

2020-04-29 Thread bugzilla
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

2020-04-29 Thread bugzilla
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

2020-04-28 Thread bugzilla
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

2020-04-23 Thread bugzilla
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

2020-04-22 Thread bugzilla
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

2020-04-22 Thread bugzilla
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

2020-04-20 Thread bugzilla
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

2020-04-20 Thread bugzilla
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

2020-04-14 Thread bugzilla
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

2020-04-14 Thread bugzilla
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

2020-04-13 Thread bugzilla
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

2020-04-12 Thread bugzilla
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

2020-04-12 Thread bugzilla
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

2020-04-12 Thread bugzilla
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

2020-04-12 Thread bugzilla
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

2020-04-12 Thread bugzilla
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

2020-04-12 Thread bugzilla
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

2020-04-12 Thread bugzilla
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

2020-04-12 Thread bugzilla
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

2020-04-12 Thread bugzilla
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

2020-04-10 Thread bugzilla
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

2020-04-09 Thread bugzilla
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

2020-04-09 Thread bugzilla
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

2020-04-08 Thread bugzilla
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

2020-04-08 Thread bugzilla
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

2020-04-08 Thread bugzilla
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

2020-04-06 Thread bugzilla
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

2020-03-29 Thread bugzilla
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

2020-03-29 Thread bugzilla
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

2020-03-27 Thread bugzilla
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

2020-03-25 Thread bugzilla
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

2020-03-25 Thread bugzilla
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

2020-03-25 Thread bugzilla
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

2020-03-25 Thread bugzilla
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

2020-03-25 Thread bugzilla
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

2020-03-25 Thread bugzilla
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

2020-03-25 Thread bugzilla
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

2020-03-25 Thread bugzilla
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

2020-03-24 Thread bugzilla
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

2020-03-24 Thread bugzilla
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

2020-03-23 Thread bugzilla
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

2020-03-23 Thread bugzilla
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

2020-03-23 Thread bugzilla
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

2020-03-23 Thread bugzilla
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

2020-03-23 Thread bugzilla
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

2020-03-23 Thread bugzilla
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

2020-03-22 Thread bugzilla
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

2020-03-22 Thread bugzilla
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

2020-03-20 Thread bugzilla
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

2020-03-20 Thread bugzilla
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

2020-03-20 Thread bugzilla
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

2020-03-20 Thread bugzilla
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

2020-03-20 Thread bugzilla
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

2020-03-19 Thread bugzilla
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

2020-03-19 Thread bugzilla
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

2020-03-18 Thread bugzilla
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