* [bug report] ionic: dynamic interrupt moderation
@ 2020-09-18 10:24 Dan Carpenter
0 siblings, 0 replies; only message in thread
From: Dan Carpenter @ 2020-09-18 10:24 UTC (permalink / raw)
To: kernel-janitors
Hello Shannon Nelson,
This is a semi-automatic email about new static checker warnings.
The patch 04a834592bf5: "ionic: dynamic interrupt moderation" from
Sep 15, 2020, leads to the following Smatch complaint:
drivers/net/ethernet/pensando/ionic/ionic_lif.c:855 ionic_adminq_napi()
error: we previously assumed 'lif->adminqcq' could be null (see line 848)
drivers/net/ethernet/pensando/ionic/ionic_lif.c
847
848 if (lif->adminqcq && lif->adminqcq->flags & IONIC_QCQ_F_INITED)
^^^^^^^^^^^^^
Existing code checks for NULL
849 a_work = ionic_cq_service(&lif->adminqcq->cq, budget,
850 ionic_adminq_service, NULL, NULL);
851
852 work_done = max(n_work, a_work);
853 if (work_done < budget && napi_complete_done(napi, work_done)) {
854 flags |= IONIC_INTR_CRED_UNMASK;
855 lif->adminqcq->cq.bound_intr->rearm_count++;
^^^^^^^^^^^^^^^
The patch adds a new unchecked dereference
856 }
857
regards,
dan carpenter
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2020-09-18 10:24 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-18 10:24 [bug report] ionic: dynamic interrupt moderation Dan Carpenter
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).