Bug 503818 - Xen dom0 fake e820 prevents IGB driver from creating VF devices
Summary: Xen dom0 fake e820 prevents IGB driver from creating VF devices
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: kernel-xen
Version: 5.4
Hardware: All
OS: Linux
high
high
Target Milestone: rc
: 5.4
Assignee: Don Dugger
QA Contact: Red Hat Kernel QE team
URL:
Whiteboard:
Depends On:
Blocks: 484429 512913 5.4, TechnicalNotes
TreeView+ depends on / blocked
 
Reported: 2009-06-02 20:53 UTC by Don Dugger
Modified: 2014-07-25 03:22 UTC (History)
16 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Boot parameters are required to enable SR/IOV Virtual Function devices. SR/IOV Virtual Function devices can only be accessed if the parameter pci_pt_e820_access=on is added to the boot stanza in the /boot/grub/grub.conf file. For example: title Red Hat Enterprise Linux Server (2.6.18-152.el5xen) root (hd0,1) kernel /xen.gz-2.6.18-152.el5 com1=115200,8n1 console=com1 iommu=1 module /vmlinuz-2.6.18-152.el5xen ro root=LABEL=/ console=ttyS0,115200 pci_pt_e820_access=on This enables the MMCONF access method for the PCI configuration space, a requirement for VF device support.
Clone Of:
Environment:
Last Closed: 2009-09-02 08:19:41 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Give Dom0 access to machine e820 map (2.78 KB, patch)
2009-06-02 21:07 UTC, Don Dugger
no flags Details | Diff


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2009:1243 0 normal SHIPPED_LIVE Important: Red Hat Enterprise Linux 5.4 kernel security and bug fix update 2009-09-01 08:53:34 UTC

Description Don Dugger 2009-06-02 20:53:26 UTC
Description of problem:
The IGB driver cannot create VF devices.  Even though `max_vfs=7' is passed as a parameter to the driver it doesn't create any VFs.


Version-Release number of selected component (if applicable):
RHEL 5.4-149


How reproducible:
Add the line:

options igb max_vfs=7

the `/etc/modules.conf' and start the IGB driver.  Checking the `dmesg' log and `lspci' will show no VF devices.

Comment 2 Don Dugger 2009-06-02 21:07:23 UTC
Created attachment 346317 [details]
Give Dom0 access to machine e820 map

In order for a Physical Function driver to create Virtual Function devices
the PCI subsystem needs to access the extended capabilities for the PF.   
The exteneded capabilities are only accessible through the MMCONFIG PCI
access method.  Although the RHEL 5.4 kernel tries to use MMCONFIG this
access method fails for the Dom0 kernel because the Dom0 kernel is using
a PV E820 map that only maps RAM.  When the PCI subsystem tries to verify
that the MMCONFIG address is reserved in the E820 map it fails (the      
MMCONFI address space is not in the PV map).  This patch solves this problem
by giving Dom0 access to the real E820 map.                                 
                                           
Upstream status: Accepted (CS 26 of the Xen Dom0 kernel)
                                                        
Signed-off-by: Don Dugger <donald.d.dugger>

Comment 5 Chris Lalancette 2009-06-03 06:41:12 UTC
Fixed the summary to be a little more descriptive.

I'm a bit concerned about putting this into 5.4 at this stage.  I'm not against the patch per-se, just against the timing.  It changes a pretty fundamental thing in the way that dom0 works, and since it is going to miss beta, it's going to miss out partner testing on a variety of different hardware, which I consider a pre-requisite to having this in 5.4.

Chris Lalancette

Comment 6 Stephen Tweedie 2009-06-03 11:14:26 UTC
Don't we also need similar changes to the 32-bit arch/i386/kernel/setup-xen.c code?

And agreed with Chris, this would seem to be very risky going in post-beta, since e820 changes are something that we really want to see get the broadest possible hardware testing on.

Comment 10 Bill Burns 2009-06-04 14:43:22 UTC
Release note added. If any revisions are required, please set the 
"requires_release_notes" flag to "?" and edit the "Release Notes" field accordingly.
All revisions will be proofread by the Engineering Content Services team.

New Contents:
The release note that gets generated for this needs to be highlighted, per RHEL meeting discussion.

Comment 12 Don Zickus 2009-06-18 14:51:33 UTC
in kernel-2.6.18-154.el5
You can download this test kernel from http://people.redhat.com/dzickus/el5

Please do NOT transition this bugzilla state to VERIFIED until our QE team
has sent specific instructions indicating when to do so.  However feel free
to provide a comment indicating that this fix has been verified.

Comment 16 Chris Ward 2009-07-03 18:44:57 UTC
~~ Attention - RHEL 5.4 Beta Released! ~~

RHEL 5.4 Beta has been released! There should be a fix present in the Beta release that addresses this particular request. Please test and report back results here, at your earliest convenience. RHEL 5.4 General Availability release is just around the corner!

If you encounter any issues while testing Beta, please describe the issues you have encountered and set the bug into NEED_INFO. If you encounter new issues, please clone this bug to open a new issue and request it be reviewed for inclusion in RHEL 5.4 or a later update, if it is not of urgent severity.

Please do not flip the bug status to VERIFIED. Only post your verification results, and if available, update Verified field with the appropriate value.

Questions can be posted to this bug or your customer or partner representative.

Comment 17 Chris Ward 2009-07-10 19:13:51 UTC
~~ Attention Partners - RHEL 5.4 Snapshot 1 Released! ~~

RHEL 5.4 Snapshot 1 has been released on partners.redhat.com. If you have already reported your test results, you can safely ignore this request. Otherwise, please notice that there should be a fix available now that addresses this particular request. Please test and report back your results here, at your earliest convenience. The RHEL 5.4 exception freeze is quickly approaching.

If you encounter any issues while testing Beta, please describe the issues you have encountered and set the bug into NEED_INFO. If you encounter new issues, please clone this bug to open a new issue and request it be reviewed for inclusion in RHEL 5.4 or a later update, if it is not of urgent severity.

Do not flip the bug status to VERIFIED. Instead, please set your Partner ID in the Verified field above if you have successfully verified the resolution of this issue. 

Further questions can be directed to your Red Hat Partner Manager or other appropriate customer representative.

Comment 20 Don Dugger 2009-07-27 16:43:09 UTC
The proposed release notes in comment 1 for BZ 480376 actually apply to this BZ but I've added a release note that addresses the specific issue in this BZ.

Comment 21 Don Dugger 2009-07-27 16:43:09 UTC
Release note updated. If any revisions are required, please set the 
"requires_release_notes"  flag to "?" and edit the "Release Notes" field accordingly.
All revisions will be proofread by the Engineering Content Services team.

Diffed Contents:
@@ -1 +1,11 @@
-The release note that gets generated for this needs to be highlighted, per RHEL meeting discussion.+Boot parameter required to enable Virtual Function devices.
+
+Virtual Function devices can only be access if the parameter `pci_pt_e820_access=on' is added to the boot stanza in the `/boot/grub/grub.conf' file, e.g.:
+
+title Red Hat Enterprise Linux Server (2.6.18-152.el5xen)
+        root (hd0,1)
+        kernel /xen.gz-2.6.18-152.el5 com1=115200,8n1 console=com1 iommu=1
+        module /vmlinuz-2.6.18-152.el5xen ro root=LABEL=/ console=ttyS0,115200
+pci_pt_e820_access=on
+
+This will enable the MMCONF access methode to the PCI configuration space, a requirement for VF device support

Comment 22 Bill Burns 2009-07-27 17:11:52 UTC
Excellent, thanks Don!

Comment 23 Subhendu Ghosh 2009-07-28 02:10:45 UTC
Should the release note contain "SRIOV" somewhere?

Comment 24 Don Dugger 2009-07-28 02:31:06 UTC
Good point, I've updated the release not to include SR/IOV.

Comment 25 Don Dugger 2009-07-28 02:31:06 UTC
Release note updated. If any revisions are required, please set the 
"requires_release_notes"  flag to "?" and edit the "Release Notes" field accordingly.
All revisions will be proofread by the Engineering Content Services team.

Diffed Contents:
@@ -1,6 +1,6 @@
-Boot parameter required to enable Virtual Function devices.
+Boot parameter required to enable SR/IOV Virtual Function devices.
 
-Virtual Function devices can only be access if the parameter `pci_pt_e820_access=on' is added to the boot stanza in the `/boot/grub/grub.conf' file, e.g.:
+SR/IOV Virtual Function devices can only be access if the parameter `pci_pt_e820_access=on' is added to the boot stanza in the `/boot/grub/grub.conf' file, e.g.:
 
 title Red Hat Enterprise Linux Server (2.6.18-152.el5xen)
         root (hd0,1)

Comment 29 Ryan Lerch 2009-08-21 00:25:33 UTC
Release note updated. If any revisions are required, please set the 
"requires_release_notes"  flag to "?" and edit the "Release Notes" field accordingly.
All revisions will be proofread by the Engineering Content Services team.

Diffed Contents:
@@ -1,11 +1,9 @@
-Boot parameter required to enable SR/IOV Virtual Function devices.
+Boot parameters are required to enable SR/IOV Virtual Function devices. SR/IOV Virtual Function devices can only be accessed if the parameter pci_pt_e820_access=on is added to the boot stanza in the /boot/grub/grub.conf file. For example:
 
-SR/IOV Virtual Function devices can only be access if the parameter `pci_pt_e820_access=on' is added to the boot stanza in the `/boot/grub/grub.conf' file, e.g.:
-
 title Red Hat Enterprise Linux Server (2.6.18-152.el5xen)
         root (hd0,1)
         kernel /xen.gz-2.6.18-152.el5 com1=115200,8n1 console=com1 iommu=1
         module /vmlinuz-2.6.18-152.el5xen ro root=LABEL=/ console=ttyS0,115200
 pci_pt_e820_access=on
 
-This will enable the MMCONF access methode to the PCI configuration space, a requirement for VF device support+This enables the MMCONF access method for the PCI configuration space, a requirement for VF device support.

Comment 30 errata-xmlrpc 2009-09-02 08:19:41 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHSA-2009-1243.html


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