RHEL Engineering is moving the tracking of its product development work on RHEL 6 through RHEL 9 to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "RHEL project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs in the statuses "NEW", "ASSIGNED", and "POST" are being migrated throughout September 2023. Bugs of Red Hat partners with an assigned Engineering Partner Manager (EPM) are migrated in late September as per pre-agreed dates. Bugs against components "kernel", "kernel-rt", and "kpatch" are only migrated if still in "NEW" or "ASSIGNED". If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "RHEL project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/RHEL-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.
Bug 2002344 - please enable the e1000 driver again
Summary: please enable the e1000 driver again
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Linux 9
Classification: Red Hat
Component: kernel
Version: CentOS Stream
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: rc
: ---
Assignee: Ken Cox
QA Contact: Dipali
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-09-08 15:17 UTC by Davide Cavalca
Modified: 2022-08-31 03:29 UTC (History)
20 users (show)

Fixed In Version: kernel-5.14.0-1.6.1.el9
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-12-02 20:06:29 UTC
Type: Bug
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Gitlab redhat/rhel/src/kernel rhel-9 merge_requests 60 0 None None None 2021-09-17 10:23:38 UTC
Red Hat Issue Tracker RHELPLAN-96571 0 None None None 2021-09-08 15:18:14 UTC

Description Davide Cavalca 2021-09-08 15:17:09 UTC
CentOS Stream 9 currently disables the e1000 driver. This was done in https://gitlab.com/cki-project/kernel-ark/-/merge_requests/10 . e1000 is still the default for several virtualization platforms (e.g. vmware), and there's still plenty of usable physical systems with NICs that require this driver in the field. Please consider enabling it back.

Comment 1 farrotin 2021-09-08 15:33:11 UTC
FWIW, it's actually also blocking CentOS CI infra, as the hardware we have is using e1000 nic https://pagure.io/centos-infra/issue/441
We also got some people asking why they couldn't install CentOS Stream 9 through vmware workstation, only to figure out that it's the emulated nic , instead of the vmxnet3 one (which is better and works)

Comment 2 Michal Schmidt 2021-09-10 07:58:29 UTC
In RHEL 8, the driver is enabled, with the following RHEL-specific addition in e1000_init_module():

        add_taint(TAINT_SUPPORT_REMOVED, LOCKDEP_STILL_OK);
        pr_warn("E1000 MODULE IS NOT SUPPORTED\n");

IF we enable it for RHEL 9, there needs to be something similar.
The mptsas driver in RHEL 9 does it too, see drivers/message/fusion/mptsas.c:mptsas_probe(), added by commit c2b3035917b85 ("mptsas: Taint kernel if mptsas is loaded"):

#ifdef CONFIG_RHEL_DIFFERENCES
        add_taint(TAINT_SUPPORT_REMOVED, LOCKDEP_STILL_OK);
        pr_warn("MPTSAS MODULE IS NOT SUPPORTED\n");
#endif

Comment 10 John W. Linville 2021-09-13 15:20:50 UTC
https://www.intel.com/content/www/us/en/support/articles/000026530/ethernet-products/legacy-ethernet-products.html

It looks like there might be a handful of e1000 cards young enough to still qualify for Intel's 5-year warranty...but not many...

Re: comment 1, my personal advice would be for someone to buy a new NIC and stuff it in that CI server.

Comment 11 farrotin 2021-09-13 15:31:39 UTC
(In reply to John W. Linville from comment #10)
> https://www.intel.com/content/www/us/en/support/articles/000026530/ethernet-
> products/legacy-ethernet-products.html
> 
> It looks like there might be a handful of e1000 cards young enough to still
> qualify for Intel's 5-year warranty...but not many...
> 
> Re: comment 1, my personal advice would be for someone to buy a new NIC and
> stuff it in that CI server.

Well, we have 256 seamicro compute nodes/cards in 4 chassis ( 4*64 compute nodes), all using that onboard nic and no way to replace it .. :)
but I get your point. if there is no way to enable it, we'll have to decide if we want to not offer stream9 tests for projects (like rdo/openstack/ovirt/gluster, etc) or if someone can then sponsor replacement hardware (all out of warranty) , but not something I have to decide, I'll just be messenger

Comment 12 John W. Linville 2021-09-13 17:04:54 UTC
Don't mind me -- I may be grumpy about it, but this is out of my hands as well. ;-)

Comment 29 Dipali 2021-10-05 17:35:13 UTC
Ran and verified tier1 and tier2 tests for kernel version 5.14.0-1.el9.x86_64

Following tests are run

/kernel/networking/nic/sanity_check
/kernel/networking/nic/functions/packet_statistics 
/kernel/networking/nic/functions/link_check 
/kernel/networking/nic/functions/ring 
/kernel/networking/nic/functions/offload 
/kernel/networking/nic/functions/multiqueue
/kernel/networking/nic/functions/devlink 
/kernel/networking/nic/functions/scaling 
/kernel/networking/nic/functions/stress
/kernel/networking/nic/functions/abnormal
/kernel/networking/nic/functions/specific 
/kernel/networking/nic/functions/vlan_geneve
/kernel/networking/nic/functions/vlan_vxlan
/kernel/networking/nic/functions/topo_reboot
/kernel/networking/nic/functions/wol
/kernel/networking/nic/functions/coalesce 
/kernel/networking/nic/functions/pause 
/kernel/networking/nic/functions/hashkey 
/kernel/networking/nic/functions/ethtool/Regression 
/kernel/networking/nic/functions/ethtool/sanity
/kernel/networking/nic/functions/nic_driver_load 
/kernel/networking/netperf/regular_netperf

Comment 30 farrotin 2021-10-06 09:32:33 UTC
Just for my own planning (centos infra), can we get an ETA when it will be enabled for Stream 9 kernel ? The kernel versions available on https://kojihub.stream.centos.org/koji/packageinfo?packageID=800 are either lower or higher so I guess the you mention was branched off for rhel9 itself .. but I guess it should be then reflected in stream itself ?

Comment 31 Jan Stancek 2021-10-06 09:55:14 UTC
(In reply to farrotin from comment #30)
> Just for my own planning (centos infra), can we get an ETA when it will be
> enabled for Stream 9 kernel ? The kernel versions available on
> https://kojihub.stream.centos.org/koji/packageinfo?packageID=800 are either
> lower or higher so I guess the you mention was branched off for rhel9 itself
> .. but I guess it should be then reflected in stream itself ?

It's been already merged to stream 9 kernel:
  https://gitlab.com/redhat/centos-stream/src/kernel/centos-stream-9/-/commit/d334fb74543bacbec67428b69a6c55a75d2d7cc5
so I estimate couple days until Herton runs next build.

Comment 32 farrotin 2021-10-06 09:57:44 UTC
(In reply to Jan Stancek from comment #31)
> 
> It's been already merged to stream 9 kernel:
>  
> https://gitlab.com/redhat/centos-stream/src/kernel/centos-stream-9/-/commit/
> d334fb74543bacbec67428b69a6c55a75d2d7cc5
> so I estimate couple days until Herton runs next build.

Thanks a lot ! really looking forward to test from a next compose with that kernel and then unblock the stream 9 tests behind ci.centos.org

Comment 33 farrotin 2021-10-13 10:27:50 UTC
Status update : I see that now we have a kernel that was built on Stream koji : https://kojihub.stream.centos.org/koji/buildinfo?buildID=14663
from the changelog :

- Enable e1000 in rhel9 as unsupported (Ken Cox) [2002344]

So let's test from a compose (https://composes.stream.centos.org/development/latest-CentOS-Stream/compose/BaseOS/x86_64/os/) and see if that works.
Can give some feedback here and then ask Stream team to also have compose pushed as "official" one and so landing on mirror.stream.centos.org (and everywhere else)

Comment 34 farrotin 2021-10-13 12:09:54 UTC
I confirm that it works now from that compose : was able to deploy Stream 9 : 

uname -a ; modinfo e1000|egrep 'filename|description|rhelversion'
Linux test-9s.ci.centos.org 5.14.0-6.el9.x86_64 #1 SMP Fri Oct 8 20:23:25 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
filename:       /lib/modules/5.14.0-6.el9.x86_64/kernel/drivers/net/ethernet/intel/e1000/e1000.ko.xz
description:    Intel(R) PRO/1000 Network Driver
rhelversion:    9.0

I'll ask Brian Stinson about when that compose will switch from 'development' to 'production' and so rolled out to mirror.stream.centos.org network (and third-party mirrors fetching from there)

Comment 35 farrotin 2021-10-16 13:58:09 UTC
I see that it landed on mirror.stream.centos.org and so I think this request can now be closed ?

Comment 36 Herton R. Krzesinski 2021-10-18 13:49:17 UTC
(In reply to farrotin from comment #35)
> I see that it landed on mirror.stream.centos.org and so I think this request
> can now be closed ?

I think it'll be auto-closed once 9-Beta is released and errata is closed/released. This is a 9-Beta bug where changes were synced/merged up to CentOS.


Note You need to log in before you can comment on or make changes to this bug.