d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

cxgbe(4): Fix "set but not used [-Wunused-but-set-variable]" warnings.

Browse Source 
MFC after:	1 week
Sponsored by:	Chelsio Communications
This commit is contained in:
Navdeep Parhar 2022-01-05 12:23:06 -08:00
parent 6792568fe2
commit 39d5cbdc1b
11 changed files with 72 additions and 31 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
sys/dev/cxgbe/common/t4_hw.c
View File

@ -6648,7 +6648,6 @@ int t4_set_trace_filter(struct adapter *adap, const struct trace_params *tp,
{
int i, ofst = idx * 4;
u32 data_reg, mask_reg, cfg;
u32 multitrc = F_TRCMULTIFILTER;
u32 en = is_t4(adap) ? F_TFEN : F_T5_TFEN;
if (idx < 0 || idx >= NTRACE)
@ -6683,7 +6682,6 @@ int t4_set_trace_filter(struct adapter *adap, const struct trace_params *tp,
* maximum packet capture size of 9600 bytes is recommended.
* Also in this mode, only trace0 can be enabled and running.
*/
multitrc = 0;
if (tp->snap_len > 9600 || idx)
return -EINVAL;
}
@ -9076,7 +9074,6 @@ int t4_handle_fw_rpl(struct adapter *adap, const __be64 *rpl)
int i;
int chan = G_FW_PORT_CMD_PORTID(be32_to_cpu(p->op_to_portid));
struct port_info *pi = NULL;
struct link_config *lc;
for_each_port(adap, i) {
pi = adap2pinfo(adap, i);
@ -9084,7 +9081,6 @@ int t4_handle_fw_rpl(struct adapter *adap, const __be64 *rpl)
break;
}
lc = &pi->link_cfg;
PORT_LOCK(pi);
handle_port_info(pi, p, action, &mod_changed, &link_changed);
PORT_UNLOCK(pi);

4
sys/dev/cxgbe/cudbg/cudbg_lib.c
View File

@ -2386,7 +2386,7 @@ static int collect_cim_qcfg(struct cudbg_init *pdbg_init,
struct cudbg_buffer scratch_buff;
struct adapter *padap = pdbg_init->adap;
u32 offset;
int cim_num_obq, rc = 0;
int rc = 0;
struct struct_cim_qcfg *cim_qcfg_data = NULL;
@ -2398,8 +2398,6 @@ static int collect_cim_qcfg(struct cudbg_init *pdbg_init,
offset = scratch_buff.offset;
cim_num_obq = is_t4(padap) ? CIM_NUM_OBQ : CIM_NUM_OBQ_T5;
cim_qcfg_data =
(struct struct_cim_qcfg *)((u8 *)((char *)scratch_buff.data +
offset));

5
sys/dev/cxgbe/cudbg/fastlz_api.c
View File

@ -349,7 +349,6 @@ int decompress_buffer(struct cudbg_buffer *pc_buff,
unsigned long chunk_checksum;
unsigned long chunk_extra;
unsigned long checksum;
unsigned long total_extracted = 0;
unsigned long r;
unsigned long remaining;
unsigned long bytes_read;
@ -409,8 +408,6 @@ int decompress_buffer(struct cudbg_buffer *pc_buff,
pc_buff->offset -= chunk_size + 16;
return CUDBG_STATUS_SMALL_BUFF;
}
total_extracted = 0;
}
if (chunk_size > CUDBG_BLOCK_SIZE) {
@ -438,7 +435,6 @@ int decompress_buffer(struct cudbg_buffer *pc_buff,
switch (chunk_options) {
/* stored, simply copy to output */
case 0:
total_extracted += chunk_size;
remaining = chunk_size;
checksum = 1L;
for (;;) {
@ -485,7 +481,6 @@ int decompress_buffer(struct cudbg_buffer *pc_buff,
checksum = update_adler32(1L, compressed_buffer,
chunk_size);
total_extracted += chunk_extra;
/* verify that the chunk data is correct */
if (checksum != chunk_checksum) {

7
sys/dev/cxgbe/cxgbei/cxgbei.c
View File

@ -488,6 +488,7 @@ do_rx_iscsi_cmp(struct sge_iq *iq, const struct rss_header *rss, struct mbuf *m)
struct inpcb *inp = toep->inp;
#ifdef INVARIANTS
uint16_t len = be16toh(cpl->len);
u_int data_digest_len;
#endif
struct socket *so;
struct sockbuf *sb;
@ -496,7 +497,7 @@ do_rx_iscsi_cmp(struct sge_iq *iq, const struct rss_header *rss, struct mbuf *m)
struct icl_conn *ic;
struct iscsi_bhs_data_out *bhsdo;
u_int val = be32toh(cpl->ddpvld);
u_int npdus, pdu_len, data_digest_len, hdr_digest_len;
u_int npdus, pdu_len;
uint32_t prev_seg_len;
M_ASSERTPKTHDR(m);
@ -598,11 +599,11 @@ do_rx_iscsi_cmp(struct sge_iq *iq, const struct rss_header *rss, struct mbuf *m)
return (0);
}
#ifdef INVARIANTS
data_digest_len = (icc->ulp_submode & ULP_CRC_DATA) ?
ISCSI_DATA_DIGEST_SIZE : 0;
hdr_digest_len = (icc->ulp_submode & ULP_CRC_HEADER) ?
ISCSI_HEADER_DIGEST_SIZE : 0;
MPASS(roundup2(ip->ip_data_len, 4) == pdu_len - len - data_digest_len);
#endif
if (val & F_DDP_PDU && ip->ip_data_mbuf == NULL) {
MPASS((icp->icp_flags & ICPF_RX_FLBUF) == 0);

8
sys/dev/cxgbe/cxgbei/icl_cxgbei.c
View File

@ -784,7 +784,6 @@ int
icl_cxgbei_conn_handoff(struct icl_conn *ic, int fd)
{
struct icl_cxgbei_conn *icc = ic_to_icc(ic);
struct cxgbei_data *ci;
struct find_ofld_adapter_rr fa;
struct file *fp;
struct socket *so;
@ -836,7 +835,6 @@ icl_cxgbei_conn_handoff(struct icl_conn *ic, int fd)
if (fa.sc == NULL)
return (EINVAL);
icc->sc = fa.sc;
ci = icc->sc->iscsi_ulp_softc;
max_rx_pdu_len = ISCSI_BHS_SIZE + ic->ic_max_recv_data_segment_length;
max_tx_pdu_len = ISCSI_BHS_SIZE + ic->ic_max_send_data_segment_length;
@ -1178,19 +1176,25 @@ icl_cxgbei_conn_task_done(struct icl_conn *ic, void *arg)
static inline bool
ddp_sgl_check(struct ctl_sg_entry *sg, int entries, int xferlen)
{
#ifdef INVARIANTS
int total_len = 0;
#endif
MPASS(entries > 0);
if (((vm_offset_t)sg[--entries].addr & 3U) != 0)
return (false);
#ifdef INVARIANTS
total_len += sg[entries].len;
#endif
while (--entries >= 0) {
if (((vm_offset_t)sg[entries].addr & PAGE_MASK) != 0 ||
(sg[entries].len % PAGE_SIZE) != 0)
return (false);
#ifdef INVARIANTS
total_len += sg[entries].len;
#endif
}
MPASS(total_len == xferlen);

10
sys/dev/cxgbe/iw_cxgbe/cm.c
View File

@ -1322,10 +1322,14 @@ alloc_ep(int size, gfp_t gfp)
void _c4iw_free_ep(struct kref *kref)
{
struct c4iw_ep *ep;
#if defined(KTR) || defined(INVARIANTS)
struct c4iw_ep_common *epc;
#endif
ep = container_of(kref, struct c4iw_ep, com.kref);
#if defined(KTR) || defined(INVARIANTS)
epc = &ep->com;
#endif
KASSERT(!epc->entry.tqe_prev, ("%s epc %p still on req list",
__func__, epc));
if (test_bit(QP_REFERENCED, &ep->com.flags))
@ -2352,7 +2356,9 @@ process_mpa_request(struct c4iw_ep *ep)
*/
int c4iw_reject_cr(struct iw_cm_id *cm_id, const void *pdata, u8 pdata_len)
{
#ifdef KTR
int err;
#endif
struct c4iw_ep *ep = to_ep(cm_id);
int abort = 0;
@ -2380,7 +2386,11 @@ int c4iw_reject_cr(struct iw_cm_id *cm_id, const void *pdata, u8 pdata_len)
abort = send_mpa_reject(ep, pdata, pdata_len);
}
STOP_EP_TIMER(ep);
#ifdef KTR
err = c4iw_ep_disconnect(ep, abort != 0, GFP_KERNEL);
#else
c4iw_ep_disconnect(ep, abort != 0, GFP_KERNEL);
#endif
mutex_unlock(&ep->com.mutex);
c4iw_put_ep(&ep->com);
CTR3(KTR_IW_CXGBE, "%s:crc4 %p, err: %d", __func__, ep, err);

3
sys/dev/cxgbe/t4_main.c
View File

@ -1663,11 +1663,8 @@ notify_siblings(device_t dev, int detaching)
static int
t4_detach(device_t dev)
{
struct adapter *sc;
int rc;
sc = device_get_softc(dev);
rc = notify_siblings(dev, 1);
if (rc) {
device_printf(dev,

24
sys/dev/cxgbe/t4_sge.c
View File

@ -2679,10 +2679,13 @@ int
parse_pkt(struct mbuf **mp, bool vm_wr)
{
struct mbuf *m0 = *mp, *m;
int rc, nsegs, defragged = 0, offset;
int rc, nsegs, defragged = 0;
struct ether_header *eh;
#ifdef INET
void *l3hdr;
#endif
#if defined(INET) || defined(INET6)
int offset;
struct tcphdr *tcp;
#endif
#if defined(KERN_TLS) || defined(RATELIMIT)
@ -2790,8 +2793,14 @@ parse_pkt(struct mbuf **mp, bool vm_wr)
} else
m0->m_pkthdr.l2hlen = sizeof(*eh);
#if defined(INET) || defined(INET6)
offset = 0;
#ifdef INET
l3hdr = m_advance(&m, &offset, m0->m_pkthdr.l2hlen);
#else
m_advance(&m, &offset, m0->m_pkthdr.l2hlen);
#endif
#endif
switch (eh_type) {
#ifdef INET6
@ -2831,6 +2840,7 @@ parse_pkt(struct mbuf **mp, bool vm_wr)
goto fail;
}
#if defined(INET) || defined(INET6)
if (needs_vxlan_csum(m0)) {
m0->m_pkthdr.l4hlen = sizeof(struct udphdr);
m0->m_pkthdr.l5hlen = sizeof(struct vxlan_header);
@ -2846,7 +2856,11 @@ parse_pkt(struct mbuf **mp, bool vm_wr)
m0->m_pkthdr.inner_l2hlen = sizeof(*evh);
} else
m0->m_pkthdr.inner_l2hlen = sizeof(*eh);
#ifdef INET
l3hdr = m_advance(&m, &offset, m0->m_pkthdr.inner_l2hlen);
#else
m_advance(&m, &offset, m0->m_pkthdr.inner_l2hlen);
#endif
switch (eh_type) {
#ifdef INET6
@ -2874,12 +2888,10 @@ parse_pkt(struct mbuf **mp, bool vm_wr)
rc = EINVAL;
goto fail;
}
#if defined(INET) || defined(INET6)
if (needs_inner_tcp_csum(m0)) {
tcp = m_advance(&m, &offset, m0->m_pkthdr.inner_l3hlen);
m0->m_pkthdr.inner_l4hlen = tcp->th_off * 4;
}
#endif
MPASS((m0->m_pkthdr.csum_flags & CSUM_SND_TAG) == 0);
m0->m_pkthdr.csum_flags &= CSUM_INNER_IP6_UDP |
CSUM_INNER_IP6_TCP | CSUM_INNER_IP6_TSO | CSUM_INNER_IP |
@ -2887,7 +2899,6 @@ parse_pkt(struct mbuf **mp, bool vm_wr)
CSUM_ENCAP_VXLAN;
}
#if defined(INET) || defined(INET6)
if (needs_outer_tcp_csum(m0)) {
tcp = m_advance(&m, &offset, m0->m_pkthdr.l3hlen);
m0->m_pkthdr.l4hlen = tcp->th_off * 4;
@ -5385,14 +5396,13 @@ write_txpkt_vm_wr(struct adapter *sc, struct sge_txq *txq, struct mbuf *m0)
struct cpl_tx_pkt_core *cpl;
uint32_t ctrl; /* used in many unrelated places */
uint64_t ctrl1;
int len16, ndesc, pktlen, nsegs;
int len16, ndesc, pktlen;
caddr_t dst;
TXQ_LOCK_ASSERT_OWNED(txq);
M_ASSERTPKTHDR(m0);
len16 = mbuf_len16(m0);
nsegs = mbuf_nsegs(m0);
pktlen = m0->m_pkthdr.len;
ctrl = sizeof(struct cpl_tx_pkt_core);
if (needs_tso(m0))
@ -6567,7 +6577,6 @@ write_ethofld_wr(struct cxgbe_rate_tag *cst, struct fw_eth_tx_eo_wr *wr,
uint64_t ctrl1;
uint32_t ctrl; /* used in many unrelated places */
int len16, pktlen, nsegs, immhdrs;
caddr_t dst;
uintptr_t p;
struct ulptx_sgl *usgl;
struct sglist sg;
@ -6662,7 +6671,6 @@ write_ethofld_wr(struct cxgbe_rate_tag *cst, struct fw_eth_tx_eo_wr *wr,
m_copydata(m0, 0, immhdrs, (void *)p);
/* SGL */
dst = (void *)(cpl + 1);
if (nsegs > 0) {
int i, pad;

11
sys/dev/cxgbe/tom/t4_connect.c
View File

@ -182,11 +182,18 @@ t4_uninit_connect_cpl_handlers(void)
t4_register_shared_cpl_handler(CPL_ACT_OPEN_RPL, NULL, CPL_COOKIE_TOM);
}
#ifdef KTR
#define DONT_OFFLOAD_ACTIVE_OPEN(x) do { \
reason = __LINE__; \
rc = (x); \
goto failed; \
} while (0)
#else
#define DONT_OFFLOAD_ACTIVE_OPEN(x) do { \
rc = (x); \
goto failed; \
} while (0)
#endif
static inline int
act_open_cpl_size(struct adapter *sc, int isipv6)
@ -235,7 +242,9 @@ t4_connect(struct toedev *tod, struct socket *so, struct nhop_object *nh,
int qid_atid, rc, isipv6;
struct inpcb *inp = sotoinpcb(so);
struct tcpcb *tp = intotcpcb(inp);
#ifdef KTR
int reason;
#endif
struct offload_settings settings;
struct epoch_tracker et;
uint16_t vid = 0xfff, pcp = 0;
@ -381,7 +390,9 @@ t4_connect(struct toedev *tod, struct socket *so, struct nhop_object *nh,
}
undo_offload_socket(so);
#if defined(KTR)
reason = __LINE__;
#endif
failed:
CTR3(KTR_CXGBE, "%s: not offloading (%d), rc %d", __func__, reason, rc);

20
sys/dev/cxgbe/tom/t4_ddp.c
View File

@ -292,7 +292,10 @@ insert_ddp_data(struct toepcb *toep, uint32_t n)
struct kaiocb *job;
size_t placed;
long copied;
unsigned int db_flag, db_idx;
unsigned int db_idx;
#ifdef INVARIANTS
unsigned int db_flag;
#endif
INP_WLOCK_ASSERT(inp);
DDP_ASSERT_LOCKED(toep);
@ -307,7 +310,9 @@ insert_ddp_data(struct toepcb *toep, uint32_t n)
while (toep->ddp.active_count > 0) {
MPASS(toep->ddp.active_id != -1);
db_idx = toep->ddp.active_id;
#ifdef INVARIANTS
db_flag = db_idx == 1 ? DDP_BUF1_ACTIVE : DDP_BUF0_ACTIVE;
#endif
MPASS((toep->ddp.flags & db_flag) != 0);
db = &toep->ddp.db[db_idx];
job = db->job;
@ -694,7 +699,10 @@ handle_ddp_close(struct toepcb *toep, struct tcpcb *tp, __be32 rcv_nxt)
struct ddp_buffer *db;
struct kaiocb *job;
long copied;
unsigned int db_flag, db_idx;
unsigned int db_idx;
#ifdef INVARIANTS
unsigned int db_flag;
#endif
int len, placed;
INP_WLOCK_ASSERT(toep->inp);
@ -707,7 +715,9 @@ handle_ddp_close(struct toepcb *toep, struct tcpcb *tp, __be32 rcv_nxt)
while (toep->ddp.active_count > 0) {
MPASS(toep->ddp.active_id != -1);
db_idx = toep->ddp.active_id;
#ifdef INVARIANTS
db_flag = db_idx == 1 ? DDP_BUF1_ACTIVE : DDP_BUF0_ACTIVE;
#endif
MPASS((toep->ddp.flags & db_flag) != 0);
db = &toep->ddp.db[db_idx];
job = db->job;
@ -1798,7 +1808,9 @@ aio_ddp_requeue(struct toepcb *toep)
while (m != NULL && resid > 0) {
struct iovec iov[1];
struct uio uio;
#ifdef INVARIANTS
int error;
#endif
iov[0].iov_base = mtod(m, void *);
iov[0].iov_len = m->m_len;
@ -1810,8 +1822,12 @@ aio_ddp_requeue(struct toepcb *toep)
uio.uio_resid = iov[0].iov_len;
uio.uio_segflg = UIO_SYSSPACE;
uio.uio_rw = UIO_WRITE;
#ifdef INVARIANTS
error = uiomove_fromphys(ps->pages, offset + copied,
uio.uio_resid, &uio);
#else
uiomove_fromphys(ps->pages, offset + copied, uio.uio_resid, &uio);
#endif
MPASS(error == 0 && uio.uio_resid == 0);
copied += uio.uio_offset;
resid -= uio.uio_offset;

7
sys/dev/cxgbe/tom/t4_tls.c
View File

@ -958,7 +958,10 @@ do_rx_tls_cmp(struct sge_iq *iq, const struct rss_header *rss, struct mbuf *m)
struct mbuf *tls_data;
struct tls_get_record *tgr;
struct mbuf *control;
int len, pdu_length, rx_credits;
int pdu_length, rx_credits;
#if defined(KTR) || defined(INVARIANTS)
int len;
#endif
KASSERT(toep->tid == tid, ("%s: toep tid/atid mismatch", __func__));
KASSERT(!(toep->flags & TPF_SYNQE),
@ -966,7 +969,9 @@ do_rx_tls_cmp(struct sge_iq *iq, const struct rss_header *rss, struct mbuf *m)
/* strip off CPL header */
m_adj(m, sizeof(*cpl));
#if defined(KTR) || defined(INVARIANTS)
len = m->m_pkthdr.len;
#endif
toep->ofld_rxq->rx_toe_tls_records++;