mirror of
https://github.com/freebsd/freebsd-src
synced 2024-10-15 12:54:27 +00:00
qlxgbe: fix debug prints in ql_os.c
QL_DPRINT2 checks the debug level first and then prints. Replace device_printf with QL_DPRINT2 to check debug level first before printing out the kernel pointers. PR: 238655 MFC after: 1 week
This commit is contained in:
parent
a58b4ee025
commit
3d6c7ee87e
|
@ -322,7 +322,7 @@ static int
|
||||||
qla_pci_attach(device_t dev)
|
qla_pci_attach(device_t dev)
|
||||||
{
|
{
|
||||||
qla_host_t *ha = NULL;
|
qla_host_t *ha = NULL;
|
||||||
uint32_t rsrc_len;
|
uint32_t rsrc_len __unused;
|
||||||
int i;
|
int i;
|
||||||
uint32_t num_rcvq = 0;
|
uint32_t num_rcvq = 0;
|
||||||
|
|
||||||
|
@ -405,10 +405,10 @@ qla_pci_attach(device_t dev)
|
||||||
__func__);
|
__func__);
|
||||||
goto qla_pci_attach_err;
|
goto qla_pci_attach_err;
|
||||||
}
|
}
|
||||||
device_printf(dev, "%s: ha %p pci_func 0x%x rsrc_count 0x%08x"
|
QL_DPRINT2(ha, (dev, "%s: ha %p pci_func 0x%x rsrc_count 0x%08x"
|
||||||
" msix_count 0x%x pci_reg %p pci_reg1 %p num_rcvq = %d\n",
|
" msix_count 0x%x pci_reg %p pci_reg1 %p num_rcvq = %d\n",
|
||||||
__func__, ha, ha->pci_func, rsrc_len, ha->msix_count,
|
__func__, ha, ha->pci_func, rsrc_len, ha->msix_count,
|
||||||
ha->pci_reg, ha->pci_reg1, num_rcvq);
|
ha->pci_reg, ha->pci_reg1, num_rcvq));
|
||||||
|
|
||||||
if ((ha->msix_count < 64) || (num_rcvq != 32)) {
|
if ((ha->msix_count < 64) || (num_rcvq != 32)) {
|
||||||
if (ha->hw.num_sds_rings > 15) {
|
if (ha->hw.num_sds_rings > 15) {
|
||||||
|
@ -1274,9 +1274,9 @@ qla_send(qla_host_t *ha, struct mbuf **m_headp, uint32_t txr_idx,
|
||||||
"mbuf = %p\n", __func__, __LINE__, txr_idx, tx_idx,\
|
"mbuf = %p\n", __func__, __LINE__, txr_idx, tx_idx,\
|
||||||
ha->tx_ring[txr_idx].tx_buf[tx_idx].m_head));
|
ha->tx_ring[txr_idx].tx_buf[tx_idx].m_head));
|
||||||
|
|
||||||
device_printf(ha->pci_dev, "%s [%d]: txr_idx = %d tx_idx = %d "
|
QL_DPRINT2(ha, (ha->pci_dev, "%s [%d]: txr_idx = %d tx_idx = %d "
|
||||||
"mbuf = %p\n", __func__, __LINE__, txr_idx, tx_idx,
|
"mbuf = %p\n", __func__, __LINE__, txr_idx, tx_idx,
|
||||||
ha->tx_ring[txr_idx].tx_buf[tx_idx].m_head);
|
ha->tx_ring[txr_idx].tx_buf[tx_idx].m_head));
|
||||||
|
|
||||||
if (m_head)
|
if (m_head)
|
||||||
m_freem(m_head);
|
m_freem(m_head);
|
||||||
|
|
Loading…
Reference in a new issue