Mana: move mana polling from EQ to CQ
-Each CQ start task queue to poll when completion happens. This means every rx and tx queue has its own cleanup task thread to poll the completion. - Arm EQ everytime no matter it is mana or hwc. CQ arming depends on the budget. - Fix a warning in mana_poll_tx_cq() when cqe_read is 0. - Move cqe_poll from EQ to CQ struct. - Support EQ sharing up to 8 vPorts. - Ease linkdown message from mana_info to mana_dbg. Tested by: whu MFC after: 2 weeks Sponsored by: Microsoft
This commit is contained in:
parent
cbc3ecb7ef
commit
1833cf1373
@ -291,8 +291,6 @@ struct gdma_event {
|
||||
|
||||
struct gdma_queue;
|
||||
|
||||
#define CQE_POLLING_BUFFER 512
|
||||
|
||||
typedef void gdma_eq_callback(void *context, struct gdma_queue *q,
|
||||
struct gdma_event *e);
|
||||
|
||||
@ -339,14 +337,6 @@ struct gdma_queue {
|
||||
unsigned int msix_index;
|
||||
|
||||
uint32_t log2_throttle_limit;
|
||||
|
||||
struct task cleanup_task;
|
||||
struct taskqueue *cleanup_tq;
|
||||
int cpu;
|
||||
bool do_not_ring_db;
|
||||
|
||||
int work_done;
|
||||
int budget;
|
||||
} eq;
|
||||
|
||||
struct {
|
||||
@ -371,9 +361,6 @@ struct gdma_queue_spec {
|
||||
void *context;
|
||||
|
||||
unsigned long log2_throttle_limit;
|
||||
|
||||
/* Only used by the MANA device. */
|
||||
struct ifnet *ndev;
|
||||
} eq;
|
||||
|
||||
struct {
|
||||
@ -388,7 +375,6 @@ struct gdma_queue_spec {
|
||||
|
||||
struct mana_eq {
|
||||
struct gdma_queue *eq;
|
||||
struct gdma_comp cqe_poll[CQE_POLLING_BUFFER];
|
||||
};
|
||||
|
||||
struct gdma_irq_context {
|
||||
@ -473,7 +459,7 @@ void mana_gd_destroy_queue(struct gdma_context *gc, struct gdma_queue *queue);
|
||||
|
||||
int mana_gd_poll_cq(struct gdma_queue *cq, struct gdma_comp *comp, int num_cqe);
|
||||
|
||||
void mana_gd_arm_cq(struct gdma_queue *cq);
|
||||
void mana_gd_ring_cq(struct gdma_queue *cq, uint8_t arm_bit);
|
||||
|
||||
struct gdma_wqe {
|
||||
uint32_t reserved :24;
|
||||
|
@ -422,7 +422,7 @@ mana_gd_wq_ring_doorbell(struct gdma_context *gc, struct gdma_queue *queue)
|
||||
}
|
||||
|
||||
void
|
||||
mana_gd_arm_cq(struct gdma_queue *cq)
|
||||
mana_gd_ring_cq(struct gdma_queue *cq, uint8_t arm_bit)
|
||||
{
|
||||
struct gdma_context *gc = cq->gdma_dev->gdma_context;
|
||||
|
||||
@ -431,7 +431,7 @@ mana_gd_arm_cq(struct gdma_queue *cq)
|
||||
uint32_t head = cq->head % (num_cqe << GDMA_CQE_OWNER_BITS);
|
||||
|
||||
mana_gd_ring_doorbell(gc, cq->gdma_dev->doorbell, cq->type, cq->id,
|
||||
head, SET_ARM_BIT);
|
||||
head, arm_bit);
|
||||
}
|
||||
|
||||
static void
|
||||
@ -508,7 +508,6 @@ mana_gd_process_eq_events(void *arg)
|
||||
struct gdma_context *gc;
|
||||
uint32_t head, num_eqe;
|
||||
struct gdma_eqe *eqe;
|
||||
unsigned int arm_bit;
|
||||
int i, j;
|
||||
|
||||
gc = eq->gdma_dev->gdma_context;
|
||||
@ -565,66 +564,17 @@ mana_gd_process_eq_events(void *arg)
|
||||
bus_dmamap_sync(eq->mem_info.dma_tag, eq->mem_info.dma_map,
|
||||
BUS_DMASYNC_PREREAD);
|
||||
|
||||
/* Always rearm the EQ for HWC. */
|
||||
if (mana_gd_is_hwc(eq->gdma_dev)) {
|
||||
arm_bit = SET_ARM_BIT;
|
||||
} else if (eq->eq.work_done < eq->eq.budget &&
|
||||
eq->eq.do_not_ring_db == false) {
|
||||
arm_bit = SET_ARM_BIT;
|
||||
} else {
|
||||
arm_bit = 0;
|
||||
}
|
||||
|
||||
head = eq->head % (num_eqe << GDMA_EQE_OWNER_BITS);
|
||||
|
||||
mana_gd_ring_doorbell(gc, eq->gdma_dev->doorbell, eq->type, eq->id,
|
||||
head, arm_bit);
|
||||
}
|
||||
|
||||
#define MANA_POLL_BUDGET 8
|
||||
#define MANA_RX_BUDGET 256
|
||||
|
||||
static void
|
||||
mana_poll(void *arg, int pending)
|
||||
{
|
||||
struct gdma_queue *eq = arg;
|
||||
int i;
|
||||
|
||||
eq->eq.work_done = 0;
|
||||
eq->eq.budget = MANA_RX_BUDGET;
|
||||
|
||||
for (i = 0; i < MANA_POLL_BUDGET; i++) {
|
||||
/*
|
||||
* If this is the last loop, set the budget big enough
|
||||
* so it will arm the EQ any way.
|
||||
*/
|
||||
if (i == (MANA_POLL_BUDGET - 1))
|
||||
eq->eq.budget = CQE_POLLING_BUFFER + 1;
|
||||
|
||||
mana_gd_process_eq_events(eq);
|
||||
|
||||
if (eq->eq.work_done < eq->eq.budget)
|
||||
break;
|
||||
|
||||
eq->eq.work_done = 0;
|
||||
}
|
||||
}
|
||||
|
||||
static void
|
||||
mana_gd_schedule_task(void *arg)
|
||||
{
|
||||
struct gdma_queue *eq = arg;
|
||||
|
||||
taskqueue_enqueue(eq->eq.cleanup_tq, &eq->eq.cleanup_task);
|
||||
head, SET_ARM_BIT);
|
||||
}
|
||||
|
||||
static int
|
||||
mana_gd_register_irq(struct gdma_queue *queue,
|
||||
const struct gdma_queue_spec *spec)
|
||||
{
|
||||
static int mana_last_bind_cpu = -1;
|
||||
struct gdma_dev *gd = queue->gdma_dev;
|
||||
bool is_mana = mana_gd_is_mana(gd);
|
||||
struct gdma_irq_context *gic;
|
||||
struct gdma_context *gc;
|
||||
struct gdma_resource *r;
|
||||
@ -659,39 +609,6 @@ mana_gd_register_irq(struct gdma_queue *queue,
|
||||
|
||||
gic = &gc->irq_contexts[msi_index];
|
||||
|
||||
if (is_mana) {
|
||||
struct mana_port_context *apc = if_getsoftc(spec->eq.ndev);
|
||||
queue->eq.do_not_ring_db = false;
|
||||
|
||||
NET_TASK_INIT(&queue->eq.cleanup_task, 0, mana_poll, queue);
|
||||
queue->eq.cleanup_tq =
|
||||
taskqueue_create_fast("mana eq cleanup",
|
||||
M_WAITOK, taskqueue_thread_enqueue,
|
||||
&queue->eq.cleanup_tq);
|
||||
|
||||
if (mana_last_bind_cpu < 0)
|
||||
mana_last_bind_cpu = CPU_FIRST();
|
||||
queue->eq.cpu = mana_last_bind_cpu;
|
||||
mana_last_bind_cpu = CPU_NEXT(mana_last_bind_cpu);
|
||||
|
||||
/* XXX Name is not optimal. However we have to start
|
||||
* the task here. Otherwise, test eq will have no
|
||||
* handler.
|
||||
*/
|
||||
if (apc->bind_cleanup_thread_cpu) {
|
||||
cpuset_t cpu_mask;
|
||||
CPU_SETOF(queue->eq.cpu, &cpu_mask);
|
||||
taskqueue_start_threads_cpuset(&queue->eq.cleanup_tq,
|
||||
1, PI_NET, &cpu_mask,
|
||||
"mana eq poll msix %u on cpu %d",
|
||||
msi_index, queue->eq.cpu);
|
||||
} else {
|
||||
|
||||
taskqueue_start_threads(&queue->eq.cleanup_tq, 1,
|
||||
PI_NET, "mana eq poll on msix %u", msi_index);
|
||||
}
|
||||
}
|
||||
|
||||
if (unlikely(gic->handler || gic->arg)) {
|
||||
device_printf(gc->dev,
|
||||
"interrupt handler or arg already assigned, "
|
||||
@ -700,10 +617,7 @@ mana_gd_register_irq(struct gdma_queue *queue,
|
||||
|
||||
gic->arg = queue;
|
||||
|
||||
if (is_mana)
|
||||
gic->handler = mana_gd_schedule_task;
|
||||
else
|
||||
gic->handler = mana_gd_process_eq_events;
|
||||
gic->handler = mana_gd_process_eq_events;
|
||||
|
||||
mana_dbg(NULL, "registered msix index %d vector %d irq %ju\n",
|
||||
msi_index, gic->msix_e.vector, rman_get_start(gic->res));
|
||||
@ -810,15 +724,6 @@ mana_gd_destroy_eq(struct gdma_context *gc, bool flush_evenets,
|
||||
|
||||
mana_gd_deregiser_irq(queue);
|
||||
|
||||
if (mana_gd_is_mana(queue->gdma_dev)) {
|
||||
while (taskqueue_cancel(queue->eq.cleanup_tq,
|
||||
&queue->eq.cleanup_task, NULL))
|
||||
taskqueue_drain(queue->eq.cleanup_tq,
|
||||
&queue->eq.cleanup_task);
|
||||
|
||||
taskqueue_free(queue->eq.cleanup_tq);
|
||||
}
|
||||
|
||||
if (queue->eq.disable_needed)
|
||||
mana_gd_disable_queue(queue);
|
||||
}
|
||||
@ -1602,10 +1507,8 @@ mana_gd_setup_irqs(device_t dev)
|
||||
if (max_queues_per_port > MANA_MAX_NUM_QUEUES)
|
||||
max_queues_per_port = MANA_MAX_NUM_QUEUES;
|
||||
|
||||
max_irqs = max_queues_per_port * MAX_PORTS_IN_MANA_DEV;
|
||||
|
||||
/* Need 1 interrupt for the Hardware communication Channel (HWC) */
|
||||
max_irqs++;
|
||||
max_irqs = max_queues_per_port + 1;
|
||||
|
||||
nvec = max_irqs;
|
||||
rc = pci_alloc_msix(dev, &nvec);
|
||||
|
@ -377,7 +377,7 @@ mana_hwc_comp_event(void *ctx, struct gdma_queue *q_self)
|
||||
bus_dmamap_sync(q_self->mem_info.dma_tag, q_self->mem_info.dma_map,
|
||||
BUS_DMASYNC_POSTREAD);
|
||||
|
||||
mana_gd_arm_cq(q_self);
|
||||
mana_gd_ring_cq(q_self, SET_ARM_BIT);
|
||||
}
|
||||
|
||||
static void
|
||||
|
@ -115,11 +115,7 @@ enum TRI_STATE {
|
||||
#define EQ_SIZE (8 * PAGE_SIZE)
|
||||
#define LOG2_EQ_THROTTLE 3
|
||||
|
||||
#if 1 /* XXX */
|
||||
#define MAX_PORTS_IN_MANA_DEV 1
|
||||
#else
|
||||
#define MAX_PORTS_IN_MANA_DEV 16
|
||||
#endif
|
||||
#define MAX_PORTS_IN_MANA_DEV 8
|
||||
|
||||
struct mana_send_buf_info {
|
||||
struct mbuf *mbuf;
|
||||
@ -351,6 +347,8 @@ struct mana_tx_comp_oob {
|
||||
|
||||
struct mana_rxq;
|
||||
|
||||
#define CQE_POLLING_BUFFER 512
|
||||
|
||||
struct mana_cq {
|
||||
struct gdma_queue *gdma_cq;
|
||||
|
||||
@ -370,8 +368,18 @@ struct mana_cq {
|
||||
*/
|
||||
struct mana_txq *txq;
|
||||
|
||||
/* Pointer to a buffer which the CQ handler can copy the CQE's into. */
|
||||
struct gdma_comp *gdma_comp_buf;
|
||||
/* Taskqueue and related structs */
|
||||
struct task cleanup_task;
|
||||
struct taskqueue *cleanup_tq;
|
||||
int cpu;
|
||||
bool do_not_ring_db;
|
||||
|
||||
/* Budget for one cleanup task */
|
||||
int work_done;
|
||||
int budget;
|
||||
|
||||
/* Buffer which the CQ handler can copy the CQE's into. */
|
||||
struct gdma_comp gdma_comp_buf[CQE_POLLING_BUFFER];
|
||||
};
|
||||
|
||||
#define GDMA_MAX_RQE_SGES 15
|
||||
@ -451,6 +459,8 @@ struct mana_context {
|
||||
|
||||
uint16_t num_ports;
|
||||
|
||||
struct mana_eq *eqs;
|
||||
|
||||
struct ifnet *ports[MAX_PORTS_IN_MANA_DEV];
|
||||
};
|
||||
|
||||
@ -467,8 +477,6 @@ struct mana_port_context {
|
||||
|
||||
uint8_t mac_addr[ETHER_ADDR_LEN];
|
||||
|
||||
struct mana_eq *eqs;
|
||||
|
||||
enum TRI_STATE rss_state;
|
||||
|
||||
mana_handle_t default_rxobj;
|
||||
@ -503,7 +511,10 @@ struct mana_port_context {
|
||||
bool port_st_save; /* Saved port state */
|
||||
|
||||
bool enable_tx_altq;
|
||||
|
||||
bool bind_cleanup_thread_cpu;
|
||||
int last_tx_cq_bind_cpu;
|
||||
int last_rx_cq_bind_cpu;
|
||||
|
||||
struct mana_port_stats port_stats;
|
||||
|
||||
|
@ -108,7 +108,7 @@ mana_ifmedia_status(struct ifnet *ifp, struct ifmediareq *ifmr)
|
||||
|
||||
if (!apc->port_is_up) {
|
||||
MANA_APC_LOCK_UNLOCK(apc);
|
||||
mana_info(NULL, "Port %u link is down\n", apc->port_idx);
|
||||
mana_dbg(NULL, "Port %u link is down\n", apc->port_idx);
|
||||
return;
|
||||
}
|
||||
|
||||
@ -142,19 +142,6 @@ mana_get_counter(struct ifnet *ifp, ift_counter cnt)
|
||||
}
|
||||
}
|
||||
|
||||
static void
|
||||
mana_drain_eq_task(struct gdma_queue *queue)
|
||||
{
|
||||
if (!queue || !queue->eq.cleanup_tq)
|
||||
return;
|
||||
|
||||
while (taskqueue_cancel(queue->eq.cleanup_tq,
|
||||
&queue->eq.cleanup_task, NULL)) {
|
||||
taskqueue_drain(queue->eq.cleanup_tq,
|
||||
&queue->eq.cleanup_task);
|
||||
}
|
||||
}
|
||||
|
||||
static void
|
||||
mana_qflush(struct ifnet *ifp)
|
||||
{
|
||||
@ -439,13 +426,11 @@ mana_xmit(struct mana_txq *txq)
|
||||
struct mana_tx_package pkg = {};
|
||||
struct mana_stats *tx_stats;
|
||||
struct gdma_queue *gdma_sq;
|
||||
struct gdma_queue *gdma_eq;
|
||||
struct mana_cq *cq;
|
||||
int err, len;
|
||||
|
||||
gdma_sq = txq->gdma_sq;
|
||||
cq = &apc->tx_qp[txq->idx].tx_cq;
|
||||
gdma_eq = cq->gdma_cq->cq.parent;
|
||||
tx_stats = &txq->stats;
|
||||
|
||||
packets = 0;
|
||||
@ -476,8 +461,7 @@ mana_xmit(struct mana_txq *txq)
|
||||
|
||||
drbr_putback(ndev, txq->txq_br, mbuf);
|
||||
|
||||
taskqueue_enqueue(gdma_eq->eq.cleanup_tq,
|
||||
&gdma_eq->eq.cleanup_task);
|
||||
taskqueue_enqueue(cq->cleanup_tq, &cq->cleanup_task);
|
||||
break;
|
||||
}
|
||||
|
||||
@ -1183,67 +1167,57 @@ mana_destroy_wq_obj(struct mana_port_context *apc, uint32_t wq_type,
|
||||
}
|
||||
|
||||
static void
|
||||
mana_init_cqe_poll_buf(struct gdma_comp *cqe_poll_buf)
|
||||
{
|
||||
int i;
|
||||
|
||||
for (i = 0; i < CQE_POLLING_BUFFER; i++)
|
||||
memset(&cqe_poll_buf[i], 0, sizeof(struct gdma_comp));
|
||||
}
|
||||
|
||||
static void
|
||||
mana_destroy_eq(struct gdma_context *gc, struct mana_port_context *apc)
|
||||
mana_destroy_eq(struct mana_context *ac)
|
||||
{
|
||||
struct gdma_context *gc = ac->gdma_dev->gdma_context;
|
||||
struct gdma_queue *eq;
|
||||
int i;
|
||||
|
||||
if (!apc->eqs)
|
||||
if (!ac->eqs)
|
||||
return;
|
||||
|
||||
for (i = 0; i < apc->num_queues; i++) {
|
||||
eq = apc->eqs[i].eq;
|
||||
for (i = 0; i < gc->max_num_queues; i++) {
|
||||
eq = ac->eqs[i].eq;
|
||||
if (!eq)
|
||||
continue;
|
||||
|
||||
mana_gd_destroy_queue(gc, eq);
|
||||
}
|
||||
|
||||
free(apc->eqs, M_DEVBUF);
|
||||
apc->eqs = NULL;
|
||||
free(ac->eqs, M_DEVBUF);
|
||||
ac->eqs = NULL;
|
||||
}
|
||||
|
||||
static int
|
||||
mana_create_eq(struct mana_port_context *apc)
|
||||
mana_create_eq(struct mana_context *ac)
|
||||
{
|
||||
struct gdma_dev *gd = apc->ac->gdma_dev;
|
||||
struct gdma_dev *gd = ac->gdma_dev;
|
||||
struct gdma_context *gc = gd->gdma_context;
|
||||
struct gdma_queue_spec spec = {};
|
||||
int err;
|
||||
int i;
|
||||
|
||||
apc->eqs = mallocarray(apc->num_queues, sizeof(struct mana_eq),
|
||||
ac->eqs = mallocarray(gc->max_num_queues, sizeof(struct mana_eq),
|
||||
M_DEVBUF, M_WAITOK | M_ZERO);
|
||||
if (!apc->eqs)
|
||||
if (!ac->eqs)
|
||||
return ENOMEM;
|
||||
|
||||
spec.type = GDMA_EQ;
|
||||
spec.monitor_avl_buf = false;
|
||||
spec.queue_size = EQ_SIZE;
|
||||
spec.eq.callback = NULL;
|
||||
spec.eq.context = apc->eqs;
|
||||
spec.eq.context = ac->eqs;
|
||||
spec.eq.log2_throttle_limit = LOG2_EQ_THROTTLE;
|
||||
spec.eq.ndev = apc->ndev;
|
||||
|
||||
for (i = 0; i < apc->num_queues; i++) {
|
||||
mana_init_cqe_poll_buf(apc->eqs[i].cqe_poll);
|
||||
|
||||
err = mana_gd_create_mana_eq(gd, &spec, &apc->eqs[i].eq);
|
||||
for (i = 0; i < gc->max_num_queues; i++) {
|
||||
err = mana_gd_create_mana_eq(gd, &spec, &ac->eqs[i].eq);
|
||||
if (err)
|
||||
goto out;
|
||||
}
|
||||
|
||||
return 0;
|
||||
out:
|
||||
mana_destroy_eq(gd->gdma_context, apc);
|
||||
mana_destroy_eq(ac);
|
||||
return err;
|
||||
}
|
||||
|
||||
@ -1294,6 +1268,9 @@ mana_poll_tx_cq(struct mana_cq *cq)
|
||||
comp_read = mana_gd_poll_cq(cq->gdma_cq, completions,
|
||||
CQE_POLLING_BUFFER);
|
||||
|
||||
if (comp_read < 1)
|
||||
return;
|
||||
|
||||
next_to_complete = txq->next_to_complete;
|
||||
|
||||
for (i = 0; i < comp_read; i++) {
|
||||
@ -1360,7 +1337,7 @@ mana_poll_tx_cq(struct mana_cq *cq)
|
||||
txq_idx, next_to_complete, txq->next_to_use,
|
||||
txq->pending_sends, pkt_transmitted, sa_drop,
|
||||
i, comp_read);
|
||||
continue;
|
||||
break;
|
||||
}
|
||||
|
||||
wqe_info = &tx_info->wqe_inf;
|
||||
@ -1430,6 +1407,8 @@ mana_poll_tx_cq(struct mana_cq *cq)
|
||||
mana_err(NULL,
|
||||
"WARNING: TX %d pending_sends error: %d\n",
|
||||
txq->idx, txq->pending_sends);
|
||||
|
||||
cq->work_done = pkt_transmitted;
|
||||
}
|
||||
|
||||
static void
|
||||
@ -1468,13 +1447,11 @@ mana_rx_mbuf(struct mbuf *mbuf, struct mana_rxcomp_oob *cqe,
|
||||
uint32_t pkt_len = cqe->ppi[0].pkt_len;
|
||||
uint16_t rxq_idx = rxq->rxq_idx;
|
||||
struct mana_port_context *apc;
|
||||
struct gdma_queue *eq;
|
||||
bool do_lro = false;
|
||||
bool do_if_input;
|
||||
|
||||
apc = if_getsoftc(ndev);
|
||||
eq = apc->eqs[rxq_idx].eq;
|
||||
eq->eq.work_done++;
|
||||
rxq->rx_cq.work_done++;
|
||||
|
||||
if (!mbuf) {
|
||||
return;
|
||||
@ -1688,6 +1665,7 @@ static void
|
||||
mana_cq_handler(void *context, struct gdma_queue *gdma_queue)
|
||||
{
|
||||
struct mana_cq *cq = context;
|
||||
uint8_t arm_bit;
|
||||
|
||||
KASSERT(cq->gdma_cq == gdma_queue,
|
||||
("cq do not match %p, %p", cq->gdma_cq, gdma_queue));
|
||||
@ -1698,7 +1676,54 @@ mana_cq_handler(void *context, struct gdma_queue *gdma_queue)
|
||||
mana_poll_tx_cq(cq);
|
||||
}
|
||||
|
||||
mana_gd_arm_cq(gdma_queue);
|
||||
if (cq->work_done < cq->budget && cq->do_not_ring_db == false)
|
||||
arm_bit = SET_ARM_BIT;
|
||||
else
|
||||
arm_bit = 0;
|
||||
|
||||
mana_gd_ring_cq(gdma_queue, arm_bit);
|
||||
}
|
||||
|
||||
#define MANA_POLL_BUDGET 8
|
||||
#define MANA_RX_BUDGET 256
|
||||
#define MANA_TX_BUDGET MAX_SEND_BUFFERS_PER_QUEUE
|
||||
|
||||
static void
|
||||
mana_poll(void *arg, int pending)
|
||||
{
|
||||
struct mana_cq *cq = arg;
|
||||
int i;
|
||||
|
||||
cq->work_done = 0;
|
||||
if (cq->type == MANA_CQ_TYPE_RX) {
|
||||
cq->budget = MANA_RX_BUDGET;
|
||||
} else {
|
||||
cq->budget = MANA_TX_BUDGET;
|
||||
}
|
||||
|
||||
for (i = 0; i < MANA_POLL_BUDGET; i++) {
|
||||
/*
|
||||
* If this is the last loop, set the budget big enough
|
||||
* so it will arm the CQ any way.
|
||||
*/
|
||||
if (i == (MANA_POLL_BUDGET - 1))
|
||||
cq->budget = CQE_POLLING_BUFFER + 1;
|
||||
|
||||
mana_cq_handler(cq, cq->gdma_cq);
|
||||
|
||||
if (cq->work_done < cq->budget)
|
||||
break;
|
||||
|
||||
cq->work_done = 0;
|
||||
}
|
||||
}
|
||||
|
||||
static void
|
||||
mana_schedule_task(void *arg, struct gdma_queue *gdma_queue)
|
||||
{
|
||||
struct mana_cq *cq = arg;
|
||||
|
||||
taskqueue_enqueue(cq->cleanup_tq, &cq->cleanup_task);
|
||||
}
|
||||
|
||||
static void
|
||||
@ -1709,6 +1734,17 @@ mana_deinit_cq(struct mana_port_context *apc, struct mana_cq *cq)
|
||||
if (!cq->gdma_cq)
|
||||
return;
|
||||
|
||||
/* Drain cleanup taskqueue */
|
||||
if (cq->cleanup_tq) {
|
||||
while (taskqueue_cancel(cq->cleanup_tq,
|
||||
&cq->cleanup_task, NULL)) {
|
||||
taskqueue_drain(cq->cleanup_tq,
|
||||
&cq->cleanup_task);
|
||||
}
|
||||
|
||||
taskqueue_free(cq->cleanup_tq);
|
||||
}
|
||||
|
||||
mana_gd_destroy_queue(gd->gdma_context, cq->gdma_cq);
|
||||
}
|
||||
|
||||
@ -1798,7 +1834,8 @@ mana_destroy_txq(struct mana_port_context *apc)
|
||||
static int
|
||||
mana_create_txq(struct mana_port_context *apc, struct ifnet *net)
|
||||
{
|
||||
struct gdma_dev *gd = apc->ac->gdma_dev;
|
||||
struct mana_context *ac = apc->ac;
|
||||
struct gdma_dev *gd = ac->gdma_dev;
|
||||
struct mana_obj_spec wq_spec;
|
||||
struct mana_obj_spec cq_spec;
|
||||
struct gdma_queue_spec spec;
|
||||
@ -1850,7 +1887,6 @@ mana_create_txq(struct mana_port_context *apc, struct ifnet *net)
|
||||
|
||||
/* Create SQ's CQ */
|
||||
cq = &apc->tx_qp[i].tx_cq;
|
||||
cq->gdma_comp_buf = apc->eqs[i].cqe_poll;
|
||||
cq->type = MANA_CQ_TYPE_TX;
|
||||
|
||||
cq->txq = txq;
|
||||
@ -1859,8 +1895,8 @@ mana_create_txq(struct mana_port_context *apc, struct ifnet *net)
|
||||
spec.type = GDMA_CQ;
|
||||
spec.monitor_avl_buf = false;
|
||||
spec.queue_size = cq_size;
|
||||
spec.cq.callback = mana_cq_handler;
|
||||
spec.cq.parent_eq = apc->eqs[i].eq;
|
||||
spec.cq.callback = mana_schedule_task;
|
||||
spec.cq.parent_eq = ac->eqs[i].eq;
|
||||
spec.cq.context = cq;
|
||||
err = mana_gd_create_mana_wq_cq(gd, &spec, &cq->gdma_cq);
|
||||
if (err)
|
||||
@ -1942,12 +1978,39 @@ mana_create_txq(struct mana_port_context *apc, struct ifnet *net)
|
||||
goto out;
|
||||
}
|
||||
taskqueue_start_threads(&txq->enqueue_tq, 1, PI_NET,
|
||||
"mana txq %d", i);
|
||||
"mana txq p%u-tx%d", apc->port_idx, i);
|
||||
|
||||
mana_alloc_counters((counter_u64_t *)&txq->stats,
|
||||
sizeof(txq->stats));
|
||||
|
||||
mana_gd_arm_cq(cq->gdma_cq);
|
||||
/* Allocate and start the cleanup task on CQ */
|
||||
cq->do_not_ring_db = false;
|
||||
|
||||
NET_TASK_INIT(&cq->cleanup_task, 0, mana_poll, cq);
|
||||
cq->cleanup_tq =
|
||||
taskqueue_create_fast("mana tx cq cleanup",
|
||||
M_WAITOK, taskqueue_thread_enqueue,
|
||||
&cq->cleanup_tq);
|
||||
|
||||
if (apc->last_tx_cq_bind_cpu < 0)
|
||||
apc->last_tx_cq_bind_cpu = CPU_FIRST();
|
||||
cq->cpu = apc->last_tx_cq_bind_cpu;
|
||||
apc->last_tx_cq_bind_cpu = CPU_NEXT(apc->last_tx_cq_bind_cpu);
|
||||
|
||||
if (apc->bind_cleanup_thread_cpu) {
|
||||
cpuset_t cpu_mask;
|
||||
CPU_SETOF(cq->cpu, &cpu_mask);
|
||||
taskqueue_start_threads_cpuset(&cq->cleanup_tq,
|
||||
1, PI_NET, &cpu_mask,
|
||||
"mana cq p%u-tx%u-cpu%d",
|
||||
apc->port_idx, txq->idx, cq->cpu);
|
||||
} else {
|
||||
taskqueue_start_threads(&cq->cleanup_tq, 1,
|
||||
PI_NET, "mana cq p%u-tx%u",
|
||||
apc->port_idx, txq->idx);
|
||||
}
|
||||
|
||||
mana_gd_ring_cq(cq->gdma_cq, SET_ARM_BIT);
|
||||
}
|
||||
|
||||
return 0;
|
||||
@ -2144,7 +2207,6 @@ mana_create_rxq(struct mana_port_context *apc, uint32_t rxq_idx,
|
||||
|
||||
/* Create RQ's CQ */
|
||||
cq = &rxq->rx_cq;
|
||||
cq->gdma_comp_buf = eq->cqe_poll;
|
||||
cq->type = MANA_CQ_TYPE_RX;
|
||||
cq->rxq = rxq;
|
||||
|
||||
@ -2152,7 +2214,7 @@ mana_create_rxq(struct mana_port_context *apc, uint32_t rxq_idx,
|
||||
spec.type = GDMA_CQ;
|
||||
spec.monitor_avl_buf = false;
|
||||
spec.queue_size = cq_size;
|
||||
spec.cq.callback = mana_cq_handler;
|
||||
spec.cq.callback = mana_schedule_task;
|
||||
spec.cq.parent_eq = eq->eq;
|
||||
spec.cq.context = cq;
|
||||
err = mana_gd_create_mana_wq_cq(gd, &spec, &cq->gdma_cq);
|
||||
@ -2192,7 +2254,34 @@ mana_create_rxq(struct mana_port_context *apc, uint32_t rxq_idx,
|
||||
|
||||
gc->cq_table[cq->gdma_id] = cq->gdma_cq;
|
||||
|
||||
mana_gd_arm_cq(cq->gdma_cq);
|
||||
/* Allocate and start the cleanup task on CQ */
|
||||
cq->do_not_ring_db = false;
|
||||
|
||||
NET_TASK_INIT(&cq->cleanup_task, 0, mana_poll, cq);
|
||||
cq->cleanup_tq =
|
||||
taskqueue_create_fast("mana rx cq cleanup",
|
||||
M_WAITOK, taskqueue_thread_enqueue,
|
||||
&cq->cleanup_tq);
|
||||
|
||||
if (apc->last_rx_cq_bind_cpu < 0)
|
||||
apc->last_rx_cq_bind_cpu = CPU_FIRST();
|
||||
cq->cpu = apc->last_rx_cq_bind_cpu;
|
||||
apc->last_rx_cq_bind_cpu = CPU_NEXT(apc->last_rx_cq_bind_cpu);
|
||||
|
||||
if (apc->bind_cleanup_thread_cpu) {
|
||||
cpuset_t cpu_mask;
|
||||
CPU_SETOF(cq->cpu, &cpu_mask);
|
||||
taskqueue_start_threads_cpuset(&cq->cleanup_tq,
|
||||
1, PI_NET, &cpu_mask,
|
||||
"mana cq p%u-rx%u-cpu%d",
|
||||
apc->port_idx, rxq->rxq_idx, cq->cpu);
|
||||
} else {
|
||||
taskqueue_start_threads(&cq->cleanup_tq, 1,
|
||||
PI_NET, "mana cq p%u-rx%u",
|
||||
apc->port_idx, rxq->rxq_idx);
|
||||
}
|
||||
|
||||
mana_gd_ring_cq(cq->gdma_cq, SET_ARM_BIT);
|
||||
out:
|
||||
if (!err)
|
||||
return rxq;
|
||||
@ -2210,12 +2299,13 @@ mana_create_rxq(struct mana_port_context *apc, uint32_t rxq_idx,
|
||||
static int
|
||||
mana_add_rx_queues(struct mana_port_context *apc, struct ifnet *ndev)
|
||||
{
|
||||
struct mana_context *ac = apc->ac;
|
||||
struct mana_rxq *rxq;
|
||||
int err = 0;
|
||||
int i;
|
||||
|
||||
for (i = 0; i < apc->num_queues; i++) {
|
||||
rxq = mana_create_rxq(apc, i, &apc->eqs[i], ndev);
|
||||
rxq = mana_create_rxq(apc, i, &ac->eqs[i], ndev);
|
||||
if (!rxq) {
|
||||
err = ENOMEM;
|
||||
goto out;
|
||||
@ -2234,20 +2324,12 @@ mana_destroy_vport(struct mana_port_context *apc)
|
||||
{
|
||||
struct mana_rxq *rxq;
|
||||
uint32_t rxq_idx;
|
||||
struct mana_cq *rx_cq;
|
||||
struct gdma_queue *cq, *eq;
|
||||
|
||||
for (rxq_idx = 0; rxq_idx < apc->num_queues; rxq_idx++) {
|
||||
rxq = apc->rxqs[rxq_idx];
|
||||
if (!rxq)
|
||||
continue;
|
||||
|
||||
rx_cq = &rxq->rx_cq;
|
||||
if ((cq = rx_cq->gdma_cq) != NULL) {
|
||||
eq = cq->cq.parent;
|
||||
mana_drain_eq_task(eq);
|
||||
}
|
||||
|
||||
mana_destroy_rxq(apc, rxq, true);
|
||||
apc->rxqs[rxq_idx] = NULL;
|
||||
}
|
||||
@ -2336,16 +2418,11 @@ int
|
||||
mana_alloc_queues(struct ifnet *ndev)
|
||||
{
|
||||
struct mana_port_context *apc = if_getsoftc(ndev);
|
||||
struct gdma_dev *gd = apc->ac->gdma_dev;
|
||||
int err;
|
||||
|
||||
err = mana_create_eq(apc);
|
||||
if (err)
|
||||
return err;
|
||||
|
||||
err = mana_create_vport(apc, ndev);
|
||||
if (err)
|
||||
goto destroy_eq;
|
||||
return err;
|
||||
|
||||
err = mana_add_rx_queues(apc, ndev);
|
||||
if (err)
|
||||
@ -2363,8 +2440,6 @@ mana_alloc_queues(struct ifnet *ndev)
|
||||
|
||||
destroy_vport:
|
||||
mana_destroy_vport(apc);
|
||||
destroy_eq:
|
||||
mana_destroy_eq(gd->gdma_context, apc);
|
||||
return err;
|
||||
}
|
||||
|
||||
@ -2430,16 +2505,13 @@ mana_dealloc_queues(struct ifnet *ndev)
|
||||
txq = &apc->tx_qp[i].txq;
|
||||
|
||||
struct mana_cq *tx_cq = &apc->tx_qp[i].tx_cq;
|
||||
struct gdma_queue *eq = NULL;
|
||||
if (tx_cq->gdma_cq)
|
||||
eq = tx_cq->gdma_cq->cq.parent;
|
||||
if (eq) {
|
||||
/* Stop EQ interrupt */
|
||||
eq->eq.do_not_ring_db = true;
|
||||
/* Schedule a cleanup task */
|
||||
taskqueue_enqueue(eq->eq.cleanup_tq,
|
||||
&eq->eq.cleanup_task);
|
||||
}
|
||||
struct mana_cq *rx_cq = &(apc->rxqs[i]->rx_cq);
|
||||
|
||||
tx_cq->do_not_ring_db = true;
|
||||
rx_cq->do_not_ring_db = true;
|
||||
|
||||
/* Schedule a cleanup task */
|
||||
taskqueue_enqueue(tx_cq->cleanup_tq, &tx_cq->cleanup_task);
|
||||
|
||||
while (atomic_read(&txq->pending_sends) > 0)
|
||||
usleep_range(1000, 2000);
|
||||
@ -2461,8 +2533,6 @@ mana_dealloc_queues(struct ifnet *ndev)
|
||||
|
||||
mana_destroy_vport(apc);
|
||||
|
||||
mana_destroy_eq(apc->ac->gdma_dev->gdma_context, apc);
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
@ -2550,6 +2620,8 @@ mana_probe_port(struct mana_context *ac, int port_idx,
|
||||
apc->port_handle = INVALID_MANA_HANDLE;
|
||||
apc->port_idx = port_idx;
|
||||
apc->frame_size = DEFAULT_FRAME_SIZE;
|
||||
apc->last_tx_cq_bind_cpu = -1;
|
||||
apc->last_rx_cq_bind_cpu = -1;
|
||||
|
||||
MANA_APC_LOCK_INIT(apc);
|
||||
|
||||
@ -2637,6 +2709,10 @@ int mana_probe(struct gdma_dev *gd)
|
||||
ac->num_ports = 1;
|
||||
gd->driver_data = ac;
|
||||
|
||||
err = mana_create_eq(ac);
|
||||
if (err)
|
||||
goto out;
|
||||
|
||||
err = mana_query_device_cfg(ac, MANA_MAJOR_VERSION, MANA_MINOR_VERSION,
|
||||
MANA_MICRO_VERSION, &ac->num_ports);
|
||||
if (err)
|
||||
@ -2682,6 +2758,9 @@ mana_remove(struct gdma_dev *gd)
|
||||
|
||||
if_free(ndev);
|
||||
}
|
||||
|
||||
mana_destroy_eq(ac);
|
||||
|
||||
out:
|
||||
mana_gd_deregister_device(gd);
|
||||
gd->driver_data = NULL;
|
||||
|
Loading…
Reference in New Issue
Block a user