]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commitdiff
be2net: Patch to determine if function is VF while running in guest OS.
authorSarveshwar Bandi <sarveshwarb@serverengines.com>
Fri, 9 Jul 2010 01:43:55 +0000 (01:43 +0000)
committerDavid S. Miller <davem@davemloft.net>
Sun, 11 Jul 2010 02:42:05 +0000 (19:42 -0700)
When driver is loaded in guest OS, the pci variables is_virtfn and is_physfn are
both set to 0. This change uses registers in controller to determine the same.

Signed-off-by: Sarveshwar Bandi <sarveshwarb@serverengines.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/benet/be.h
drivers/net/benet/be_main.c

index 1a0d2d037c4e232958a568e2ba841ab1753aa7cc..f17428caecf1974a390e789a896056ad77ecdf2d 100644 (file)
@@ -291,9 +291,10 @@ struct be_adapter {
        u32 vf_if_handle[BE_MAX_VF];
        u32 vf_pmac_id[BE_MAX_VF];
        u8 base_eq_id;
+       u8 is_virtfn;
 };
 
-#define be_physfn(adapter) (!adapter->pdev->is_virtfn)
+#define be_physfn(adapter) (!adapter->is_virtfn)
 
 /* BladeEngine Generation numbers */
 #define BE_GEN2 2
@@ -393,6 +394,15 @@ static inline u8 is_udp_pkt(struct sk_buff *skb)
        return val;
 }
 
+static inline void be_check_sriov_fn_type(struct be_adapter *adapter)
+{
+       u8 data;
+
+       pci_write_config_byte(adapter->pdev, 0xFE, 0xAA);
+       pci_read_config_byte(adapter->pdev, 0xFE, &data);
+       adapter->is_virtfn = (data != 0xAA);
+}
+
 extern void be_cq_notify(struct be_adapter *adapter, u16 qid, bool arm,
                u16 num_popped);
 extern void be_link_status_update(struct be_adapter *adapter, bool link_up);
index b63687956f2be87b7a214d3b4d3494144c542176..e6ca92334d6d0bbb72de8ddc03930388986efaf5 100644 (file)
@@ -1631,6 +1631,7 @@ static void be_sriov_enable(struct be_adapter *adapter)
 {
 #ifdef CONFIG_PCI_IOV
        int status;
+       be_check_sriov_fn_type(adapter);
        if (be_physfn(adapter) && num_vfs) {
                status = pci_enable_sriov(adapter->pdev, num_vfs);
                adapter->sriov_enabled = status ? false : true;