diff options
author | Sathya Perla <sathya.perla@emulex.com> | 2012-09-28 00:39:44 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2012-09-30 02:15:34 -0400 |
commit | d379142bc4d9b78cdd9fc5aa696ca1ea083fb7d4 (patch) | |
tree | f726f3074df4a26edd498f3293dabed94972869e | |
parent | 323ff71e566994f77761e221361de5b4a52a612f (diff) |
be2net: fixup log messages
Added and modified a few log messages mostly in probe path.
Signed-off-by: Sathya Perla <sathya.perla@emulex.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/ethernet/emulex/benet/be_cmds.c | 3 | ||||
-rw-r--r-- | drivers/net/ethernet/emulex/benet/be_main.c | 34 |
2 files changed, 31 insertions, 6 deletions
diff --git a/drivers/net/ethernet/emulex/benet/be_cmds.c b/drivers/net/ethernet/emulex/benet/be_cmds.c index 46a19affbc21..af60bb26e330 100644 --- a/drivers/net/ethernet/emulex/benet/be_cmds.c +++ b/drivers/net/ethernet/emulex/benet/be_cmds.c | |||
@@ -2425,6 +2425,9 @@ int be_cmd_req_native_mode(struct be_adapter *adapter) | |||
2425 | struct be_cmd_resp_set_func_cap *resp = embedded_payload(wrb); | 2425 | struct be_cmd_resp_set_func_cap *resp = embedded_payload(wrb); |
2426 | adapter->be3_native = le32_to_cpu(resp->cap_flags) & | 2426 | adapter->be3_native = le32_to_cpu(resp->cap_flags) & |
2427 | CAPABILITY_BE3_NATIVE_ERX_API; | 2427 | CAPABILITY_BE3_NATIVE_ERX_API; |
2428 | if (!adapter->be3_native) | ||
2429 | dev_warn(&adapter->pdev->dev, | ||
2430 | "adapter not in advanced mode\n"); | ||
2428 | } | 2431 | } |
2429 | err: | 2432 | err: |
2430 | mutex_unlock(&adapter->mbox_lock); | 2433 | mutex_unlock(&adapter->mbox_lock); |
diff --git a/drivers/net/ethernet/emulex/benet/be_main.c b/drivers/net/ethernet/emulex/benet/be_main.c index 4855dd627c7f..6accb0c0273a 100644 --- a/drivers/net/ethernet/emulex/benet/be_main.c +++ b/drivers/net/ethernet/emulex/benet/be_main.c | |||
@@ -1896,6 +1896,8 @@ static int be_tx_qs_create(struct be_adapter *adapter) | |||
1896 | return status; | 1896 | return status; |
1897 | } | 1897 | } |
1898 | 1898 | ||
1899 | dev_info(&adapter->pdev->dev, "created %d TX queue(s)\n", | ||
1900 | adapter->num_tx_qs); | ||
1899 | return 0; | 1901 | return 0; |
1900 | } | 1902 | } |
1901 | 1903 | ||
@@ -1946,10 +1948,9 @@ static int be_rx_cqs_create(struct be_adapter *adapter) | |||
1946 | return rc; | 1948 | return rc; |
1947 | } | 1949 | } |
1948 | 1950 | ||
1949 | if (adapter->num_rx_qs != MAX_RX_QS) | 1951 | dev_info(&adapter->pdev->dev, |
1950 | dev_info(&adapter->pdev->dev, | 1952 | "created %d RSS queue(s) and 1 default RX queue\n", |
1951 | "Created only %d receive queues\n", adapter->num_rx_qs); | 1953 | adapter->num_rx_qs - 1); |
1952 | |||
1953 | return 0; | 1954 | return 0; |
1954 | } | 1955 | } |
1955 | 1956 | ||
@@ -2187,6 +2188,7 @@ static void be_msix_enable(struct be_adapter *adapter) | |||
2187 | { | 2188 | { |
2188 | #define BE_MIN_MSIX_VECTORS 1 | 2189 | #define BE_MIN_MSIX_VECTORS 1 |
2189 | int i, status, num_vec, num_roce_vec = 0; | 2190 | int i, status, num_vec, num_roce_vec = 0; |
2191 | struct device *dev = &adapter->pdev->dev; | ||
2190 | 2192 | ||
2191 | /* If RSS queues are not used, need a vec for default RX Q */ | 2193 | /* If RSS queues are not used, need a vec for default RX Q */ |
2192 | num_vec = min(be_num_rss_want(adapter), num_online_cpus()); | 2194 | num_vec = min(be_num_rss_want(adapter), num_online_cpus()); |
@@ -2211,6 +2213,8 @@ static void be_msix_enable(struct be_adapter *adapter) | |||
2211 | num_vec) == 0) | 2213 | num_vec) == 0) |
2212 | goto done; | 2214 | goto done; |
2213 | } | 2215 | } |
2216 | |||
2217 | dev_warn(dev, "MSIx enable failed\n"); | ||
2214 | return; | 2218 | return; |
2215 | done: | 2219 | done: |
2216 | if (be_roce_supported(adapter)) { | 2220 | if (be_roce_supported(adapter)) { |
@@ -2224,6 +2228,7 @@ done: | |||
2224 | } | 2228 | } |
2225 | } else | 2229 | } else |
2226 | adapter->num_msix_vec = num_vec; | 2230 | adapter->num_msix_vec = num_vec; |
2231 | dev_info(dev, "enabled %d MSI-x vector(s)\n", adapter->num_msix_vec); | ||
2227 | return; | 2232 | return; |
2228 | } | 2233 | } |
2229 | 2234 | ||
@@ -3797,6 +3802,23 @@ static bool be_reset_required(struct be_adapter *adapter) | |||
3797 | return be_find_vfs(adapter, ENABLED) > 0 ? false : true; | 3802 | return be_find_vfs(adapter, ENABLED) > 0 ? false : true; |
3798 | } | 3803 | } |
3799 | 3804 | ||
3805 | static char *mc_name(struct be_adapter *adapter) | ||
3806 | { | ||
3807 | if (adapter->function_mode & FLEX10_MODE) | ||
3808 | return "FLEX10"; | ||
3809 | else if (adapter->function_mode & VNIC_MODE) | ||
3810 | return "vNIC"; | ||
3811 | else if (adapter->function_mode & UMC_ENABLED) | ||
3812 | return "UMC"; | ||
3813 | else | ||
3814 | return ""; | ||
3815 | } | ||
3816 | |||
3817 | static inline char *func_name(struct be_adapter *adapter) | ||
3818 | { | ||
3819 | return be_physfn(adapter) ? "PF" : "VF"; | ||
3820 | } | ||
3821 | |||
3800 | static int __devinit be_probe(struct pci_dev *pdev, | 3822 | static int __devinit be_probe(struct pci_dev *pdev, |
3801 | const struct pci_device_id *pdev_id) | 3823 | const struct pci_device_id *pdev_id) |
3802 | { | 3824 | { |
@@ -3901,8 +3923,8 @@ static int __devinit be_probe(struct pci_dev *pdev, | |||
3901 | 3923 | ||
3902 | be_cmd_query_port_name(adapter, &port_name); | 3924 | be_cmd_query_port_name(adapter, &port_name); |
3903 | 3925 | ||
3904 | dev_info(&pdev->dev, "%s: %s port %c\n", netdev->name, nic_name(pdev), | 3926 | dev_info(&pdev->dev, "%s: %s %s port %c\n", nic_name(pdev), |
3905 | port_name); | 3927 | func_name(adapter), mc_name(adapter), port_name); |
3906 | 3928 | ||
3907 | return 0; | 3929 | return 0; |
3908 | 3930 | ||