Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add patches from the Debian package #74

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

thomasgoirand
Copy link

Hi,

Here's 4 patches that we are currently using in the Debian version of lshw. It would be very helpful if these could be merged upstream (even better if a new release can happen, and even awesome if you could from now on use a better version number with letters in the tags...).

Cheers,

Thomas Goirand (zigo)

This is needed with more recent GCC versions.
Independant -> Independent
lshw was only capable of reporting link types up to SUPPORTED_10000baseT_Full,
meaning that nothing above 10Gbits/s plus many other new link types where not
reported at all, leading to faster cards not reporting their speeds.

This patch adds new capacities, with the information taken from the libc
usr/include/linux/ethtool.h (ie: the linux-libc-dev package).

FYI, I needed this patch because lshw was not reporting any speed for our
MT27710 Family [ConnectX-4 Lx] (Mellanox Technologies) 25Gbits/s cards, so I
thought I would add the speed reporting for that one, then since I was doing
the job, I added all standards documented in ethtool.h.
In Debian, we need to be able to override pkg-config, so we can
cross compile.
@yarrick
Copy link
Contributor

yarrick commented Oct 17, 2021

Looks like most of this is merged.

For network adding modes above 31 doesn't help (it only returns 32 bits). I added support for longer bitmaps in #73 and just added 25/40G in #75 - maybe you can help test it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants