Bug 771485

Summary: Review Request: libcxgb4 - Chelsio T4 iWARP HCA Userspace Driver
Product: [Fedora] Fedora Reporter: Doug Ledford <dledford>
Component: Package ReviewAssignee: Albert Strasheim <fullung>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: fenlason, fullung, notting, package-review
Target Milestone: ---Flags: fenlason: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
URL: http://people.redhat.com/dledford/Package%20Review/
Whiteboard:
Fixed In Version: libcxgb4-1.2.0-1.fc16 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-02-01 19:23:43 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Doug Ledford 2012-01-03 21:21:38 UTC
This package is a hardware driver that enables Chelsio T4 hardware to be used as an iWARP capable device in combination with the libibverbs package.  The libibverbs package is the hardware independent RDMA library, and the libcxgb4 package is one of multiple possible hardware driver modules that are abstracted out by libibverbs.

The package for review can be found at the URL above.

Comment 1 Albert Strasheim 2012-01-09 13:30:07 UTC
$ rpmlint libcxgb4*
libcxgb4.x86_64: W: spelling-error Summary(en_US) Chelsio -> Chelsea
libcxgb4.x86_64: W: spelling-error Summary(en_US) iWARP -> i Warp, warp, antiwar
libcxgb4.x86_64: W: spelling-error %description -l en_US Userspace -> User space, User-space, Users pace
libcxgb4.x86_64: W: spelling-error %description -l en_US libibverbs -> verbalizes
libcxgb4.x86_64: W: spelling-error %description -l en_US iWARP -> i Warp, warp, antiwar
libcxgb4.x86_64: W: obsolete-not-provided libcxgb4-devel
libcxgb4.x86_64: W: non-conffile-in-etc /etc/modprobe.d/libcxgb4.conf
libcxgb4.x86_64: W: non-conffile-in-etc /etc/libibverbs.d/cxgb4.driver
libcxgb4.spec:13: W: unversioned-explicit-obsoletes %{name}-devel
libcxgb4.spec:15: W: unversioned-explicit-provides libibverbs-driver.%{_arch}
libcxgb4-static.x86_64: W: no-documentation
2 packages and 1 specfiles checked; 0 errors, 11 warnings.

I don't have this hardware, so I can't test further unfortunately.

I noticed that libmlx4-static provides libmlx4-devel-static. What's up with that? Should libcxgb4-static do the same?

Comment 2 Doug Ledford 2012-01-09 19:29:05 UTC
libmlx4 provides libmlx4-devel-static because early versions of the package provided that.  We switched the package naming over to the more standard compliant base base-devel base-static long ago and the old provides is in libmlx4 to keep any software that didn't get updated from breaking.  Libcxgb4 has no such broken history and need not provide -devel-static as a package.

Comment 3 Doug Ledford 2012-01-11 19:08:11 UTC
Albert, are you good with signing off on this package at this point?

Comment 4 Albert Strasheim 2012-01-11 19:23:24 UTC
Looks good to me.

Comment 5 Doug Ledford 2012-01-11 20:10:53 UTC
New Package SCM Request
=======================
Package Name: libcxgb4
Short Description: Driver for Chelsio T4 hardware
Owners: dledford
Branches: f15 f16
InitialCC:

Comment 6 Gwyn Ciesla 2012-01-11 20:36:16 UTC
Git done (by process-git-requests).

Comment 7 Fedora Update System 2012-01-11 20:52:27 UTC
libcxgb4-1.2.0-1.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/libcxgb4-1.2.0-1.fc15

Comment 8 Fedora Update System 2012-01-11 20:52:36 UTC
libcxgb4-1.2.0-1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/libcxgb4-1.2.0-1.fc16

Comment 9 Fedora Update System 2012-01-15 20:05:21 UTC
libcxgb4-1.2.0-1.fc16 has been pushed to the Fedora 16 testing repository.

Comment 10 Fedora Update System 2012-02-01 19:23:43 UTC
libcxgb4-1.2.0-1.fc15 has been pushed to the Fedora 15 stable repository.

Comment 11 Fedora Update System 2012-02-01 19:28:07 UTC
libcxgb4-1.2.0-1.fc16 has been pushed to the Fedora 16 stable repository.